Klasa walidacji, Nie dziala jesli jest wiecej zmiennych Opcje

[marcio]

Witam napisalem sobie szkielet klasy do walidacji formularzy.
Prosze nie patrzec na sam kod ani ze sa error'y na sztywno bo to sie zmienie lecz na sama logike:

[PHP] pobierz, plaintext 
<?php
 
class Validation {
 
  protected $type;
 
  public $rules = array();
 
  public $msg;
 
  //$fieldname = 'login'
  //$label = 'Username'
  //$rules = lenght => 1,4
 
  public function __construct($type = '') {
 
      if(isset($type) && !empty($type)) {
 
          $this -> type = $type;
 
      }
 
      else {
 
          $this -> type = 'post';
 
      }
 
  }
 
  public function getTypeValidation() {
 
      return $this -> type;
 
  }
 
  public function Submitted() {
 
    $submitType = ($this -> getTypeValidation() == 'post') ? $_POST : $_GET;
 
      if(count($submitType) == 0)
        return false;
 
      else
        return true;
 
  }
 
  public function AddRules($fieldname, $label, $rules = array()) {
 
      if(empty($label)) {
 
          $label = $fieldname;
 
      }
 
      if(empty($fieldname)) {
 
          $fieldname = $label;
 
      }
 
      $submitType = ($this -> getTypeValidation() == 'post') ? $_POST : $_GET;
 
      $this -> rules[$fieldname] = array($submitType[$fieldname] => $rules);
 
  }
 
 
  public function Validation() {
 
      foreach($this -> rules as $rule) {
 
        foreach($rule as $post => $validRule) {
 
          $methodRule = explode('|', $validRule);
          $methodRule = $methodRule[count($methodRule)-1];
 
          switch($methodRule) {
 
            case 'required':
 
                            if(method_exists(get_class($this), $methodRule)) {
 
                                 $required = $this -> $methodRule($post);
 
                            }
 
                            if($required)
                              return true;
 
                            else {
 
                              $key = array_keys($this -> rules);
                              $this -> msg =  'Pole '.$key[0].' nie moze byc puste';
 
                            }
 
            break;
 
          }
 
       }
 
    }
 
  }
 
 
  protected function required($string) {
 
      return (!is_null($string) && $string != false && !empty($string)) ? true : false;
 
  }
 
 
}
 
?>
[PHP] pobierz, plaintext 

Testuje jak narazie tylko na jednej funkcji analogicznie potem bedzie dzialalo i na reszcie.
No i robie taki maly test:

[PHP] pobierz, plaintext 
$_POST['logins'] = 'vodka';
$_POST['haslos'] = '';
 
$valid = new Validation;
$valid -> AddRules('logins', 'username', 'required');
$valid -> AddRules('haslos', 'username', 'required');
 
$valid -> Validation();
if(!empty($valid -> msg)) echo('cos jest nie tak');
else echo('Wszystko good');
 
print_r($valid);
[PHP] pobierz, plaintext 

Ten print_r() daje mi:

Validation Object ( [type:protected] => post [rules] => Array ( [logins] => Array ( [vodka] => required ) [haslos] => Array ( [] => required ) ) [msg] => )

No i jak widac i msg jest puste i wyswietli mi sie info ze wszystko good.
Gdy ustawie tylko jedna zmienna np logins i ustawie ja jako pusta to dziala wyswietla error dla tego pola lub gdy nie jest puste nie wyswietla wiec robie cos zle w funckji Validation() jednak nie wiem zabardzo co chyba ze ogolnie zle zaprojektowalem tablice z danymi.

Prosze o pomoc i z gory dzieki.

EDIT:
Teraz tak przegladajac post z drugiego kompa zauwazylem ze jest blad:

[PHP] pobierz, plaintext 
$methodRule = $methodRule[count($methodRule)-1];
[PHP] pobierz, plaintext 

Powinno byc:

[PHP] pobierz, plaintext 
$methodRule = $methodRule[0];
[PHP] pobierz, plaintext 

Podajac jednak tylko required jako 3 argument to nic nie zmienia.

Ten post edytował marcio 3.11.2009, 01:43:29

[Nattfarinn]

Twoja sekcja:

[PHP] pobierz, plaintext 
case 'required':
    (...)
[PHP] pobierz, plaintext 

Jest źle napisana. Sprawdzanie walidacji jest ograniczone do pierwszego poprawnego elementu (linia 89 i 90: return true;). Ten warunek jest tam zupełnie nie potrzebny. Nie zagłębiając się w resztę:

[PHP] pobierz, plaintext 
  (...)
  public function Validation() {
    foreach($this -> rules as $key => $rule) {
      foreach($rule as $post => $validRule) {
        $methodRule = explode('|', $validRule);
        $methodRule = $methodRule[count($methodRule)-1];
        switch($methodRule) {
          case 'required':
            if(method_exists(get_class($this), $methodRule)) {
              $required = $this -> $methodRule($post);
              if(!$required) {
                $this -> msg =  'Pole '.$key.' nie moze byc puste';
              }
            } 
            break;
        }
      }    
    }
  }
  (...)
[PHP] pobierz, plaintext 

I będzie śmigało...

Ten post edytował Nattfarinn 3.11.2009, 10:29:24

[marcio]

Ogolnie zrobilem dokladnie to co ty, spojrzalem jeszcze raz na klasy walidacji z kohany,CI i Mohebo i wtedy sie skaplem ze fakt faktem return konczy dzialanie metody validation() eh bylo juz pozno i robilem takie byki ze hej.




Ale dzieki za fatyge masz + tylko dzis dokoncze klase i jak bede mial jeszcze jakies watpliwosci to napisze.