[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]

Chudy kontroller i gruby model - taka filozofie wyznaje (IMG:style_emoticons/default/smile.gif)
W kontrolerze tworząc instancje klasy przekazuje do modelu dane z post i get

[PHP] pobierz, plaintext 
public function before()
    {
        parent::before();
        $this->model = new Model_Area(
                    array(
                        'post' => $this->request->post(),
                        'get'  => $this->request->query()
                    )
                );
    }
[PHP] pobierz, plaintext 

i w modelu w metodzie save

[PHP] pobierz, plaintext 
public function save()
    {
        $valid = Validation::factory($this->post)
                ->rule('name', 'not_empty')
                ->rule('color', 'not_empty');
 
        if($valid->check() == false)
        {
            $this->errors = $valid->errors('error');
            return false;
        }
        // dodawanie lub edycja w zależności od parametru
    }
[PHP] pobierz, plaintext 

a w kontrolerze mam tak

[PHP] pobierz, plaintext 
private function form($values = array())
    {
        if($this->request->post() !== array())
        {
            if($this->model->save() == false)
            {
                $error = $this->model->error();
                $values = $this->model->post();
            }
            else
            {
                //przekierowanie
            }
        }
 
        return View::factory('area/form')
                      ->set('values', $values);
    }
[PHP] pobierz, plaintext