Czy return false moze dzialac w metodzie __set ? Opcje

[marekc12]

[PHP] pobierz, plaintext 
 
class Uzytkownik
{
	public $dane = array();
 
	function __set($parametr, $wartosc)
	{
switch ($parametr)
		{
			case 'id':
if(!sprawdzCzyliczbaNaturalna($wartosc)) throw new Exception();
			break;
case 'mail':
if(!(filtrMail($wartosc))) return false;
			break;
			case 'haslo':
 if( !( strlen($wartosc) >= 8 && strlen($wartosc) <= 32 ) )return false;
				$wartosc = $this->szyfrujHaslo($wartosc);
			break;
			default:
throw new Exception();
		}
 
		$this->dane[$parametr] = $wartosc;
}
 
	function zapisz()
	{
if(isset($this->dane[id]))
		{
			wyslijZapytanie("UPDATE users SET ".pobierzListeDanych($this->dane)." WHERE id=$this->dane[id]");
		}
		else
		{
			$this->dane[data] = date("m.d.y");
 		wyslijZapytanie("INSERT INTO users (id,mail,haslo,data) VALUES ('','0','".$this->dane[mail]."','".$this->dane[haslo]."','".$this->dane[data]."')");
		}
	}
 
	function usun()
	{
		wyslijZapytanie("DELETE FROM users WHERE id=$this->dane[id])");
	}
 
	function wczytaj()
	{
		if(count($this->dane))
 		$wynik = wyslijZapytanieSelect("SELECT id,mail,haslo,data FROM users WHERE ".pobierzListeDanych($this->dane));
		else
 		$wynik = wyslijZapytanieSelect("SELECT id,mail,haslo,data FROM users");
 
  return $wynik;
	}
 
	function szyfrujHaslo($haslo)
	{
		if( !( strlen($haslo) >= 8 && strlen($haslo) <= 32 ) )
			die('Dostalem haslo z bledna iloscia znakow!');
 
		$haslo = sha1( $haslo.'yk[d' );
 
		return $haslo;
	}
}
 
 
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 
if (count($_POST))
{
	$bledy = array();
 
 $Mail = filtrZmienna($_POST['Mail']);
 $Haslo = filtrZmienna($_POST['Haslo']);
 
	$uzytkownikSprawdz = new Uzytkownik;
if( $uzytkownikSprawdz->mail=$Mail )
	{
		if( count( $uzytkownikSprawdz->wczytaj() ) !== 0 )
 		$bledy[]='Mail';
	}
	else
 	$bledy[]='Mail';
 
	if(empty($bledy))
	{
		$uzytkownikNowy = new Uzytkownik;
if( ($uzytkownikNowy->mail = $Mail) && ($uzytkownikNowy->haslo=$Haslo) )
			$uzytkownikNowy->zapisz();
 	else $wyswietlFormularz = true;
	}
	else
		$wyswietlFormularz = true;
}
else
$wyswietlFormularz = true;
 
 
[PHP] pobierz, plaintext 

Witam,

powyzej zamiscilem kod ktory pisalem sobie na wlasne potrzeby do tworzenia,usuwania,edytkowania uzytkownikow w bazie danych. Nie jestem bardzo doswiadczony w rozwiazaniach obiektowych. 

Kombinowałem sobie w taki sposob zeby w klasie Uzytkownik poprzez funckcje __set sprawdzac wszystkie wprowadzane dane i ona przy wprowadzaniu jakiejs danej zwracalaby true lub false w zaleznosci od tego czy dana byla poprawna: 

if( !($uzytkownikNowy->mail = $Mail) echo "Wprowadzono niepoprawny adres e-mail";

Niestety nie działa Myslalem ze moge tak zrobic i z tego "($uzytkownikNowy->mail = $Mail)" wyjdzie false... niestety mimo polecenia "return false;" w metodzie __set nie zwraca ona false.




Da sie jakos to zrobic zeby ($uzytkownikNowy->mail = $Mail) zwrocilo false lub true , jezeli wiecie o co mi chodzi?? Czy musze sobie zrobic inna metode odpowiedzialna za wprowadzanie wartosci do klasy?

Ten post edytował marekc12 19.10.2009, 18:13:59

[LBO]

Wprowadź mutatory (a idąc za ciosem akcesory), czyli metody stworzone specjalnie do wprowadzania danych do obiektu.

[PHP] pobierz, plaintext 
if($user->setMail($mail)) echo "Wprowadzono niepoprawny adres e-mail";
[PHP] pobierz, plaintext 

[-=Peter=-]

Możesz zaimplementować magiczną metodę __isset i po próbie ustawieniu tego meila, sprawdzać czy faktycznie został on ustawiony. Jednakże jest to dodatkowa linijka kodu. Możesz również wyrzucać wyjątek w metodzie __set jeśli podano niepoprawne dane.

[PHP] pobierz, plaintext 
//w definicji klasy
//ciach
public function __isset($name){
  return isset($this->dane[$name]);
}
//ciach
 
$user->mail = $mail;
if(!isset($user->mail)){
  echo 'podano niepoprawny mail';
}
[PHP] pobierz, plaintext 

[erix]

Ostatecznie możesz wysypywać wyjątek przy próbie wprowadzenia błędnych danych. Obejdzie się bez mutatorów, których osobiście nie znoszę.

[marekc12]

Też mi się nie podoba użycie oddzielnej funkcji dla każdej jednej wprowadzanej zmiennej (mutatory) ... Ale chyba to jednak najlepsze wyjście, poniważ jak będę z zewnątrz klasy Uzytkownik wprowadzał każdą jedną zmienną do klasy to będę za każdym razem wiedział czy została ona przyjęta. A jak zrobię wyjątki to już przy pierwszej błędnej zmiennej kod zostanie zatrzymany...
A ja chciałem powiedzieć użytkownikowi które pola w formularzu źle podał, a nie tylko pierwsze pole które podał źle.

Z drugiej strony mógłbym sprawdzać to przed wrzuceniem do klasy... Ale od tego jest klasa, że to się sprawdzało w niej...
Jak z kilku różnych miejsc w kodzie będę wrzucał do tej klasy zmienne to lepiej żeby tam w środku było weryfikowane niż za każdym razem na zewnątrz (łatwo się pomylić i zapomnieć zweryfikować).

Mam nadzieje, że zrozumieliście co napisalem, bo troche zamotałem chyba ))

Chyba mutatory wykorzystam.

Ten post edytował marekc12 21.10.2009, 14:27:16

[phpion]

Tworzenie setterów/getterów jest strasznie upierdliwe. Gdy dodamy do tego jeszcze komentowanie to staje się to jeszcze bardziej upierdliwe. Nie zmienia to jednak faktu, że chyba warto je pisać. Mamy wówczas pełną kontrolę nad wypełnianiem obiektów danymi. Tworząc w setterze $this->pole = (string)$wartosc; mamy pewność, że zawsze będzie tam siedział string. Nie wspomnę już o wymuszeniach typu (set_pole(Moj_Obiekt $wartosc)). Tak więc moim zdaniem warto poświęcić nieco czasu na utworzenie setterów/getterów nawet jeśli mają po prostu ograniczać się do return $this->pole.

[erix]

Z drugiej strony mógłbym sprawdzać to przed wrzuceniem do klasy... Ale od tego jest klasa, że to się sprawdzało w niej...
Jak z kilku różnych miejsc w kodzie będę wrzucał do tej klasy zmienne to lepiej żeby tam w środku było weryfikowane niż za każdym razem na zewnątrz (łatwo się pomylić i zapomnieć zweryfikować).

Zawsze możesz użyć interfejsu ArrayAccess. Będziesz miał wówczas wszystko wydzielone i bez problemu sprawdzisz. A i chyba szybciej działa niż settery/gettery.