PHP if, esleif else zusammenziehen?

Unregistriert

Hallo Forum,

ich versuche die Bedingungen ordnungsgemäß auszugeben.

PHP

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


	echo "<p>Username: ".$user. "</p>";
	echo "<p>Passwort: ".$pass. "</p>";


	if( $user == "" ) 
	{
		echo "<p>Bitte Usernamen eingeben!</p>";
	}
	elseif ( $pass == "" )
	{
		echo "<p>Bitte Passwort angeben!</p>";
	}
	else 
	{
		echo "Bitte Usernamen und Passwort eingeben!";
	}
	?>

Alles anzeigen

Er zeigt mir aber nun nicht beides an wenn beide Felder leer gelassen wurden.

Gruß Nico

synaptic

naja stopf des doch in eine bedingung

PHP

if( $user == "" && $pass == ""){
     echo "Bitte Usernamen und Passwort eingeben!"; 
 }else{
    if( $user == "" ) {
        echo "<p>Bitte Usernamen eingeben!</p>"; 
    } 
    if ( $pass == "" ) {
        echo "<p>Bitte Passwort angeben!</p>"; 
    } 
  }

-Bambino-

Ja, wenn beide Fehlen, wird er bei dir ja schon bei der ersten abfrage was ausgebene, und aufgrund der elseifs ja dann aufhören.

Mit Synaptics Methode klappt es =)

Unregistriert

danke für die Hilfe, eventuell bin ich nu auch schon ein Stück weiter

PHP

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


	 if( $user == "" && $pass == "")
	{
		echo "<p>Username:</p>";
		echo "<form action=\"login.php\" method=\"post\">";
		echo "<p><input type=\"text\" name=\"user\" /></p>";
		echo "<p>Passwort:</p>";
		echo "<p><input type=\"password\" name=\"pass\" /></p>";
		echo "<p><input type=\"submit\" value=\"Senden\" /></p>";
		echo "</form>";
     		echo '<p>Bitte Usernamen und Passwort eingeben!</p>'; 

 	}
	else
	{
    		if( $user == "" ) 
		{
        			echo '<p>Bitte Usernamen eingeben!</p>'; 
    		} 
   		if ( $pass == "" ) 
		{
        			echo '<p>Bitte Passwort angeben!</p>'; 
    		} 
 	 } 
	if ( $user == "User" || $pass == "xxx" )
	{
		echo "<p>Username: ".$user. "</p>";
		echo "<p>Passwort: ".$pass. "</p>";
		echo '<p>Zugang gestattet</p>';
	}
	?>

Alles anzeigen

Wenn jetzt nur der Username eingegeben wird dann erscheint trotzdem folgende Meldung

Zugang gestattet!

Ich versuche eben mal das Formular in eine extra Datei zu packen , is glaub besser...

Ich werde mich hier wohl registrieren --)

-Bambino-

PHP

if ( $pass == "" )  
        { 
                    echo '<p>Bitte Passwort angeben!</p>';  
            }  
      }  
   if ( $user == "User" || $pass == "xxx" ) /// Hier liegt der Fehler
    { 
        echo "<p>Username: ".$user. "</p>"; 
        echo "<p>Passwort: ".$pass. "</p>"; 
        echo '<p>Zugang gestattet</p>'; 
    }

Alles anzeigen

Die Abfrage muss mit einem && oder and kommen.
Andernfalls reicht es ihm ja schon, wenn nur der Name eingegeben wird.

PHP

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


     if( $user == "" && $pass == "") 
    { 
        echo "<p>Username:</p>"; 
        echo "<form action=\"login.php\" method=\"post\">"; 
        echo "<p><input type=\"text\" name=\"user\" /></p>"; 
        echo "<p>Passwort:</p>"; 
        echo "<p><input type=\"password\" name=\"pass\" /></p>"; 
        echo "<p><input type=\"submit\" value=\"Senden\" /></p>"; 
        echo "</form>"; 
             echo '<p>Bitte Usernamen und Passwort eingeben!</p>';  

     } 
    else 
    { 
            if( $user == "" )  
        { 
                    echo '<p>Bitte Usernamen eingeben!</p>';  
            }  
           if ( $pass == "" )  
        { 
                    echo '<p>Bitte Passwort angeben!</p>';  
            }  
      }  
    if ( $user == "User" && $pass == "xxx" ) /// Jetzt mit &&
    { 
        echo "<p>Username: ".$user. "</p>"; 
        echo "<p>Passwort: ".$pass. "</p>"; 
        echo '<p>Zugang gestattet</p>'; 
    } 
    ?>

Alles anzeigen

Bandit

Und hier mal eine elegantere Lösung:

PHP

<?php
  $errors = array();
 
  if (empty($_POST['user']))
    $errors[] = "Username";
 
  if (empty($_POST['pass']))
    $errors[] = "Passwort";
 
  if (count($errors))
    echo "Bitte " . implode (" und ", $errors) . " eingeben";
?>

Alles anzeigen

<edit>
Und so könnte das komplett aussehen:

PHP

<?php
  $errorText = "";
 
  if (isset($_POST['user']))
  {
     $errors = array();
 
     if (empty($_POST['user']))
       $errors[] = "Username";
 
     if (empty($_POST['pass']))
       $errors[] = "Passwort";
 
     if (count($errors) == 0)
     {
        if($_POST['user'] == "user" && $_POST['pass'] == "pass")
          echo '<p>Zugang gestattet</p>'; 
        else
          $errorText = "Zugangsdaten falsch"; 
     }
     else
       $errorText = "Bitte " . implode (" und ", $errors) . " eingeben";
  }
 
  if (! isset($_POST['user']) || strlen ($errorText))
  {
     if (strlen($errorText))
       echo "<p>$errorText</p>\n";
 
     echo "<form action='login.php' method='post'>  
            <p>Username: <input type='text' name='user' /></p>
            <p>Passwort: <input type='password' name='pass' /></p>
            <p><input type='submit' value='Senden' /></p>
           </form>";  
 
  }  
?>

Alles anzeigen

</edit>

Unregistriert

Zitat von bandit600

Und hier mal eine elegantere Lösung:

<edit>
Und so könnte das komplett aussehen:

PHP

<?php
  $errorText = "";
 
  if (isset($_POST['user']))
  {
     $errors = array();
 
     if (empty($_POST['user']))
       $errors[] = "Username";
 
     if (empty($_POST['pass']))
       $errors[] = "Passwort";
 
     if (count($errors) == 0)
     {
        if($_POST['user'] == "user" && $_POST['pass'] == "pass")
          echo '<p>Zugang gestattet</p>'; 
        else
          $errorText = "Zugangsdaten falsch"; 
     }
     else
       $errorText = "Bitte " . implode (" und ", $errors) . " eingeben";
  }
 
  if (! isset($_POST['user']) || strlen ($errorText))
  {
     if (strlen($errorText))
       echo "<p>$errorText</p>\n";
 
     echo "<form action='login.php' method='post'>  
            <p>Username: <input type='text' name='user' /></p>
            <p>Passwort: <input type='password' name='pass' /></p>
            <p><input type='submit' value='Senden' /></p>
           </form>";  
 
  }  
?>

Alles anzeigen

</edit>

Und kann ich in deinem Script die $_POST Variablen nicht in variablen abspeichern?

Gruß Nico

Bandit

Wofür?

Unregistriert

Bin Änfänger.... der Übersicht halber, ich möchte die befehle verstehen. Ich kann es für mich so besser verstehen und kürzer is es doch auch....

Gruß Nico

synaptic

es soll kürzer sein, wenn man zeilen hinzufügt um daten aus dem $_POST in variablen zu speichern?
kann deine logik da nicht nachvollziehen

Bandit

"Kürzer" ist kein Argument. So kannst du an allen Stellen des Scriptes erkennen, wo die Variable herkommt. Ist bei längeren Scripte hilfreich. Es ist also relativ sinnlos, wenn man

$blub = $_POST['blub'];

schreibt.

Unregistriert

ja in meinem Fall würde es dann wohl

PHP

$user = $_POST['user'];
$pass = $_POST['pass'];




....

heissen. Natürlich keine unsinnigen Variablennamen :p

@ Ich würde gerne noch wissen wie sich die ersten beiden if abfragen erläutern...

PHP

if (isset($_POST['user']))   // wenn variable existiert
  		{
     		$errors = array(); // erstelle ein array unter $errors
 
     		if (empty($_POST['user'])) //wenn variable nicht gesetzt erstelle automatische den Namen: Username
       		$errors[] = "Username";

Die Kommentare sind so wie ich es verstehe.
Und warum die Var errorText am Anfang leer ist?

Gruß Nico

Bandit

PHP

<?php 
  // Variable initialisieren, weil darauf später auf jeden Fall zugegriffen wird
  $errorText = ""; 

  // Prüfen, ob $_POST['user'] gesetzt ist
  if (isset($_POST['user'])) 
  { 
     // Leeres Fehler-Array erstellen
     $errors = array(); 

     // Wenn $_POST['user'] leer ist, Username ins Array schreiben
     if (empty($_POST['user'])) 
       $errors[] = "Username"; 

     // Wenn $_POST['pass'] leer ist, Passwort ins Array schreiben
     if (empty($_POST['pass'])) 
       $errors[] = "Passwort"; 

     // Prüfen, ob es einen Fehler gab
     if (count($errors) == 0) 
     { 
        // Prüfen, ob die Daten richtig sind
        if($_POST['user'] == "user" && $_POST['pass'] == "pass") 
          echo '<p>Zugang gestattet</p>';  // Alles paletti
        else 
          $errorText = "Zugangsdaten falsch";  // Fehlertext setzen
     } 
     else 
       $errorText = "Bitte " . implode (" und ", $errors) . " eingeben"; // Fehlertext setzen
  } 

  // Wenn $_POST['user'] nicht exsitiert oder wenn es einen Fehler gab
  if (! isset($_POST['user']) || strlen ($errorText)) 
  { 
     // Wenn ein Fehlertext vorhanden ist, ausgeben. Deshalb auch oben die Initialisierung
     if (strlen($errorText)) 
       echo "<p>$errorText</p>\n"; 

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

  }   
?>

Alles anzeigen

-Bambino-

Zitat von Unregistriert

Und kann ich in deinem Script die $_POST Variablen nicht in variablen abspeichern?

Gruß Nico

Auf der anderen Seite wirst du die POST oder GET Daten aber noch weiter verarbeiten müssen, denn eine ungesicherte Usereingabe ist in einem Loginscript nicht zu empfehlen.

The User

Das beste ist, wenn du Funktionen und Klassen benutzt. Dann kannst du die POST-Variablen als Argument an die Funktion login übergeben und login macht dann damit, was es für nötig hält.

Viele liebe Grüße
The User

Unregistriert

Ich habe das Script jetzt mal etwas umgeschrieben, so dass man mit den Variablen bzw usereingaben (username, passwort ) noch geschickter weiterarbeiten kann:

PHP

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


	$errorText = ""; 			// Variable initialisieren

	if ( isset ( $user ) )  		// Prüfen, ob $user gesetzt ist
  	{ 
	$errors = array();  			// Leeres Fehler-Array erstellen

     	if ( empty ( $user ) ) 		// Wenn $user leer ist, Username ins Array schreiben
       	$errors[] = "Username"; 

     	if ( empty( $pass ) ) 		// Wenn $pass leer ist, Passwort ins Array schreiben
       	$errors[] = "Passwort"; 

     	if ( count ( $errors ) == 0 ) 		// Prüfen, ob es einen Fehler gab
    	{ 

        	if( $user == "user" && "pass" == $pass )  
	{
		$_SESSION['eingeloggt'] = TRUE;
	}
	if ( $_SESSION['eingeloggt'] == TRUE )
	{
		echo "<p>Hallo ". $user. "</p>";
		echo '<ul>';
		echo '<li><a href="daten.php">Eigene Daten</a></li>';
		echo '<li><a href="logout.php">Logout</a></li>';
		echo '</ul>';
	}
        	else 
          	$errorText = "<p>Zugangsdaten falsch</p>";  					// Fehlertext setzen
     	} 
     	else 
       	$errorText = "<p>Bitte " . implode (" und ", $errors) . " eingeben</p>"; 		// Fehlertext setzen
  	} 


  	if ( ! isset ( $user ) || strlen ( $errorText ) ) 		// Wenn $user nicht exsitiert oder wenn es einen Fehler gab
  	{ 

     	if ( strlen ( $errorText ) ) 			// Wenn ein Fehlertext vorhanden ist, ausgeben. Deshalb auch oben die Initialisierung
       	echo "<p>$errorText</p>\n"; 

     	echo "<form action='login.php' method='post'>   
            	<p>Username: <input type='text' name='user' /></p> 
            	<p>Passwort: <input type='password' name='pass' /></p> 
            	<p><input type='submit' value='Senden' /></p> 
           	</form>";
	}   
?>

Alles anzeigen

Ich versuche nun eine Session Variable zu setzten und wenn diese wahr ist darf man weiter. Ich bin nicht ganz sicher ob ich verstehe was ich tue, auf jeden Fall ward ihr mir hier eine große Hilfe bisher :p

Das ist die Logout.php

PHP

<?php 
                          session_start();
		session_destroy();
		echo 'Sie wurden erfolgreich abgemeldet!';

	?>

Und das der interne bereich:

PHP

<?php 
	if ( $_SESSION['eingeloggt'] == TRUE )
	{
	echo "Hallo ".$user;
}
else 
{
echo 'NO';
}
?>

Leider funktioniert noch nicht alles so. z.b wenn ich nur Username eingebe dann erscheint trotzdem nicht mehr der Startseiten Inhalt also Herzlich Willkommen auf bla..bla..blabla

Gruß Nico

Bandit

Schreibe manan den Anfang deines Scriptes diese Zeilen:

PHP

error_reporting(E_ALL);
ini_set("display_errors", true);

Dann siehst du, wie geschickt deine Version ist.

Und wieso sieht dein Quellcode so zerschossen aus? Das sah bei mir ordentlicher aus.

Aus der Doku:

Zitat

Beispiel #1 Löschen einer Session mit $_SESSION
<?php
// Initialisierung der Session.
// Wenn Sie session_name("irgendwas") verwenden, vergessen Sie es
// jetzt nicht!
session_start();

// Löschen aller Session-Variablen.
$_SESSION = array();

// Falls die Session gelöscht werden soll, löschen Sie auch das
// Session-Cookie.
// Achtung: Damit wird die Session gelöscht, nicht nur die Session-Daten!
if (ini_get("session.use_cookies")) {
$params = session_get_cookie_params();
setcookie(session_name(), '', time() - 42000, $params["path"],
$params["domain"], $params["secure"], $params["httponly"]
);
}

// Zum Schluß, löschen der Session.
session_destroy();
?>

Alles anzeigen

Unregistriert

Zitat von bandit600
Schreibe manan den Anfang deines Scriptes diese Zeilen:
PHP
error_reporting(E_ALL);
ini_set("display_errors", true);
Dann siehst du, wie geschickt deine Version ist.

Hab ich es erscheint keine meldung bzw local ist es glaub eingestellt. vorher kamen mal fehler aber jetzt keine mehr :p

Zitat von bandit600

Und wieso sieht dein Quellcode so zerschossen aus? Das sah bei mir ordentlicher aus.

Entschuldige ich musste ihn etwas trennen um den Überblick zu bekommen was geschieht, wenn alles noch enger steht kann ich ihn fast nicht mehr lesen und die befehle auseinanderhalten. Wir machen noch nicht so lange PHP...

@ ich hatte in der logout datei noch versucht mit

PHP

Header("Location: http://127.0.0.1/index.php"); 
exit();

auf die indexseite zurückzuleiten, dies mislang auch

Bandit

Hier

PHP

$user = $_POST['user']; 
$pass = $_POST['pass'];

kommt bei hochgedrehtem Error-Reporting garantiert eine Notice-Meldung, wenn das Script das erste Mal augerufen wird!

Zitat von Unregistriert

Entschuldige ich musste ihn etwas trennen um den Überblick zu bekommen was geschieht, wenn alles noch enger steht kann ich ihn fast nicht mehr lesen und die befehle auseinanderhalten. Wir machen noch nicht so lange PHP...

Und deshalb sind die Einrückungen alle hinüber???

Unregistriert

Zitat von bandit600
Hier
PHP
$user = $_POST['user']; 
$pass = $_POST['pass'];
kommt bei hochgedrehtem Error-Reporting garantiert eine Notice-Meldung, wenn das Script das erste Mal augerufen wird!

Ist es möglich zu sagen das nur wenn das script arbeitet das die variablen dann gesetzt werden?
für weitere Uberprüfungen erscheint mir das logisch sinnvoll

Zitat von bandit600

Und deshalb sind die Einrückungen alle hinüber???

Bei mir im Quellcode sieht es ganz anders aus, alles ab <?php in einer Reihe :p

Gruß Nico

Tags