[laravel][symfony] Laravel, działanie modelu Opcje

[miccom]

Cześć.
Nie rozumiem jednej sprawy i proszę społeczność o nakreślenie co robić (IMG:style_emoticons/default/smile.gif)

Mam problem w zrozumieniu laravela, modeli itp.

Ostatnio analizowałem kurs laravela ( z uprzejmości nie podam adresu kursu (IMG:style_emoticons/default/wink.gif) ) na którym pan jawnie w kontrolerze wykonywał operacje na modelu, coś w deseń poniżej:

[PHP] pobierz, plaintext 
namespace App\Http\Controllers;
 
use App\Models\User;
$data = file_get_contents('https://jsonplaceholder.typicode.com/users');
        $fullData = json_decode($data, TRUE);
        foreach( $fullData AS $singleData ){
            $user =  new User();
            $user->name = $singleData['name'];
            $user->username = $singleData['username'];
            $user->phone = $singleData['phone'];
            $user->website = $singleData['website'];
            $user->save();
}
[PHP] pobierz, plaintext 

Jako że poszukuję pracy jako junior laravel developer to przygotowałem zadanie testowe zgodnie z tym kursem... ale moi rekruterzy napisali mi, że pomieszałem wszystko w kontrolerze itp.

Pytanie- jak poprawnie używać kontrolera i modelu ? bo nieco zagmatwałem się w tych elementach.

Jak rozumiem, kontroler steruje modelem, model wykonuje operacje ( najczęściej na bazie danych ) i zwraca dane do kontrolera, a ten kontroler dalej przekazuje dane do widoku.
Jak powinienem zmodyfikować $user = new User(); aby poprawnie używać frameworka MVC?

Proszę o pomoc w zrozumieniu tematu (IMG:style_emoticons/default/smile.gif)

[miccom]

Ogólnie, ten kod który wkleiłeś to jakaś tragedia... propnowałbym ten wątek przenieśc jednak do Przedszkola, wtedy można by było wytknąć wszystkie błędy które należy koniecznie poprawić.

Można przenieść do Przedszkola ale tam że napiszesz co jest źle ? bo pisać tragedia- i nie wyjasnić dlaczego tragedia- to zwykłe trollowanie (IMG:style_emoticons/default/smile.gif)

[netir]

Można przenieść do Przedszkola ale tam że napiszesz co jest źle ? bo pisać tragedia- i nie wyjasnić dlaczego tragedia- to zwykłe trollowanie (IMG:style_emoticons/default/smile.gif)

Pomijając dodatkowe warstwy, biorąc tylko pod uwagę logikę to koniecznie poczytaj i zrozum:
- https://laravel.com/docs/8.x/collections (opcjonalnie, zamiast foreach)
- https://laravel.com/docs/8.x/eloquent#mass-assignment (nie masz najmniejszej potrzeby tworzyć modelu tak jak to robisz)
- https://laravel.com/docs/8.x/eloquent-relat...e-create-method (j/w korzystaj z relacji przy tworzeniu)
- poczytaj o standardach formatowania kodu https://www.php-fig.org/psr/
- https://laravel.com/docs/8.x/migrations#introduction (linijki 3-8 świadczą ewidentnie o złych migracjach/fk)
- https://pl.wikipedia.org/wiki/SOLID_(progra...anie_obiektowe)

To co napisałeś powinno być max w ~10 linijkach na oko + rozbite na mniejsze części i umieszczone w odpowiednich miejscach.

Ten post edytował netir 13.09.2021, 17:17:52