mysql_num_rows() - Fehlermeldung

[SKYRUNNER]

Hallo,
ich brauche in einem Script diesen Befehl.
Ich bekomme jedoch eine Fehlermeldung zurückgegeben.
Der Code sieht so aus:

<?php        
        $conn =mysql_connect("$dbhost", "$dbuser", "$dbpass");
        $select = mysql_select_db("$dbname", $conn);
        $sql1 = mysql_query("SELECT * FROM game_user WHERE ip = $ip");
        $sql2 = mysql_query("SELECT * FROM game_user WHERE user = '$username'");
        $res1 = mysql_query("$sql1", $conn);
        $res2 = mysql_query("$sql2", $conn);
        [b]$err_1 = mysql_num_rows($res1);
        $err_2 = mysql_num_rows($res2);[/b]
             if ($err_1 > "0")
             {
             echo "Jeder User darf nur <b>einen</b> Account registrieren!";
             exit;
             }
?>

Die Fehlermeldun g ist das hier:

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /www/htdocs/v072098/gww/register.php on line 28
Die Ziele, die den Fehler verursacht/verursachen ist fett geschreiben.

MfG und danke schonmal für eure Mühe

[Stuck Mojo]

Hi !!

erstmal:

ip = '$ip'

und dann... Was soll der ganze Kram darstellen?
mysql_query() gibt eine resourceid zurück. Wozu willst du dann nochmals mit der resourceid einen Query bauen? Alles etwas durcheinander und unlogisch!
Die beiden letzten mysql_query()'s sind total überflüssig und verursachen selbstverständlich Fehler. Dann solltest du dir mal abgewöhnen Variablen immer in " zu übergeben. Mannchmal übergibt man nämlich auch Arrays und durch die " wird die Variable als String ausgegeben. Spätestens dann hast du ein Problem

Also schaus dir nochmal genau an...

Gruss
Jan

[SKYRUNNER]

So,
ich hab das alles mal ein bisschen umgeschrieben, aber dennoch kommt der gleiche fehler wie zuvor.

$conn =mysql_connect("$dbhost", "$dbuser", "$dbpass");
        $select = mysql_select_db("$dbname", $conn);
        $sql1 = "SELECT * FROM game_user WHERE ip = '$ip'";
        $sql2 = "SELECT * FROM game_user WHERE user = '$username'";


             if (mysql_num_rows($sql1) > 0)
             {
             echo "Jeder User darf nur <b>einen</b> Account registrieren!";
             exit;
             }
             if (mysql_num_rows($sql2) > 0)
             {
             echo "Der gewählt benutzername ist leider bereits vergeben.<br>
             Bitte wähle einen anderen!";
             exit;
             } 

Hmm was kann da denn noch falsch sein?

[Stuck Mojo]

Sorry, aber ein bissel selber überlegen musst du schon!
Wenn ich schon schreibe, dass die letzten beiden mysql_query()'s überflüssig sind, musst du nicht ALLE rausnehmen.

http://www.php.net -> Suchfunktion -> mysql_num_rows
Da steht, was diese Funktion für Parameter erwartet.

[Filou]

$conn =mysql_connect("$dbhost", "$dbuser", "$dbpass");
        $select = mysql_select_db("$dbname", $conn);
        $sql1 = "SELECT * FROM game_user WHERE ip = '$ip'";
        $sql2 = "SELECT * FROM game_user WHERE user = '$username'";


             if (mysql_num_rows(mysql_query($sql1)) > 0)
             {
             echo "Jeder User darf nur <b>einen</b> Account registrieren!";
             exit;
             }
             if (mysql_num_rows(mysql_query($sql2)) > 0)
             {
             echo "Der gewählt benutzername ist leider bereits vergeben.<br>
             Bitte wähle einen anderen!";
             exit;
             } 

ciao filou

[SKYRUNNER]

danke!