Не проходит

Вопрос по задаче

Архивный

Не проходит валидацию 3 пункт(

public class Cat {
    public static ArrayList<Cat> cats = new ArrayList();

    public Cat(ArrayList<Cat> cats) {
        this.cats = cats;
    }

    public static void main(String[] args) {
        for (int a = 0; a < 10; a++) {
            Cat cat = new Cat(cats);
            cats.add(cat);
        }

        printCats();
    }

    public static void printCats() {
        String catName = "";
        for (int i = 0; i < cats.size(); i++) {
            catName = catName + cats.get(i) + " ";
            System.out.println(catName);
        }
        }
    }

1. В классе Cat добавь публичную статическую переменную cats (ArrayList<Cat>).
2. Пусть при каждом создании кота (нового объекта Cat) в переменную cats добавляется этот новый кот. Создать 10 объектов Cat.
3. Метод printCats должен выводить всех котов на экран. Нужно использовать переменную cats.

Требования:

Добавь в класс Cat публичную статическую переменную cats (ArrayList<Cat>).
Переменная cats должна быть проинициализирована.
Метод main должен создавать 10 объектов Cat (используй конструктор Cat()).
Метод main должен добавить всех созданных котов в переменную cats.
Метод printCats должен выводить всех котов из переменной cats на экран. Каждого с новой строки.

package com.javarush.task.task06.task0614; import java.util.ArrayList; /* Статические коты */ public class Cat { public static ArrayList<Cat> cats = new ArrayList(); public Cat(ArrayList<Cat> cats) { this.cats = cats; } public static void main(String[] args) { for (int a = 0; a < 10; a++) { Cat cat = new Cat(ArrayList<Cat> cats); cats.add(cat); } printCats(); } public static void printCats() { String catName = ""; for (int i = 0; i < cats.size(); i++) { catName = catName + cats.get(i) + " "; System.out.println(catName); } } }

Комментарии (6)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Сергеев Виктор

Уровень 40

Master

1 марта 2018, 20:40

полезный

а зачем вы в конструктор передаете лист?

Ксюха Бышевая

Уровень 10

1 марта 2018, 21:03

В условии написано проинициализировать cats, это нужно сделать как-то по-другому?)

Стас Пасинков Software Developer в Zipy

Master

1 марта 2018, 21:10

решение

проинициализирована:

public static ArrayList<Cat> cats = new ArrayList();

не проинициализирована:

public static ArrayList<Cat> cats;

Ксюха Бышевая

Уровень 10

1 марта 2018, 21:16

ах, вон оно что :D спасибочки.

Стас Пасинков Software Developer в Zipy

Master

2 марта 2018, 10:12

нет, ну конечно проинитить можно разными способами... можно в конструкторе, как вы сделали) можно в статическом блоке инициализации, можно вообще просто в коде где-то... но конкретно в этой задаче подразумевалось именно так, как вы сделали изначально :) только без повторной инициализации в конструкторе еще раз :) вообще, инициализировать статические переменные в конструкторе - не самая крутая идея. потому что из-за того, что они статики - к ним можно обращаться просто используя сам класс, объект этого класса создавать не обязательно. то-есть, я могу спокойно сделать так

int catsSize = Cat.cats.size();

при этом, не создавая объект класса Cat перед этим вызовом нигде. а если объект я не создавал - значит и конструктор не выполнялся еще. значит переменная та в классе Cat - не проинициализирована (если бы мы только в конструкторе ее инитили) и в ней лежит null. в итоге я получаю NullPointerException в этом куске кода :) но проблема в том, что об этом я узнаю только когда уже запущу программу. во время написания кода мне в этом месте никакая ошибка не возникнет :) поэтому да, статики лучше инициализировать либо сразу же, как только их объявили:

public static ArrayList<Cat> cats = new ArrayList();

либо в статическом блоке инициализации:

public static ArrayList<Cat> cats;

static {
    cats = new ArrayList();
}

получится, что при первом же обращении к этому классу - выполнится этот блок и все будет проинициализировано и готово к работе :) кроме того, в этом блоке мы можем еще и какие-то начальные значения добавить если нам очень надо:

public static ArrayList<Cat> cats;

static {
    cats = new ArrayList();
    cats.add(new Cat("Барсик"));
    cats.add(new Cat("Мурзик"));
}

Стас Пасинков Software Developer в Zipy

Master

2 марта 2018, 10:14

в общем, статический блок инициализации - это типа как конструктор, только для статических переменных. если в конструкторе мы устанавливаем значения для не статических переменных, то тут можем сделать то же, но для статиков в нашем классе :)