- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
/**
* Getter for billbackAdjustmentType
*
* @return the billbackAdjustmentType instance
*/
public BillbackAdjustmentType getBillbackAdjustmentType() {
return billbackAdjustmentType != null ? billbackAdjustmentType : BillbackAdjustmentType.PlusBillbackAdjustment;
}
А в чем прикол?
Нормальный код. И название метода правильное: getFieldName есть аксессор к полю this.fieldName. Может, просто в комментарии следует указать, что "никогда не возвращает null, а берёт статический BillbackAdjustmentType.PlusBillbackAdjus tment"?
А мне кажется, что лучше было бы сделать проверку в сеттере. А если сеттера нет, то выставить дефолтное значение в конструкторе. IMO. А то все-таки каждый раз при запросе поля выполняется ненужная проверка на null. Хоть маленькая, но оптимизация.