Formular korrekt?

guckmael

Es gibt mir bei diesem bisherigen verfassten script immer nur hallo aus. ich komm nicht auf den fehler, meldungen gibt er keine aus. seht ihr irgendwo ein logikfehler. were nätt

PHP

<?php
session_start();
$user = $_POST['user'];
$pass = $_POST['pass'];


if($user == "admin" || $pass == "pass") 
{
	echo "Sie sind eingeloggt";
	$_SESSION['eingeloggt'] = TRUE;
}


if($_SESSION['eingeloggt'] == TRUE) 
{
	echo "Hallo".$user;
}


if($_SESSION['eingeloggt'] != TRUE) 
{
?>


<form action="index.php" method="post">
<p>Username:</p>
<p><input type="text" name="user" /></p>
<p>Passwort:</p>
<p><input type="password" name="pass" /></p>
<p>&nbsp;</p>
<p><input type="submit" value="Senden" /></p>
</form>


<?php 
} 
?>

Alles anzeigen

Es ist noch nicht fertig, aber im Tutorial sieht der dozent ein formular, ich bekomme aber nur Hallo ausgegeben und kein Formular

Dodo

PHP

echo "Sie sind eingeloggt"; 
    $_SESSION['eingeloggt'] = TRUE;
    $_SESSION['user'] = $user;

PHP

echo "Hallo".$_SESSION['user'];

</span></span>

mehlmael

Nabend, danke für deinen Post! Ich habe es mal so abgeändert, ich komme allerdings nicht mehr auf Formularseite zurück. Und jetzt muss ich noch den elseteil programmieren oder?

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 


if($user == "Admin" || $pass == "pass")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="index.php" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

Dodo

wenn du zurück willst: logout-button, oder eine zusätzliche GET-Variable, die du abfragen kannst

guckmael

Dankeschön, hab mich schon schlau gemacht. Dank PHP.net session_destroy();

Herzlichen Dank für die promte Hilfe und einen schönen Abend noch.

guckmael

Eins müsste ich noch wissen, bevor ich weiterbastle . Wie mache ich es das wenn man die hauptseite aufruft, keine meldungen das user id und passwort fehlen kommen, das wäre klasse. Und dann verswuche ich es noch abzusichern

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 


if($user == "")
{
	echo "Bitte geben SIe ihre User-ID ein!";
}
if($pass == "")
{
	echo "Bitte geben Sie ein Passwort ein!";
}
if($user == "" && $pass == "")
{
	echo "Bitte geben Sie ihre User-ID und ihr Passwort ein!";
}


if($user == "Sylvia" || $pass == "armageddon*" XOR $user == "Rene" || $pass == "xx1000xx23")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
    echo "<p><a href=\"daten.php\">Ihre Daten</a></p>";
    echo "<p><a href=\"ausloggen.php\">Ausloggen</a></p>";
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

Und wenn es eine Möglichkeit die vielen if abfragen anders zu regeln, immer raus damit.

Dodo

http://at2.php.net/manual/en/function.isset.php

guckmael

Hallo, ich habe bereits einige Teile umgebaut, will hier auch nichts vorgkaut bekommen aber komme nicht weiter. Geb mir aber Mühe. ich bekomme die ersten 3 echos nicht ausgegeben

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 




if(isset($user)) {
echo "Bitte geben Sie ihre User-ID ein!";
}
if(isset($pass)) 
{
echo "Bitte geben Sie ihr Passwort an!";
}
if(isset($user, $pass))
{
echo "Bitte beide Felder ausfüllen!";
}
if($user == "user1" || $pass == "passwort3" XOR $user == "user2" || $pass == "passwort4")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
    echo "<p><a href=\"daten.php\">Ihre Daten</a></p>";
    echo "<p><a href=\"ausloggen.php\">Ausloggen</a></p>";
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

grübel....

guckmael

Ich habe den code nochmal mit empty überarbeitet:

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 




if(isset($user)) {
echo "Bitte geben Sie ihre User-ID ein!";
}


if(isset($pass)) 
{
echo "Bitte geben Sie ihr Passwort an!";
}
if(empty($user)) {
echo "";
}
if($user == "user1" || $pass == "passwort3" XOR $user == "user2" || $pass == "passwort4")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
    echo "<p><a href=\"daten.php\">Ihre Daten</a></p>";
    echo "<p><a href=\"ausloggen.php\">Ausloggen</a></p>";
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

Dodo

du musst die POST-Variablen abfragen, ob sie existieren

guckmael

danke dir dodo, hast mir echt schon viel Hilfe geleistet. Habs so gelernt das die variableninhalte dem variablennamen zugewiesen werden. von daher müsste man die doch therotisch über den variablennamen ansprechen können. Na gut gleich mal umcoden. Danke nochmal

guckmael

Ich muss nochmal fragen, er gibt mir keinen Parse Fehler oder so aus, das habe ich schon ganz gut drauf, aber so ganz klappt es immer noch nicht.

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 




if(isset($_POST['user'] )) 
{
echo "Bitte geben Sie ihre User-ID ein!";
}


if(isset($_POST['pass'] )) 
{
echo "Bitte geben Sie ihr Passwort an!";
}
if(empty($_POST['user'] )) 
{
echo "";
}
if(empty($_POST['pass'] )) 
{
echo "";
}
if($user == "user1" || $pass == "passwort3" XOR $user == "user2" || $pass == "passwort4")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
    echo "<p><a href=\"daten.php\">Ihre Daten</a></p>";
    echo "<p><a href=\"ausloggen.php\">Ausloggen</a></p>";
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

Dodo

erste PHP-Zeile sollte während der Entwicklung immer Folgendes sein:

PHP

error_reporting(E_ALL);

guckmael

Zitat von Dodo
erste PHP-Zeile sollte während der Entwicklung immer Folgendes sein:
PHP
error_reporting(E_ALL);

Xampp und hatte doch geschrieben das es kein Parsefehler ist. Nur die Zeilen Username und Passwort fehlt erscheinen im eingeloggten Zustand immer noch

Grevas

PHP

if($user == "user1" || $pass == "passwort3" XOR $user == "user2" || $pass == "passwort4")

Muss man das verstehen? wenn user einen wert ODER das passwort den wert hat soll xy passieren NICHT-ODER ... öh?
meintest du das vielleicht so?

PHP

if($user == "user1" && $pass == "passwort3" || $user == "user2" && $pass == "passwort4")

Deine isset() abfragen machen wenig Sinn. Wenn die Variable gesetzt ist, soll eine Fehlermeldung ausgegeben werden, das es fehlt?
Wenn schon dann mit

PHP

if(empty($_POST['xy']) echo "xy fehlt.";

Die Felder werden immer mitübergegeben, mit isset kannst du überprüfen ob ein bestimmter Button gedrückt wurde. Textinputs werden immer beim Absenden abgeschickt und sind damit immer gesetzt(auch wenn sie leer sind).

/p.s. Nicht zu vergessen, dass der "eingelogt" status einfach per session variable zu speichern schon Fahrlässig ist...

guckmael

Hallo danke für deine antwort, stehe ich auf dem schlauch.

PHP

<?php 
session_start(); 
$user = $_POST['user']; 
$pass = $_POST['pass']; 




if(isset($_POST['user'] )) 
{
echo "Bitte geben Sie ihre User-ID ein!";
}


if(isset($_POST['pass'] )) 
{
echo "Bitte geben Sie ihr Passwort an!";
}
if(empty($_POST['user'] )) 
{
echo "Username fehlt";
}
if(empty($_POST['pass'] )) 
{
echo "Passwort fehlt";
}
if($user == "user1" && $pass == "passwort3" || $user == "user2" && $pass == "passwort4")  
{  
    $_SESSION['eingeloggt'] = TRUE; 
    $_SESSION['user'] = $user;
} 


if($_SESSION['eingeloggt'] == TRUE)  
{ 
    echo "Hallo ".$_SESSION['user']; 
    echo "<p><a href=\"daten.php\">Ihre Daten</a></p>";
    echo "<p><a href=\"ausloggen.php\">Ausloggen</a></p>";
} 


if($_SESSION['eingeloggt'] != TRUE)  
{ 
?> 


<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Username:</p> 
<p><input type="text" name="user" /></p> 
<p>Passwort:</p> 
<p><input type="password" name="pass" /></p> 
<p>&nbsp;</p> 
<p><input type="submit" value="Senden" /></p> 
</form> 


<?php  
}  
?>

Alles anzeigen

Wenn ich jetzt index.php aufrufe steht automatisch username und passwort fehlt, logge ich mich ein steht username und passwort fehlt und hallo user2.

Macht mich ganz kirre. aber bin schon ganz schön weiter gekommen. isset, empty ect....

guckmael

Zitat von Grevas

/p.s. Nicht zu vergessen, dass der "eingelogt" status einfach per session variable zu speichern schon Fahrlässig ist...

Na was muss ich dann tun, ich kanns noch nicht besser. hey moment mal das habe ich so in einem kurs der vhs gelernt. ernsthaft. das war erst letztes jahr im sommer

werde mir mal den login von quake.net anschauen.

Grevas

Dein Script macht genau das, was du verlangst. Wenn $_POST['user'] nicht gesetzt ist, kommt die Nachricht. Mach vorher ne Abfrage (die das ganze umschließt), die überprüft ob das Formular überhaupt gesendet wurde.

Dafür musst du dem submit-input noch n Namen verpassen..

HTML

<p><input type="submit" value="Senden" name="gesendet" /></p>

und in PHP

PHP

if(isset($_POST['gesendet'])) {


...


}

guckmael

Zitat von guckmael

Na was muss ich dann tun, ich kanns noch nicht besser. hey moment mal das habe ich so in einem kurs der vhs gelernt. ernsthaft. das war erst letztes jahr im sommer

werde mir mal den login von quake.net anschauen.

Könntest du freundlicherweise wenn du nochmal Zeit hast, mir schreiben wie man es anstelle macht. Event. mit Cookies?

Bandit

Eine von mehreren Möglichkeiten:

PHP

<?php 
  session_start(); 


  $errors = array();
  $error  = "";

  if(isset($_POST['user'])) 
  {
     if (empty($_POST['user']))
        $errors[] = "Username";

     if (empty($_POST['pass']))
        $errors[] = "Passwort";

     if (!count($errors))
     {
        if(($_POST['user'] == "user1" && $_POST['pass'] == "passwort3") || 
            ($_POST['user'] == "user2" && $_POST['pass'] == "passwort4"))
        {  
           $_SESSION['eingeloggt'] = TRUE; 
           $_SESSION['user'] = $_POST['user'];
           echo "Hallo ".$_SESSION['user'] .
                "<p><a href=\"daten.php\">Ihre Daten</a></p>
               <p><a href=\"ausloggen.php\">Ausloggen</a></p>";
        }
        else
          $error = "Zugangsdaten falsch";
     } 
  }

  if(! isset($_POST['user']) || count($errors) || $error != "") 
  {
     if (count($errors))
        echo "<p>" . implode (" und ", $errors) . " nicht eingegeben</p>";

     if ($error != "")
       echo "<p>$error</p>";

     echo "<form action='" . $_SERVER['SCRIPT_NAME']  . "' method='post'> 
            <p>Username:</p> 
            <p><input type='text' name='user' /></p> 
            <p>Passwort:</p> 
            <p><input type='password' name='pass' /></p> 
            <p>&nbsp;</p> 
            <p><input type='submit' value='Senden' /></p> 
           </form> ";


  }  
?>

Alles anzeigen

Formular korrekt?

Tags

Benutzer online in diesem Thema