__set() verhindern das erzeugen einer neuen Variablen

shocky · 20.10.2006, 16:46

Also mit __set( $name, $value ) kann ich ja variablen einner klasse setzen die als private gekenzeichnet sind.
Das Problem ist das __set aber auch eine neue Variable erzeugt falls diese nicht vorhanden ist.
kann ich das verhindern??
Hier mein Code :

PHP-Code:

   
class MyClass{

   private $MyVar;

   

   function __set( $name, $value ){

      $this->$name = $value;

   }

 
}

 
   $foo = new MyClass();

   $foo->MyVar     = "bla";

   $foo->MyErrVar = "blu";

   echo $foo->MyErrVar;

   echo $foo->MyVar;

Jetzt will ich verhindern dass MyErrVar überhaupt erzeugt wird da man sich ja immer mal verschreiben kann und somit eine Variable erzeugt und werte dort reinschreibt die für eine andere gedacht waren und somit ist man bei der fehlersuche verloren.

nikosch · 20.10.2006, 16:50

kann man innerhalb der Funktion nicht $name mit einem Array gültiger Variablennamen abgleichen und bei fehlender Angabe mit false beenden??

shocky · 20.10.2006, 16:51

hmm..
auf diese idee bin ich nicht gekommen.
Schon mal nicht schlecht obwohl einbischen unsauber wie ich finde

nikosch · 20.10.2006, 17:04

so wie die Funktion jetzt definiert ist kann man sie aber auch gleich weglassen oder?

Code:

   function __set( $name, $value ){ 
      if (!in_array ($name , 
                     array ('MyVar' ,
                            'OtherVar'
                            ))) return false;
      $this->$name = $value; 
   }

Unsauber?

Basti · 20.10.2006, 18:49

Macht echt keinen Sinn.
Warum setzt du die Vars nicht gleich auf public?

Ich vewende __get() etc. nur in Dekoratoren oder Datenobjekten, in denen die Werte in assoziativen Arrays vorliegen.

Basti

nikosch · 20.10.2006, 20:19

mal ne blöde frage: was sind'n Dekoratoren?

Basti · 20.10.2006, 20:39

xabbuh · 21.10.2006, 15:22

Abgesehen davon, dass ich __set() ebenfalls für eher schlecht halte, suchst du vermutlich get_class_vars():

PHP-Code:

  public function __set($name, $value)

{

    if(array_key_exists($name, get_class_vars(get_class($this))) {

        $this->$name = $value;

    }

}

Basti · 21.10.2006, 15:42

...müsste array_key_exists() anstatt in_array() sein, da die Property-Namen die Schlüssel im Array sind.

Und, im Falle, dass das Attribut nicht existiert sollte eine Exception oder eben ein Fehler geworfen werfen. Andernfalls wäre das kein Spaß beim Debuggen!

Basti

shocky · 21.10.2006, 15:59

Zitat:

Warum setzt du die Vars nicht gleich auf public?

weil ich soo besser kontrolieren kann welche werte reinkommen.
Bsp. : ein Vorname darf keine zahlen enthalten!
Dies kann ich abfangen und eine Exception aufwerfen falls jemand eine zahl speichert ! :wink:

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Smarty/quickform: Variablen Namen dynamisch erzeugen ?	Toxo	HTML, Usability und Barrierefreiheit	1	31.10.2007 16:55
variablen aus textfile erzeugen (shellscript)	noskule	Server, Hosting und Workstations	2	19.07.2006 15:42
Variablen übergeben bzw. auslesen?		PHP Tipps 2005	4	30.01.2005 03:56

20.10.2006, 16:46
shocky Benutzer Registriert seit: 08.11.2004 Beiträge: 70	__set() verhindern das erzeugen einer neuen Variablen Also mit __set( $name, $value ) kann ich ja variablen einner klasse setzen die als private gekenzeichnet sind. Das Problem ist das __set aber auch eine neue Variable erzeugt falls diese nicht vorhanden ist. kann ich das verhindern?? Hier mein Code : PHP-Code: `class MyClass{ private $MyVar; function __set( $name, $value ){ $this->$name = $value; } } $foo = new MyClass(); $foo->MyVar = "bla"; $foo->MyErrVar = "blu"; echo $foo->MyErrVar; echo $foo->MyVar;` Jetzt will ich verhindern dass MyErrVar überhaupt erzeugt wird da man sich ja immer mal verschreiben kann und somit eine Variable erzeugt und werte dort reinschreibt die für eine andere gedacht waren und somit ist man bei der fehlersuche verloren.


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

20.10.2006, 16:50
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	kann man innerhalb der Funktion nicht $name mit einem Array gültiger Variablennamen abgleichen und bei fehlender Angabe mit false beenden??

20.10.2006, 16:51
shocky Benutzer Registriert seit: 08.11.2004 Beiträge: 70	hmm.. auf diese idee bin ich nicht gekommen. Schon mal nicht schlecht obwohl einbischen unsauber wie ich finde

20.10.2006, 17:04
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	so wie die Funktion jetzt definiert ist kann man sie aber auch gleich weglassen oder? Code: function __set( $name, $value ){ if (!in_array ($name , array ('MyVar' , 'OtherVar' ))) return false; $this->$name = $value; } Unsauber?

20.10.2006, 18:49
Basti Erfahrener Benutzer Registriert seit: 18.07.2004 Beiträge: 2.162 PHP-Kenntnisse: Fortgeschritten	Macht echt keinen Sinn. Warum setzt du die Vars nicht gleich auf public? Ich vewende __get() etc. nur in Dekoratoren oder Datenobjekten, in denen die Werte in assoziativen Arrays vorliegen. Basti

20.10.2006, 20:19
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	mal ne blöde frage: was sind'n Dekoratoren?

20.10.2006, 20:39
Basti Erfahrener Benutzer Registriert seit: 18.07.2004 Beiträge: 2.162 PHP-Kenntnisse: Fortgeschritten

21.10.2006, 15:22
xabbuh Erfahrener Benutzer Registriert seit: 13.11.2005 Beiträge: 2.583	Abgesehen davon, dass ich __set() ebenfalls für eher schlecht halte, suchst du vermutlich get_class_vars(): PHP-Code: `public function __set($name, $value) { if(array_key_exists($name, get_class_vars(get_class($this))) { $this->$name = $value; } }`

21.10.2006, 15:42
Basti Erfahrener Benutzer Registriert seit: 18.07.2004 Beiträge: 2.162 PHP-Kenntnisse: Fortgeschritten	...müsste array_key_exists() anstatt in_array() sein, da die Property-Namen die Schlüssel im Array sind. Und, im Falle, dass das Attribut nicht existiert sollte eine Exception oder eben ein Fehler geworfen werfen. Andernfalls wäre das kein Spaß beim Debuggen! Basti