[MYSQL/PHP/HTML]Problem mit einem Query

Acrio · 9. Februar 2013

Hi. Ich habe mal wieder ein Problem mit meinem Query. Ich weis nicht wieso aber es wird anscheinend nicht ausgeführt. Hier.

SQL

loggedin = mysql_query("SELECT `LoggedIn` FROM `accounts` WHERE `Benutzername`='$uname' AND `Passwort`='$pw'");

Und hier.

PHP

<?php if($loggedin == 1) { ?>
                        <label id="FehlerLabel2">Der Benutzer ist bereits eingeloggt!</label>
                  <?php } ?>

Außerdem werde ich nicht weiter geleitet.

PHP

$_SESSION['Benutzername'] = $uname;
                                                        $_SESSION['Passwort'] = $pw;
                                                        header("location: index.php");

Bitte um Hilfe.

MFG Acrio

d30af26d · 9. Februar 2013

Das Ergebnis der Funktion mysql_query ist eine Resource/das Ergebnis, dass du mit mysql_fetch verarbeiten kannst.

Acrio · 9. Februar 2013

Und wie mache ich das?

Kaliber · 9. Februar 2013

Zum Beispiel so:

PHP

$loggedin = mysql_query("SELECT * FROM `accounts` WHERE `Benutzername`='$uname' AND `Passwort`='$pw'");
$row = mysql_fetch_array($loggedin);
$loggedin = $row["LoggedIn"];

mfg.

Acrio · 9. Februar 2013

Ok aber ich werde trotzdem nicht weitergeleitet.

PHP

header("location: index.php");

Jedoch ist der Code doch richtig oder etwa nicht?

Kaliber · 9. Februar 2013

Zitat von Acrio

Jedoch ist der Code doch richtig oder etwa nicht?

Naja, ich mache das immer mit einem meta-Tag:

PHP

echo '<meta http-equiv="refresh" content="0; URL=index.php">';

mfg.

Acrio · 9. Februar 2013

Ok. Und eine Frage zur Codeoptimierung.
So?

PHP

if(..) {
      //...
}

oder so?

PHP

if(..)
      //...

also mit oder ohne {} ?

d30af26d · 9. Februar 2013

Ich weiß nicht, ob es ein großen Unterschied macht, aber probier doch mal "Location" statt "location".

Kaliber · 9. Februar 2013

Du brauchst die { } sobald du mehr als 1ne Sache machen willst

mfg.

Acrio · 9. Februar 2013

d30af26d bei meiner register seite habe ich es genauso nur mit anmelden.php anstatt index.php

Backspace · 9. Februar 2013

./ Editiert \.

PHP

## $uname = $_POST['username'];
## $pw = $_POST['password'];
$loggedin = mysql_fetch_object(mysql_query("SELECT * FROM accounts WHERE Benutzername='$uname' AND Passwort='$pw' LIMIT 1"));
if($loggedin->LoggedIn >= 1)
{
    echo '<label id="FehlerLabel2">Der Benutzer ist bereits eingeloggt!</label>';
} else {
    $_SESSION['Benutzername'] = $uname;
    $_SESSION['Passwort'] = $pw;
    header("Location: index.php");
}

Alles anzeigen

Acrio · 9. Februar 2013

Backspace kannst du mir mal bitte erklären was du da jetzt gemacht hast und was das ## heist?

Backspace · 9. Februar 2013

Zitat von Acrio

Backspace kannst du mir mal bitte erklären was du da jetzt gemacht hast und was das ## heist?

Ich habe den Code kürzer gestalltet und den Query verschnellert.
Wenn du bei dem Query "SELECT *" schreibst, sucht er in jeder Spalte bzw. jedes Feld, da du nur ein Feld auslesen willst, reicht der Feldname.
Mit "mysql_fetch_object" zu arbeiten ist besser und übersichtlicher.

Die "##" Zeichen sind Kommentare. Ich habe die definierten Variablen auskommentiert, du musst die Textboxnamen bei dem "$_POST['..'];" ersetzen und die Kommentarzeichen entfernen.

Bei weiteren Fragen kannst du dich per Skype bei mir unter dem Namen "BackSpace.xL" melden.

breadfish.de 11. März 2022