Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
@Nullable
public static BigDecimal getWidth(final ITagSpecification spec, final TagSize size) {
switch (size) {
case S:
return spec.getSmallWidth();
case M:
return spec.getMediumWidth();
case L:
return spec.getLargeWidth();
default:
throw new AssertionError();
}
}
@Nullable
public static BigDecimal getHeight(final ITagSpecification spec, final TagSize size) {
switch (size) {
case S:
return spec.getSmallHeight();
case M:
return spec.getMediumHeight();
case L:
return spec.getLargeHeight();
default:
throw new AssertionError();
}
}
public static BigDecimal getDiameter(final ITagSpecification spec, final TagSize size) {
switch (size) {
case S:
return spec.getSmallDiameter();
case M:
return spec.getMediumDiameter();
case L:
return spec.getLargeDiameter();
default:
throw new AssertionError();
}
}
@Nullable
public static BigDecimal getWeight(final ITagSpecification spec, final TagSize size) {
switch (size) {
case S:
return spec.getSmallWeight();
case M:
return spec.getMediumWeight();
case L:
return spec.getLargeWeight();
default:
throw new AssertionError();
}
}
Бойлерплейт - он такой. А можно ли с JPA это как-то изящнее сделать? Дело в том, что {small|medium|large}{Width|Height|Diamet er|Weight} - это двенадцать столбцов таблицы в БД. Пока единственное, что приходит на ум - это сделать три одинаковых @Embedded'а на каждый размер.
TagSize енумом пахнет.
>ITagSpecification
Если тут по три метода захардкожено, то сушите весла, только рефлексия.
Должно либо три инстанса быть ITagSpecification, либо в каждом методе параметр TagSize.
>ITagSpecification
Если тут по три метода захардкожено, то сушите весла, только рефлексия.
Должно либо три инстанса быть ITagSpecification, либо в каждом методе параметр TagSize.
Знаете, кто такой этот конардо?
> двенадцать столбцов таблицы в БД
Добавить колонку size (со значениями 'L', 'M' или 'S') и разбить запись на 3 отдельных?