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 (IMG:style_emoticons/default/sad.gif) 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

[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 (IMG:style_emoticons/default/smile.gif) ))

Chyba mutatory wykorzystam.

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