commentbox - ocena paczki, co poprawić, laravelowa paczka Opcje

[starf]

Hej,

Ogólnie od jakiegoś czasu pracuję jako programista, ale jako że nie mam nikogo kto by mi zrobił jakikolwiek codereview albo pomógł powiedział co poprawić to stwierdziłem że zapytam tutaj (IMG:style_emoticons/default/smile.gif)
Jak każdy szanujący się programista staram się rozwijać samodzielnie itp. Postanowiłem napisać własną paczkę do Laravela, i bardzo bym prosił nie tyle o ocene bo mam świadomość tego że nie koniecznie pięknie jest napisane, samemu mi się nie podoba to raczej wiadomości co mógłbym zmienić poprawić no i oczywiście jak.

Podsyłam link

https://github.com/19lyczkul91/CommentBox

ponadto zależy mi też żeby ktoś mi doradził (tutorial lub po prostu napisał) jak pisać testy jednostkowe. Pod jak nie mam na myśli coś w stylu "O Panie tu musisz wywołać tą metodę i zawsze ma Ci zwracać to i to" tylko raczej coś w stylu
tu powinieneś napisać testy, tu nie.
Konkretne moje pytanie czy to metod store/update itp powinno się robić testy ? I jeśli tak to jak zrobić np do metody store test żeby nie utworzyło nowego elementu ? nie wiem co jeszcze jak wymyślę to dodam.

[nospor]

Bbyloby naprawde milo, jakbys jeszcze napisal co ta paczka robi (IMG:style_emoticons/default/smile.gif)

W kontrolerze stworzyles metode renderError(), ktora raz uzywasz a raz nie. Widac, ze stworzyles ja w polowie kodu a potem zapomniales uzyc wszedzie (IMG:style_emoticons/default/wink.gif)

Generalnie nie lubie, lapac wyjatkow jak ty to robisz i za kazdym razem zwracac to samo. Zbedna duplikacja kodu.
W laravel na pewno jest jakis ExceptionHandler, w ktorym to jedbym mozesz to wszystko obsluzyc. Wowczas kody jak ten:

[PHP] pobierz, plaintext 
try {
            $articles = Article::all();
        } catch (ModelNotFoundException $e) {
            $restExceptionHandeler = new RestExceptionHandler();
            return $restExceptionHandeler->render($e);
        }
[PHP] pobierz, plaintext 

Wygladaja poprostu tak:

[PHP] pobierz, plaintext 
            $articles = Article::all();
[PHP] pobierz, plaintext 

Prawda ze ladniej i czytylniej ? (IMG:style_emoticons/default/wink.gif)