[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

[sabat24]

Z czysto konwencyjnych rzeczy, to z tego co pamiętam, Kohana nie lubi takich konstrukcji:

[PHP] pobierz, plaintext 
if (warunek)
instrukcja
else
instrukcja
[PHP] pobierz, plaintext 

poza użyciem continue i return zaleca się zawsze stosowanie klamer { } -> http://kohanaframework.org/3.3/guide/kohana/conventions

Jeśli używasz kohany 3.3, to nigdy nie używaj gołego $_POST, tak jak tu:

[PHP] pobierz, plaintext 
$_post = Validation::factory($_POST)
[PHP] pobierz, plaintext 

W przypadku takiej konstrukcji:

[PHP] pobierz, plaintext 
$view  = View::factory("admin/news/message")->set("message", null);
[PHP] pobierz, plaintext 

Nie używaj set, tylko bind. Jeśli do zmiennej zostanie coś wpisanego później w kodzie 'na dole', to zostanie to 'podłączone' dzięki bind 'na górze'.

Ten post edytował sabat24 26.02.2013, 14:10:03