Mam mały dylemacik dotyczący OOP. Gdzie najlepiej umiescic walidacje danych dostarczonych przez uzytkownika? Ja widze 3 rozwiązania
1) Przy "setach"

[PHP] pobierz, plaintext 
<?php
class user
{
  public function setLogin( $login )
  {
	if ( !UserValidate::login( $login ) )
	{
	  throw new ValidateException( 'login' );
	}
	$this->setField( 'login', $login );
  }
}
?>
[PHP] pobierz, plaintext 

2) Przy zapisywaniu danych

[PHP] pobierz, plaintext 
<?php
class user
{
  public function save()
  {
	if ( !UserValidate::login( $this->login ) )
	{
	  throw new ValidateException( 'login' );
	}
	if ( !UserValidate::pass( $this->pass ) )
	{
	  throw new ValidateException( 'pass' );
	}....
 
	// zapisywanie
  }
}
?>
[PHP] pobierz, plaintext 

3) W kontrolerze zanim wprowadzimy dane do obiektu

[PHP] pobierz, plaintext 
<?php
class UserController
{
  public function save()
 {
   try
   {
	  if ( !UserValidate::login( $_POST['login'] ) )
	  {
		throw new ValidateException( 'login' );
	  }
	  if ( !UserValidate::pass( $_POST['pass'] ) )
	  {
		throw new ValidateException( 'pass' );
	  }....
	}catch(ValidateException $e)
	{
	  echo $e->getMessage();
	}
  $user = new user;
  $user->setLogin( $_POST['login'] )
  $user->setPass( $_POST['passs'] )
  $user->save();
 } 
}
?>
[PHP] pobierz, plaintext 

Jak wy rozwiązaliscie ten problem? Z góry dzięki za odpowiedz.

Hmm mi się wydaje, że w kontrolerze

Witam !
Wydaje mi się, że najlepszym miejscem będzie kontroler

Pozdrawiam
Darektbg

Walidujesz model -- to w modelu.
Pozatym pomysl jak bedzie wygladac akcja tworzenia obiektu i edycji + walidacja wszystkich pol w kontrolerze.

dr_bonzo wyjaśnił gdzie powinna zajść walidacja, ja jeszcze dołożę swoje 2 grosze. Mianowicie używanie statycznych metod do walidacji jest dość mało obiektowe, jak i mało wygodne. Zaraz wrzucę Ci moją małą klasę do walidacji.

Mało obiektowe? Powiedz mi dlaczego i jak to ma być obiektowe - nie potrafię sobie tego wyobrazić. Może chodzi o zastosowanie pewnej implementacji dekoratora?

Pozdrawiam.

Otworzyłem masę zakładek a zapomniałem o tym, ale już jestem wraz z moim obiecanym kodem: http://forum.php.pl/index.php?showtopic=58987


Cysiaczek a co obiektowego jest w używaniu klasy która wszystkie metody ma statyczne? To tak samo jak strukturalnie, po za tym ze jest class, public przed metodą, oraz inaczej się to wywołuje

ja dodam tez cos o czym nie wszyscy wiedzą (wiedzieli:P) a strzalek wie

http://strzalek.net/blog/3/input-filter

@strzałek - nic nie ma, ale czy wogóle musi być? Moim zdaniem nie musi, bo nie ma sensu na siłę wciskać obiektów wszędzie. Ubierając zastaw filtrów nawet w zwykłą klasę zyskujesz ładny interfejs : )

Pozdrawiam.

Własną klase do obsługi walidacji juz mam. Zresztą umiesciłem ją juz jakis czas temu na tym forum. http://forum.php.pl/index.php?showtopic=56211&hl=

Owe statyczne metody to implementacja dekoratora.
Dzięki za rozwianie moich wątpliwości.