[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

[skowron-line]

Troche za dużo kodu w kontrolerze

[PHP] pobierz, plaintext 
public function action_index() 
{		
	//Pobieranie wszystkich newsów
	$_news = Model::factory('News')->news_get_all();
 
	if (count($_news) > 0)
	{	
		//Tabela newsów
        $content = View::factory("admin/news/news_table")->set('array_news', $_news);
	}
	else
	{
		//Błąd w przypadku braku newsów
        $content = $this->message("no_news_found");		
	}
	//Ustawienie widoku
	$this->template->content = 	View::factory("admin/news/news")
	->set('content', $content);
}
 
[PHP] pobierz, plaintext 

Validacja i dodawanie rekorów to w modelu się odbywa.
Moje metody odpowiedzialne za dodawanie i edycje rekordu wyglądają tak

[PHP] pobierz, plaintext 
public function action_add()
    {
        $this->template->content = $this->form();
    }
 
    public function action_edit()
    {
        $this->template->content = $this->form($this->model->get()->current());
    }
 
    private function form($values = array())
    {
        // tu sprawdzam czy został wysłany post
        // i co zwraca metoda save w modelu
        return View::factory('form')->set('values', $values);
     }
[PHP] pobierz, plaintext 

Dodatkowo przenieś sobie metode message do kontrollera po którym dziedziczysz żebyś w każdym nie musiał sobie czegoś takiego dopisywać.
A model jak model nic szczególnego, tylko dlaczego wszystko na bool rzutujesz nie jest Ci potrzebne ID rekordu ktory dodałeś ? Liczba usuniętych / zmienionych rekordów

[PHP] pobierz, plaintext 
/**
	 * Odnalezienie newsa o podanym ID
	 */
return (DB::select()
		->from("parys_news")
		->where("ID", "=", $id)
		->execute()
		->count() > 0);
[PHP] pobierz, plaintext