[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

[r4xz]

nie piszę w kohanie, ale z mojej strony powiem że preferuję zapis (zmienne, metody itp. cuda - zmyślone):

[PHP] pobierz, plaintext 
<?php
 
class ...
 
public function foo() {
 
	// 1 warunek
	$id = ...
	if( !$id )
 
		return false;
 
	// 2 warunek
	if( !$this -> request -> is('post'))
 
		return false;
 
	// i dopiero kod
	return View::....
 
}
[PHP] pobierz, plaintext 

zamiast:

[PHP] pobierz, plaintext 
<?php
 
class ...
 
public function foo() {
 
	// 1 warunek
	$id = ...
	if( $id ) {
 
		// 2 warunek
		if( $this -> request -> is('post')) {
 
			// i dopiero kod
			return View::....
 
		}
		else
 
			return false;
 
	}
	else
 
		return false;
 
}
[PHP] pobierz, plaintext 

musisz mieć tylko wtedy pewność np. jeśli zamiast return false masz przekierowanie/widok etc. że ta czynność blokuje wykonywanie dalszego kodu

PS pewnie ma tylu zwolenników, co i przeciwników? (IMG:style_emoticons/default/smile.gif)