🚀 go-pugleaf

Witam

a1 wrote:

> Order o1 = new Order();
>         customer.addOrder(o1);
>          OrderLine line1 = new OrderLine(6, Product.find("TAL"));
>          o1.addLine(line1);
>          OrderLine line2 = new OrderLine(5, Product.find("HPK"));
>          o1.addLine(line2);
>          OrderLine line3 = new OrderLine(3, Product.find("LGV"));
>          o1.addLine(line3);
>          line2.setSkippable(true);
>          o1.setRush(true);
> 
> vs.
> 
>          customer.newOrder()
>                  .with(6, "TAL")
>                  .with(5, "HPK").skippable()
>                  .with(3, "LGV")
>                  .priorityRush();

To jest tendencyjna konstrukcja. Bo w wersji drugiej jakoś nie widać metody
Product.find, nie ma też powiedziane że są tworżone nowe obiekty co widać w
pierwszej wersji. A druga może sugerować tylko podmianę wartości.
Przedstawiłeś kawałek kodu, bez pokazania jego faktycznej implementacji, bo
Stosując sugestie Adama Karpierza powinieneś przeparafrazować swój kod na
taki:

        o1 = customer.newOrder()
        o1.with(6, "TAL");
        o1.with(5, "HPK");
        o1.get("HPK").skippable(); //ewentualnie to jeszcze rozbić na 2
        o1.with(3, "LGV");
        o1.get("LGV").priorityRush(); //ewentualnie to jeszcze rozbić na 2



Pozdrawiam.

-- 

|\/\/|   Seweryn Habdank-Wojewódzki
`\/\/