oop - funktionsaufruf + argumente?

[greener]

hallo, ich beschäftige mich gerade mit meinem ersten oop-projekt, und habe ein problem (...)

ich habe eine beliebige klasse (CLASS_NAME) mit einer funktion drinne (funktion($arg1,$arg2,$arg3))...

ich starte den konstruktor mittels

PHP-Code:

$var = new CLASS_NAME;
$funktions_aufruf = $var->funktion($arg1,$arg2,$arg3); 


jetzt bekomme ich ständig die fehlermeldung, Warning: Missing Argument... usw.

die argumente sind aber definitiv angegeben...

was muss ich beachten, damit das ganze wie gewünscht funkt?

gruß + dank!

[rechtschreibfan]

wenn dein konstruktor ein argument erwartet, darfst du ihn nicht so aufrufen

nehmen wir also an:

PHP-Code:

class myClass {
  
  function myClass() {
    // 
  }
   function myFunction($arg1, $arg2) {
    //
  }
}

// gültige aufrufe:
$cla = new myClass();
$cla2 = new myClass;

$cla -> myFunction("hallo","welt"); 


sieht aber der konstruktor so aus:

PHP-Code:


class myClass2 {
 
  function myClass2($arg) {

  }
  function myFunction($arg1,$arg2,$arg3) {

  }

}

// ungültig:
$my = new myClass2;

// gültig:
$my = new myClass2("");
// gültig:
$my = new myClass2($hjell);
$my -> myFunction($one,$two,$three); 


solltest dir vielleicht gleich angewöhnen, eine klasse immer mit klammern zu initialisieren, also nicht $my = new Class; sondern $my = new Class(); -- auch wenn die klammern manchmal nicht nötig sind, ist es schönerer und eindeutigerer code.

[greener]

Zitat von rechtschreibfan

PHP-Code:

// gültig:
$my = new myClass2("");
// gültig:
$my = new myClass2($hjell);
$my -> myFunction($one,$two,$three); 


hm, erschließt sich mir noch nicht ganz:

warum muss ich die klasse mit klammern aufrufen? [new myClass2("")]
und
was bewirkt: $my = new myClass2($hjell);

?

vielen dank aber schon mal


edit:tatsächlich: wenn ich die klasse per new CLASS_NAME('','',''); aufrufe funktioniert es, aber warum es so sein muss, leuchtet mir nicht ein?!

[rechtschreibfan]

naja... es geht nur darum: der konstruktor einer klasse wird, so er denn existiert (einfach eine funktion in der klasse, die den selben namen wie die klasse selbst trägt), auf jeden fall aufgerufen, wenn du die klasse mit "new" initialisierst; und zwar immer bevor alles andere passiert.

wenn also dein konstruktor argumente erwartet, du die klasse aber ohne argumente aufrufst, erzeugst du ein warning.

vielleicht solltest du deine klasse mal hier posten - was intern passiert ist jetzt auch nicht wichtig, das könntest du zu zwecken der lesbarkeit rauslassen. nur die funktions-köpfe sind wichtig. dann können wir mal anschauen, was passiert und wo der fehler liegen könnte.

[greener]

ok:

PHP-Code:

class DB_ACTION {
    function db_connect($server,$user,$password,$database)
    {
        //
    }
    function db_action($action,$table,$extra,$array_type,$addkey=0)
    {
        //
    }
} 


mein aufruf:

PHP-Code:

        // so funkts:
        $db = new DB_ACTION('','','','');
        // so nicht:
        $db = new DB_ACTION;
        $connection = $db->db_connect('localhost','root','','zeiterfassung');

        //$act = new DB_ACTION;
        $result = $db->db_action('INSERT','work_entries',$tmp_arr,''); 


die db-funktionen werden in einer weiteren klasse aufgerufen, was aber keine rolle spielen sollte, oder?

[rechtschreibfan]

na da haben wir's ja schon.

die klasse db_action hat einen konstruktor, der db_action($action,$table,$extra,$array_type,$addke y=0) heißt und einige argumente haben will ... hier ist es wohl etwas verwirrend, dass die klasse großgeschrieben ist und der konstruktor 1. nicht an der ersten stelle steht und 2. klein geschrieben ist. php wird wohl den groß- kleinschreibefehler ignorieren (was es eigentlich nicht sollte ) und bei initialisierung mit "new" versuchen, die funktion db_action aufzurufen. und weil die mehrere argumente erwartet, bekommst du ein warning.


tipp:

1. achte peinlichst genau auf die groß- und kleinschreibung. das spart dir viele bugs
2. die mehrheit der oop-leute haben sich eine bestimmte art angewöhnt, klassennamen auszuwählen. die form lautet:
"DiesIstEinGuterKlassenName" und nicht "dies_ist_ein_guter_klassenname" oder "DIES_IST_EIN_GUTER_KLASSENNAME".

großbuchstaben mit underscore ist für "konstanten" reserviert.

richtig wäre also eher "DbAction", und dein konstruktor sollte am besten die erste funktion innerhalb der klasse sein, und ebenfalls "DbAction" heißen.

alle anderen methoden in der klasse sollten mit kleinbuchstaben beginnen, und dann diese groß-klein-schreibung beherzigen. underscores "_" in funktionsnamen sind für echte "funktionen" reserviert, die nicht an klassen gebunden sind, z.b. integrierte funktionen in php oder selbst geschriebene funktionen.

mit diesen tipps erhöhst du die lesbarkeit deines codes enorm ... und andere können dir leichter und schneller helfen.

[greener]

hm, das leuchtet ein, ich danke vielmalös und experimentiere mal weiter

[elias]

das problem liegt daran das du dir bisher nicht bewusst warst das eine funktion innerhalb der klasse die genau so heißt wie deine klasse (case ist egal) als konstruktor gild. du musst lediglich die zweite funktion umbenennen und es wird erstmal so laufen wie gewünscht

[greener]

schon geschehen bzw hab die klasse umbenannt

[Nightflyer]

Gibts irgendwo ein gutes Tut für OOP in PHP? Ich hab irgendwie das Gefühl da was zu verpassen

[elias]

war nur zu langsam @greener

@nightflyer
schreibe gerade an einem PHP OOP tutorial, wennde bischen geduldig bist kannste mir vielleicht sagen obs brauchbar ist

[Nightflyer]

Zitat von binary blade

@nightflyer
schreibe gerade an einem PHP OOP tutorial, wennde bischen geduldig bist kannste mir vielleicht sagen obs brauchbar ist

Unbedingt, wenn ich's dann schnalle ist's gut, sonst nicht