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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) 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? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

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