Symfony4 Program dla warsztatu samochodowego

pytanie zadane 18 czerwca 2018 w Nasze projekty przez `Krzychuu Stary wyjadacz (13,940 p.)

Witam

Uczę się symfony4, wpadłem na pomysł stworzenia programu dla warsztatu samochodowego, wcześniej nie chciałem nikomu pokazywać tego projektu ale, ten projekt spodobał się dla mojego kolegi który pracuje jako mechanik, i powiedział że chciałby mieć coś takiego, chciałbym żebyście ocenił kod backendu oraz frontu, projekt nie jest jeszcze skończony, ale stwierdziłem że wstawię do oceny kodu, ponieważ nie wiem czy dobrze robię.

Funkcje:

Dodawanie użytkowników (pracowników)
Usuwanie użytkowników
Wyświetlanie profilu użytkownika
Dodawanie klienta
Szukanie klienta
Dodawanie samochodu
Wyświetlenie samochodu
Szukanie samochodu po rejestracji
Dodawanie stanowiska (Mechanik, Lakiernik)
Usuwanie stanowiska
Wyświetlenie wszystkich stanowisk
Logowanie

Kod: Github

1 odpowiedź

odpowiedź 18 czerwca 2018 przez Ehlert Ekspert (212,670 p.)
wybrane 18 czerwca 2018 przez `Krzychuu

Najlepsza

Zero walidacji po stronie be
Fajnie byłoby zrobić api. Front oddzielnie
Serio bez Fosuserbundla?
Cała logika w kontrolerach
Brak jakichkolwiek testów. Nawiasem mówiąc z Twoimi kontrolerami w grę wchodzą chyba tylko funkcjonalne.
Brak admin panelu.
Nie będę podawać konkretów ale schema db mogłaby być bardziej spójna.
W repo nie powinno być kodu zakomentowanego. Komentarze nie będące dokumentacją, todo, fixme albo stricte notką wprowadzają zamęt.

Podsumowując ode mnie up za migracje, ambicje i dążenie w dobrym kierunku. Tak trzymaj

komentarz 18 czerwca 2018 przez `Krzychuu Stary wyjadacz (13,940 p.)
edycja 18 czerwca 2018 przez `Krzychuu

dziękuje, możesz podać link do Fosuserbudle bo nie mogę znaleźć pod symfony4, gdzie mogę trzymać logikę muszę stworzyć jakiś nowy folder?, możesz bardziej wyjaśnić punkt 2?

komentarz 18 czerwca 2018 przez Ehlert Ekspert (212,670 p.)

Link do userbundla. Z tego co widzę to wersja 2.1 śmiga z sf4. https://packagist.org/packages/friendsofsymfony/user-bundle

Możesz stworzyć taki folder jaki Ci się podoba. Trzymaj logikę aplikacji w osobnych klasach. Pisząc ją nie powinno Cię już obchodzić czy dane są poprawne i walidowane. Skupiasz się na samej esencji tego jaką funkcjonalność udostępnia aplikacja. Takie podejście ułatwia potem rozwijanie kodu, upgrade frameworka, a nawet jego bezbolesną zmianę. Do logiki korzystaj z symfonowego DI i odpowiedniego podziału na klasy spełniające określone zadania. Jeśli ogarniesz może się przydać tactician, albo wgl cały wzorzec CQRS.

Jeszcze niedawno też byłem fanem backendu renderującego widok. Jednak to rozwiązanie staje się przeżytkiem. Teraz pisze się api które operują na czystych danych (w formacie json, xml itp). Dzięki temu piszesz serwer raz i masz możliwość podpięcia go pod SPA, aplikację mobilną lub co chyba najlepsze dać innym projektom, stronom lub developerom możliwość integracji. Standardem w na tym polu jest REST api, ale szeroko używany jest też GraphQL (patrz fb).

Ps. dołącz do projektu code_sniffera

komentarz 18 czerwca 2018 przez `Krzychuu Stary wyjadacz (13,940 p.)

czyli w logice aplikacji powinienem przenieść np. tworzenie formularza, operacje na bazie danych, a w kontrolerze walidacje?, masz może jakieś projekty które stosują api?

komentarz 18 czerwca 2018 przez Ehlert Ekspert (212,670 p.)

Ja w kontrolerze trzymam tworzenie formularza. Jeśli nie jest walid to rzucam 400. W przeciwnym wypadku odpalam klasy odpowiedzialne za samą logikę czyli w moim przypadku są to LogicServices.

Mam swój projekt api, ale nie jest to kod otwarty, więc niestety nie mogę się podzielić.