Skrypt pokazuje tylko jeden komunikat, (powinien 4) Opcje

[jarekbutek]

Dlaczego poniższy skrypt wyrzuca tylko jeden komunikat (Nie podałeś emailu) po pozostawieniu wszystkich pól forma pustych, chociaż powinien pokazać ich trzy.

Oto kod forma:

[PHP] pobierz, plaintext 
<?
include "mysql_config.php";
$action = $_GET['action'];
$tabela = "users";
 
if(isset($action))
{
  if($action=="rejestruj")
  {
    if(!isset($_POST['submit']))
    {
      echo '<form name="rejestracja" action="index.php?op=user&action=rejestruj" method="post">
      <table>
      <tr>
      <td>Nazwa użytkownika <font color="red">*</font></td><td><input name="nazwa_usera" type="text" value="" /></td>
      </tr>
      <tr>
      <td>Hasło <font color="red">*</font></td><td><input type="password" name="haslo1" value="" /></td>
      </tr>
      <tr>
      <td>Potwierdź hasło <font color="red">*</font></td><td><input type="password" name="haslo2" value="" /></td>
      </tr>
      <tr>
      <td>Email <font color="red">*</font></td><td><input name="email" type="text" value="" /></td>
      </tr>
      <tr>
      <td><input type="submit" name="submit" value="Rejestruj" /></td><td><input type="reset" value="Wyczyść" /></td>
      </tr>
      </table>
      </form>
      <br>Pola z <font color="red">*</font> gwiazdką są wymagane.';
    }
    if(isset($_POST["submit"]))
    {
      $login = $_POST["nazwa_usera"];
      $haslo1 = $_POST["haslo1"];
      $haslo2 = $_POST["haslo2"];
      $email = $_POST["email"];
      
      $sql_sprawdz_login = mysql_query("SELECT * FROM users WHERE login='$login'");
      $sql = mysql_fetch_array($sql_sprawdz_login);
      if(empty($login))
        {
   $blad = "Nie wpisałeś nazwy użytkownika<br>";
     }
  
      if($login==$sql["login"])
      {
        $blad ="Użytkownik istnieje.<br>";
      }
if(empty($haslo1))
 {
  $blad = "Nie wpisałeś hasła<br>";
}
   
     if($haslo1!=$haslo2)
     {
       $blad = "Podane hasła się nie zgadzają<br>";
     }
     
   
   if(empty($email))
   {
     $blad = "Nie podałeś emailu<br>";
   }
   
     function  ValidEmail($email) {
     if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$" , $email)) {
      return false;
  }
  return true;
 }
     if(!empty($email) && ValidEmail($email))
     {
       $emailtrue = true;
     }
     elseif(!empty($email))
     {
       $emailtrue = false;
       $blad = "Niepoprawny email<br>";
     }
   
   
   if(!empty($login) && !empty($haslo1) && !empty($haslo2)&& $haslo1==$haslo2 && !empty($email) && $emailtrue==true)
   {
     mysql_query("INSERT INTO users SET login='$login', pass='".md5($haslo1)."', email='$email', activation_key='".base64_encode($login.$haslo1.$email)."', active='no';");
     echo"Jesteś już zarejestrowany";
   }
   
  
     }
      
    if($blad)
    {
      echo "$blad";
    }
    }
}
  
 
?>
[PHP] pobierz, plaintext 

[porady-it.pl]

Komunikaty z błędami powinny być doklejane do zmiennej $blad np.

[PHP] pobierz, plaintext 
<?php
$blad .= "Jakiś błąd.";
?>
[PHP] pobierz, plaintext 

[gebp]

[PHP] pobierz, plaintext 
<?php
$a=1;
echo $a; // 1
$a=2;
echo $a; // 2
 
$a=4;
$a=5;
 
echo $a; // 5
 
$a=array();
 
$a[]=6;
$a[]=7;
$a[]=8;
 
foreach($a AS $val)
 { echo $val."<br/>"; }
 
// 6
// 7
// 8
 
?>
[PHP] pobierz, plaintext 

[tb82]

$sql_sprawdz_login = mysql_query("SELECT * FROM users WHERE login='$login'") - tu jest błąd

Powinno być: SELECT * FROM users WHERE login='" . $login . "'";

[Spawnm]

tb82, wiesz czym się różni " od '

Ten post edytował Spawnm 25.04.2009, 18:20:58

[tb82]

tb82, wiesz czym się różni " od '

wiem..a czemu ?

[Spawnm]

skoro wiesz to czemu proponujesz zamianę z
"SELECT * FROM users WHERE login='$login'"
na
SELECT * FROM users WHERE login='" . $login . "'";

działanie będzie to samo , a " używamy aby nie bawić się z kropkami.

[Kildyt]

tb82 polecam podstawy kursu php.

Zauważ, że za każdym razem gdy chcesz zapisać treść komunikatu do zmiennej $blad (na przyszłość używaj angielskich wyrazów) to stara zawartość zostaje nadpisana. Musisz dopisać kolejny komunikat do poprzedniego.

Rozwiązanie:

[PHP] pobierz, plaintext 
<?php
$blad = $blad.'tresc';
 
// czyli
$blad .= 'tresc';
?>
[PHP] pobierz, plaintext 

To zauważyłem na pierwszy rzut oka. Jeżeli nie o to chodzi o sory.

[jarekbutek]

Dzięki Kildyt o to chodziło.