2

Подскажите пожалуйста. Имеется массив ArrayList. Его я объявляю в классе переменных:

private ArrayList<Part> parts;

Как написать проверку инициализирован ли он или нет? и если нет, то инициализировать. Следующий код неверный:

public ArrayList<Part> initArrayList(Part part){
    if (parts.isEmpty()) {
        parts = new ArrayList<>();
        parts.add(part);
        }
    else parts.add(part);

    return parts;
1
  • Пустой (isEmpty) это уже инициализированный. Надо проверять равенством с null как в ответе ниже. new ArrayList() каждый раз создаёт аррай, который будет isEmpty (если в него ничего не поместить конечно)
    – Sergey
    21 окт 2016 в 12:35

3 ответа 3

6

Как написать проверку инициализирован ли он или нет? и если нет, то инициализировать.

Например, так:

public ArrayList<Part> initArrayList(ArrayList<Part> parts) {
    if (parts == null) {
        return new ArrayList<>();
    }
    return parts;
}

UPD.

Ввиду уровня заданного вопроса, специально не хотел путать автора тернарным оператором, но если Вы про него знаете, то метод initArrayList(...) можно реализовать несколько короче:

public static ArrayList<Part> initArrayList(ArrayList<Part> parts) {
    return parts == null ? new ArrayList<>() : parts;
}

А вообще, это т.н. syntactic sugar, не более того.

4
  • return parts == null ? new ArrayList() : parts; да и лишний переброс ссыли в метод чтоб сделать провекру, нуууууу... уж жирновато нет? Чем это отличается от if (parts != null) parts.add(new Part); в теории, хоть и маловероятно, есть возможность словить палку причем ударить может обеими сторонами, ну и ладно, плюсуют, и то хорошо)) 21 окт 2016 в 12:57
  • @ShwarzAndrei, 1. Я специально не стал путать человека, который задает подобного уровня вопросы, тернарным оператором. 2. У автора код был оформлен в виде метода – так и оставил.
    – post_zeew
    21 окт 2016 в 14:55
  • @m.vokhm, Поясните, пожалуйста, по поводу «если parts не инициализирован, он так и останется null после выполнения такого кода».
    – post_zeew
    21 окт 2016 в 15:13
  • @post_zeew, я невнимательно посмотрел, не заметил, что parts - это аргумент метода, показалось, что речь идет о поле объекта. Снимаю свое замечание.
    – m. vokhm
    21 окт 2016 в 17:53
3
// Запрос списка через добавление элемента с ленивой инициализацией списка
public ArrayList<Part> initArrayList(Part part){
    if (parts == null) parts = new ArrayList<>();
    parts.add(part);
    return parts;
}

    // Работать будет, но правильнее разнести две различные функции по разным методам

// Запрос списка с ленивой инициализацией
public List<Part> getParts() {
    if (parts == null) parts = new ArrayList<>();
    return parts;
}
// Добавление элемента в список
public void addPart(Part part) {
    getParts().add(part);
}
2
  • Комментируйте код в ответах - обосновывайте ваше решение 21 окт 2016 в 14:41
  • Добавил комментарии в код и пояснение, почему так делать не совсем правильно.
    – gmatagmis
    21 окт 2016 в 14:53
2

Вообще по идее ваше поле parts проинициализировано с самого начала, т.к. все поля объекта в java инициалищируются их значением по-умолчанию. Вот только так как ArrayList - объект, то дефолтное значение будет null. А если вы хотите, чтоб у вас был именно рабочий экземпляр ArrayList, то можно поступить как в предыдущем ответе, либо вообще задать значение поля при объявлении:

private ArrayList<Part> parts = new ArrayList<>();
2
  • 1
    Иногда бывает нужно различать, работали мы уже с ним или нет, и тога есть смысл держать там null, как признак того, что еще не обращались. А при обращении к нему делать так: if (parts == null) parts = new ArrayList(); return parts;
    – m. vokhm
    21 окт 2016 в 13:30
  • Это понятно, просто в данном случае автор поста по всей видимости пытается забежать вперед, толком не разобравшись, что же из себя представляет ArrayList, да и объект в целом, поэтому я попробовал объяснить
    – selya
    21 окт 2016 в 14:32

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.