Witam

Zastanawiam się jak rozmieszczać poprawnie metody w kontrolerze. Mam swój sposób, który stosuje od dawna lecz nie jestem do niego przekonany.
Załóżmy, że chcę dodać nowego użytkownika, wykorzystując kontoler User.
Dalsze wyjaśnienia w komentarzu

[PHP] pobierz, plaintext 
 
class User extends Controller{
 
    add(){
        //tutaj jest metoda add, ładuje ona odpowiedni widok z formularzem
       //zajmuje się ona tylko załadowaniem widoku
 
      //w formularzu action='addUser', czyli formularz wysyłany jest do poniższej metody
    }
 
    addUser(){
         //tutaj zostaja wyslane dane z formluarza
         //nastepuje tutaj walidacja i jezeli jest okej jest tworzony obiekt modelu User ii wywolana metoda add()l
         $dane='xx'; //jakies dane z formularza
         $Model_User->add($dane);
         //powyżej zostaje dodany user do bazy
    }
}
[PHP] pobierz, plaintext 

czy taka konwencja jest poprawna?
że jest osobna metoda na załadowanie widoku?
Bo teraz chcąc dodać usera muszę stworzyć dwie metody pierwsza-ładuje widok, druga-wysyła dane do modelu

Jak chcesz zgodnie z MVC to nie kontroler powinien zajmować się walidacją danych z formularza.

Kontroler -> wywołuje widok formularza -> przesłanie POST -> model interpretuje dane i zapisuje do bazy -> kontroler wywołuje odpowiedni widok

ja bym zrobił tak:

[PHP] pobierz, plaintext 
class User extends Controller{
 
public function actionCreate() {    // akcja tworząca użytkownika
 
$model = new User;                    // model user
 
if (isset($_POST['formualarz'])) {   // jeżeli pobierze wartości to wykona dalszy skrypt
$model->attributes = $_POST['Formularz'];     // zapisuje wartości z postu
 
if($model->validate()){  // walidujesz model
$model->save();           // jeżeli zwaliduje to zapisuje
$this->redirect ();         // i wyświetla widok
}
 
}
[PHP] pobierz, plaintext 

hmmm, a ja zawsze walidowałem w kontolerze.

Generalnie kontroler ma pośredniczyć między widokiem, a modelem, a także zarządzać całością. Ale sam w sobie nie powinien ingerować w żaden sposób w zawartość przekazywanych zmiennych. Od tego jest model. Kontroler ma go tylko uruchomić oraz wywołać odpowiednie metody.