[Kohana]Kontrola jakości kodu Opcje

[MateuszS]

Witam. Skończyłem mały projekt w Kohanie (prosta strona z newsami, galerią itp.). Nie ma sensu pokazywać strony bo najważniejsza jest w tym właśnie Kohana i szlifowanie jej. Dopiero zacząłem się go uczyć i chciałbym przejść swego rodzaju kontrolę jakości. Wiadomo, prawdopodobnie "forumowy sanepid" zamknąłby moją fabrykę kodu ale głównie chodzi mi o uzyskanie rad, co robię źle i jak to poprawić żeby było dobrze (albo chociaż lepiej). Przymierzam się w niedalekiej przyszłości do stworzenia większego projektu i chciałbym żeby to było zrobione PRO.

Chciałbym abyście przyjrzeli się kontrolerowi i modelowi newsów (dla panelu administracyjnego). Proszę nie traktować tego jako "kodu do oceny". Ja wiem że to do oceny się jeszcze nie nadaje, z tym że ja już nie bardzo wiem co mógłbym poprawić. Stąd moja prośba o "kontrolę" kodu. Wybaczcie durnowate komentarze w kodzie, czasami się gubiłem.

Kontroler:
http://wklej.org/id/966403/

Model:
http://wklej.org/id/966404/

Pozdrawiam

Ten post edytował MateuszS 26.02.2013, 01:07:52

[toffiak]

Parę uwag z którymi nie wszyscy się zgodzą:
Kod jest rozwlekły, to znaczy twój podany przez Ciebie kod nie potrzebuje tylu komentarzy, tak wiem że często wymagane jest na uczelaniach aby komentować wszystko co się da ale komentarze odwracają uwagę od kodu właściwego. Tak naprawdę gdyby usunąć je wszystkie to i tak kod jest czytelny. Jedynie warto zachować komentarze nad metodami.
W przypadku "if-ów" i "else-ów" zawierających po jednej instrukcji powinieneś skorzystać z operatora trójargumentowego.
Kod kontrolera nie powinien przekraczać 30 linijek, choć oczywiście nie jest to reguła a raczej zalecenie.
Uwagi dotyczą kodu przedstawionego, w przypadku rozwiniętego modelu kod musi mieć więcej komentarzy, ponieważ są to sprawy indywidualne a nie zwykły CRUD.