Forum PHP.pl

Witam. Jest to moj pierwszy walidator, dlatego prosze o konstruktywne opinie i sugestie, co poprawic

[PHP] pobierz, plaintext 
<?php
 
// Klasa z podstawowymi metodami walidacji
class valid {
 
    public function required($value) {
        return !empty($value);
    }
 
    public function alpha($value) {
        return (bool) preg_match('/[a-zA-Z]/', $value);
    }
 
}
 
// Klasa do walidacji konkretnego formularza, w tym przypadku formularza logowania
class user_valid extends valid {
 
    public function username_valid($value) {
        return false;
    }
 
    public function errors() {
        return array(
            'username' => array(
                'required' => 'Musisz podac nazwe uzytkownika',
                'alpha' => 'Nazwa moze skladac sie jedynie ze znakow alfabetu',
                'username_valid' => 'Test obiektow',
            ),
            'password' => array(
                'not_sended' => 'Nie otrzymano hasla',
                'required' => 'Musisz podac haslo',
            ),
        );
    }
}
 
 
// Klasa validacji
class validation {
 
    private
        $fields = array(),
        $rules = array(),
        $errors = array(),
        $valid_object;
 
    public function __construct($post, $object_name) {
        $this->fields = $post;
        $this->valid_object = new $object_name;
    }
 
    public function add_rules($field_name, $rules) {
        if (isset($this->rules[$field_name])) {
            $this->rules[$field_name] = array_merge($this->rules[$field_name], $rules);
        } else {
            $this->rules[$field_name] = $rules;
        }
    }
 
    public function add_error($field_name, $error_name) {
        if (isset($this->errors[$field_name])) {
            $this->errors[$field_name][] = $error_name;
        } else {
            $this->errors[$field_name] = array($error_name);
        }
    }
 
    public function validate() {
        foreach ($this->rules as $field_name => $rules) {
            foreach ($rules as $method) {
                if (isset($this->fields[$field_name])) {
                    if ($this->valid_object->$method($this->fields[$field_name]) === false) {
                        $this->add_error($field_name, $method);
                    }
                } else {
                    $this->add_error($field_name, 'not_sended');
                }
            }
        }
        return empty($this->errors);
    }
 
    public function errors() {
        $errors_array = $this->valid_object->errors();
        $output_errors = array();
        foreach ($this->errors as $field_name => $errors) {
            foreach ($errors as $error) {
                if (isset($errors_array[$field_name])) {
                    $output_errors[$field_name][] = $errors_array[$field_name][$error];
                } else {
                    $output_errors[$field_name] = array($errors_array[$field_name][$error]);
                }
            }
        }
 
        return $output_errors;
    }
 
}
 
 
// Obsluga
$post = array(
    'username' => '123',
 
);
 
$validation = new validation($post, 'user_valid');
$validation->add_rules('username', array('required', 'alpha', 'username_valid'));
$validation->add_rules('password', array('required'));
if ($validation->validate()) {
    echo 'Poprawna nazwa';
} else {
    echo 'Bledy:';
    echo '<ul>';
    foreach ($validation->errors() as $field_name => $errors) {
        foreach ($errors as $error) {
            echo '<li>'.$error.'</li>';
        }
    }
    echo '</ul>';
}
[PHP] pobierz, plaintext 

Umiescilem wszystko w 1 pliku, aby bylo latwiej. Domyslnie klasy _valid beda w osobnym folderze

Widze ze nie masz na sztywno metod walidacji lecz mozesz tworzyc klasy dla danego forma wiec jest good
Ja bym wyrzucil tresci error'ow do osobnych plikow i do tego zadbal o mozliwosc wielojezykowosci

Tak ogolnie calego kodu nie analizowalem ale polecam poczytac:
http://forum.php.pl/index.php?showtopic=14...t=0&start=0
Temat: klasaphp Validation

klasa z bledami bedzie w osobnym pliku, tylko teraz jest w tym samym, co walidator

Albo mi się wydaje, albo zaczerpnąłeś "inspirację" z Kohany. Tak czy siak interesuje mnie czy da się (a chyba nie) napisać walidator matches, czyli sprawdzający czy podane pola mają taką samą wartość (najczęściej wykorzystywany do sprawdzania podanych haseł).

z kohany zaczerpnalem jedynie nazwy. Nie da sie, to ma byc prosty walidator bez zbednych funkcji, ktore wykorzysta sie max 2 razy w calek aplikacji. Wystarczy zrobic $haslo1 === $haslo2 i juz jest matches

Mi się nie podoba metoda required()
Wstaw jako jej argument 0 a dowiesz się czemu

Nospor@

zamiast

[PHP] pobierz, plaintext 
return !empty($value);
[PHP] pobierz, plaintext 

zrobilem

[PHP] pobierz, plaintext 
return (strlen($value) > 0) ? true : false;
[PHP] pobierz, plaintext 

To jak już jesteśmy przy wytykaniu błędów, to mi nie podoba się metoda alpha(). Dlaczego? Podaj jako parametr np. abc123 - przejdzie walidację, a zapewne nie powinno. Musisz dodać ograniczniki ^ oraz $, czyli:

[PHP] pobierz, plaintext 
    public function alpha($value) {
        return (bool) preg_match('/^[a-zA-Z]$/', $value);
    }
[PHP] pobierz, plaintext 

phpion: skoro już wytykamy błędy to trzeba jeszcze dodać gwiazdkę ^[a-zA-Z]*$

@up:
Nie trzeba, ale można. Nie wiem czy nie lepiej byłoby dodać + zamiast *, no ale decyzja należy do autora. Wówczas dany element byłby równocześnie wymagany.

Po co preg ? Jest ctype-alpha i zwraca bool.