Witam,
Chciałem dziś napisać skrypt sprawdzający poprawność loginu hasła oraz adres e-mail nowego użytkownika przed dodaniem go do bazy. Wymodziłem coś takiego, wiem że każdy z Was pewnie napisałby to lepiej, ale chciałem aby to było moje "dzieło" a nie znalezione w necie
Mam problem.. funkcje nie działają, nie zwracają żadnych "kodów_błedu" bez względu co im zapodam..
Wygląda to tak:


functions.php

[PHP] pobierz, plaintext 
 
<?php
 
// sprawdzenie loginu
 
function sprawdz_user($user) {
 
$user_dlugosc = strlen($user);
 
 if($user_dlugosc < 4) {  // za krotki
 $kod_bledu = 1;
 
 }
 if($user_dlugosc > 10) {  // za dlugi
 $kod_bledu = 2;
 }
 
 else{
  $kod_bledu = 0;  // jest ok
 
  }
 
  return $kod_bledu;
 
}
 
 
 
// haslo
 
function sprawdz_haslo($haslo, $haslo2) {
 
$haslo_dlugosc = strlen($haslo);
 
 if($haslo_dlugosc < 4) { // za krotkie
 $kod_bledu = 3;
 }
 if($haslo_dlugosc > 10) {  // za dlugie
 $kod_bledu = 4;
 }
 if(!strcmp($haslo, $haslo2)) {
 $kod_bledu = 5;                          // nie zgadzaja sie oba hasla
 }
 if(!preg_match("/[0-9A-Za-z]/",$haslo)) {
  $kod_bledu = 6;         // zawiera zle znaki
 }
 
  else {
    $kod_bledu=0;                                         //  haslo ok, mozna je zakodowac przed wrzuceniem do bazy
    $haslo_kod = sha1($haslo . SEED);
    }
 
   return $kod_bledu;
}
 
 
 
// email
 
function  sprawdz_email($email) {
 
 if (!preg_match("/^( [a-zA-Z0-9] )+( [a-zA-Z0-9\._-] )*@( [a-zA-Z0-9_-] )+( [a-zA-Z0-9\._-] +)+$/" , $email)) {
  $kod_bledu = 7;        // zly format e-maila
 }
 
 else {
 $kod_bledu=0;           // e-mail ok
 }
 
  return $kod_bledu;
}
 
 
 
 
// wartosci bledow w zaleznosci od kodu
 
function raportuj_blad($kod_bledu) {
 
 if($kod_bledu == 0) {
  return true;
 }
 if($kod_bledu == 1) {
 $alert = "login za krótki - podaj nową nazwę użytkownika, min 4 znaki";
 }
 if($kod_bledu == 2) {
 $alert = "login za długi - podaj nową nazwę użytkownika, max 10 znaków";
 }
 if($kod_bledu == 3) {
 $alert = "haslo za krótkie - podaj nowe hasło, min 4 znaki";
 }
 if($kod_bledu == 4) {
 $alert = "hasło za długie - podaj nowe hasło, max 10 znaków";
 }
 if($kod_bledu == 5) {
 $alert = "podałeś dwa inne hasła";
 }
 if($kod_bledu == 6) {
 $alert = "hasło zawiera zabronione znaki";
 }
 if($kod_bledu == 7) {
 $alert = "niezgodny format adres e-mail";
 }
   return $alert;
?>
}
 
[PHP] pobierz, plaintext 

testowe wywołanie:
index.php

[PHP] pobierz, plaintext 
 
<?php require_once 'functions.php'; 
 
$user = "piotrek";
$haslo = "haselko";
$haslo2 = "haselko";
$email = "mail@o2.pl";
 
sprawdz_user($user);
raportuj_blad($kod_bledu);
echo $alert;
 
 
//sprawdz_haslo($haslo, $haslo2);
//raportuj_blad($kod_bledu);
//echo $alert;
 
//sprawdz_email($email);
//raportuj_blad($kod_bledu);
//echo $alert;
 
?>
[PHP] pobierz, plaintext 

pewnie jest jakis głupi błąd.. ale nie mogę fo znaleść wygląda to tak jakby funkcje nie miały zamiaru się wykonać

[PHP] pobierz, plaintext 
$a=sprawdz_user($user);
$b=raportuj_blad($a);
echo $b;
 
echo raportuj_blad(sprawdz_user($user));
[PHP] pobierz, plaintext 

1.Nie przypisujesz nigdzie wartości zwracanych przez funkcje, więc $kod_bledu w wywołaniu f-cji raportuj_blad() nie ma wartości, tak samo, jak póxniej $alert, który chcesz wyświetlić:

[PHP] pobierz, plaintext 
sprawdz_user($user);
raportuj_blad($kod_bledu);
echo $alert;
 
[PHP] pobierz, plaintext 

Powinno być:

[PHP] pobierz, plaintext 
$kod_bledu = sprawdz_user($user);
$alert = raportuj_blad($kod_bledu);
echo $alert;
[PHP] pobierz, plaintext 

2. W functions.php ostatni nawias zamykający masz ZA ?> a powinien byc PRZED.
Zainstaluj sobie IDE podświetlające błędy składni, będzie Ci łatwiej (np. NetBeans)

aaaa dziękuje bardzo
Składnia jest ok, przepisując ją tutaj pomyliłem ten nawias..

Wielkie dzięki, taki prosty błąd..

Jeszcze pytanie z mojej strony, licząc na Wasze doświadczenie.. Takie sprawdzenie danych przed zapisem do bazy jest wystarczające? na co moge jesze rzucic warunki?

no bo nawet jeśli błędnych danych jest więcej niż 1 to

[PHP] pobierz, plaintext 
echo $alert;
[PHP] pobierz, plaintext 

poda tylko jeden komunikat

Generalnie skrypt jest przekombinowany i wszystko dałoby się zapisać w 20% linijek tego, co Ty napisałeś, ale dobra, uczysz się.
Co robisz źle?
- niepotrzebnie eliminujesz z hasła znaki inne niz litery i cyfry. dobre hasło wręcz powinno zawierac np. znaki specjalne (!,#,$ itp.)
- brakuje walidacji, czy dany login i email juz istnieją w bazie (generalnie taka sytuacja nie powinna miec miejsca, no chyba, że logika aplikacji dopuszcza taką sytuację).


Co do pytania, czy jest wystarczające: to zależy od wymagań aplikacji.

a to nie powinno być tak?

[PHP] pobierz, plaintext 
$blad = sprawdz_user($user);
$komunikat = raportuj_blad($blad);
echo $komnikat;
[PHP] pobierz, plaintext 

jeszcze raz dziękuje, wiem że ktoś bardziej obyty z PHP napisałby to w kilku linijkach.. ale chciałem zrobić to sam
Pozdr