[Erledigt] call_user_func_array und bind_param Problem

Ballamann · 25.05.2010, 20:28

Hey
Ich möchte sämtliche MySQLi-Anweisungen in eine einzige Klasse packen. Das klappt soweit ganz gut, bei den Prepared-Statements allerdings stoße ich auf ein Problem. Folgende Methode soll die Anweisung bind_param() übernehmen. Dafür generiere ich ein Array mit allen übergebenen Parametern, welche ich dann an die bind_param() weiterleite, so dass man dann später bequem mit $objekt->set_parameter($bind, $value) die "Variablen" befüllen kann.
Der Code dazu sieht so aus:

PHP-Code:

  function bind()
    {
        //Minimum 2 Argumente werden benötigt
        $arg_num = func_num_args();
        if ($arg_num < 1) return false;
        
        $datastring = func_get_arg(0);
        $i = 1; //0 wird übersprungen da im $datastring
        while ($i < $arg_num)
        {
            $this->params[func_get_arg($i)] = "test"; //Das Array mit den "gebindeten" Variablen
            $i++;
        }
        
        //$datastring an das array anfügen, zwecks funktionsaufruf
        array_unshift($this->params, &$datastring);
        //bind_param aufrufen und Parameter übergeben 
                if (!call_user_func_array(array($this->prep_obj, "bind_param"), &$this->params))
            $this->exc("Parameter konnten nicht gebunden werden!");
                }

Die Variablen werden dann so befüllt:

PHP-Code:

  function set_parameter($parameter, $value)
    {
        $this->params[$parameter] = &$value;
        
    }

Das Array wird auf diese Art richtig angepasst (das ergab ein die() ), wenn ich dann allerdings ein execute() aufrufe wird bei einem INSERT-Kommando nicht das eingetragen was ich über set_parameter() festgelegt habe, sondern das was beim initialisieren des arrays eingetragen wurde in die datenbank geschrieben (in dem fall hier also immer "test").

Beim Nachforschen bin ich auf einen Hinweis auf php.net gestoßen:

Zitat:

Hinweis: Care must be taken when using mysqli_stmt_bind_param() in conjunction with call_user_func_array(). Note that mysqli_stmt_bind_param() requires parameters to be passed by reference, whereas call_user_func_array() can accept as a parameter a list of variables that can represent references or values.

Aber das hilft mir nicht weiter, meiner Meinung nach werden alle Werte korrekt by reference übergeben?!

Danke für Hilfe

nikosch · 25.05.2010, 21:43

Versteh nicht, was das für einen SInn haben soll?

Ballamann · 25.05.2010, 22:09

Das ich alles was mit Datenbanken zu tun hat in einer Klasse habe. Fehlerüberprüfungen etc. kann ich mir damit schon vereinfachen.
Wie würdest du es denn machen?

Und selbst wenn das Unsinn sein sollte - warum funktioniert es nicht?

nikosch · 25.05.2010, 22:16

mysqli ist bereits eine Klasse mit allen Funktionen in einem Objekt.

Ich verstehe schon allein Deinen Plan mit set_parameter nicht, weil da ja das Type-Binding fehlt.

Ballamann · 25.05.2010, 22:38

Hmm es wäre vielleicht einfacher gewesen einfach von der Klasse zu erben und zu erweitern oder was meinst du?

Und zu set_parameter:
mysqli->bind_param($typ, $variable1, $variable2,...) erwartet als ersten parameter ja den typ für die variablen die ihm danach übergeben werden. Also ist mir im Moment ja nicht bekannt welcher Type erwartet wird oder?! Und das ganze muss überhaupt erst so kompliziert werden weil ich ja nicht weiß wieviele Parameter an das bind_param übergeben werden müssen, deswegen der Umweg über ein Array das alle "Variablen" enthält die an die methode übergeben werden sollen, und die Werte dieses Arrays ändert man über set_parameter().

nikosch · 25.05.2010, 22:44

Aus genau diesem Grund ist es zwecklos, einen Wrapper zu schreiben.

Zitat:

Also ist mir im Moment ja nicht bekannt welcher Type erwartet wird oder?!

Wem, wenn nicht Dir! Gerade die Typvorgabe ist wichtig, da ja dadurch die Sicherheit erreicht wird. Wenn Du die Parameter einzeln binden willst, kannst DU das ja machen, die Typvorgabe musst Du trotzdem angeben, sonst ist Dein gesamtes Konzept nichts wert.
Allerdings frage ich mich auch ernsthaft, welchen Wert ein Objekt mit dynamischen Parametern hat, wenn Du sowieso irgendwo die zugrundeliegende Query statisch angeben musst.

Ballamann · 25.05.2010, 23:17

naja Ziel des Ganzen sollte es eigentlich nur sein den ganzen Datenbankteil auslagern zu können.
Für einen Profi mag das vielleicht sinnlos erscheinen, mir in ein, zwei monaten hoffentlich auch, aber im moment hab ich vorallemdingen nen relativ großen lerneffekt dadurch!

Deswegen, egal ob das sinnvoll sein sollt eoder nicht, aber ich würds schon gern zum Laufen bringen.

nikosch · 25.05.2010, 23:19

Dann beschreibe die Anforderung. Mir ist immer noch nicht klar, wie der Aufruf mit Deinem neuen Objekt aussehen soll.

Ballamann · 25.05.2010, 23:33

PHP-Code:

  $db = new db;

$db->prepare_query("INSERT INTO user (Feld1, Feld2, Feld3, Feld4) VALUES (?, ?, ?, ?)");

$db->bind("ssss", "var1", "var2", "var3", "var4");

$db->set_parameter("var1", "Testnickname");

$db->set_parameter("var2", "hat keins");

$db->set_parameter("var3", "100");

$db->set_parameter("var4", "ÄttDot");

$db->execute();

$ergebnis = $db->fetch_result()

So funktioniert der Aufruf meines neuen Objekts.
In der Funktion bind() werden alle übergebenen Variablen in ein assoziatives Array geschrieben (als Offsets) und ihnen wird probehalber der Wert "test" zugewiesen. Über set_parameter kann man dann die einzelnen Werte des Arrays ändern, man gibt den Namen und den Wert an. Das funktioniert soweit auch, wenn ich mir danach das Array ausgebe sind die vorher initialisierten Felder geändert. Wenn ich dann aber execute() aufrufe werden diese Änderungen ignoriert und nur der Wert "test" wird in die Datenbank geschrieben - also nur der Wert der bei der Initialisierung zugewiesen wurde. Da liegt mein Problem.

ChrisB · 25.05.2010, 23:37

Zitat:

Zitat von Ballamann

So funktioniert der Aufruf meines neuen Objekts.

Das Erzeugen des neuen DB-Objektes, Vorbereiten der Query, Binden der Variablen und Ausführen - all das läuft ganz genau so, wenn du wirklich das native MySQLi nimmst.

Im Grunde hast du nur die Funktionen „umbenannt“ - ausführen musst du sie nach wie vor alle (und bei deinem Code noch ein paar zusätzliche).

Gewinn, und damit Sinn des ganzen Unterfangens - absolut keiner.
Ausser du betrachtest es als Gewinn, dass dein Code nicht funktioniert. Nicht lauffähige Scripte kann man sich aber auch mit weniger Aufwand basteln.

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
Problem mit Ajax - Session Problem	halskrause	PHP Tipps 2009	10	12.01.2010 02:27
[Erledigt] Problem mit Mehrsprachigkeit	Sniper_Kotaro	PHP Tipps 2009	2	16.12.2009 11:47
Problem beim Download Management	mistermint	PHP Tipps 2009	8	24.10.2009 11:34
Umlaut Problem	nokieone	PHP Tipps 2009	7	16.10.2009 19:30
Problem mit Basedir in PhP	cmuch05	Datenbanken	9	08.01.2009 14:17
[Erledigt] Problem: PHP Selbe Datei mehr als 1 Mal	soulan	PHP Tipps 2008	13	28.11.2008 22:51
Problem Upload mehrere Dateien und schreiben in DB	Lapje	PHP Tipps 2008	1	22.10.2008 13:12
utf8 Problem (Daten aus DB auslesen)	go1denboy	PHP Tipps 2008	1	23.09.2008 12:19
Problem mit swfobjects	R4v3r	JavaScript, Ajax und mehr	8	21.09.2008 14:17
[Erledigt] Galerie Problem...	coraplanet	PHP Tipps 2008	4	06.06.2008 13:42
datensätze defekt oder problem mit dem einlesen?	Ministry	Datenbanken	4	06.07.2006 18:42
problem!!!		PHP Tipps 2006	6	08.02.2006 11:06
Smarty und PHP-Skript Problem		PHP Tipps 2004-2	2	03.12.2004 22:27
Problem mit alter JavaScript-Funktion	woods	PHP Tipps 2004	1	13.08.2004 13:34


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

25.05.2010, 21:43
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.990 PHP-Kenntnisse: Fortgeschritten	Versteh nicht, was das für einen SInn haben soll? __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

25.05.2010, 22:09
Ballamann Neuer Benutzer Registriert seit: 21.01.2010 Beiträge: 23 PHP-Kenntnisse: Anfänger	Das ich alles was mit Datenbanken zu tun hat in einer Klasse habe. Fehlerüberprüfungen etc. kann ich mir damit schon vereinfachen. Wie würdest du es denn machen? Und selbst wenn das Unsinn sein sollte - warum funktioniert es nicht?

25.05.2010, 22:16
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.990 PHP-Kenntnisse: Fortgeschritten	mysqli ist bereits eine Klasse mit allen Funktionen in einem Objekt. Ich verstehe schon allein Deinen Plan mit set_parameter nicht, weil da ja das Type-Binding fehlt. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

25.05.2010, 22:38
Ballamann Neuer Benutzer Registriert seit: 21.01.2010 Beiträge: 23 PHP-Kenntnisse: Anfänger	Hmm es wäre vielleicht einfacher gewesen einfach von der Klasse zu erben und zu erweitern oder was meinst du? Und zu set_parameter: mysqli->bind_param($typ, $variable1, $variable2,...) erwartet als ersten parameter ja den typ für die variablen die ihm danach übergeben werden. Also ist mir im Moment ja nicht bekannt welcher Type erwartet wird oder?! Und das ganze muss überhaupt erst so kompliziert werden weil ich ja nicht weiß wieviele Parameter an das bind_param übergeben werden müssen, deswegen der Umweg über ein Array das alle "Variablen" enthält die an die methode übergeben werden sollen, und die Werte dieses Arrays ändert man über set_parameter().

25.05.2010, 23:17
Ballamann Neuer Benutzer Registriert seit: 21.01.2010 Beiträge: 23 PHP-Kenntnisse: Anfänger	naja Ziel des Ganzen sollte es eigentlich nur sein den ganzen Datenbankteil auslagern zu können. Für einen Profi mag das vielleicht sinnlos erscheinen, mir in ein, zwei monaten hoffentlich auch, aber im moment hab ich vorallemdingen nen relativ großen lerneffekt dadurch! Deswegen, egal ob das sinnvoll sein sollt eoder nicht, aber ich würds schon gern zum Laufen bringen.

25.05.2010, 23:19
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.990 PHP-Kenntnisse: Fortgeschritten	Dann beschreibe die Anforderung. Mir ist immer noch nicht klar, wie der Aufruf mit Deinem neuen Objekt aussehen soll. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

25.05.2010, 23:33
Ballamann Neuer Benutzer Registriert seit: 21.01.2010 Beiträge: 23 PHP-Kenntnisse: Anfänger	PHP-Code: `$db = new db; $db->prepare_query("INSERT INTO user (Feld1, Feld2, Feld3, Feld4) VALUES (?, ?, ?, ?)"); $db->bind("ssss", "var1", "var2", "var3", "var4"); $db->set_parameter("var1", "Testnickname"); $db->set_parameter("var2", "hat keins"); $db->set_parameter("var3", "100"); $db->set_parameter("var4", "ÄttDot"); $db->execute(); $ergebnis = $db->fetch_result()` So funktioniert der Aufruf meines neuen Objekts. In der Funktion bind() werden alle übergebenen Variablen in ein assoziatives Array geschrieben (als Offsets) und ihnen wird probehalber der Wert "test" zugewiesen. Über set_parameter kann man dann die einzelnen Werte des Arrays ändern, man gibt den Namen und den Wert an. Das funktioniert soweit auch, wenn ich mir danach das Array ausgebe sind die vorher initialisierten Felder geändert. Wenn ich dann aber execute() aufrufe werden diese Änderungen ignoriert und nur der Wert "test" wird in die Datenbank geschrieben - also nur der Wert der bei der Initialisierung zugewiesen wurde. Da liegt mein Problem.