[yersey]

Zrobiłem aplikacje SPA a la wykop, użyłem Laravela i Vue. Dajcie jakiś feedback co źle, co poprawić itp. Oczywiście nie miałem w zamiarze robić drugiego lurkera, to tylko taki projekt edukacyjny.
Czy z takimi umiejętnościami miałbym szansę dostać gdzieś prace jako junior?

Link do repo


#programowanie #it #php #laravel #js #vue #programista15k

[A.....2]

Mirko Ci chyba popsułem ( ͡° ʖ̯ ͡°)

[yersey]

@Anonymous123232: Wlaśnie widzę :)

[yersey]

@Anonymous123232: naprawione

[Piekny_Krolewicz]

@yersey: gościu daj linka normalnie bo ze spojlera nie można skopiować na telefonie (ꖘ⏏ꖘ)

[yersey]

@Piekny_Krolewicz: http://vps-ba61f06e.vps.ovh.net

[Piekny_Krolewicz]

@yersey: można poprawić to

[Hipodups]

@yersey: zerkam Ci w kod i o Panie.

katalog app, wszyatkie klasy po nagielaku nazwane, jedna nazywa sie wpis - brak spojnosci.

metoda created-at w tej klasie - cisnie sie slowo - co to ***** ma być?
ogarnij to za pomocą obiektu datetime w max 5

[aso824]

@yersey: kurde, sporo rzeczy jest nie tak jak być powinno, ale jak popracujesz nad tym to będzie ok. Lista uwag w losowej kolejności, niektóre pierdoły niektóre solidne:

- metody relacji z dużej litery - nope
- używaj ::class zamiast podawania namespace w ten sposób
- metoda created_at w modelu Wpis - wtf? Użyj Carbon
- model się nazywa Wpis albo Wykop a obok masz User albo Comment - wiem że to nazwa domenowa ale

[aso824]

I dalej mi się nie chce ( ͡° ͜ʖ ͡°) wróć jak poprawisz i poproś ponownie o CR, bo mógłbym pisać i pisać na ten moment

[Hipodups]

@Hipodups: patrzac na kontrolery - robisz to źle.

W kontroleach ma być max odebranie danych, przekazanie ich dalej, zwrocenie wyniku.

logikę biznesową powinieneś zamknąć w fasade lub jakiś serwis.

[aso824]

@aso824: ale w sumie jeszcze dodam parę:

- niepotrzebne findOrFail w kontrolerze, używaj route-model binding
- nie możesz robić jednej metody w stylu show a tuż obok wpisy, cmon
- README spoko jeżeli to projekt na studia, ale w realnej aplikacji schemat bazy generujesz sobie kliknięciem, a opis nie powinien być potrzebny

Generalnie jeżeli nie przerabiałeś to zdecydowanie weź darmowy kurs larwy na laracasts.com przerób cały a potem sobie

[Hipodups]

@yersey: + protip jezeli nie ma kto zrobic Ci cr, uzywaj statycznej analizy kodu, np sonar-cube.

[aso824]

+ protip jezeli nie ma kto zrobic Ci cr, uzywaj statycznej analizy kodu, np sonar-cube.

@Hipodups: sonar to armata dla początkującego ( ͡° ͜ʖ ͡°) phpstan/larastan (tutaj to drugie) to znacznie przyjemniejszy wstęp

[Hipodups]

@aso824: moze i racja, ja w larvie nie pisalem juz ponad 3lata wiec smialo moge twierdzic ze go nie znam, natomiast sonar to bardzo pożyteczna armata. Nic tak nie motywuje zespołu do utrzymywania czystego i utrzymywalnego kodu jak dobrze ustawione reguły, rosnący dlug , smelsy czy poziom duplikacji kodu i malejacy coverage.

cr przepusci wiele rzeczy, sonar wiele wyłapie, ale kombo cr + sonar to wg. mnie musthave w kazdym

[olekturbo]

@yersey kawał gówna, ale pewnie nie różni się za dużo od wykopu

[yersey]

@olekturbo: (╥﹏╥)

[yersey]

@Hipodups: @aso824: Trochę poprawiłem, ale nie rozumiem kilku rzeczy.

w przypadku laravela używaj IDE helpera który wygeneruje ci ładne komentarze

Chodzi o te komentarze przy metodach w kontrolerach? Jak to zrobić? Po co

[yersey]

@Hipodups: Czy teraz robię to dobrze?

https://github.com/yersey/laravel-vuejs-spa/blob/service/app/Http/Controllers/CommentController.php
https://github.com/yersey/laravel-vuejs-spa/blob/service/app/Services/CommentService.php