Login Script problem...

[sourceforge]

Hallo alle zusammen,

ich hab ein Script geschrieben, womit sich die Besucher anmelden können, um Zugriff auf geschützte Bereiche zu haben.
Soweit so gut aber nun hab ich ein Problem mit der Login Datei.
Die Benutzerdaten werden in der Datenbank gespeichert.

Folgende Fehlermeldung möchte ich noch einbauen: "Benutzername oder Passwort ist falsch!" was mir aber nicht gelingt.


So sieht login.php aus:

<?php
 if(!isset($_SESSION['username'])) {
echo "Hallo Gast [<a href='login.php' class='link'>Login</a> | <a href='register.php' class='link'>Registrieren</a>]";
}
if(isset($_SESSION['username'])) {
echo "Hallo ".$_SESSION['username']." <a href='login.php?action=logout' class='link'>[Abmelden]</a>";
}
?>

<?php
include('functions.php');
echo connect();

$user=$_POST['user'];
$pass=$_POST['pass'];
$pass=md5($pass);
$send=$_POST['s'];

$action=$_GET['action'];
if(!isset($action)) {
$action="form";
}

if($action=="form") {
echo '<form action="login.php?action=login" method="post">

<table width="496" border="0" cellspacing="0" cellpadding="0">
  <tr>
    <td width="154">Username*</td>
    <td width="342"><input type="text" name="user" class="inputs" size="45" /></td>
  </tr>
  <tr>
    <td>Passwort*</td>
    <td><input type="password" name="pass" class="inputs" size="45" /></td>
  </tr>
  <tr>
    <td>&nbsp;</td>
    <td><input type="submit" value="Einloggen" class="submit_register" /><input type="hidden" name="s" value="1" /></td>
  </tr>
</table>
</form>';
}

if($action=="login") {
if(($user and $pass)=="") {
echo "Alle Felder mit einem * sind erfolderlich.<br>";
}

$abfrage="SELECT user, pass FROM userdata";

$ergebnis=mysql_query($abfrage) or die (mysql_error());

while($row=mysql_fetch_assoc($ergebnis)) {

if($user == $row['user'] and $pass == $row['pass']) {

$_SESSION['username'] = $user;

echo 'Login war erfolgreich. Hallo '.$_SESSION['username'].'';
     } 
   }
 }

if($action=="logout") {

session_destroy();

unset($_SESSION['username']);

echo "<br>Logout war erfolreich!<br>";
}
?>

Kann mir jemand sagen wie ich das ganze anstellen soll, wäre nett.
Danke im Voraus!
Mfg

[penizillin]

suche

if(...){
  echo 'login erfolgreich';
} 

ergänze durch

else { echo 'nicht erfolgreich'; } 

[sourceforge]

ähm... wie meinst du das?

etwa so?

if(...) {
echo "login erfolgreich";
} else {
echo "nicht erfolgreich";
} 

[penizillin]

welche if-abfrage aus deinem code habe ich wohl damit gemeint?

[sourceforge]

so funktioniert es nicht....
ich kann mich zwar einloggen aber die Meldung "login nicht erfolgreich" erscheint mit dem login erfolgreich meldung.

$abfrage="SELECT user, pass FROM userdata";

$ergebnis=mysql_query($abfrage) or die (mysql_error());

while($row=mysql_fetch_assoc($ergebnis)) {

if($user == $row['user'] and $pass == $row['pass']) {

$_SESSION['username'] = $user;

echo 'Login war erfolgreich. Hallo '.$_SESSION['username'].'';
      } 
      else {
echo "Login nicht ergolgreich";

   } 

[penizillin]

weißt du warum das so ist? weil du bei jedem login _alle_ benutzer ausliest und mit der eingabe vergleichst, anstatt mit WHERE eine bedingung einzufügen und somit die anzahl der ausgelesenen datensätze auf 0 (login fehlgeschlagen) oder 1 (erfolgreich) zu reduzieren.

anderer ansatz:

überprüfe nach der while schleife, ob $_SESSION['username'] gesetzt wurde. wenn nicht, fehlermeldung ausgeben.

[sourceforge]

Session wird gesetzt...

[penizillin]

=> login erfolgreich.

[sourceforge]

cool, das funktioniert ... vielen dank!

ich weiß nicht wie ich das anstellen soll:

weißt du warum das so ist? weil du bei jedem login _alle_ benutzer ausliest und mit der eingabe vergleichst, anstatt mit WHERE eine bedingung einzufügen und somit die anzahl der ausgelesenen datensätze auf 0 (login fehlgeschlagen) oder 1 (erfolgreich) zu reduzieren.

[penizillin]

http://tut.php-q.net/login.html

[sourceforge]

Sehr Gut.. vielen dank^^