Parameterübergabe

[HStev]

Was haltet ihr davon noch auf die Vorteile von einer Array- bzw. Objekt-Übergabe einzugehen?
Lesbarkeit und Verständnis des Codes werden ja durch Konstrukte wie

<?php
myFunc(array("color"=>"green","some"=>"thing"))

erhöht.
Wenn man nach ein paar Monaten noch mal was ändern möchte oder fremden Code bekommt,
muss man nicht ständig in der Dokumentation suchen.

Im prinzip nicht schlecht weil man so auch in PHP leichter überladen kann

[nikosch]

Im prinzip nicht schlecht weil man so auch in PHP leichter überladen kann

was dann aber eher in einen gleichnamigen artikel gehörte.

Wenn man nach ein paar Monaten noch mal was ändern möchte oder fremden Code bekommt,
muss man nicht ständig in der Dokumentation suchen.

Gerade doch, oder? Am Funktionsparameter erkennt man die Schnittstelle ja nicht mehr. In Objekt-Init Methoden benutze ich das auch öfter, aber generell ist das ein zweischneidiger Tipp.

[HStev]

In Objekt-Init Methoden benutze ich das auch öfter, aber generell ist das ein zweischneidiger Tipp.

leider hat vor- und nachteile

[phpdummi]

Gerade durch die assoziativen Arrays erkenne ich doch jeden Parameter der übergeben wird.
Wieso erkennt man dann die Schnittstelle nicht mehr?

[nikosch]

Gerade durch die assoziativen Arrays erkenne ich doch jeden Parameter der übergeben wird.

Ja, an der Stelle des Aufrufs. Und nur bei vollständiger Initialisierung. Verwendest Du nur eine Teilmenge der möglichken Parameterschlüssel (=Überladen), dann sind Dir auch nur diese Parameter bekannt.

vgl.

function listFolder ($sFolder , $aExtension = array () , $bSubFolders = true , $bKeyAssoc = true)
  {
  }

...

$aStruct = listFolder ('./'); 

vs.

function listFolder ($aSettings)
  {
  }

...

$aStruct = listFolder (array (
                             'folder' => './'
                             )); 

[phpdummi]

Ok nun hab ich auch kapiert =)

[nikosch]

Bei guter Dokumenatation ist die Diskussion auch eher akademisch. Aber letztendlich ists das selbe Argument, das auch immer bei getter/setter Methoden angeführt wird.

[HStev]

Ich hoffe ja darauf das überladen in PHP6 endlich ohne __call mögich ist ... ein wichtiger Schritt dahin sind ja schon mal die TypeHints im Prinzip wäre ich auch dafür wenn es endlich feste Datentypen geben würde und man diese erst inizieren muss wie in C/C++

also praktich:

integer $i;
float $f = 1.5;

class Foo {
   public function bar(integer $i)
   {
      echo "method bar with int param";
   }
   
   public function bar(float $f)
   {
       echo "method bar with float param";
   }
} 

geht derzeit ja leider nur per __call

[Manko10]

Nur um Missverständnissen vorzubeugen: Typehinting ist mit primitiven Datentypen nicht möglich.

[agrajag]

Man müsste eben in dem Artikel auf die Vor- und Nachteile eingehen und aufzeigen wo es besonders sinnvoll ist.

Parameter die zwingend notwendig sind und nicht durch einen "sensible default" ersetzt werden können: normaler Parameter.
Für Nikoschs listFolder-Beispiel also zum Beispiel:

function listFolder($folder, $options = array())
{
    $default_options = array(
        'extensions' => array('*'),
        'recursive' => true
    );
    $options = array_merge($default_options, $options);
    //do stuff
}
  
listFolder('./', array('recursive' => false)); 

Ich zieh jedenalls sowas im allgemeinen einer ellenlangen Parameterliste mit optionalen Parametern vor. Die Optionen müssen natürlich schön dokumentiert sein...

[nikosch]

Ich benutze für sowas im allgemeinen binäre Flags, wie die PHP System-Funktionen.

[Manko10]

Wenn mit Strings verschiedene Settings angesprochen werden und sich dieses Schema in mehreren Methoden wiederholt, kann man auch entsprechende Klassenkonstanten definieren, um Tippfehler zu vermeiden.

[nikosch]

So sähe das bei mir aus:

define ('FOLDER_LIST_SUBFOLDERS' , 1);
define ('FOLDER_LIST_ASSOC' , 2);

function listFolder ($sFolder , $aExtension = array () , $iFlags = 0)
  {
  }

..
$aStruct = listFolder ('./' , null , FOLDER_LIST_ASSOC); 

[Manko10]

Und so bei mir mit Klasse:

Class Folder
{
    const LIST_SUBFOLDERS = 1;
    const LIST_ASSOC      = 2;
    
    public function listFolder($sFolder, $aExtension = array(), $iFlags = 0)
    {
    }
}
...
$folder = new Folder('.');
$struct = $folder->listFolder('./' , null , Folder::LIST_ASSOC); 

[nikosch]

In so'nem Fall selbstredend. Aber, Mist, ich benutze viel zu selten Klassenkonstanten.