Forum PHP.pl > [ZF] Przerwanie skryptu po wysłaniu formularza

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [ZF] Przerwanie skryptu po wysłaniu formularza

IceManSpy

5.04.2011, 18:15:46

Witam

Cały czak kontynuuję naukę Zenda i natknąłem się na fajny blog:
http://www.karolnowicki.pl/zend-framework/artykuly/
i wykonuje w nim krok po kroku wszystko z serii "Praca z bazą danych". W części pierwszej pominąłem dopisek do pliku Boostrap, ale dopisuje wszędzie przedrostek Application. Ale do rzeczy.
Model:

[PHP] pobierz, plaintext 
class Application_Model_Uzytkownicy
{
    protected $_name = 'uzytkownicy';
 
    public function nowyUzytkownik($email, $haslo, $imie, $nazwisko, $rola, $status)
    {
      $row = $this->createRow();
      if ($row) {
        $row->email = $email;
        $row->haslo = $haslo;
        $row->imie = $imie;
        $row->nazwisko = $nazwisko;
        $row->rola = $rola;
        $row->status = $status;
        $row->save();
        return TRUE;
      } else {
        throw new Zend_Exception('Nie można utworzyć użytkownika. Błąd bazy danych!'); // warto tworzyć własne komunikaty o błędach, kiedy wykonujemy funkcje na tabelach
      }
    }
 
    public function listaUzytkownikow()
    {
        $select = $this->select();
        $select->order('email ASC');
        $adapter = new Zend_Paginator_Adapter_DbTableSelect($select);
        return $adapter;
    }
}
[PHP] pobierz, plaintext

Kontrolery (ustawiłem sobie w nim echo, aby wiedzieć gdzie się wysypuje):

[PHP] pobierz, plaintext 
<?php
 
class UzytkownicyController extends Zend_Controller_Action
{
 
    public function init()
    {
        /* Initialize action controller here */
    }
 
    public function indexAction()
    {
 
    }
    public function nowyUzytkownikAction()
    {
        $form = new Application_Form_Uzytkownik();
      $form->setAction('/uzytkownicy/nowy-uzytkownik')
              ->setMethod('post');
 
      print_r($_POST);
      if($this->getRequest()->isPost()) {
          echo "odebrałem forma";
          if($form->isValid($_POST)) {
              echo "po walidacji";
              $modelUzytkownicy = new Application_Model_Uzytkownicy();
              $email = $form->getValue('email');
              $haslo = md5($form->getValue('haslo'));
              $imie = $form->getValue('imie');
              $nazwisko = $form->getValue('nazwisko');
              $rola = $form->getValue('rola');
              $status = $form->getValue('status');
              echo "odebrałem values";
              echo $email;
              $results = $modelUzytkownicy->nowyUzytkownik($email, $haslo, $imie, $nazwisko, $rola, $status);
              echo "obiekt stworzony";
              print_r($results);
              echo "po pront r";
              if($results) {
                  echo "rezultat";
                  return $this->_redirect('/uzytkownicy/lista-uzytkownikow');
                  //$this->_helper->redirector('/uzytkownicy/lista-uzytkownikow'); // taka wersja też nie działa
              }
          }
          if($form->isErrors()) {
              $form->populate($_POST);
          }
      }
 
      $this->view->form = $form;
    }
 
}
[PHP] pobierz, plaintext

Formularz:

[PHP] pobierz, plaintext 
class Application_Form_Uzytkownik extends Zend_Form
{
 
    public function init()
    {
        $email = $this->createElement('text', 'email');
        $email->setLabel('Adres e-mail:')
                ->setRequired(TRUE)
                ->setAttrib('size', 30)
                ->addFilters(array(
                    new Zend_Filter_StringToLower(),
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_NotEmpty(),
                    new Zend_Validate_EmailAddress(Zend_Validate_Hostname::ALLOW_DNS)
                ));
        $haslo = $this->createElement('password', 'haslo');
        $haslo->setLabel('Hasło:')
                ->setRequired(TRUE)
                ->setAttrib('size', 30)
                ->addFilters(array(
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_NotEmpty(),
                    new Zend_Validate_StringLength(3, 12)
                ));
        $imie = $this->createElement('text', 'imie');
        $imie->setLabel('Imię:')
                ->setRequired(FALSE)
                ->setAttrib('size', 30)
                ->addFilters(array(
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_StringLength(2, 50)
                ));
        $nazwisko = $this->createElement('text', 'nazwisko');
        $nazwisko->setLabel('Nazwisko:')
                ->setRequired(FALSE)
                ->setAttrib('size', 30)
                ->addFilters(array(
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_StringLength(2, 50)
                ));
        $rola = $this->createElement('select', 'rola');
        $rola->setLabel('Rola użytkownika:')
                ->setRequired(TRUE)
                ->addFilters(array(
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_NotEmpty(),
                    new Zend_Validate_StringLength(4, 5)
                ))
                ->addMultiOptions(array(
                    ''  =>  'Wybierz rolę z listy',
                    'user' =>  'Gość',
                    'autor' =>  'Autor',
                    'admin' =>  'Administrator'
                ));
        $status = $this->createElement('select', 'status');
        $status->setLabel('Status użytkownika:')
                ->setRequired(TRUE)
                ->addFilters(array(
                    new Zend_Filter_StringTrim(),
                    new Zend_Filter_StripNewlines(),
                    new Zend_Filter_StripTags()
                ))
                ->addValidators(array(
                    new Zend_Validate_NotEmpty(),
                    new Zend_Validate_StringLength(1, 1)
                ))
                ->addMultiOptions(array(
                    ''  =>  'Wybierz status z listy',
                    '0' =>  'Zawieszony',
                    '1' =>  'Aktywny'
                ));
        $id = $this->createElement('hidden', 'id');
        $this->addElements(array(
            $email,
            $haslo,
            $imie,
            $nazwisko,
            $rola,
            $status,
            array(
                'submit', 'submit', array(
                    'label' => 'zapisz'
                )
            ),
            $id
            ));
    }
 
 
}
[PHP] pobierz, plaintext

I teraz gdy wypełnię formularz to zatrzymuje mi się na wykonaniu metody nowyUzytkownik. Wyświetla się tylko wszystko do echo "odebrałem values"; echo $email; a potem jest pusta, biała strona. Gdy błędnie wypełnię formularz, to pokazują się komunikaty o błędach i wraca do strony z formularzem. W bazie nie ma żadnych nowych rekordów. To samo się dzieje (białą, pusta strona), gdy chcę wyświetlić listę użytkowników (adres/uzytkownicy/lista-uzytkownikow).

Plik applacation.ini:

Kod

[production]
phpSettings.display_startup_errors = 0
phpSettings.display_errors = 0
includePaths.library = APPLICATION_PATH "/../library"
bootstrap.path = APPLICATION_PATH "/Bootstrap.php"
bootstrap.class = "Bootstrap"
appnamespace = "Application"
resources.frontController.controllerDirectory = APPLICATION_PATH "/controllers"
resources.frontController.params.displayExceptions = 0

resources.layout.layoutPath = APPLICATION_PATH "/layouts/scripts/"
resources.db.adapter = "pdo_mysql"
resources.db.params.host = "localhost"
resources.db.params.username = "user"
resources.db.params.password = "haslo"
resources.db.params.dbname = "naukazend2"
resources.db.params.charset = "utf8"
resources.db.isDefaultTableAdapter = true

[staging : production]

[testing : production]
phpSettings.display_startup_errors = 1
phpSettings.display_errors = 1

[development : production]
phpSettings.display_startup_errors = 1
phpSettings.display_errors = 1
resources.frontController.params.displayExceptions = 1

Gdzie jest błąd?

ROZWIĄZAŁEM:
Model ma być dziedziczony z Zend_Db_Table

NEO.pl

7.04.2011, 10:38:35

Na przyszlosc: error_reporting(E_ALL) + error.log serwera Twoim przyjacielem

KrzysiekWildfire

14.04.2011, 21:15:20

Łoooo, w zendzie tak nie wyświetlasz błędów, możesz sobie zmienić poniższe trzy linijki w pliku config.ini z:

phpSettings.display_startup_errors = 0
phpSettings.display_errors = 0
resources.frontController.params.displayExceptions = 0

na

phpSettings.display_startup_errors = 1
phpSettings.display_errors = 1
resources.frontController.params.displayExceptions = 1

Ale wtedy musisz pamiętać o tym, aby przy puszczaniu strony live przywrócić standardowe ustawienia

Powyższy model jest bleeee (oczywiście moim zdaniem)!

Dla mnie dobry model ma następujące funkcje:
toArray();
__set();
__get();
setOptions($array = null);

Dobry model wygląda tak (w zend tools: "create model User"):

[PHP] pobierz, plaintext 
class Application_Model_User{
    protected $_email;
    protected $_haslo;
    protected $_imie;
    protected $_nazwisko;
    protected $_rola;
    protected $_status;
 
    public function __construct (array $options = null)
    {
        if (is_array($options))
            $this->setOptions($options);
    }
    public function __set ($name, $value)
    {
        $method = 'set' . $name;
        if (! method_exists($this, $method))
            throw new Exception('Nie znaleziono metody "'.$method.'"!');
        $this->$method($value);
        return $this;
    }
    public function __get ($name)
    {
        $method = 'get' . $name;
        if (! method_exists($this, $method))
            throw new Exception('Nie znaleziono metody "'.$method.'"!');
        return $this->$method();
    }
    public function setOptions (array $options = null)
    {
        $methods = get_class_methods($this);
        foreach ($options as $key => $elem) {
            $method = 'set' . ucfirst($key);
            if (! in_array($method, $methods))
                throw new Exception('Nie nzaleziono metody "'.$method.'"!');
            $this->$method($elem);
        }
        return $this;
    }
    public function toArray(){
        $vars = get_class_vars(get_class($this));
        $array = array();
        foreach ($vars as $key=>$value){
            $method = 'get' . str_replace('_','',$key);
            $array[str_replace('_', '', $key)] = $this->$method();
        }
        return $array;
    }
 
    public function setEmail($email){
        $this->_email = $email;
        return $this;
    }
 
    public function getEmail(){
        return $this->_email;
    }
 
    public function setHaslo($haslo){
        $this->_haslo = $haslo;
        return $this;
    }
 
    public function getHaslo(){
        return $this->_haslo;
    }
 
    public function setImie($imie){
        $this->_imie = $imie;
        return $this;
    }
 
    public function getImie(){
        return $this->_imie;
    }
 
    public function setNazwisko($nazwisko){
        $this->_nazwisko = $nazwisko;
        return $this;
    }
 
    public function getNazwisko(){
        return $this->_nazwisko;
    }
 
    public function setRola($rola){
        $this->_rola = $rola;
        return $this;
    }
 
    public function getRola(){
        return $this->_rola;
    }
 
    public function setStatus($status){
        $this->_status = $status;
        return $this;
    }
 
    public function getStatus(){
        return $this->_status;
    }
}
[PHP] pobierz, plaintext

I do tego potrzebny jest jeszcze mapper (Komenda w zend_tools "create model UserMapper"):

[PHP] pobierz, plaintext 
class Application_Model_UserMapper
{
    protected $_dbTable = null;
    protected $_dbTableName = 'Application_Model_DbTable_User';
 
    public function setDbTable ($dbTable)
    {
        if (is_string($dbTable))
            $dbTable = new $dbTable();
        if (! $dbTable instanceof Zend_Db_Table_Abstract)
            throw new Exception('Zła baza danych!');
        $this->_dbTable = $dbTable;
    }
    public function getDbTable ()
    {
        if (null === $this->_dbTable)
            $this->setDbTable($this->_dbTableName);
        return $this->_dbTable;
    }
    public function  __construct($dbTable = null) {
        if ($dbTable !== null){
            if (is_string($dbTable)){
                $this->_dbTableName = $dbTable;
            }else{
                throw new Exception('Nazwa bazy danych powinna być ciągiem znaków!');
            }
        };
    }
 
    public function fetchAll($select = null){
        $result = $this->getDbTable()->fetchAll($select);
        if (count($result) === 0)
            return null;
        $results = array();
        foreach ($result as $row){
            $results[] = new Application_Model_User($row->toArray());
        }
        return $results;
    }
    public function findLast($count = 5, $user = null){
        return $this->fetchAll($this->getDbTable()->select()->limit(5)->where('creator = ?',$user));
    }
    public function save(Application_Model_User $app){
        if (null === $app->getId()){
            $this->getDbTable()->insert($app->toArray());
        }else{
            $this->getDbTable()->update($app->toArray(), 'id = '.$app->getId());
        }
    }
    public function find($id){
        $result = $this->getDbTable()->find($id);
        if (count($result) === 0)
            return null;
        return new Application_Model_User($result->current()->toArray());
    }
[PHP] pobierz, plaintext

potrzebujesz jeszcze bazy danych (w zend tools: "create db-table User users");

I stworzenie nowego użytkownika wygląda tak (w kontrolerze):

[PHP] pobierz, plaintext 
public function nowyUzytkownikAction(){
    $form = new Application_Form_NewUser(); //to jest twój formularz - konieczne jest, aby pola formularza miały takie same nazwy jak nazwy zmiennych w modelu
    if ($this->_request->isPost()){
        if ($form->isValid($this->_request->getPost())){
            $user = new Application_Model_User($form->toArray());
            $user->haslo = md5($user->haslo);
            $userMapper = new Application_Model_UserMapper();
            $userMapper->save($user);
            $this->_redirect('/sciezka/do/przeniesienia/uzytkownika');
            exit;
        }
    }
}
[PHP] pobierz, plaintext

I powinno wszystko elegancko śmigać.

Pilsener

16.04.2011, 22:40:11

A po co taki kombajn? Przecież to wszystko jest gotowe w Zend_Db_Table
Wystarczy prosty model:

[PHP] pobierz, plaintext 
class Model_Users extends Zend_Db_Table_Abstract{
 
	protected $_name = 'users';
 
	public function getById ($id) {
		$select = $this->select();
		$select->where('id = ?', $id);
		return $this->fetchRow($select);
	}
}
[PHP] pobierz, plaintext

Wykorzystanie:

[PHP] pobierz, plaintext 
//pobieranie danych
$model = new Model_Users();
$user = $model->getById(12);
echo $user->nick; 
 
//dodanie nowego usera
$data = array(
   'email'=>$email,
   'nick'=>$nick
);
$model->insert($data);
[PHP] pobierz, plaintext

KrzysiekWildfire

17.04.2011, 12:10:03

Jestem już tak przyzwyczajony - tym bardziej, że nie piszę małych projektów a tworząc tak mam wszystko na swoim miejscu - i można w takiej klasie tworzyć metody do zarządzania danymi. Potworzyłem sobie klasy w których mam te najczęściej wykorzystywane funkcje (__get, __set, itd) i stworzyłem sobie generator - Czysto, i wszystko z łatwością mogę stworzyć w kilka chwil.

cudny

19.04.2011, 11:19:29

Cytat(Speedy @ 9.04.2011, 14:45:39 )

Nie wiem, czy prościej, bo do wykonania tak banalnej czynności jest załączany zewnętrzny plik i jakaś klasa. Moje rozwiązanie mieści się w 1 linijce i robi to samo.

Cytat(Pilsener @ 16.04.2011, 21:40:11 )

A po co taki kombajn? Przecież to wszystko jest gotowe w Zend_Db_Table
Wystarczy prosty model:

[PHP] pobierz, plaintext 
class Model_Users extends Zend_Db_Table_Abstract{
 
	protected $_name = 'users';
 
	public function getById ($id) {
		$select = $this->select();
		$select->where('id = ?', $id);
		return $this->fetchRow($select);
	}
}
[PHP] pobierz, plaintext

Wykorzystanie:

[PHP] pobierz, plaintext 
//pobieranie danych
$model = new Model_Users();
$user = $model->getById(12);
echo $user->nick; 
 
//dodanie nowego usera
$data = array(
   'email'=>$email,
   'nick'=>$nick
);
$model->insert($data);
[PHP] pobierz, plaintext

Eee no,

jak ma być prosto to raczej tak:

[PHP] pobierz, plaintext 
// pobieranie
$result = $model->find($id)->current();
// a co do tworzenia tablicy do pamieci to lepiej utrworzyc ja jednak do wymazania przez sama metode czyli
$model->insert(array('email' => $email, 'nick' => $nick));
[PHP] pobierz, plaintext

KrzysiekWildfire

19.04.2011, 14:16:19

wykorzystanie mojego modelu wygląda tak:

[PHP] pobierz, plaintext 
  $model = new Application_Model_UserMapper();
  $user = $model->find($id); //znalezienie 1 użytkownika
  $users = $model->fetchAll(); //znalezienie wszystkich
 
  //modyfikacja użytkownika:
  $user->setMail($mail);
  $model->save($user);
 
  //jeżeli potrzeba wyciągnąć dane:
  user->toArray();
[PHP] pobierz, plaintext

No a poza tym, jeżeli tak wyciągniesz te dane, np do kontrolera - to chyba nie manipulujesz w nim tymi danymi - Powstaje potężny zgrzyt - kontroler nie służy do manipulowania danymi - funkcje pisze się w modelach.
MVC

cudny

21.04.2011, 12:41:47

Cytat(KrzysiekWildfire @ 19.04.2011, 14:16:19 )

W modelach operujesz na danych z bazy, zapisywaniu ew. szybkim przetwarzaniu - resztę robisz w kontrolerach, ja używam do tego jeszcze pisanych przeze mnie bibliotek ale całość i tak trzeba wypluć do widoku z kontrolera wiec... nie rozumiem jak przetwarzasz to wszystko w modelu.
Ogólnie tworzę sobie w library obok Zend katalog powiedzmy Project i w nim daje funkcje związane powiedzmy z XML czy przetwarzanie danych czy jakieś stałe walidacje formularzy, które często wykorzystuje.
Nie wyobrażam sobie tego mieszać z zapytaniami do bazy - model jest tylko do użytku od strony biznesowej kodu, a nie do umieszczania w nim jakiś modułów.
Bardzo często robię rozbudowane zapytania w pracy i nie wiem jak można do klas modeli wstawiać jakieś inne metody wykorzystywane w kontrolerze ?
Przecież w modelu dziedziczysz sporo danych z innych klas i co - jeśli chce sobie napisać funkcję powiedzmy, no nie wiem... between:

[PHP] pobierz, plaintext 
public function between($from, $to, $value) {
if($value > $to || $value < $from) return false;
return true;
}
[PHP] pobierz, plaintext

I co - walne to do modelu ?
Chyba inaczej rozumiemy MVC

KrzysiekWildfire

21.04.2011, 16:15:52

Może inaczej:
Zend Framework: Documentation
Zapraszam do lektury bo ciekawa.

cudny

21.04.2011, 20:06:57

Ooo, super książka, skąd masz taką

Niestety tak jak napisałem, wszytkie opisane w niej metody dotyczą jedynie danych związanych z modelem czyli... wow

dane do walidacji w tym przypadku

Niemniej jednak jeśli musisz zrobić walidację, której zend nie posiada to co zrobisz ? wrzucisz do modelu jako metoda i ... kurcze - cały model będziesz musiał wywołać żeby tej metody użyć innym modelu ?
Model jest tylko do obróbki danych używanych do insetów, selectów i updatów. Po seleccie z bazy danych obrabiasz to już w controlerach i własnych klasach !

Acha - żeby nie było - odwołuję się tutaj do twoich słów:

Cytat

Powstaje potężny zgrzyt - kontroler nie służy do manipulowania danymi - funkcje pisze się w modelach.

A jeśli chodzi o __get __set to jak najbardziej używać należy ale... funkcji nie pisze się w modelach

kalipek

21.04.2011, 23:40:53

taaa, obrabiaj sobie w kontrolerze, obrabiaj

http://blog.astrumfutura.com/2008/12/the-m...-unappreciated/

http://www.littlehart.net/atthekeyboard/20...nny-controllers

cudny

22.04.2011, 11:30:39

Cytat(kalipek @ 21.04.2011, 23:40:53 )

taaa, obrabiaj sobie w kontrolerze, obrabiaj

http://blog.astrumfutura.com/2008/12/the-m...-unappreciated/

http://www.littlehart.net/atthekeyboard/20...nny-controllers

Wszystkie moje posty w tym temacie to bzdura.
Ogólnie cieszę się tylko, że większość aplikacji i tak pisałem na osobnych modułach tworząc własną bibliotekę (

Cytat

taaa, obrabiaj sobie w kontrolerze, obrabiaj

- nic nie obrabiałem w kontrolerze

) więc nie będę miał kłopotów z przerobieniem obecnie pisanych aplikacji. Niestety przychodzi też taki czas, że trzeba się przyznać do błędu i opuścić głowę

Oczywiście pojmując (błędnie) MVC poszedłem na skróty i uczyłem się od kolegów z pracy którzy... chyba zrobili tak jak ja

Ten błąd pojmowania MVC jest dokładnie opisany w podanym przez kalipka linku

(tym drugim)
Od 2 lat żyłem z klapkami na oczach, a wystarczyło jednak samemu troszki się pouczyć

Artykuł: http://blog.astrumfutura.com/2008/12/the-m...-unappreciated/ otworzył mi oczy

KrzysiekWildfire

27.04.2011, 11:47:15

Ja też z początku miałem ZŁE podejście do M. Wiadomo, nie od razu Rzym zbudowano. Życzę powodzonka przy przyszłych projektach.

Apo

29.04.2011, 22:06:30

Cytat(KrzysiekWildfire @ 14.04.2011, 20:15:20 )

public function fetchAll($select = null){
$result = $this->getDbTable()->fetchAll($select);
if (count($result) === 0)
return null;
$results = array();
foreach ($result as $row){
$results[] = new Application_Model_User($row->toArray());
}
return $results;
}

Mam pytanie @KrzysiekWildfire.
Jak rozwiązałeś opakowanie zwracanych danych w metodzie fetchAll, gdy używasz klasy Zend_Paginator?
Próbowałem robić:

[PHP] pobierz, plaintext 
<?php
foreach ($result as $row) {
            $entries[] = new Application_Model_Product(
                             $result instanceof Zend_Paginator ? $row : $row->toArray());
?>
[PHP] pobierz, plaintext

Niestety po zwróceniu wyników w widoku nie mogę wygenerować paginatora, ponieważ zwracane wyniki nie
są instancją klasy Zend_Paginator.

// edycja
Już rozwiązałem problem. Należy stworzyć filtr:

Kod

$paginator>setFilter(new Zend_Filter_Callback(array($this, 'getProductObjects')));

następnie tworzymy metodę:

Kod

<?php
public function getProductObjects($result)
    {
        $entries = array();
        foreach($result as $row) {
            $entries[]  = new Application_Model_Product($row instanceof Zend_Db_Table_Row
                              ? $row->toArray(): $row);
        }
        return $entries;
    }
?>

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.