Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+24
/**
* Getter for billbackAdjustmentType
*
* @return the billbackAdjustmentType instance
*/
public BillbackAdjustmentType getBillbackAdjustmentType() {
return billbackAdjustmentType != null ? billbackAdjustmentType : BillbackAdjustmentType.PlusBillbackAdjustment;
}...и попробуй такое найди под дебагом :)
Запостил: 
guest,
04 Декабря 2008
Follow us!
А в чем прикол?
Нормальный код. И название метода правильное: getFieldName есть аксессор к полю this.fieldName. Может, просто в комментарии следует указать, что "никогда не возвращает null, а берёт статический BillbackAdjustmentType.PlusBillbackAdjus tment"?
А мне кажется, что лучше было бы сделать проверку в сеттере. А если сеттера нет, то выставить дефолтное значение в конструкторе. IMO. А то все-таки каждый раз при запросе поля выполняется ненужная проверка на null. Хоть маленькая, но оптимизация.