Шаблоны GRASP: Information Expert (информационный эксперт)

[GraDea]

нового курса «Архитектура и шаблоны проектирования»

Ссылка обещает новый курс, а приводит на регистрацию на непонятный вебинар.
Так задумано?

[MaxRokatansky]

Да. Вебинар абсолютно бесплатный и имеет непосредственное отношение к курсу. Посмотрев его можно в принципе оценить процесс изложения плюс, возможно, получить какие-то новые знания. Если же не хочется смотреть вебинар, а есть цель сразу зарегистрироваться на курс, можно закрыть крестиком этот попап и под ним будет кнопочка прохождения тестирования и регистрации

[GraDea]

На момент написание коммента открывался не вебинар, а страница регистрации на него. И это не выглядело как попап, поэтому и спросил.

[bRUtality]

Мне кажется, стоит добавить тот код, что будет учитывать «скидка пенсионерам, скидка по праздничным дням, скидка из-за покупки оптом» (хоть что-нибудь одно), иначе преимущество кода после рефакторинга не очевидно.

[DenisovP]

@Getter
@AllArgsConstructor
public class OrderItem {
    private Good good;
    private int amount;

    public int getPrice() {
        return good.getPrice();
    }
}

rodinvv, здесь случайно не забыли учесть количество товара в элементе заказа?

[rodinvv]

Забыл, спасибо!

[ldss]

public int getPrice() {
return good.getPrice();
}

Наверное, good.getPrice()*amount, не?

Но вообще, боже мой, GRASP паттерны! Я их еще в 2001м году учил:) Хуже они не стали, конечно.

[rodinvv]

Да, конечно, спасибо!

[MRomaV]

@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?