Carl: Login-Problem

Moin

hab ein problem mit nem login-script, das so aussieht:

<?
if (isset($username) && isset($passwort))
{
require("db_connect.php");
$query = "SELECT user, pass FROM doomin_users where pass ='".$passwort."' AND user = '".$username."'";
$result = mysql_query($query);
if(mysql_num_rows($result) == 1)
include("bearbeiten.php");
else
echo "pech";
}
else
echo "<br> Bitte fülle alle Felder aus.";
?>

Allerdings funktionierts nicht, ich gebe zwar den namen und das passwort genau so wie in der DB angegeben an, aber ich bekomme nur den else fall "pech" zurück.

Weiß jemand, woran das liegt?

Gruss
Darky

  1. Moin

    hab ein problem mit nem login-script, das so aussieht:

    <?
    if (isset($username) && isset($passwort))

    was steht in den Variablen(echo)? Außerdem solltest Du lieber $_POST["username"] bzw. $_GET verwenden.

    {
    require("db_connect.php");
    $query = "SELECT user, pass FROM doomin_users where pass ='".$passwort."' AND user = '".$username."'";
    $result = mysql_query($query);
    if(mysql_num_rows($result) == 1)
    include("bearbeiten.php");
    else

    Wo beendest Du den Teil vor else mit "}" ?

    echo "pech";
    }

    wo öffnest Du mit "{" für dieses "}"?

    else

    analog zu oben. ich weiß das man die Klammenrn nicht immer braucht, aber wenn muß man das auch bewußt und richtig machen.

    echo "<br> Bitte fülle alle Felder aus.";
    ?>

    Weiß jemand, woran das liegt?

    da gibt es viele Gründe, s.o.

    Grüße
    Andreas

    1. Hallo Andreas,

      Ich finde zwar auch, dass man Klammern _immer_ setzten sollte -
      also auch, wenn sie eigentlich nicht nötig wären, aber die
      Klammersetzung in dem Script stimmt (leider).

      Ich hab das mal durch Einrückung (solltest Du Dir auch
      angewöhnen, Carl) verdeutlicht:

      <?
      if (isset($username) && isset($passwort))
      {
          require("db_connect.php");
          $query = "SELECT user, pass FROM doomin_users where pass ='".$passwort."' AND user = '".$username."'";
          $result = mysql_query($query);

      if(mysql_num_rows($result) == 1)
              include("bearbeiten.php");
          else
              echo "pech";
      }
      else
          echo "<br> Bitte fülle alle Felder aus.";
      ?>

      Gruß,
      dimde