0

Если при создания сущности, его поля я инициализирую с помощью геттеров и сеттеров, нужен ли мне в таком случае конструктор?

package Family;

public class Dad {


    private String name;
    private int age;
    private String keyPhrase;


    public String getName() {
        return name;
    }

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

    public int getAge() {
        return age;
    }

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

    public String getKeyPhrase() {
        return keyPhrase;
    }

    public void setKeyPhrase(String keyPhrase) {
        this.keyPhrase = "Ломать и крушить!";
    }



    private String say(){
        return "Меня зовут " + getName() + ", и мне " + getAge();
    }

    private String phrases(){
        return getKeyPhrase();
    }

    private void kick(){ //Суперспособность Удар

    }

    private void laserEyes(){ //Суперспособность Летать

    }

    private void fly(){ //Суперспособность Полет

    }
}
6
  • 1
    А что вы будете делать, если вам нужно будет присвоить полю age значение 20? Как ваш setAge() с этим справится? Какой бы аргумент вы не задали, этот сеттер все равно присвоит 45. Как и другие ваши сеттеры, которые присваивают конкретные значения. Когда ответите на этот вопрос, сразу станет понятно на счет конструктора. 6 дек 2019 в 11:07
  • А что, если мне не надо в дальнейшем больше присваивать значения полю age? Мне нужно, чтобы оно проинициализировалось раз и не изменилось. 6 дек 2019 в 11:28
  • 3
    А тогда зачем вам вообще сеттер для этого поля? Как раз конструктор и инициализирует поля класса. Вы создаете объект, где в параметрах конструктора перечисляете все необходимые значения для полей. И все. Не нужно вызывать никаких сеттеров. 6 дек 2019 в 11:39
  • Хорошо. Тогда я делаю поля файнал и инициализирую там же? А так же убираю сеттер? 6 дек 2019 в 12:21
  • Ну можно и так, если не хотите устанавливать значения полей во время создания объекта. Но сеттеры однозначно не нужны в этом случае. 6 дек 2019 в 16:21

2 ответа 2

2
public void setName(String name) {
        this.name = "Big Dad";
    }

Зачем вы в параметрах вообще тогда запрашиваете String. Если имя всё равно будет Big Dad. Если придерживаться данной концепции, будущий ваш коллега будет долго ломать голову, почему при любом setName у всех одно и то же имя!

Так делать ни в коем случае нельзя! Для подобных случаев вам необходимо указать пустой конструктор где

class Dad(){
this.name = "Big Dad";
this.age = 45;
}
3
  • 1
    человек видимо слабо понимает, что такое сеттер и лля чего он нужен. Было бы неплохо, если бы вы в своем ответе прояснили его назначение, а не только сделали замечание о неправильном использовании
    – pavlofff
    8 дек 2019 в 2:58
  • было бы неплохо! 8 дек 2019 в 12:21
  • Наверно ключевое слово тут должно быть, нужно проинициализировать, а не присвоить... 17 дек 2019 в 2:40
1

Вам нужен в этом случае конструктор только для того, чтобы вы знали, что если объект создан, то переменные, которые должны быть инициализированы, уже инициализированы. В случае с пустым конструктором, вы не можете быть уверены в том, что они инициализированы. Если не обязательно инициализировать некоторые поля или вам не нравится то, что список параметров слишком длинен, воспользуйтесь паттерном "строитель". Если вам не нравится то, что список параметров слишком длинен - паттерн "замена параметров объектом" также вам поможет.
Если же вам нужно инициализировать константы - язык java даст вам это сделать только в конструкторе(и в блоке инициализации). Что логично

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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