PHP, Funktion wird nicht wiedrgeben

Sean_Caprez · 4. Juni 2015

Hallo,

sollte kein Account gefunden werden, wiedergibt er mir den print 0 und nicht print 99.
Weiß jemand woran das liegt?

PHP

$username = $_GET['user'];
$m= md5($_GET["pass"]);
$login= "SELECT * FROM Accounts WHERE Benutzername LIKE ' $username '";
$res = mysql_query($login);
if(mysql_affected_rows() > 0)
{
                  $query = "SELECT Benutzername, Passwort FROM Accounts WHERE Benutzername LIKE ' $username '";
                  $ress = mysql_query($query);
                  $row = mysql_fetch_object($ress);
                  if($row->Passwort == $m) 
                  {
                        die("1");
                  }
                  else
                  {
                       die("0");
                  }
}
else
{
     die("99");
}
?>

Alles anzeigen

Mr.Hot4Ever · 4. Juni 2015

Wahrscheinlich weil man in einer if Abfrage nur einmal else nutzen kann.
Quasi du fängst mich if an und dann else if dann else dann müsste das gehen.
So kenn ich das weis aber nicht ob ich da falsch liege.

Edit:
//korrigiert.

whitetiiger · 4. Juni 2015

Zitat von Mr.Hot4Ever

Wahrscheinlich weil man in einer if Abfrage nur einmal else nutzen kann.

Ne, dass wäre hier Falsch, da er hat die Klammer anders hat. Bin mir gerade aber nicht sicher, ob die Funktion "die" überhaupt was ausgibt.

Mr.Hot4Ever · 4. Juni 2015

Hab mein post korrigiert.
1 <?php
2 $var=1;
3 if($var<0){
4 // wird nicht ausgeführt da $var größer 0
5 // ist ein "if" darf nur einmal vorkommen
6 echo "$var ist kleiner 0";
7 } else if($var>0){
8 // dieser Ausdruck ist wahr, darum
9 // wird das nachfolgende "echo" ausgeführt
10 // "else if" Anweisungen können
11 // beliebig hinzu gefügt werden
12 echo "$var ist größer 0";
13 } else {
14 // wird nur ausgeführt wenn alle vorangegangenen
15 // "else if" sowie das einleitende "if" als false
16 // ausgewertet wurden. "else" darf nur einmal vorkommen.
17 echo "$var ist gleich 0";
18 }
19 ?>

Caglar · 4. Juni 2015

PHP

$username = $_GET['user'];
$m= md5($_GET['pass']);
$login= "SELECT NULL FROM `Accounts` WHERE `Benutzername` = '{$username}' LIMIT 1";
$res = mysql_query($login);
if(mysql_affected_rows() == 1)
{
                  $query = "SELECT Benutzername, Passwort FROM `Accounts` WHERE `Benutzername` = '{$username}'";
                  $ress = mysql_query($query);
                  $row = mysql_fetch_object($ress);
                  if($row->Passwort == $m) 
                  {
                        die("1");
                  }
                  else
                  {
                       die("0");
                  }
}
else
{
     die("99");
}
?>

Alles anzeigen

ChristianW · 4. Juni 2015

Viel leichter wäre eso so

$login= "SELECT NULL FROM `Accounts` WHERE `Benutzername` = '" . $username . "' AND `Passwort` = '" . $m . "' LIMIT 1";
$res = mysql_query($login);
if(mysql_affected_rows() == 1) {
//Daten richtig
} else {
//Daten falsch
}

palwal · 4. Juni 2015

Zitat von ChristianW

Viel leichter wäre eso so

$login= "SELECT NULL FROM `Accounts` WHERE `Benutzername` = '" . $username . "' AND `Passwort` = '" . $m . "' LIMIT 1";
$res = mysql_query($login);
if(mysql_affected_rows() == 1) {
//Daten richtig
} else {
//Daten falsch
}

Alles anzeigen

Deine Abfrage ist durchaus sinnvoll.
Das macht aber wenig Unterschied.

Da es auch klappen wird wenn @Sean_Caprez statt LIKE, ein = schreiben würde. Weil LIKE findet gern mehrere Accounts die sich vom Namen ähneln.

ChristianW · 4. Juni 2015

Zitat von palwal

Deine Abfrage ist durchaus sinnvoll.
Das macht aber wenig Unterschied.

1 Query abzusenden wäre besser als 2, ich denke immer an die Performance von daher

xGreekz7x · 4. Juni 2015

Zitat von palwal

Deine Abfrage ist durchaus sinnvoll.
Das macht aber wenig Unterschied.

Da es auch klappen wird wenn @Sean_Caprez statt LIKE, ein = schreiben würde. Weil LIKE findet gern mehrere Accounts die sich vom Namen ähneln.

LIKE findet dann mehrere accounts wenn zum beispiel:

LIKE '%xGreekz7x%' stehen würde
Weil dann sucht er nur nach etwas wo xGreekz7x drin steht, also würde er xGreekz7x1, xGreekz7x2 usw mit finden.

breadfish.de 12. März 2022