4. Session

Session

  • ?

    Ok, hab mir das Bsp. angeschaut und mein Script ein wenig geändert, trotzdem funktioniert es nicht:

    <?
session_start();
if(session_is_registered("user") && session_is_registered("psw") && session_is_registered("level"))
{
echo("Sie sind eingeloggt als " . $_session["user"] . " !");
}
else if(isset($psw) && isset($user))
{
include("inc/db.inc.php");
$result = mysql_query("SELECT * FROM user");
while($row = mysql_fetch_array($result, MYSQL_ASSOC))
{
if($user == $row["name"] && $psw == $row["passwort"])
{$enter = 1;}
}
if($enter == 1)
{
$_session["user"] = $row["name"];
$_session["psw"] = $row["passwort"];
$_session["level"] = $row["level"];
header("Location: login.php");
}
else
{die("Ihre Benutzerdaten sind falsch !!!");}
}
else
{
// Login
}
?>
    0
  • F

    Ich bin mir zwar grad net sicher, ob PHP Groß-/Kleinschreibung unterscheidet, aber normalerweise wird $_session großgeschrieben, also $_SESSION
    Außerdem sollte man kein SELECT * machen, sondern die einzelnen Spalten aufzählen
    Falls das oben genannte nicht klappt: Wo genau hängt er denn bzw. wie sieht das "funktioniert nicht" aus? 🙄

    0
  • ?

    Ich schicke meine Logindaten ab, diese werden verarbeitet und es wird eine Session erstellt, allerdings gelange ich dann einfach wieder zurück zur Loginseite.

    0
  • F

    Bist du dir sicher, dass die richtig in die Session geschrieben werden?
    Hast du das mit $_SESSION mal ausprobiert? Was für ne PHP Version läuft bei dir? (> 4.1.0?)

    0
  • M
     

    include("inc/db.inc.php");
$result = mysql_query("SELECT * FROM user");
while($row = mysql_fetch_array($result, MYSQL_ASSOC))
{
if($user == $row["name"] && $psw == $row["passwort"])
{$enter = 1;}
}

    Wenn mich meine Augen nicht trügen, dann holst Du Dir hier alles mögliche aus Deiner user-Tabelle raus. Warum tust Du sowas??? Wozu gibt es die WHERE-Klausel bei SQL?!

    VORSCHLAG:

    $query = "SELECT name,passwort FROM user WHERE name='$user' AND passwort='$psw'"; 
$result = mysql_query($query);

    Dann solltest Du im Normalfall genau 1 Zeile oder gar keine zurückgeliefert bekommen. Dann kannst Du Deine $enter-Variable davon abhängig machen.
    Und Deine $psw und $user kommen sicher aus nem Form per POST oder GET. Also wäre es schick, wenn Du da $_POST['xxx'] oder $_GET['xxx'] verwendest.

    Und flenders hat schon Recht, Du solltest auch tunlichst $_SESSION groß schreiben. 😉

    Und eines Sache hab ich noch. *g*
    Ich weiß nicht, ob das mit session_is_registered("psw") schlechter ist, aber ich würde das so abfragen: isset($_SESSION['psw'])

    Falls Du php kleiner 4.1.0. hast, dann verwende statt $_ABC $HTTP_ABC_VARS

    mfgr,
    mid

    [ Dieser Beitrag wurde am 21.05.2003 um 22:24 Uhr von Midnighter editiert. ]

    0
  • U

    [PHP/MySQL] Login

    Hier wurde das ausführlich behandelt.
    Und auch warum man keinen SELECT * und einen WHILE-Schleife macht.

    [ Dieser Beitrag wurde am 22.05.2003 um 06:33 Uhr von Unix-Tom editiert. ]

    0
  • ?

    Ich habe das ganze nochmal überarbeitet, habe aber immer noch den alten Fehler, nachdem ich mich eingeloggt habe, gelange ich immer wieder zur Loginseite zurück, könnte da vielleicht ein Problem mit den Sessioncookies vorliegen ?

    <?
session_start();
if(session_is_registered('user'))
{
echo("Sie sind eingeloggt als " . $_SESSION["user"] . " !");
}
else if(isset($psw) && isset($user))
{
include("inc/db.inc.php");
$result = mysql_query("SELECT passwort, level, name FROM user WHERE name = '$user'");
$row = mysql_fetch_array($result,MYSQL_ASSOC);
if (!$row)
{
die("Dieser Name ist leider nicht bekannt !");
}
if ($row["passwort"] <> $psw)
{
die("Dieses Passwort passt nicht zum Usernamen !");
}
$_SESSIOM["user"] = $row["name"];
$_SESSION["psw"] = $row["passwort"];
$_SESSION["level"] = $row["level"];
header("Location: login.php");
}
else
{
// Login
}
?>
    0
  • F

    1. elseif schreibt man zusammen
    2. würde ich immer { } setzen, um die einzelnen Böcke klar zuzuordnen

    0
  • ?

    Das hilft mir jetzt aber auch nicht, wo könnte das Problem liegen, eventuell wird die Session nicht per Cookie übertragen oder ist ne alte Version ?

    0
  • M

    Ich bin mir nicht sicher, ob PHP <> für ein != anerkennt. Probier ich aber jetzt auch nicht aus.
    Und dann würde ich auch nicht if(!row) abfragen. 😮

    <?
    session_start();
    if(session_is_registered('user'))
    {
        echo("Sie sind eingeloggt als " . $_SESSION["user"] . " !");
    }
    else if(isset($psw) && isset($user))
    {
        include("inc/db.inc.php");
        $result = mysql_query("SELECT passwort, level, name FROM user WHERE name = '$user'");
        $row = mysql_fetch_array($result,MYSQL_ASSOC);
        if ( mysql_num_rows($result)==0 )
        {
            die("Dieser Name ist leider nicht bekannt !");
        }
        if ( $row["passwort"] != $psw )
        {
            die("Dieses Passwort passt nicht zum Usernamen !");
        }
        $_SESSIOM["user"] = $row["name"];
        $_SESSION["psw"] = $row["passwort"];
        $_SESSION["level"] = $row["level"];
        header("Location: login.php");
    }
    else
    {
        echo "Login";
    }
?>

    Und noch ein Vorschlag. Mach Dir mal ne weitere PHP-Seite:

    check.php

    <?php
    session_start();
    print_r( $_SESSION );
?>

    Die rufst Du dann mal auf, nachdem die Session Userdaten enthalten soll.
    Dann kannst Du mal checken, was überhaupt in Deiner Session steht.

    Was für eine PHP-Version fährst Du eigentlich???

    Und nochwas, vielleicht hilft Dir das hier bei Deinen Session-Probs weiter: http://www.tool-garage.de/de/snip_ses.php

    Gruß,
    mid

    0
Anmelden zum Antworten