1

К примеру есть простой класс без конструктора

public class Pojo {

private String name;
private int age;

public String getName() {
    return name;
}

public void setName(String name) {
    this.name = name;
}

public int getAge() {
    return age;
}

public void setAge(int age) {
    this.age = age;
}}

Если вызвать getName у объекта данного класса, он возвращает null и вызов методов класса String например таких как length, isEmpty приводит к NullPointerException. Хотелось бы узнать, если внести вот такую модификацию в getName:

public String getName() {
    if (name == null) {
        name = "";
    }
    return name;
}

Является ли такой метод хорошим тоном? Или проверку на null нужно проводить при каждом вызове getName в первом варианте?

3
  • 1
    Меня поправят надеюсь если не прав, но может задать дефолтное имя и возраст, тогда доп методом не надо писать минимум 23 мар 2016 в 19:42
  • Ваш код и есть "проверка на null при каждом вызове getName".
    – user176262
    23 мар 2016 в 19:49
  • Если вам дан исчерпывающий ответ, отметьте его как принятый. 24 мар 2016 в 22:56

1 ответ 1

6

Не очень хорошее решение: будут проблемы с тестированием и т.п. При возврате пустой строки вы не сможете проверить, реально ли в поле ничего не было записано, или же была записана пустая строка.

Логично проверять на null в том месте, где вызов происходит. А вообще такие объекты надо делать немутабельными.

public class Pojo {

    private final String name;
    private final int age;

    public Pojo (String name, int age) {
        this.name = name;
        this.age = age;
    }
    public String getName() {
        return name;
    }

    public int getAge() {
        return age;
    }
}

Ваш ответ

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

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