Witam,
dla większości to pewnie będzie banał ale ja dopiero zaczynam z tym przygodę.
Chcę zbudować formularz kontaktowy w oparciu o wzorzec MVC.
Nie wiem tylko gdzie mają być przekazywane dane z formularza: do kontrolera czy do modelu.
Czy przekierowanie danych bezpośrednio z formularza na klasę poprzez znacznik <form> jest poprawne?
Pierwsze co mi przyszło na myśl to zbudowanie pliku(ale nie opartego na klasie) który by przejmował dane z tablicy $_POST a następnie dopiero przekazywał je do odpowiedniej klasy aby dokonać walidacji itp.
Wiem, że pewnie mi ktoś poleci jakiś framework ale ja chcę poznać zasadę działania od podszewki.

1. Formularz to właśnie model.
2. Dane do przekazania formularzowi "przyjdą" w żądaniu, więc będziesz musiał je mu przekazać. Pierwszy uruchomiony zostaje kontroler, który tworzy formularz, więc to on przekaże mu dane. Skończysz z czymś w stylu:

[PHP] pobierz, plaintext 
$form = new ThreadForm(new Thread());
 
if ($request->getMethod() === 'POST') {
  $form->bindData($request->get('my-form')); // Request::get() zwróci tablicę z danymi do formularza.
}
[PHP] pobierz, plaintext 

a wyświetlenie i zdefiniowanie formularza nie powinno się znajdować w widoku?

Definicją formularza jest... formularz. Co do jego wyświetlania - raczej ciężko zrobić to gdzie indziej.

Czyli formularz ma być oddzielną klasą?

Tak, zazwyczaj tak się właśnie to robi. Wszystko, co da się nazwać i co powinno odpowiadać za jakieś działania może być obiektem.

a posiada ktoś może jakiś przykładowy skrypt formularza na wzorcu mvc?
Bo jakoś teraz nie mogę sobie tego wyobrazić

Formularz z zend framework:

[PHP] pobierz, plaintext 
class Form_Announce_Search extends Zend_Form
{
	public function init(){	
		$this->setMethod('Get');
 
		$search = new Zend_Form_Element_Text('search');
		$search->setRequired(true);
		$search->setLabel('Search');
                $search->addFilter('StripTags');
		$this->addElement($search);
 
                $submit = new Zend_Form_Element_Submit('save','Search');
		$this->addElement($submit);			
	}
} 
[PHP] pobierz, plaintext 

W kontrolerze tworzysz obiekt formularza i przekazujesz go do widoku:

[PHP] pobierz, plaintext 
	$form = new Form_Announce_Search();
	$this->view->form = $form;
[PHP] pobierz, plaintext 

Formularz sam generuje standardowy HTML, można też podpiąć do niego własny widok. Odbieranie, walidacja, karmienie formularza danymi odbywa się poprzez zestaw metod oczywiście.

Symfony2 ma całkiem ciekawie rozwiązany mechanizm obsługi formularzy.

Aktualnie jestem w trakcie wdrażania się w symfony2, ale bez ORMa. Nie do końca to tam super wygląda, bo jednak fajniejszy był poprzedni FW i trochę szkoda, że tak wiążą formy z modelami. Trochę nadmiarowej pracy trzeba wykonać.

ok, mam klasę która generuje formularz, wiem że z poziomu kontrolera ładuję się ją do widoku.
Tam ta klasa wyświetla formularz. Ale co się dzieje dalej - a mianowicie po kliknięciu przycisku submit?
Akcja ma wskazywać na model i również w modelu dokonywana jest walidacja?

Model to model, wszystkie żądania trafiają do index.php, potem do jakiegoś dispathera i tak dalej aż do wybranej metody:

[PHP] pobierz, plaintext 
//kod w kontrolerze wykonany po wpisaniu adresu strona.pl/kontroler/pokaz_strone
public function pokaz_strone(){
      $form = new Form_Announce_Search();
      if($this->request->isPost() && $form->isValid($this->request->getPostParams())){
         $cos_tam = $form->getValue('search');
     }
      $this->view->form = $form;
 
}
[PHP] pobierz, plaintext 

Kontroler sprawdza, czy wysłano tablicę POST i jeśli tak, to wsadza te parametry do metody isValid, która wykonuje walidatory dodane do formularza.
Walidatory dodaje się podobnie, jak inne obiekty:

[PHP] pobierz, plaintext 
$postcode = new Zend_Form_Element_Text('postcode');
$validator = new Zend_Validate_PostCode('pl_PL');
$postcode->addValidator($validator);
[PHP] pobierz, plaintext 

Podjąłem próbę wykonania formularza opartego o model MVC - oceńcie czy prawidłowo:


index.php - kontroler

[PHP] pobierz, plaintext 
 
<?php
function __autoload($class){
  require $class.'.php';
}
 
class index{
  private $form; 
  private $view;
 
  private $name; //przechowuje wartość pola z formularza
  private $msg; // przechowuje alerty
 
  //konstruktor renderuje widok
  //oraz sprawdza czy został wysłany formularz
  function __construct(){
    if(isset($_POST['send'])){
      $this->name = trim($_POST['name']);
      $this->val();
    }
    $this->render();
  }
  //przygotowanie modelu - formularz
  private function prepareModel(){
    $this->form = new form();
  }
  //przygotowanie widoku
  private function prepareView(){
    $this->view = new View($this->msg,$this->form->st_form);
  }
  //renderowanie widoku
  private function render()
  {
    $this->prepareModel();
    $this->prepareView();
  }
  //walidacja formularza
  private function val(){
    if(strlen($this->name)>0){
      $this->msg = "Operacja wykonana poprawnie";
    }else{
      $this->msg = "Wypełnij puste pole";
    }
 
 
  }
}
new index();	
?>
[PHP] pobierz, plaintext 

View.php - widok

[PHP] pobierz, plaintext 
<?php 
class View{
 
  public $msg;
  public $body;
 
 
function __construct($msg,$body){
  $this->msg = $msg;
  $this->body = $body;
  $this->body();
}
public function body(){
  echo $this->msg."<br />";
  echo $this->body;
}
 
}   
 
?> 	
[PHP] pobierz, plaintext 

form.php - model

[PHP] pobierz, plaintext 
<?php
class form
{
	public $st_form;
 
	public function __construct()
	{
   	   $this->showForm();		
 	}
 	//przpisywanie formularza do zmiennej
	public function showForm()
	{
		$this->st_form = "<form action=\"index.php\" method=\"post\" >
   		<input type=\"hidden\" name=\"send\" >
   		<input type\"text\" name=\"name\" >
   		<input type=\"submit\" value=\"send\" >
   		</form>";
 	}   
}
 
?>
[PHP] pobierz, plaintext 

Zanim weźmiesz się za MVC, poducz się podstaw OOP: http://pl.wikipedia.org/wiki/Zasada_jednej_odpowiedzialności

"Nigdy nie powinno być więcej niż jednego powodu do modyfikacji klasy"

Ale jak to się odnosi do mojego kodu?
Myślałem, że każda klasa wykonuje to co do niej należy....

Spróbuj sobie odpowiedzieć co robią poszczególne obiekty, za co są odpowiedzialne. Jeżeli w odpowiedzi użyjesz spójnika "i" to już masz odpowiedź co jest nie tak.

Czy teraz formularz nadal zawiera błędy?



Index.php - kontroler

[PHP] pobierz, plaintext 
<?php
function __autoload($class){
  require $class.'.php';
}	
 
class index{
  private $model; //
  private $view;
  private $val;
 
  private $name;
  private $form;
  private $msg;
 
  function __construct(){
    $this->startModel();
    $this->setData();
    $this->getData();
    $this->val();
    $this->getData();
    $this->startView();
}
private function startModel(){
  $this->model = new form();
}
private function setData(){
 
  if(isset($_POST['send'])){
 
   $this->model->name = $_POST['name'];
 
   }
}
private function getData(){
  $this->name = $this->model->name;
  $this->form = $this->model->st_form;
  $this->msg = $this->val->msg;
}
private function val(){
  if(isset($_POST['send'])){
    $this->val = new val($this->name);
 
  }
}
private function startView(){
  $this->widok = new view($this->form,$this->msg);
 
}
}
new index();
?>
[PHP] pobierz, plaintext 

form.php - model

[PHP] pobierz, plaintext 
<?php
class form
{
	public $st_form;
	public $name;
 
	public function __construct()
	{
   	  $this->showForm();		
 	}
 	//przpisywanie formularza do zmiennej
	public function showForm()
	{
		$this->st_form = "<form action=\"index.php\" method=\"post\" >
   		<input type=\"hidden\" name=\"send\" >
   		<input type\"text\" name=\"name\" >
   		<input type=\"submit\" value=\"send\" >
   		</form>";
 	}  
 
}
 
?>
[PHP] pobierz, plaintext 

view.php - widok

[PHP] pobierz, plaintext 
<?php 
class view{
  private $form;
  private $msg;
 
  function __construct($form,$msg){
    $this->form = $form;
    $this->msg = $msg;
    $this->render();
  }
  private function render(){
    echo $this->msg."<br/>";
    echo $this->form."<br/>";
  }
}
?> 	
[PHP] pobierz, plaintext 

val.php - klasa odpowiedzialna za walidację

[PHP] pobierz, plaintext 
<?php
 
class val{
  private $name;
  public  $msg;	
 
  public function __construct($name){
    	$this->name = $name;
	$this->check();	
  }
  public function check(){
    if(strlen(trim($this->name))>0){
      $this->msg = "pole nie jest puste";
    } else{
      $this->msg = "pole jest puste";
    }
  }
}
?>
[PHP] pobierz, plaintext