Implementierung vieler Getter bzw. Setter-Methoden

axelf · 29.05.2011, 12:51

Hallo,

mitunter enthält eine Klasse ja sehr viele Eigenschaften.
Benutzt man die magischen Mehoden __set bzw. __get, muss man nicht für jeder Eigenschaft eine Methode schreiben. Jedoch kann man dann auch die jeweiligen Werte nicht mehr beeinflussen (evtl. möchte ich ja einen Wert in der Setter-Methode prüfen). Bei 20 Eigenschaften hat man jeweils 20 Getter und 20 Setter.

Ausserdem soll verhindert werden, das man direkt auf eine Variable von aussen zugreifen kann. Das geht meines Wissens ja sowieso nur mit einem Getter / Setter und nicht mit magischen Methoden

Was ist hier ein guter Weg? Für jede Eigenschaft einen Setter / Getter oder gibts auch einen Weg, der die Vorteile von beiden Varianten verbinden kann?

ChrisB · 29.05.2011, 12:58

Zitat:

Zitat von axelf

Benutzt man die magischen Mehoden __set bzw. __get, muss man nicht für jeder Eigenschaft eine Methode schreiben. Jedoch kann man dann auch die jeweiligen Werte nicht mehr beeinflussen (evtl. möchte ich ja einen Wert in der Setter-Methode prüfen).

Sowohl __get als auch __set bekommen den Namen der Property übergeben.
Daran, per switch o.ä. für einige oder auch alle Properties, die darüber angesprochen werden, eine Sonderbehandlung zu implementieren, hindert dich also niemand.

Zitat:

Ausserdem soll verhindert werden, das man direkt auf eine Variable von aussen zugreifen kann. Das geht meines Wissens ja sowieso nur mit einem Getter / Setter und nicht mit magischen Methoden

Auch daran, eine Exception zu schmeißen, wenn dir der Zugriff nicht passt, hindert dich niemand.

axelf · 29.05.2011, 13:13

Hi,

also ist es letztlich die persönliche Entscheidung wie man sowas löst?
Die switch-Anweisung würde ja mitunter auch sehr groß werden.

Was ist denn besserer Stil? Die Verwendung der magischen Methoden oder das alles selbst zu schreiben?

ChrisB · 29.05.2011, 13:30

Diese magischen Methoden kommen sowieso nur zum Zug, wenn die Sichtbarkeit den direkten Zugriff nicht erlaubt.

Getter und Setter nur zum „Durchreichen“ zu nutzen, ist Unsinn. Wenn keinerlei Kontrolle erforderlich ist - dann wählt man public als Sichtbarkeit, und gut.

Bei den Eigenschaften, die aber aus gutem Grund private oder protected deklariert sind, wirst du dir aber sicher auch etwas dabei gedacht haben. Dass die dann jeweils eigene explizite Getter/Setter haben, wenn erforderlich, da spricht m.E. nichts gegen.
Wenn die magischen Methoden also nur aus „Faulheit“ exzessiv genutzt werden - dann sollte erst mal das Sichtbarkeits-Konzept der Eigenschaften überdacht werden.

mimomamu · 29.05.2011, 13:45

Zitat:

Zitat von ChrisB

Getter und Setter nur zum „Durchreichen“ zu nutzen, ist Unsinn. Wenn keinerlei Kontrolle erforderlich ist - dann wählt man public als Sichtbarkeit, und gut.

Insbesondere wenn sich Überprüfungen später doch als notwendig erweisen, leisten magische Methoden gute Dienste.

Vorher:

PHP-Code:

  class Foo {
  public $Bar;
}

Nachher:

PHP-Code:

  class Foo {
  public function __set($name, $value) {
    switch ($name) {
      case 'Bar':
        if (is_int($value)) $this->bar = $value; else throw new Exception('Integer expected');
        break;
    }
  }
  public function __get($name) {
    switch ($name) {
      case 'Bar':
        return $this->bar;
    }
  }
  private $bar;
}

Beide Klassen sehen von außen betrachtet gleich aus.

axelf · 29.05.2011, 14:02

Hi,

danke erstmal für die Antworten.
Ich hätte hier:

Zitat:

Zitat von mimomamu

PHP-Code:

  class Foo {

  public function __set($name, $value) {

    switch ($name) {

      case 'Bar':

        if (is_int($value)) $this->bar = $value; else throw new Exception('Integer expected');

        break;

    }

  }

  public function __get($name) {

    switch ($name) {

      case 'Bar':

        return $this->bar;

    }

  }

  private $bar;

}

wohl sowas geschrieben:

PHP-Code:

  public function setBar($value){

  if (is_int($value)){

    $this->bar = $value; 

  }else {

    throw new Exception('Integer expected');

  }

}

 
public function getBar(){

    return $this->bar;

}

Aber hier wurde ja schon gesagt, das es immer auf den jeweiligen Fall ankommt.

Eine Frage noch:
Wie kann man denn eigene Getter / Setter und die magischen Methoden mischen? also wenn ich für manche Variablen die magische Methode benutzen will und für andere eigene Methoden? Reicht es dann aus einfach die Sichtbarkeit von public auf private / protected zu setzen?

Chriz · 29.05.2011, 14:07

Faulheit sollte kein Argument sein, schlechten Programmierstil anzuwenden. Du kannst wunderbar mit einem kleinen Codegenerator die Methoden generieren.

ChrisB · 29.05.2011, 14:07

Zitat:

Zitat von axelf

Reicht es dann aus einfach die Sichtbarkeit von public auf private / protected zu setzen?

Nein.

axelf · 29.05.2011, 14:11

Zitat:

Zitat von Chriz

Faulheit sollte kein Argument sein, schlechten Programmierstil anzuwenden. Du kannst wunderbar mit einem kleinen Codegenerator die Methoden generieren.

Hatte ich bisher auch immer so gemacht, bzw. geht das über die IDE (Netbeans). Der generiert aus den Eigenschaften die jeweiligen getter und setter.

php1704 · 29.05.2011, 18:04

Zitat:

Wie kann man denn eigene Getter / Setter und die magischen Methoden mischen?

Du könntest __call verwenden:

PHP-Code:

  class Foo {

    protected $_properties = array(

        'date' => 0,

        'name' => '',

        'active' => false,

        'counter' => 0

    );

    

    public function __call($name, $arguments)

    {

        switch (substr($name, 0, 3)) {

            case "set":

                $name = lcfirst(substr($name, 3));

                if (array_key_exists($name, $this->_properties)) {

                    $this->_properties[$name] = $arguments[0];

                } else {

                    throw new Exception("Unknown property '" . $name . "'");

                }

                break;

                

            case "get":

                $name = lcfirst(substr($name, 3));        

                if (array_key_exists($name, $this->_properties)) {

                    return $this->_properties[$name];

                }

                return null;

                break;

                

            default:

                if(method_exists($this, $name)) {

                    call_user_func_array($this->$name(), $arguments);

                } else {

                    throw new Exception("Unknown method '" . $name . "'");

                }

        }

    }

    

    public function setCounter($value)

    {

        if ($value < 0) {

            $value = 0;

        }

        

        if ($value > 10) {

            $value = 10;

        }

        

        $this->_options['counter'] = $value;

    }

}

 
/*

$bar = new Foo();

$bar->setName('Heinz');

$bar->setCounter(15);

$bar->setHugo(2); // -> Exception "unknown property"

$bar->michtGibtsNicht(); // -> Exception "unknown method"

echo "\n";

echo "Name: " . $bar->getName() . "\n";

echo "Counter: ". $bar->getCounter() . "\n";

*/

Grüße.

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

29.05.2011, 12:51
axelf Benutzer Registriert seit: 29.05.2011 Beiträge: 58 PHP-Kenntnisse: Anfänger	Implementierung vieler Getter bzw. Setter-Methoden Hallo, mitunter enthält eine Klasse ja sehr viele Eigenschaften. Benutzt man die magischen Mehoden __set bzw. __get, muss man nicht für jeder Eigenschaft eine Methode schreiben. Jedoch kann man dann auch die jeweiligen Werte nicht mehr beeinflussen (evtl. möchte ich ja einen Wert in der Setter-Methode prüfen). Bei 20 Eigenschaften hat man jeweils 20 Getter und 20 Setter. Ausserdem soll verhindert werden, das man direkt auf eine Variable von aussen zugreifen kann. Das geht meines Wissens ja sowieso nur mit einem Getter / Setter und nicht mit magischen Methoden Was ist hier ein guter Weg? Für jede Eigenschaft einen Setter / Getter oder gibts auch einen Weg, der die Vorteile von beiden Varianten verbinden kann?


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

29.05.2011, 13:13
axelf Benutzer Registriert seit: 29.05.2011 Beiträge: 58 PHP-Kenntnisse: Anfänger	Hi, also ist es letztlich die persönliche Entscheidung wie man sowas löst? Die switch-Anweisung würde ja mitunter auch sehr groß werden. Was ist denn besserer Stil? Die Verwendung der magischen Methoden oder das alles selbst zu schreiben?

29.05.2011, 13:30
ChrisB Moderator¹ Registriert seit: 28.03.2010 Beiträge: 7.470 PHP-Kenntnisse: Fortgeschritten	Diese magischen Methoden kommen sowieso nur zum Zug, wenn die Sichtbarkeit den direkten Zugriff nicht erlaubt. Getter und Setter nur zum „Durchreichen“ zu nutzen, ist Unsinn. Wenn keinerlei Kontrolle erforderlich ist - dann wählt man public als Sichtbarkeit, und gut. Bei den Eigenschaften, die aber aus gutem Grund private oder protected deklariert sind, wirst du dir aber sicher auch etwas dabei gedacht haben. Dass die dann jeweils eigene explizite Getter/Setter haben, wenn erforderlich, da spricht m.E. nichts gegen. Wenn die magischen Methoden also nur aus „Faulheit“ exzessiv genutzt werden - dann sollte erst mal das Sichtbarkeits-Konzept der Eigenschaften überdacht werden. __________________ RGB is totally confusing - I mean, at least #C0FFEE should be brown, right?

29.05.2011, 14:07
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.269	Faulheit sollte kein Argument sein, schlechten Programmierstil anzuwenden. Du kannst wunderbar mit einem kleinen Codegenerator die Methoden generieren. __________________ "Nuschel ich?" - "Was?"