Problem mit mysqli

[smoon]

Hallo,

ich versuche mich gerade daran eine Klasse für MySql zu schreiben, jedoch komme ich an einer Stelle nicht weiter. Meine "SELECT"-Funktion sieht folgender Maßen aus:

Code: Select all

function select($tabelle, $keys, $optionen = "") {
         $spalten = "";
         
         foreach($keys as $k) {
            $spalten .= $k.", ";
         }
         
         $spalten = substr($spalten, 0, -2);
      
         $sql = "SELECT ".$spalten." FROM ".$tabelle." ".$optionen;
                  
         if (!$this->db->query($sql)) {
            $this->error = "Konnte Query nicht ausführen: ".$this->db->error." (".$this->db->errno.")<br>[".$sql."]";
            return false;
         }
         
         if(!$ergebnis = $this->db->store_result()) {
            $this->error = "Konnte Ergebnis nicht lesen: ".$this->db->error." (".$this->db->errno.")";
            return false;
         }
         else {         
            $i = 0;
            
            while ($zeile = $ergebnis->fetch_assoc()) {
               $daten[$i++] = $zeile;
            }
            
            return $daten;
         }
      }

Jetzt wird jedoch immer der Fehler "Konnte Ergebnis nicht lesen" ausgegeben. Jedoch liefert mysqli selber keinen Fehler zurück, daher weiß ich grad nicht wo dieser liegt. Lasse ich diese Abfrage weg, liefert PHP beim Aufruf von "$ergebnis->fetch_assoc()" den Fehler, dass "$ergebnis" kein Objekt sei.

Mein Testaufruf sieht so aus:

Code: Select all

$spalten = array("id", "eintrag");
   if($ergebnis = $dbase->select("tabelle", $spalten)) {
      foreach($ergebnis as $zeile) {
         echo "(".$zeile['id'].") ".$zeile['eintrag']."<br>";
      }
   }
   else {
      die($dbase->error());
   }

Vllt. kann mir einer von euch mal wieder helfen

[smoon]

Konnte den Fehler jetzt doch selbst finden. "store_result()" wird anscheinend nur bei "multi_query()" verwendet.

[Nobbie]

Das ist aber nicht der wahre Fehler, store_result() sollte es auch bei einzelnen Selects tun.

Ganz sicher jedoch ist diese Zeile syntaktischer Blödsinn:

Code: Select all

if(!$ergebnis = $this->db->store_result()) {

Schau mal genau hin, was da entscheidendes fehlt (und dadurch vollkommen sinnentstellt wird).

[Wiedmann]

Schau mal genau hin, was da entscheidendes fehlt (und dadurch vollkommen sinnentstellt wird).

Der Zuweisungsoperator ist Syntaktisch korrekt und an dieser Stelle auch gewollt.

Zuerst wird der Wert von "$this->db->store_result()" in der Variablen $ergebnis abgespeichert ("="), dann die Variable $ergebnis negiert ("!"), und dieses Resultat dann für das "if" benutzt.
--> "!" und "=" werden jeweils von rechts nach links ausgewertet.

(eben dadurch kann man dann gleich im "else", wenn also $ergebnis nicht false ist, auf den Inhalt von $ergebnis zugreifen.)

[Nobbie]

Zuerst wird der Wert von "$this->db->store_result()" in der Variablen $ergebnis abgespeichert ("="), dann die Variable $ergebnis negiert ("!"), und dieses Resultat dann für das "if" benutzt.
--> "!" und "=" werden jeweils von rechts nach links ausgewertet.

Mit Sicherheit nicht - und es ist auch kein Zufall, dass diese Abfrage in die "Hose" geht, aber dennoch keinerlei MySQL Error ausgegeben werden (das müßte ja der Fall sein, wenn diese Syntax richtig wäre). Es fehlt die alles entscheidende Klammer, da nach allgemeinen Programmierrichtlinien boolsche Operator vor Zuweisungen ausgeführt werden. Richtig müßte es so aussehen:

Code: Select all

if (!($ergebnis = $this->db->store_result())) {

Es wird ja die if-Abfrage ausgeführt (also die eigene Fehlermeldung wird angezeigt), aber dennoch keine MySQL-Fehlermeldung ausgegeben - genau das ist der Effekt der falschen Syntax.

[Wiedmann]

Mit Sicherheit nicht

Ganz wie du meinst.
Oder machst einfach mal ein "var_dump(!$a = true);" bzw. "var_dump(!$a = false);"

a nach allgemeinen Programmierrichtlinien boolsche Operator vor Zuweisungen ausgeführt werden.

Die Wertigkeit von "!" ist tatsächlich höher als "=". HIer hat man aber einen "Sonderfall". Du kannst das natürlich auch selbst im PHP Manual nachlesen (Operator Precedence), wo genau das beschrieben ist.

Richtig müßte es so aussehen

Ist in dem Fall das selbe, auch wenn es meist, nicht unbedingt hier, einfacher zu lesen ist, wenn man passende Klammern setzt.)

[Nobbie]

Tatsächlich steht da dieses: (man kann es echt nicht glauben).

Note: Although = has a lower precedence than most other operators, PHP will still allow expressions similar to the following: if (!$a = foo()), in which case the return value of foo() is put into $a.

Das PHP Team gibt sich ja wirklich Mühe - aber das ist kein "Sonderfall", sondern schlicht falsch programmiert. Die richtige Reaktion wäre ein Fatal Error, weil man einem "L-Value" bzw. Konstanten keinen Wert zuweisen kann (ähnlich falsch wie so eine Zuweisung: 'Hallo' = $a;). In jeder anderen Sprache (sowieso in C) ist das ein Compiler Abbruch.

Bleibt dann die Frage, wieso MySQL ein FALSE zurückgibt und dennoch weder Errno gesetzt noch eine Fehlermeldung erfolgt?