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

[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.