mrhappiness login script

[roman]

Ich krieg folgenden Fehler:
mysql_num_rows(): supplied argument is not a valid MySQL ... on line 61

Das ist die Stelle:

function logged_in()
{
    $sql="SELECT id
    FROM $usertbl
    WHERE session='".session_id()."'
    LIMIT 1";
    $result= mysql_query($sql);
      return ( mysql_num_rows($result)==1); // hier ist Linie 61
} 

Ich hab bei allen Zeilennamen das User vorne weggenommen und alles kleingeschrieben. Das ist auch in der MySQL Tabelle so.
Ausserdem hab ich die Variable $usertbl in einer konfigurationsdatei definiert.

gruss
roman

[MaxP0W3R]

ich schätze dass esson_id() in diesem fall keinen wert zurückgibt

[Hopka]

Änder die Funktion mal so, dass sie dir
a) den Query ausgibt und
b) eine MySQL-Fehlermeldung ausgibt, wenn der Query fehlgeschlagen ist.

Ungefähr so:

function logged_in()
{
    $sql="SELECT id
    FROM $usertbl
    WHERE session='".session_id()."'
    LIMIT 1";
    echo "Query: " . $sql;
    $result= mysql_query($sql) or die(mysql_error());
      return ( mysql_num_rows($result)==1); // hier ist Linie 61
} 

Dann sag mal, was der Code so an Ausgaben produziert, oder noch besser finde anhand der Ausgaben selbst heraus, was falsch ist.

[roman]

Query: SELECT id FROM WHERE session='9151f2573af1d4c562b8799f89d768be' LIMIT 1You have an error in your SQL syntax near 'WHERE session='9151f2573af1d4c562b8799f89d768be' LIMIT 1' at line 3

[Hopka]

Tja, was sagt uns das?

Richtig, die Variable $usertbl ist nicht definiert.

Woran liegt das?

Richtig, sie ist eine globale Variable, und daher innerhalb der Funktion unsichtbar.


Hättest du den Anfang des PHP-Handbuchs gelesen (in dem sehr viele nützliche Grundlagen stehen), wüsstest du das auch selbst.

Insbesondere ist der Geltungsbereich von Variablen für dich interessant, aber ich empfehle dir trotzdem mal die kompletten ersten 4 Kapitel des PHP-Handbuchs zu lesen (ist nicht so viel, wie es sich anhört)

[roman]

Danke.

Ich sollte mir mal komplett ein PHP-Handbuch lesen.


Ich hab es so gemacht dass die Variable definiert ist.
Es gibt immernoch eine Fehlermeldung:

Query: SELECT id FROM user WHERE session='4e64913e2c180f53ad1a87e5fe8f0060' LIMIT 1

[Hopka]

das ist keine Fehlermeldung, das kommt von dem echo.

Kannst du wieder wegmachen (die Zeile mit dem echo).

[roman]

Danke