[PHP]Problem ze skryptem rejestracyjnym Opcje

[e-milnet]

Skrypt po uzupełnieniu wszystkich pól w formularzu nic nie zwraca i nie zapisuje użytkownika w bazie.

[PHP] pobierz, plaintext 
  $email=$_POST['email'];
  $name_us=$_POST['name_us'];
  $password=$_POST['password'];
  $password2=$_POST['password2'];
 
  session_start();
 
  require_once('functions.php'); 
  $web->Add_Stylesif("style.css"); 
 
  try
  {
 
    if (!filled($_POST))
    {
      throw new Exception('Formularz wypełnony nieprawidłowo - proszę wrócić i spróbować ponownie.');  
    }
 
    if (!normal_email($email))
    {
      throw new Exception('Nieprawidłowy adres poczty elektronicznej - proszę wrócić i spróbować ponownie.');
    }
 
    if ($password != $passowrd2)
    {
      throw new Exception('Niepasujące do siebie hasła - proszę wrócić i spróbować ponownie.');
    } 
 
    if (strlen($name_user) > 16)
    {
      throw new Exception('Nazwa użytkownika nie może mieć więcej niż 16 znaków - proszę wrócić i spróbować ponownie.');
    } 
 
    if((strlen($password) < 6) || (strlen($password) > 16))
    {
      throw new Exception('Hasło musi mieć co najmniej 6 i maksymalnie 16 znaków - proszę wrócić i spróbować ponownie.');
    }
 
    register($name_user, $email, $password);
    $_SESSION['normal_user'] = $name_us;
 
    $web->site_name="Rejestracja pomyślna";
    $xyz="<center>Rejestracja zakończyła się sukcesem!!! <br>".create_HTML_URL('member.php', 'Konfiguracja użytkownika')."</center>";
    $web->Add_textbox($xyz);
 
  }
  catch (Exception $e)
  {
    $web->site_name="Problem";
    $web->Add_textbox($e->getMessage());
    exit;
  }
[PHP] pobierz, plaintext 

[mortus]

W liniach 29 i 39 masz zmienną $name_user, która nie jest nigdzie zdefiniowana. session_start() daj na sam początek. Walidacja powinna odbywać się w momencie pobierania danych z tablicy $_POST, a nie później. Błędy raczej zapisywałbym do odpowiedniej tablicy, zamiast rzucać non stop wyjątkami. Dlaczego? Choćby dlatego, że jeśli email będzie nieprawidłowy i jednocześnie hasła nie będą się zgadzać, to wyjątki zostaną przechwycone dwa razy, za każdym razem z innym komunikatem błędu. Jeśli zatem użytkownik wszystkie pola wypełni nieprawidłowo, to aż cztery razy otrzyma komunikat błędu, za każdym razem inny, a chyba powinien od razu wiedzieć, które ze wszystkich pól wypełnił źle. Mnie osobiście skłoniłoby to do opuszczenia strony. Blok try ... catch ... nie jest dobrym pomysłem na walidację danych z formularza.

Radzę zapoznać się z tematem Jak poprawnie zadać pytanie - są tam zawarte podstawy dotyczące debugowania skryptów, z którymi warto się oswoić.

[e-milnet]

Wstawiłem funkcję włączające wszystkie komunikaty o błędach do skryptu z pliku functions.php. Poprawiłem swój skrypt do postaci:

[PHP] pobierz, plaintext 
  session_start();
 
  $email=$_POST['email'];
  $name_us=$_POST['name_us'];
  $password=$_POST['password'];
  $password2=$_POST['password2'];
 
  require_once('functions.php'); 
  $web->Add_Stylesif("style.css"); 
 
    if (!filled($_POST))
    {
      $errors[]='Formularz wypełnony nieprawidłowo - proszę wrócić i spróbować ponownie.';  
    }
 
    if (!normal_email($email))
    {
      $errors[]='Nieprawidłowy adres poczty elektronicznej - proszę wrócić i spróbować ponownie.';
    }
 
    if ($password != $password2)        
    {
      $errors[]='Niepasujące do siebie hasła - proszę wrócić i spróbować ponownie.';
    } 
 
    if (strlen($name_us) > 16)
    {
      $errors[]='Nazwa użytkownika nie może mieć więcej niż 16 znaków - proszę wrócić i spróbować ponownie.';
    } 
 
    if((strlen($password) < 6) || (strlen($password) > 16))
    {
      $errors[]='Hasło musi mieć co najmniej 6 i maksymalnie 16 znaków - proszę wrócić i spróbować ponownie.';
    }
 
    if(!$errors)
    {
      register($name_us, $email, $password);
      $_SESSION['normal_user'] = $name_us;
      $web->site_name="Rejestracja pomyślna";
      $xyz="<center>Rejestracja zakończyła się sukcesem!!! <br>".create_HTML_URL('member.php', 'Konfiguracja użytkownika')."</center>";
      $web->Add_textbox($xyz);
    }
    else
    {
      $xyz = "";
      $web->site_name="Problem";
      foreach($errors as $a)
      {
       $xyz .= $a."\n 
      }
      $web->Add_textbox($xyz);
    }
[PHP] pobierz, plaintext 

Teraz wyskakuje mi błąd:
Notice: Undefined variable: menu in /home/alleg/public_html/e-milnet/functions/functions_show.php on line 14
W 8 lini tego kodu:

[PHP] pobierz, plaintext 
function head_menu()
  {
    $table["start"]="index.php";
    $table["CV"]="cv.php";
 
    foreach ($table as $name => $value)
    {
      $menu .= "\t\t\t\t <a href=\"$value\"><div id=\"button\">$name</div></a>\n";
    }
    return $menu;
  }
[PHP] pobierz, plaintext 

i dalej skrypt nic nie wykonuje.

[mortus]

W funkcji head_menu() w linii 5 powinieneś zmienną $menu zainicjować pustym łańcuchem znaków: $menu = '';. Zamiast if(!errors) użyj if(empty($errors)). No i zobacz na linię 50, bo nie domknąłeś tam łańcucha znaków, w wyniku czego reszta się sypie. Linie 3-6 włącznie powinny się znaleźć w bloku if(empty($errors)) w linii 37.

EDIT
I jeszcze

[PHP] pobierz, plaintext 
if(!normal_email($_POST['email'])) {
// ...
if($_POST['password'] !=  $_POST['password2']) {
// ...
[PHP] pobierz, plaintext 

Ten post edytował mortus 4.09.2011, 11:41:24

[e-milnet]

Teraz podczas rejestracji wyskakuje mi błąd:
Notice: Undefined property: mysqli::$num_rows in /home/alleg/public_html/e-milnet/functions/functions_authen.php on line 39
W 13 lini funkcji:

[PHP] pobierz, plaintext 
  function register($name_user, $email, $password)
  {
 
    $connect = connect_bd();
 
    $result = $connect->query("select * from users where name_user='".$name_user."'");
 
    if (!$result)
    {
      $errors[]='Wykonanie zapytania nie powiodło się.';
    }
 
    if ($connect->num_rows>0)
    {
      $errors[]='Nazwa użytkownika zajęta - proszę wrócić i wybrać inną.';
    }
 
    $result = $connect->query("insert into users values ('".$name_user."', sha1('".$password."'), '".$email."')");
 
    if (!$result)
    {
      $errors[]='Rejestracja w bazie danych niemożliwa - proszę spróbować później.';
    }
 
    if(empty($errors))
    {
      return true;
    }
    else
    {
      return $errors;
    }
 
  }
[PHP] pobierz, plaintext 

ale dodaje uzytkownika do bazy.

Ten post edytował e-milnet 5.09.2011, 15:55:00