Poprawność MVC Opcje

[Ubermade]

Witam! Napisałem ostatnio wzorzec MVC w php i mam pewną obawę, że ciut źle zacząłem go używać. Może pokażę na przykładzie.

Mam tutaj funkcję w kontrolerze, która odpowiada za dodawanie kategorii:

[PHP] pobierz, plaintext 
public function add(){
    Access::logged();
    $view = new \Ubermade\view\CategoryView();
    if(empty($_POST)){
        $view->renderHTML('add', 'front/category/');
    }else{
        $model = new \Ubermade\model\CategoryModel();
        $utilities = new \Ubermade\utilities\CategoryUtilities();
        $items = $utilities->add($_POST['name'], $model->getByName($_POST['name']));
        if($items){
            $model->add($_POST);
            RouterUtilities::redirect(RouterUtilities::generateUrl('category/list'));
        }else{
            $view->renderHTML('add', 'front/category/');
        }
    }
}
[PHP] pobierz, plaintext 

W swoim projekcie zastosowałem pewnego rodzaju "pomocników", którzy zajmują się przetwarzaniem, oraz uwierzytelnianiem podanych informacji. W tym przypadku odnoszę się do klasy "CategoryUtilities" do funkcji"add", która wygląda następująco:

[PHP] pobierz, plaintext 
public function add($name, $database){
    $check = true;
 
    if($database){
        $check = false;
        Session::set('e_category_repeat', true);
    }
    if(strlen($name) < 1){
        $check = false;
        Session::set('e_category_length', true);
    }
 
    iF($check){
        return true;
    }else{
        return false;
    }
}
[PHP] pobierz, plaintext 

I tutaj się rodzi pytanie... Czy używanie takich "pomocników" jest zgodny ze wzorcem MVC, czy jednak taki kod powinno się pakować gdzie indziej, np modelu.
Jeżeli chodzi o nazwę "items" w kontrolerze, mimo, że może mieć ona jedynie wartość 1, to czasami, np: w dodawaniu użytkownika, mój "pomocnik" hashuje hasło, zwraca je do kontrolera, który później przesyła je modelu i do bazy danych.

Z góry dziękuję za pomoc (IMG:style_emoticons/default/smile.gif)

[athabus]

Jesli to ma byc podstawa takiego miniframeworka mvc to wielu rzeczy mi tu brakuje typu obiekty request/response itd. Tacy "pomocnicy" sa ok, ale strasznie wszystko mieszasz w klasach - wyglada to troche jak programowanie strukturalne przeniesione do obiektow.
Robilem ostatnio podobne zadanie "szkoleniowe" - jak chcesz to moge Ci na pw pokazac jak.taki framework mvc strony moglby wygladac.

[Ubermade]

Jesli to ma byc podstawa takiego miniframeworka mvc to wielu rzeczy mi tu brakuje typu obiekty request/response itd. Tacy "pomocnicy" sa ok, ale strasznie wszystko mieszasz w klasach - wyglada to troche jak programowanie strukturalne przeniesione do obiektow.
Robilem ostatnio podobne zadanie "szkoleniowe" - jak chcesz to moge Ci na pw pokazac jak.taki framework mvc strony moglby wygladac.

Jasne, fajnie by było (IMG:style_emoticons/default/wink.gif)

[athabus]

Ok to postaram sie jutro po pracy podeslac. Napisz mi tylkoczy ogarniasz takie rzeczy jak git, composer, doctrine, kontener di - bo mam wersje basic i taka bardziej oop po refactoringu z moim przelozonym. Wersja poczatkowa jest taka "poprawna ale prosta jak kij od szczotki", koncowa bardziej oop. Nawet ta poczatkowa mysle ze by Ci pozwolila troche poprawic kod, ale koncowa to juz calkiem fajny kod oop przynajmniej jak na moj obecny poziom, tyle ze juz bardziej zlozona.

[nospor]

Z pierd, ale z takich pierd co ladnie ci poprawia kod to

nie:

[PHP] pobierz, plaintext 
    iF($check){
        return true;
    }else{
        return false;
    }
[PHP] pobierz, plaintext 

a poprostu:

[PHP] pobierz, plaintext 
return $check;
[PHP] pobierz, plaintext 

[athabus]

Haha mój popisowy numer. Na pierwszym code review potrzebowałem minuty, żeby odpowiedzieć na pytanie co tu jest nie tak (IMG:style_emoticons/default/smile.gif)

[Ubermade]

jest to jedynie napisany na szybko przykład (IMG:style_emoticons/default/wink.gif) Bardzo często zwracam wartości takie jak zahashowane hasło itp (IMG:style_emoticons/default/wink.gif) Ale sam fakt, taka pierdoła poprawiłaby kod (IMG:style_emoticons/default/wink.gif)

Ok to postaram sie jutro po pracy podeslac. Napisz mi tylkoczy ogarniasz takie rzeczy jak git, composer, doctrine, kontener di - bo mam wersje basic i taka bardziej oop po refactoringu z moim przelozonym. Wersja poczatkowa jest taka "poprawna ale prosta jak kij od szczotki", koncowa bardziej oop. Nawet ta poczatkowa mysle ze by Ci pozwolila troche poprawic kod, ale koncowa to juz calkiem fajny kod oop przynajmniej jak na moj obecny poziom, tyle ze juz bardziej zlozona.

Cóż, ogarniam takie rzeczy, jak git i composer, ale co do reszty to tylko słyszałem o nich (IMG:style_emoticons/default/biggrin.gif)