[JezelyPanPozwoly]

Ogarniam sobie laravela i robię tutorial. Mam klasę kontrolera z metodą update jak picrel, tu wiem o co chodzi, natomiast ciężko mi zrozumieć co się dzieje w metodzie complete klasy modelu. Co robi $completed = true? w zależności od tego czy przekażę true lub false?
#naukaprogramowania #laravel #php

[DanioPL]

public function complete($completed = true) ...

@JezelyPanPozwoly:

powoduje, że argument $completed nie musi być podany przy wywołaniu metody, a jeśli się tak stanie to jego wartość to true. Metoda compact bierze wszystkie podane mu nazwy zmiennych i tworzy z nich array, czyli w tym przypadku to będzie coś w

[JezelyPanPozwoly]

$completed nie musi być podany przy wywołaniu metody, a jeśli się tak stanie to jego wartość to true

@DanioPL: dzięki! Compact ogarniam co robi.

Swoją drogą ktoś Cię uczy pisać gównokod i robisz sobie w ten sposób krzywdę.

Co masz na myśli?

[DanioPL]

@JezelyPanPozwoly: Ostatnio przeglądałem zadania rekrutacyjne ludzi, którzy zaczynali właśnie w Laravelu (nawet nie musieli tego pisać, bo to po prostu widać). Uczy on bardzo wielu złych schematów i chodzenia na skróty, które są spoko jak piszesz apkę, która będzie miała 500 linii kodu i w razie jakiekolwiek rozwoju wywalisz to i napiszesz od początku.

Wchodząc bardziej w szczegóły ta metoda complete chociażby. Kod musi się dobrze czytać to po pierwsze.

[nowiutki]

Co masz na myśli?

@JezelyPanPozwoly:

1. Brak zadeklarowanego typu, więc jak ktoś sobie poda stringa jako te $completed to wywali update.
2. Brak jakiegokolwiek sensu tworzyć metodę, której logika to jest zwykłe odwołanie do update z repo Eloquenta.
3. Takie rzeczy powinny być nie w modelu, a w serwisie.

[zolwixx]

@JezelyPanPozwoly: kiedyś robiłem dokładnie tak jak Ty, teraz robię dokładnie tak jak opisał @nowiutki

3. Takie rzeczy powinny być nie w modelu, a w serwisie.