Можно ли использовать конструкцию if…else в setter'e?

269

07 июля 2017, 04:46

Всем привет! Простите за нубский вопрос, но можно ли использовать условие в сеттере? В решаемой задаче получаемое значение должно соответствовать некоему условию, в противном случае должно подставляться дефолтное значение. Я хочу сделать так, но не уверен, что это правильно:

public void setHouseName(String houseName) {
    if (houseName.length()<4 || houseName.length()>20){
        this.houseName = "House#" + collectorNumber;
    }
    else{
        this.houseName = houseName;
    }
}

Answer 1

Да, конечно можно. Только это будет сеттер, который устанавливает значение. Геттер наоборот, возвращает его

public void setHouseName(String houseName) {
    if (houseName.length() < 4 || houseName.length() > 20) {
        this.houseName = "House#" + collectorNumber;
    }
    else {
        this.houseName = houseName;
    }
}

Answer 2

Ты точно отличаешь getter от setter'а?
В обоих можно, т. к. это просто обычные функции. Теоретически, для того и нужны.
Код в вопросе неверный, поскольку присваивание this.houseName = houseName; происходит безусловно. Нужно оставить присваивания только в ветвях if'а, а последнее убрать.