Mehrfachvererbung, trotzdem DRY? - Seite 2

robo47 · 15.02.2010, 01:23

Get es hier nur um ganz einfache setter und getter ? die alle im grund gleich sind ? Oder enthalten die auch unterschiedliche Validierung/Filterung/Castings, etc ?

Also sehen die alle so aus:

PHP-Code:

  public function setFoo($foo) {

$this->_foo = $foo;

return $this;

}

public function getFoo()

{

return $this->_foo;

}

Wenn ja, verstehe ich nicht warum du nicht __call mit einem generischen setter nutzt, du sparst dir einen haufen arbeit, test-aufwand, viel weniger code und der performance-verlust dürfte recht gering sein.

Einfach intern in der Klasse ein array, ein generischer setter:

PHP-Code:

  class Foo {

protected $_existingProperties = array('foo' => 'foo', 'baa' => 'baa');

protected $_properties = array();

protected function _genericSetter($name, $value)

{

    $this->_properties[$name] = $value;

    return $this;

}

 
public function __call($method, $params)

{

    /*

    falls $method mit set anfängt schauen ob das was danach kommt in $this->_existingProperties vorhanden ist (wenn du indizes benutzt kannst du direkt isset nutzen, ansonsten muss die langsamere variante mit in_array() herhalten) 

und dann via return call_user_func_array(array($this, '_genericSetter'), $params); ausführen

*/

// ... get...

 
// ansonsten exception schmeißen.

}

}

Dazu dann halt noch das gleiche für den getter.

Und für die Code-Completion kannst du dann mit @property/@method arbeiten und keiner merkts solange er nicht in den quellcode schaut

Für DRY ist das definitiv ein gutes Stück besser.

raw-venture-ad · 16.02.2010, 22:29

Alle Paragraphen sind bei meiner Klasse (PHPWord) vom Typ PHPWord_Section_Text und haben eine eigenschaft style die eine Instanz der Klasse PHPWord_Style_Text beinhaltet. Bei Listen und Tabellen nutze ich dann die gleiche Klasse.

Ich habe es bei PHPWord dann so gemacht, dass ich einem Paragraph entweder ein Array mit Style-Attributen übergebe...

PHP-Code:

  $section->addText('Lorem ipsum ...', array('color'=>'006699'));

...oder einen String der auf eine Style Eigenschaft in der styles.xml zeigt:

PHP-Code:

  $PHPWord->addTextStyle('Factory_CICD', array('color'=>'006699', 'size'=>18, 'spaceBefore'=>60));
$section->addText('Lorem ipsum...', 'Factory_CICD');

Da die Eigenschaften alle private sind, kann man alternativ auch über getter/setter die entsprechenden Werte setzen:

PHP-Code:

  $paragraph = $section->addText('Lorem ipsum...');
$paragraph->setName('Arial');
$paragraph->setBold();
$paragraph->setColor('006699');

Chriz · 16.02.2010, 23:41

Du "musst" dabei aber unterscheiden zwischen Formatvorlagen und Inlinestyles. Ich kann ja einem Paragraphen eine Formatvorlage zuweisen, die den Text fett macht, dieses "fett" dann aber per Inline wieder wegnehmen.

Ala:
<span class="bold_text" style="font-weight:normal">..</span>

So abwegig ist das auch nicht mal. Deshalb will ich das schon unterscheiden.

raw-venture-ad · 17.02.2010, 09:13

Ja, genau !

Diese Unterscheidung mache ich auch indem der Konstruktor von PHPWord_Section_Text prüft ob ein style-array (führt zur inline Formatierung) oder ein Formatvorlagen-String übergeben wurde. Alle Formatvorlagen stehen in einer statischen Klasse und können daher auch auf ihre Existenz geprüft werden.

Dem entsprechend wird dann entweder ein <w: pStyle id="MeinFormat" /> oder ein <w:rPr>...</w:rPr> geschrieben.

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

15.02.2010, 01:23
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.798 PHP-Kenntnisse: Fortgeschritten	Get es hier nur um ganz einfache setter und getter ? die alle im grund gleich sind ? Oder enthalten die auch unterschiedliche Validierung/Filterung/Castings, etc ? Also sehen die alle so aus: PHP-Code: `public function setFoo($foo) { $this->_foo = $foo; return $this; } public function getFoo() { return $this->_foo; }` Wenn ja, verstehe ich nicht warum du nicht __call mit einem generischen setter nutzt, du sparst dir einen haufen arbeit, test-aufwand, viel weniger code und der performance-verlust dürfte recht gering sein. Einfach intern in der Klasse ein array, ein generischer setter: PHP-Code: class Foo { protected $_existingProperties = array('foo' => 'foo', 'baa' => 'baa'); protected $_properties = array(); protected function _genericSetter($name, $value) { $this->_properties[$name] = $value; return $this; } public function __call($method, $params) { /* falls $method mit set anfängt schauen ob das was danach kommt in $this->_existingProperties vorhanden ist (wenn du indizes benutzt kannst du direkt isset nutzen, ansonsten muss die langsamere variante mit in_array() herhalten) und dann via return call_user_func_array(array($this, '_genericSetter'), $params); ausführen */ // ... get... // ansonsten exception schmeißen. } } Dazu dann halt noch das gleiche für den getter. Und für die Code-Completion kannst du dann mit @property/@method arbeiten und keiner merkts solange er nicht in den quellcode schaut Für DRY ist das definitiv ein gutes Stück besser. __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework


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

16.02.2010, 22:29
raw-venture-ad Neuer Benutzer Registriert seit: 29.07.2009 Beiträge: 17 PHP-Kenntnisse: Fortgeschritten	Alle Paragraphen sind bei meiner Klasse (PHPWord) vom Typ PHPWord_Section_Text und haben eine eigenschaft style die eine Instanz der Klasse PHPWord_Style_Text beinhaltet. Bei Listen und Tabellen nutze ich dann die gleiche Klasse. Ich habe es bei PHPWord dann so gemacht, dass ich einem Paragraph entweder ein Array mit Style-Attributen übergebe... PHP-Code: `$section->addText('Lorem ipsum ...', array('color'=>'006699'));` ...oder einen String der auf eine Style Eigenschaft in der styles.xml zeigt: PHP-Code: `$PHPWord->addTextStyle('Factory_CICD', array('color'=>'006699', 'size'=>18, 'spaceBefore'=>60)); $section->addText('Lorem ipsum...', 'Factory_CICD');` Da die Eigenschaften alle private sind, kann man alternativ auch über getter/setter die entsprechenden Werte setzen: PHP-Code: `$paragraph = $section->addText('Lorem ipsum...'); $paragraph->setName('Arial'); $paragraph->setBold(); $paragraph->setColor('006699');`

16.02.2010, 23:41
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.069	Du "musst" dabei aber unterscheiden zwischen Formatvorlagen und Inlinestyles. Ich kann ja einem Paragraphen eine Formatvorlage zuweisen, die den Text fett macht, dieses "fett" dann aber per Inline wieder wegnehmen. Ala: <span class="bold_text" style="font-weight:normal">..</span> So abwegig ist das auch nicht mal. Deshalb will ich das schon unterscheiden. __________________ "Nuschel ich?" - "Was?"

17.02.2010, 09:13
raw-venture-ad Neuer Benutzer Registriert seit: 29.07.2009 Beiträge: 17 PHP-Kenntnisse: Fortgeschritten	Ja, genau ! Diese Unterscheidung mache ich auch indem der Konstruktor von PHPWord_Section_Text prüft ob ein style-array (führt zur inline Formatierung) oder ein Formatvorlagen-String übergeben wurde. Alle Formatvorlagen stehen in einer statischen Klasse und können daher auch auf ihre Existenz geprüft werden. Dem entsprechend wird dann entweder ein <w: pStyle id="MeinFormat" /> oder ein <w:rPr>...</w:rPr> geschrieben.