Login -.-

[glen]

Der Senden-Button vom Formular hat den Namen submit.... in der selben Datei wie man oben sieht..!

 

[won_gak]

Verwende doch lieber $_POST["submit"].

Ist kürzer und übersichtlicher. Achte auf den _ nach $

 

[cybergnom]

hmm... geht das so überhaupt?

Also ich würde ein "hidden"-Textfeld dafür einsetzten.

Und wenn Du die Seite zum ersten mal öffnest, gibt's ja noch keinen Parameter "submit".

Der wird erst übergeben (falls es so funktionieren sollte, wie Du Dir das vorstellst), wenn Du den Submit-Button geklickt hast.

Somit musst Du erst einmal überprüfen, ob's "submit" überhaupt gibt:

if (isset($_POST["submit"]) AND  $_POST["submit"] == "Login") {
.
.
.
}
else {
.
.
.
}

edit:

eigentlich reicht es auch, einfach nur zu testen, ob der Paramter existiert. Der Inhalt von "submit" spielt dabei ja keine Rolle.
also geht's auch kürzer:

if (isset($_POST["submit"])){
.
.
.
}
else {
.
.
.
}

 

[Fabian H]

Original geschrieben von boelkstoff
kleiner tipp entweder html oder php und kein mischmasch mit dem ständigen öffnen und schließen! nimm also echo für sowas

Quatsch! Für längere Texte und HTML-Stücke nach Möglichkeit kein echo nehmen!

 

[glen]

Parse error: parse error, expecting `','' or `')'' in /data/members/free/tripod/de/j/o/c/jock15/htdocs/Comm/loginform.php on line 7


aber ist egal ob ich 'submit' oder "submit" mache....

zum verzweifeln :|

 

[cybergnom]

zeig mal deinen Code...

 

[glen]

<?

error_reporting(E_ALL);

require_once("dbconnect.php");

if ($_POST["submit"])
{

$nick = $HTTP_POST_VARS["nick"];
$pw = $HTTP_POST_VARS["pw"];


if ((!isset($nick)) OR (!isset($pw)))
{
echo ("Sorry ohne Nick oder Passwort kommst du hier nicht rein!");
}


$query = @mysql_query("SELECT * FROM userdaten WHERE Nick = '$nick'") or die('Select ist fehlgeschlagen!');
$result = @mysql_fetch_array($query) or die('Sorry, aber dieser Benutzername existiert nicht!');

if ($result['Passwort'] != $pw OR $result['Nick'] != $nick)
{
echo "Sorry, aber das Passwort gehört nicht zu diesem Nick!";
}
{
header ("Location: index2.php");
}
{
echo "<FORM name='login' action='left.php' method='post'>";
echo "<center>Nickname</center>";
echo "<input type='text' name='nick' maxlength='15' size='15'><br>";
echo "<center>Passwort</center>";
echo "<input type='password' name='pw' maxlength='15' size='15'><br><br>";
echo "<input type='submit' value='Login' name='submit'>";
echo "</FORM>";
}
}

?>


Hab ihn nun wieder so umgeformt,....mit deinem ersten Code ging es aufeinmal, aber dann zeigte er überhaupt kein Formular mehr an...er soll ja wenn der submit-button noch nicht gedrückt wurde das formular anzeigen ....und erst wenn man den button drückt überprüfen und alles...

 

[won_gak]

Ich sehe nur ein Problem bei dem

{
}

Fehlt da nicht ein "else" oder sonst ein logischer Operator?

 

[cybergnom]

Jo... hab's schon erkannt.
Hatte bei meinem Code oben eine Klammer vegessen.
So müsste es nun eigentlich klappen.

<?

error_reporting(E_ALL);

require_once("dbconnect.php");

if (isset($_POST["submit"])){
{

$nick = $_POST["nick"];
$pw = $_POST["pw"];


if ((!isset($nick)) OR (!isset($pw))) 
{
echo ("Sorry ohne Nick oder Passwort kommst du hier nicht rein!");
}


$query = @mysql_query("SELECT * FROM userdaten WHERE Nick = '$nick'") or die('Select ist fehlgeschlagen!');
$result = @mysql_fetch_array($query) or die('Sorry, aber dieser Benutzername existiert nicht!'); 

if ($result['Passwort'] != $pw OR $result['Nick'] != $nick)
{
echo "Sorry, aber das Passwort gehört nicht zu diesem Nick!";
}
header ("Location: index2.php");
}

else {
echo "<FORM name='login' action='left.php' method='post'>";
echo "<center>Nickname</center>";
echo "<input type='text' name='nick' maxlength='15' size='15'><br>";
echo "<center>Passwort</center>";
echo "<input type='password' name='pw' maxlength='15' size='15'><br><br>";
echo "<input type='submit' value='Login' name='submit'>";
echo "</FORM>";
}


?>

P.S.: Du hast euch das "else" vergessen.

 

[won_gak]

Ein } am Ende zu viel, aber sonst siehts gut aus.