Forum PHP.pl

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 

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