[Symfony]Wyjątki try catch Opcje

[blackroger]

Mam problem. Otóż mam takie coś:

[PHP] pobierz, plaintext 
 try
              {
                  $con->beginTransaction();
 
                  $r = new Rejestracja();
 
                  $r->setLogin($request->getParameter('login'));
                  $r->setHaslo($request->getParameter('haslo'));
                  $r->setImie($request->getParameter('imie'));
                  $r->setNazwisko($request->getParameter('nazwisko'));
                  $r->setMiasto($request->getParameter('miasto'));
                  $r->setMail($request->getParameter('mail'));
                  $r->setData(date("Y-m-d"));
 
                  $r->save();
                  $con->commit();
 
                  return $this->forward('register', 'thank_you');                
              } catch(Exception $e)
              {
                  $con->rollback();                                           
 
                  return $this->forward('register', 'error');
                  //throw($e);
              }
             //return $this->forward('register', 'thank_you');       
[PHP] pobierz, plaintext 

Wszystko działa dobrze (bez throw()), gdy daje return $this->forward('register', 'thank_you'); poza blokiem try. Gdy jest w środku - pojawia się warning, dochodzi do poprawnego przekierowania a następnie catch przechwytuje i przekierowuje do error:


Dziękujemy!

Warning: Cannot modify header information - headers already sent by (output started at /usr/share/php/symfony/response/sfResponse.class.php:105) in /usr/share/php/symfony/response/sfWebResponse.class.php on line 335

Warning: Cannot modify header information - headers already sent by (output started at /usr/share/php/symfony/response/sfResponse.class.php:105) in /usr/share/php/symfony/response/sfWebResponse.class.php on line 349
Błąd transakcji!


Gdy używam throw() problem rozwiązuje się częściowo, gdyż warningi nie występują, ale w przypadku błędu blok catch przechwytuje wyjątek a throw wyrzuca błąd uniemożliwiając przeforwardowanie return $this->forward('register', 'error'); Ustawienie throw() przed czy po return forward nie załatwia sprawy.

Zaznaczam raz jeszcze, że problem występuje tylko i wyłącznie przy zastosowaniu try, catch...
Symfony Framework.

Ten post edytował Ociu 7.09.2009, 11:12:18

Powód edycji: [Ociu]: dodałem tag.

[krowal]

Gdzieś niepotrzebnie wyrzucasz 'Dziękujemy!' od razu na ekran a symfony nie lubi echo w modelach i w akcjach

[Ociu]

Przenoszę do Framoworki.

[destroyerr]

Na początku napiszę tylko, że związek z symfony moim zdaniem jest mizerny.

Zakładam, że korzystasz z Propela, a kod który podałeś to kod Twojej akcji. Zakładam też, że klasa Rejestracja to klasa modelu, jeśli tak, to w takim razie otwórz sobie plik: lib/model/om/BaseRejestracja.php i poszukaj sobie metody save (zwróć uwagę na obsługę transakcji).

Cały Twój problem bierze się z braku walidacji danych (lepiej gdybyś stworzył do tego formularz). Trafiają się pewnie dane, których baza nie chce przyjąć dlatego dostajesz wyjątek. Przy dobrej walidacji, wyjątek będzie tylko w sytuacji wyjątkowej i wtedy użytkownik może zobaczyć stronę błędu (dodatkowa obsługa wyjątku może zostać pominięta).

Po poprawnym wykonaniu akcji z zapisem lepiej zrobić redirect, a nie forward.
Co do ostrzeżenia - nie będę wróżył skąd się wzięło.

[blackroger]

Jeżeli chodzi o walidację to jest, była i będzie w lib/form. Nie będę jej całej tu umieszczać, chyba że ktoś stwierdzi, że to może pomóc w rozwiązaniu problemu. Przy redirect problem nie występuje. Zastanawia mnie tylko dlaczego przy forward dzieją się takie dziwne rzeczy i to tylko w przypadku zastosowania try,catch...

Przed chwilą znalazłem link na forum:
http://forum.php.pl/lofiversion/index.php/t124562.html

Jest tu poruszany dokładnie ten sam problem, jakkolwiek nie został on tu również rozwiązany (przyczyna warningu jest nieznana).

Ten post edytował blackroger 7.09.2009, 20:21:06

[destroyerr]

Tylko co z tego, że ta walidacja jest w jakimś tam folderze, skoro w podanym przez Ciebie kodzie nie jest ona wykorzystana?!

W podanym przez Ciebie wątku problem nie został rozwiązany z przyczyn oczywistych, autor nie podał informacji o tym jaki dostaje wyjątek. Obaj chcecie złapać wszystkie wyjątki, co nie ma najmniejszego sensu. Symfony jest skonstruowane w ten sposób, że forward jest wymuszony przez wyjątek: sfForwardException. Ty go przechwytujesz i forward się nie odbywa, a więc winne jest to o czym pisałem między innymi w poprzednim poście.

[blackroger]

Tak dla wyjaśnienia sprawy z walidacją:

[PHP] pobierz, plaintext 
public function executeIndex(sfWebRequest $request)
  {
      $form = new RegisterForm();
 
      if($this->getRequest()->isMethod('post'))
      {
          $form->bind(array('login'=>$this->getRequest()->getParameter('login'),
                            'haslo'=>$this->getRequest()->getParameter('haslo'),
                            'powtorz'=>$this->getRequest()->getParameter('powtorz'),
                            'imie'=>$this->getRequest()->getParameter('imie'),
                            'nazwisko'=>$this->getRequest()->getParameter('nazwisko'),
                            'miasto'=>$this->getRequest()->getParameter('miasto'),
                            'mail'=>$this->getRequest()->getParameter('mail')
                     ));
 
 
            if($form->isValid())
          {
              $data = $form->getValues();
 
              $con = Propel::getConnection();
 
(.....tą część już podałem wcześniej)
 
           }//koniec valid if
      }//koniec request if
 
      $this->form = $form;
  }//koniec executeIndex
 
[PHP] pobierz, plaintext 

Nie będę pokazywał registerForm bo jest za długi. Dzięki destroyerr za zainteresowanie tematem. Jeżeli w symfony forward wymusza wyjatek przekierowania sfForwardException to jak teraz zrobić aby przechwytywać wszystkie wyjątki z wyjątkiem niego. Wiem, że można sprawę załatwić na 10 innych sposobów ale mnie interesuje konkretnie ten...

Ten post edytował blackroger 8.09.2009, 10:13:59

[-=Peter=-]

A po co wszystkie? wysterczy jeden (dla ułatwienia: bodajże PropelException)...

Ten post edytował -=Peter=- 8.09.2009, 10:28:13

[destroyerr]

Oh, moim zdaniem robisz błąd i pakujesz się w głupią sytuację, no ale jeśli tak bardzo chcesz:

[PHP] pobierz, plaintext 
<?php
 
 
try
{
	//...
	$this->forward('register', 'thank_tou'); //Upierasz się w tym miejscu na forward, a powinien być redirect!
}
catch(PropelException $exception)
{
	$this->forward('register', 'error');
}
catch(sfForwardException $exception)
{
	throw $exception;
}
 
?>
[PHP] pobierz, plaintext 

Twoja akcja jest w tym momencie niesamowice długa, a wystarczyło:

[PHP] pobierz, plaintext 
<?php
 
public function executeIndex(sfWebRequest $request)
{
	$form = new RegisterForm();
	//$form->getWidgetSchema()->setNameFormat('rejestracja[%s]'); //Ewentulnie możesz dodać to tutaj, choć lepiej w formularzu.
	if($request->isMethod('post')
	{
		$form->bind($request->getParameter('rejestracja'));
		if($form->isValid())
		{
			$form->save(); //Taki zapis jeśli dziedziczy Twój formularz po sfFormPropel
			/**
				a jeśli nie to tak (łapanie wyjątku dodałem specjalnie dla Ciebie):
				$rejestracja = new Rejestracja();
				$rejestracja->fromArray($form->getValues(), BasePeer::TYPE_FIELDNAME);
				try
				{
					$rejestracja->save();
				}
				catch(PropelException $exception)
				{
				}
			*/
			$this->getUser()->setFlash('notice', 'Rejestracja przebiegła pomyślnie');
 
			$this->rediret('@register_thank_you');
		}
		else
		{
			$this->forward('register', 'error');
		}
	}
 
	$this->setVar('form', $form);
}
?>
[PHP] pobierz, plaintext 

[blackroger]

Ok...dzięki za odpowiedzi...wiem, że może głupio się upierałem ale takie rozwiązanie znalazłem w pewnej książce (dla symfony 1.1) i po prostu wywnioskowałem, że autor zamieścił kod, który zawiera błędy i nie jest stricte edukacyjny Zresztą to nie pierwszy, który tam znalazłem.

[destroyerr]

Mały OT: nie wiem jak ktoś mógł wydać książkę dla symfony 1.1. Już sam pomysł na takie wydawnictwo sugeruje niepełną znajomość tematu. Jeśli faktycznie autor wypisuje tam takie rzeczy, to tylko potwierdza się moje zdanie.