Комментарии 9
нового курса «Архитектура и шаблоны проектирования»
Ссылка обещает новый курс, а приводит на регистрацию на непонятный вебинар.
Так задумано?
0
Да. Вебинар абсолютно бесплатный и имеет непосредственное отношение к курсу. Посмотрев его можно в принципе оценить процесс изложения плюс, возможно, получить какие-то новые знания. Если же не хочется смотреть вебинар, а есть цель сразу зарегистрироваться на курс, можно закрыть крестиком этот попап и под ним будет кнопочка прохождения тестирования и регистрации
0
Мне кажется, стоит добавить тот код, что будет учитывать «скидка пенсионерам, скидка по праздничным дням, скидка из-за покупки оптом» (хоть что-нибудь одно), иначе преимущество кода после рефакторинга не очевидно.
+1
public int getPrice() {
return good.getPrice();
}
Наверное, good.getPrice()*amount, не?
Но вообще, боже мой, GRASP паттерны! Я их еще в 2001м году учил:) Хуже они не стали, конечно.
0
@Getter
@AllArgsConstructor
public class OrderItem {
private Good good;
private int amount;
public int getPrice() {
return amount * good.getPrice();
}
}
@Getter
@AllArgsConstructor
public class Good {
private String name;
private int price;
}
good.getPrice(); — забыли добавить getPrice() в class Good?
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Шаблоны GRASP: Information Expert (информационный эксперт)