Skrócenie sprawdzania if(isset($_POST['zmienna']) dla kilku zmiennych, Optymalizacja kodu pod względem czytelności i objętności :) Opcje

[Apocalyptiq]

Dosyć uciążliwe jest sprawdzanie wszystkich potrzebnych zmiennych dla obsługi każdego formularza danego serwisu w php (szczególnie dotkneło mnie to podczas pisania gry via www, gdzie jest pełno formularzy):

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['login']) || empty($_POST['pwd']) || empty($_POST['pwd2']) || empty($_POST['email']) || empty($_POST['teamName']) || empty($_POST['teamColor']) || empty($_POST['teamSubColor']))
        $_SESSION['mesg']='Wypełnij wszystkie wymagane pola!';
    else{...}
?>
[PHP] pobierz, plaintext 

I tego typu fragmentów kodu mam tam sporo, ale to pytam ogólnie, co do wszelkich projektów Narazie wymyśliłem coś takiego:

[PHP] pobierz, plaintext 
<?php
# isset dla określonych zmiennych w tablicy
function isset_array_vars($array,$vars){
    $notset_vars=array();
    foreach($vars as $key => $var)
        if(!isset($array[$key])) $notset_vars[]=$var;
    if($notset_vars) return implode(', ',$notset_vars);
    else return true;
}
 
# zastosowanie:
    $check_isset=isset_array_vars($_POST,array('login'=>'login','pwd'=>'hasło','pwd_repeat'=>'powtórzone hasło'));
    if($check_isset!==true)
        $_SESSION['mesg']='Nie podałeś wszystkich wymaganych danych ('.$check_isset.')';
    else{...}
?>
[PHP] pobierz, plaintext 

Jeżeli wszystkie podane zmienne są ustawione w danej tablicy, funkcja zwraca true, w przeciwnym wypadku - wyprzcinkowane nazwy (spolszczone, zrozumiałe dla człowieka) niepodanych zmiennych. No ale jest to jakoś mi sie wydaje na siłe zrobione, mam przeczucie że można to zrobić lepiej :-)
Macie jakieś pomysły albo patenty ze swoich projektów?

Ten post edytował Apocalyptiq 19.04.2009, 10:27:46

[kantek]

Czy na pewno to działa??

Bo tam masz sprawdzanie isset($tablica['klucz']) np. $_POST['login'],
a jak zatwierdzisz submitem formularz i pola są niewypełnione
to i tak te zmienne z tablicy post 'są set' tyle ze są empty.

Pozdrawiam

[Crozin]

Yyy...

[PHP] pobierz, plaintext 
<?php
 
if(!isset($_POST['username'], $_POST['password'], $_POST['abc'])){
  
}
?>
[PHP] pobierz, plaintext 

A co do tego by to automatycznie zwracało jakieś komunikaty... zainteresuj się pojęciem walidatora bo żadko kiedy będziesz ograniczął się do sprawdzenia podano/nie podano.

[Kildyt]

Po co foreach? Ja bym to zrobił na pętli. For będzie najlepsze.
Jak już kantek napisał: rozróżnij isset i empty.

[PHP] pobierz, plaintext 
<?php
$check_isset=isset_array_vars($_POST,array('login'=>'login','pwd'=>'hasło','pwd_repeat'=>'powtórzone hasło'));
   if($check_isset!==true)
       $_SESSION['mesg']='Nie podałeś wszystkich wymaganych danych ('.$check_isset.')';
?>
[PHP] pobierz, plaintext 

Zmieniłbym na:

[PHP] pobierz, plaintext 
<?php
if($check_isset=isset_array_vars($_POST,array('login'=>'login','pwd'=>'hasło','pwd_repeat'=>'powtórzone hasło')))
       $_SESSION['mesg']='Nie podałeś wszystkich wymaganych danych ('.$check_isset.')';
?>
[PHP] pobierz, plaintext 

TRUE powinno zwracać, jeżeli któreś pole zostało niewypełnione, a FALSE gdy wszystko jest dobrze.

[pejott]

TRUE powinno zwracać, jeżeli któreś pole zostało niewypełnione, a FALSE gdy wszystko jest dobrze.

Trochę to nielogiczne, zważywszy na nazwę funkcji.
Poza tym, foreach to też pętla.
Już ktoś napisał, abyś zainteresował się walidatorami, bo zazwyczaj sprawdzasz kilka rzeczy na raz.

Na przykład, http://framework.zend.com/manual/en/zend.validate.html.

Pozdrawiam.

[Kildyt]

Trochę to nielogiczne, zważywszy na nazwę funkcji.
Poza tym, foreach to też pętla.

No tak, to już zależy od programisty jak sobie wszystko ustali, jednak napisałem tak, gdyż moim zdaniem prościej jest najpierw podawać komunikaty o błędzie nie "wykrzykując" warunku.
Wiem, że forach to pętla. Trochę nielogicznie napisałem posta. Przepraszam. Chodzi mi o to, że IMHO lepiej i łatwiej można by to zrobić na pętli for.

[#luq]

Ja bym napisał tak:

[PHP] pobierz, plaintext 
<?php
function validArray( $arr, $value, &$error ){
    $cntValue = count( $value );
    for( $i = 0; $i < $cntValue; $i++ ){
       if( !array_key_exists( $value[$i], $arr ) ){
           $error[] = $value[$i];    
       }
       elseif( empty( $arr[ $value[$i] ] ) ){
           $error[] = $value[$i];
       }
    }
 
    // return
    if( count( $error ) == 0 ){
        return true;
    }
    else{
        return false;
    }
}
?>
[PHP] pobierz, plaintext 

Przykład:

[PHP] pobierz, plaintext 
<?php
$table = array( 
    'login' => 'luq',
    'pass' => 'asdf',
    'data' => ''       
);
 
$mustBe = array(
    'login', 'pass', 'data', 'id'
);
 
if( !validArray( $table, $mustBe, $err ) ){
    echo '<pre>';
    print_r( $err );
    echo '</pre>';
}
?>
[PHP] pobierz, plaintext 

Zwróci false i wyświetli:

Kod

   Array
   (
       [0] => data
       [1] => id
   )