Aktualnie piszę moją pierwszą aplikację w laravelu 5 i chcę abyście ocenili kod napisali co tam trzeba zmienić, aplikacja jest jeszcze nie napisana więc się nie zdziwcie jak coś będzie nie działało. Tutaj jest link. I jeszcze jedno trochę struktura katalogów jest zła bo ręcznie wrzucałem pliki.
0
0
Jak zrobisz z tego normalne repo laravela a nie jakieś pliki powrzucane nie wiadomo do czego się odnoszące to mogę ocenić
0
Otworzyłem losowy plik i straszna **ujnia.
if ($page == "views_product" or $page == "views_user" or $page == "add_product" or $page == "views_product" or $page == "change_setting" or $page == "buy_product" or $page == "auction" or $page == "issuing_product" or $page == "change_product" or $page == "delete_product" or $page == "views_statistics" or $page == "banning_users") return true;
Jeśli if ma więcej niż dwa "and" lub "or" to robisz to źle i prosisz się o płacz przy debuggowaniu/naprawianiu. W tym przypadku możesz stworzyć po prostu tablicę/kolekcję dozwolonych podstron i sprawdzać czy $page się w niej znajduje. Bardziej ogólniej: wieloczęściowe warunki wynosi się do oddzielnej funkcji.
0
Lista 'słabych' stron:
- Słabo wyglądają te polskie wcinki wśród jednak większości angielskiego kodu
- Trochę śmietnik z mnóstwem zakomentowanego kodu
- Nazwy klas pisane z małej litery
- Do funkcji polecam camelCase
- App\Http\Controllers\funkcje::check_flap() - to samo co w komentarzu @datdata (płacz)
- Mnóstwo printów z htmlem w controllerach (BTW. niedawno doradzałeś na forum w temacie od mvc)