Method chaining

nikosch · 09.09.2008, 20:56

Diskussionsbeitrag zum Wiki Eintrag: Method chaining.

Die Diskussionsplattform des PHP.de Wiki wurde ins Forum integriert. Durch Klicken des Buttons "Antwort" kannst du an diesem Thema teilnehmen.

nikosch · 09.09.2008, 21:01

Hallo Flo. Cool, endlich mal ein neuer Autor im Wiki!
Hab bisher nur überflogen, habe aber ein paar Anmerkungen.

1/ pochenden Zeigefingers verweise ich auf unsere niegelnagelneueln Coding Standards

2/ das Argument "sondern ergibt auch einen logischen Sinn beim lesen vom Code" sehe ich durch das verwendete Codebeispiel nicht getragen:

Zitat:

PHP-Code:

  $person->setName('Flo')->setAge(21)->displayPerson();

Etwas wie

PHP-Code:

  $oDB->connect()->selectDB('bla')->query ('xyz');

wäre vielaussagekräftiger.

3/ Ich würde auf jeden Fall Negativargumente mit in den Artikel nehmen:
- es wird ständig eine Objekt herumgereicht.
- Fehlerbehandlung ist schwierig, da die Methoden nicht vorzeitig abgebrochen werden können sondern immer $this zurückgeben müssen

robo47 · 10.09.2008, 00:14

Zitat:

Zitat von nikosch

3/ Ich würde auf jeden Fall Negativargumente mit in den Artikel nehmen:
- es wird ständig eine Objekt herumgereicht.
- Fehlerbehandlung ist schwierig, da die Methoden nicht vorzeitig abgebrochen werden können sondern immer $this zurückgeben müssen

Was macht die Fehlerbehandlung denn schwerer ?
Ich denke die Fehlerbehandelung erfolgt, wenn man solche Methoden einsetzt eh über Exceptions und wo ist da dann der Unterschied ob ich

PHP-Code:

  $obj = new MyClass();

try {

$obj->setX(1);

$obj->setY(2);

$obj->setZ(3);

} catch (Exception $e) {

// handle Error

}

oder

PHP-Code:

  $obj = new MyClass();

try {

$obj->setX(1)

      ->setY(2)

      ->setZ(3);

} catch (Exception $e) {

// handle Error

}

benutzte?

nikosch · 10.09.2008, 01:01

Ich rede von Fehlern, die innerhalb von beteiligten Methoden abgefangen werden müssen.

Flor1an · 10.09.2008, 11:27

Ja Codingstandards hab ich schon gelesen, Text war allerdings schon fertig ... werd ihn nochmal überarbeiten wollte aber einfach mal ausprobieren wie ich den Text in die Wiki bekomme

Manko10 · 10.09.2008, 20:58

Kleine Anmerkung: vielleicht könntest du die PHP-Tags bei Code-Ausschnitten ganz weglassen. Meines Erachtens sollten die nur bei Komplett-Codes gesetzt werden. Wird nur eine einzelne Funktion oder Zeile herausgepickt, wirken sie eher störend und lenken vom eigentlich Wichtigen ab.

nikosch · 10.09.2008, 21:13

Guter Punkt. Den werden wir mal noch in die Konventionen oder Coding Standards aufnehmen.

Flor1an · 11.09.2008, 09:58

Ja das würde Sinn machen. Ich entferne mal alle PHP Tags. Ich war es nur gewohnt die öffnenden Tags zu schreiben da in irgend einem Board der [ php] BBTag nur dann den Code highlighted wenn ein <?php am Anfang steht.

Manko10 · 11.09.2008, 18:49

Das ist aber nur noch in einigen Boards so. vBulletin benutzt zwar auch highlight_string(), aber setzt bei Bedarf noch ein <?php davor und entfernt es anschließend wieder.
Beim Highlighter auf meiner Seite habe ich es ebenso gemacht.

nikosch · 05.05.2010, 16:00

2 neue Fragen:

- displayPerson() sollte besser den String zurückliefern, oder? Vielleicht sollten wir im Wiki eine gewisse Konsistenz an Positivpatterns (EVA, immer valideren, auch Trivialcode) einhalten.

- Das Validierungsbeispiel würde ich so nicht umsetzen, weil i.A. ein Fehler für die Negativvalidierung ausreicht. Also entweder man sammelt auch Infos zur Fehlersituation mit oder bricht Folgefehler schon vor der Prüfung mit return $this ab.

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
Sagt dies jemand was? Invalid method in request QUIT	tomx992	PHP-Fortgeschrittene	7	28.02.2006 06:27
method post frage	test022	PHP Tipps 2007	10	23.12.2005 13:02
Training von neuronalen Netzen		PHP Tipps 2005	5	28.09.2005 15:12
[Erledigt] Problem: The requested method POST is not allowed for the		Server, Hosting und Workstations	17	31.05.2005 15:15
requested method POST is not allowed for the URL /index.html		PHP-Fortgeschrittene	10	30.05.2005 19:36
" und \ mit method post ohne maskierung übergeben		PHP Tipps 2005	9	21.04.2005 14:55
Form method post Problem	test022	PHP Tipps 2005	15	05.01.2005 20:20
PostgreSQL	Orolhawion	Server, Hosting und Workstations	2	14.12.2004 21:58
[Erledigt] SOAP/Client.php: PHP Fatal error: Method SOAP_Client::__call		PHP-Fortgeschrittene	6	25.11.2004 09:48

09.09.2008, 20:56
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 34.241 PHP-Kenntnisse: Fortgeschritten	Method chaining Diskussionsbeitrag zum Wiki Eintrag: Method chaining. Die Diskussionsplattform des PHP.de Wiki wurde ins Forum integriert. Durch Klicken des Buttons "Antwort" kannst du an diesem Thema teilnehmen.


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

10.09.2008, 01:01
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 34.241 PHP-Kenntnisse: Fortgeschritten	Ich rede von Fehlern, die innerhalb von beteiligten Methoden abgefangen werden müssen. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

10.09.2008, 11:27
Flor1an ¯\_(ツ)_/¯ Registriert seit: 18.06.2008 Beiträge: 8.814 PHP-Kenntnisse: Fortgeschritten	Ja Codingstandards hab ich schon gelesen, Text war allerdings schon fertig ... werd ihn nochmal überarbeiten wollte aber einfach mal ausprobieren wie ich den Text in die Wiki bekomme __________________ ▇█▓▒░◕‿‿◕░▒▓█▇

10.09.2008, 20:58
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.425 PHP-Kenntnisse: Fortgeschritten	Kleine Anmerkung: vielleicht könntest du die PHP-Tags bei Code-Ausschnitten ganz weglassen. Meines Erachtens sollten die nur bei Komplett-Codes gesetzt werden. Wird nur eine einzelne Funktion oder Zeile herausgepickt, wirken sie eher störend und lenken vom eigentlich Wichtigen ab. __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

10.09.2008, 21:13
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 34.241 PHP-Kenntnisse: Fortgeschritten	Guter Punkt. Den werden wir mal noch in die Konventionen oder Coding Standards aufnehmen. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

11.09.2008, 09:58
Flor1an ¯\_(ツ)_/¯ Registriert seit: 18.06.2008 Beiträge: 8.814 PHP-Kenntnisse: Fortgeschritten	Ja das würde Sinn machen. Ich entferne mal alle PHP Tags. Ich war es nur gewohnt die öffnenden Tags zu schreiben da in irgend einem Board der [ php] BBTag nur dann den Code highlighted wenn ein <?php am Anfang steht. __________________ ▇█▓▒░◕‿‿◕░▒▓█▇

11.09.2008, 18:49
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.425 PHP-Kenntnisse: Fortgeschritten	Das ist aber nur noch in einigen Boards so. vBulletin benutzt zwar auch highlight_string(), aber setzt bei Bedarf noch ein <?php davor und entfernt es anschließend wieder. Beim Highlighter auf meiner Seite habe ich es ebenso gemacht. __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

05.05.2010, 16:00
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 34.241 PHP-Kenntnisse: Fortgeschritten	2 neue Fragen: - displayPerson() sollte besser den String zurückliefern, oder? Vielleicht sollten wir im Wiki eine gewisse Konsistenz an Positivpatterns (EVA, immer valideren, auch Trivialcode) einhalten. - Das Validierungsbeispiel würde ich so nicht umsetzen, weil i.A. ein Fehler für die Negativvalidierung ausreicht. Also entweder man sammelt auch Infos zur Fehlersituation mit oder bricht Folgefehler schon vor der Prüfung mit return $this ab. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --