Formular Validierung - Seite 3

#Avedo · 31.12.2008, 15:11

Eben eben und unnütze Zeitverschwendung ist nicht so schön. Habe das $errors Array noch in der abstrakten Klasse hinzugefügt. Danke. Mich würde noch interessieren, welche Validatoren ihr für den täglichen Gebrauch ihr für vorerst ausreichend haltet. Welche sollte ich also erstmal implementieren? Meine Idee dazu:

Validator_Alnum (nur alphanumerische Zeichen)
Validator_Alpha (nur alphabetische Zeichen)
Validator_Between ( zwischen Minimum und Maximum)
Validator_Date (YYYY-MM-DD)
Validator_Num ( nur numerische Werte )
Validator_Int ( numerischer Wert ohne "." )
Validator_Float ( numerischer Wert mit "." )
Validator_Mail ( Email Addresse )
Validator_Greater ( größer als )
Validator_Less ( kleiner als )
Validator_Hex ( Hexacode )
Validator_Domain ( gültige Doamin )
Validator_Ip ( gültige IP-Addresse )
Validator_Regex ( stimmt mit Regex überein )
Validator_Notempty ( nicht leer)
Validator_Inarray ( ist im array entahlten)
Validator_Length ( Stringlänge zwischen Minimum und Maximum )

Habe mal ein Beispiel für eine solche Klasse geschrieben, wie es mir von dir gezeigt wurde. Ist ds so ok mit meinen Anpassungen? Bin mir nicht mehr sicher, ob Methoden und Eigenschaften die in abstrakten Klassen bereits implementiert wurden, in der erbenden Klasse erneut implementiert werden müssen.

PHP-Code:

  <?php
error_reporting(E_ALL | E_STRICT);

require_once('Validator.php');

/***
* Class Validator_Length
*
* The Validator_Length class 
* checks if the length of the 
* given String is between the given
* maximm and minimum length.
*  
* @package Validator
* @subpackage Validator_Length
* @version 0.1
**/  
Class Validator_Length extends Validator
{
    // protected class variables
    protected $min = 0;
    protected $max = 0;

    /**
    * Constructor - Stes protected class values
    *
    * @access public
    * @param Int $min
    * @param Int $max
    * @return NONE
    */
    public function __construct($min = 3, $max = 10) 
    {
        $this->min = $min;
        $this->max = $max;
    }

    /**
    * isValid() - Checks if inpout has a valid length
    *
    * @access public
    * @param Str $value
    * @return Boolean
    */
    public function isValid($value) 
    {
        // check min
        if( strlen($value) < $this->min ) 
        {
            $this->errors[] = sprintf('Input is too short, should be at least %s signs long.', $this->min);
            return false;
        }

        // check max
        if( strlen($value) < $this->max ) 
        {
            $this->errors[] = sprintf('Input is too long, should be at least %s signs long.', $this->max);
            return false;
        }
        
        return true;
    }
}
?>

MfG, Andy

robo47 · 31.12.2008, 17:13

Int macht wenn du Numeric hast für formulare keinen sinn

Manko10 · 01.01.2009, 13:55

@#Avedo: der gleiche Validator mit unterschiedlich Optionen unterscheidet sich hier aber nicht, da der Hash mit get_class() erzeugt wird und get_class() ist immer gleich, wenn es sich um die gleiche Klasse handelt.
Wenn du erreichen möchtest, dass der Hash auf Basis des Objekts erzeugt wird, musst du es entweder so:

PHP-Code:

  $hash = md5(serialize($validator));

oder so machen:

PHP-Code:

  $hash = md5(serialize(get_object_vars($validator)));

Eine Alternativ ohne Hashs ist diese (das ist wohl das, was du zuerst erreichen wolltest):

PHP-Code:

  $validatorAlreadySet = in_array($validator, $this->validators);

#Avedo · 01.01.2009, 15:39

Hallo!

Ich wünsche euch allen ein frohes neues Jahr!

Ich glaube ich werde wohl auf die erste Methode, also das erzeugen eines Hashes auf Basis des Objekts, zurückgreifen.

PHP-Code:

  $hash = md5(serialize($validator));

Ich habe boch eine Frage zu den Validator_ Klassen. Ist es sinnvoll bei einem Validator, der einen Wertevergleich durchführt, zum Beispiel mit Validator_Between, zuerst auf den richtigen Eingabetyp zu testen, oder sollte man einfach vorraussetzen, dass die entsprechenden Validatoren vorher genutzt worden sind?

Wieso ist es eigentlich nicht sinnvoll zu testen, ob ein Eingabewert ein Integer ist, wenn man is_numeric() nutzt? Diese Funktion gibt auch dann true zurück, wenn der Eingabewert Hexadezimal ist. Von daher macht es schon Sinn.

Habe im übrigen noch eine weitere Klasse geschrieben (Validator_Notempty) ist das in der Regel so ok? Und war die Validator_Length so ok? Erkundige mich nur bei eins zwei, keine Angst frage nun nicht bei jeder kleinen Klasse. Möchte nur das Prinzip verstehen. Prüft man auf alphanumerische Eingabe auch am betsen über Regex? Ja bestimmt oder? Was haltet ihr für die bewährtesten Mail und Domain Regex? Waren meine ok?

PHP-Code:

  <?php
error_reporting(E_ALL | E_STRICT);

require_once('Validator.php');

/***
* Class Validator_Notempty
*
* The Validator_Notempty class 
* checks if a given input
* is not empty.
*  
* @package Validator
* @subpackage Validator_Notempty
* @version 0.1
**/  
Class Validator_Notempty extends Validator
{
    /**
    * isValid() - Checks if input is not empty
    *
    * @access public
    * @param Str $value
    * @return Boolean
    */
    public function isValid($value) 
    {
        // check min
        if( empty($value) ) 
        {
            $this->errors[] = 'Empty Input.';
            return false;
        }
        
        return true;
    }
}
?>

Ich bedanke mich schonmal für eure Hilfen und Mühen. Seid spitze.

MfG, Andy

Manko10 · 01.01.2009, 19:47

Zitat:

Wieso ist es eigentlich nicht sinnvoll zu testen, ob ein Eingabewert ein Integer ist, wenn man is_numeric() nutzt? Diese Funktion gibt auch dann true zurück, wenn der Eingabewert Hexadezimal ist. Von daher macht es schon Sinn.

Wie meinst du das? Ich persönlich habe is_numeric() noch nie benötigt, eben weil es jede nur erdenkliche Art von Nummer und String und… als gültig erkennt, solange es nur in irgendeinem der hunderttausend Zahlensysteme dieser Erde vorkommt.
Ich validiere meine Integer immer so:

PHP-Code:

  $isValidInt = (string)((int)$eingabe) === $eingabe;

Da es sich um eine User-Eingabe handelt, liegt diese immer als String vor und ein is_int() würde somit vollkommen versagen.

Alphanumerische Werte würde ich mit einem RegEx validieren, ja.

Der erste Validator sieht in Ordnung aus, wobei ich nicht verstehe, weshalb $min und $max die Standardbelegungen 3 und 10 haben. Ich würde den ersten Parameter als erforderlich deklarieren und den zweiten per Default mit -1 belegen, was so viel wie „beliebig lang“ bedeutet.
Der zweite Validator tut ja nicht viel. Du solltest dir aber überlegen, ob du das Verhalten von empty() 1:1 übernehmen willst oder nicht doch lieber eigene Kriterien festlegst, wann eine Variable leer ist und wann nicht.

Die Fehlerbehandlung selbst könnte vielleicht etwas zentraler geregelt werden (z.B. mit Exceptions). Du kannst, wenn du willst, ja auch noch einen Parameter $throwException für den Validator anlegen, um das Verhalten bei einem Fehler zu kontrollieren.

#Avedo · 01.01.2009, 21:00

Naja in einem füheren Beitrag wurde mir gesagt, dass es nicht sinnvoll sein auf ein Integer-Input zu testen, wenn zuvor erfolgreich mittels is_numeric() getested wurde. Aber wie du richtig gesagt hast, gibt is_numeric() auf so zuemlich jedes Zahlensystem true zurück. Deine Art der Validierung von Integern gefällt mir sehr. Werde ich so auch machen. So ist es auch sehr einfach möglich auf Floats zu testen.

Habe den ersten Validator so, wie von dir vorgeschlagen, geändert.

Den zweiten - Validator_Notempty - habe ich nochmals komplett überarbeitet, weil du auf jeden Fall Recht hattest, dass es nicht sinnvoll ist, diese Validierung mittels empty() vorzunehmen. Eine Integereingabe 0 würde zum Beispiel einen Fehler erzeugen. Sehr schlecht, da dies doch mal vorkommt. Daher habe ich mir überlegt auf einen Leerstring, ein leeres Array oder NULL zu prüfen. Halte ich persönlich für sinnvoller.

Zudem hatte ich vergessen, dass ja nicht immer, wenn ein Feld leer ist ein Fehler erzeugt werden soll. Daher sollte ich wohl doch noch einen zusätzlichen Parameter beim Instanzieren dieser Klasse übergeben, damit festgelegt werden kann, ob dieses Feld gesetzt werden muss oder nicht. Habe die Klasse daher nochmal überarbeitet. Das Porblem ist jetzt, dass ich nicht weiß, wie ich im Allgemeinen verhindern kann, dass leere Eingaben durch weitere Validatoren gehen. Ist eine Eingabe leer, muss aber nicht gesetzt sein, sollen ja auch keine weiteren Validatoren drüberlaufen. Ist die Eingabe leer soll aber gesetzt sein, so sollen ja aber trotzdem keine weiteren Validatoren drüberlaufen, macht ja keinen Sinn. Würde das ganze denn mit meinen Anpassungen richtig funktionieren, oder habe ich vielleicht etwas übersehen?

PHP-Code:

  <?php
error_reporting(E_ALL | E_STRICT);

require_once('Validator.php');

/***
* Class Validator_Notempty
*
* The Validator_Notempty class 
* checks if a given input
* is not empty.
*  
* @package Validator
* @subpackage Validator_Notempty
* @version 0.1
**/  
Class Validator_Notempty extends Validator
{
    // private class variables
    private $binding = true;

    /**
    * Constructor - Stes protected class values
    *
    * @access public
    * @param Bool $binding
    * @return NONE
    */
    public function __construct($binding = true) 
    {
        $this->binding = $binding;
    }

    /**
    * isValid() - Checks if input is not empty
    *
    * @access public
    * @param Str $value
    * @return Boolean
    */
    public function isValid($value) 
    {
        // check min
        if( (($value == "") || ($value = array()) || ($value == NULL)) && ($binding === true) ) 
        {
            $this->errors[] = 'Empty Input.';
            return false;
        }

        if( binding === false )
            return false;
        
        return true;
    }
}
?>

Zudem würde mich noch interessieren, ob es nicht sinnvoller wäre, anstatt protected doch lieber private in den Klassen für die eigentschaften zu nutzen, da man ja verschiedene Parameter angeben können soll.

Wie hast du das mit der Fehlerbehandlung gemeint, Manko10. Verstehe ich leider nicht. Wie willst du die Klassen denn auf Exceptions umstellen? Hört sich für mich gut an, weiß aber nicht so richtig, wie ich diese dann so zentral verwalten und die Ausgabe Feld für Feld steueren soll. Zudem möchte ich ja alle Fehlermeldungen erhalten und nicht nur die erste für jedes Feld.
MfG, Andy

Manko10 · 01.01.2009, 21:54

Zitat:

Wie hast du das mit der Fehlerbehandlung gemeint, Manko10. Verstehe ich leider nicht. Wie willst du die Klassen denn auf Exceptions umstellen? Hört sich für mich gut an, weiß aber nicht so richtig, wie ich diese dann so zentral verwalten und die Ausgabe Feld für Feld steueren soll. Zudem möchte ich ja alle Fehlermeldungen erhalten und nicht nur die erste für jedes Feld.

Du erstellst eigene Exceptionklassen, die du werfen und abfangen kannst.
Innerhalb der Klasse handelst du die Fehler weiterhin wie bisher auch, aber statt einem return false bei einem Fehler am ende der isValid()-Methode wirfst du eine Exception, der du das Fehler-Array übergibst.

#Avedo · 01.01.2009, 22:22

Ah habs verstanden. Werds mir mal ansehen. Mich würde noch interessieren, wie ich es implementieren kann, dass ich durch die Ausgabe eines Objekts der Klasse Form via

PHP-Code:

<?php echo $form; ?>

auch das Formular, dass zu diesem Objekt gehört ausgaben kann. Ich weiß nicht, wie ich festlegen soll, wann welcher Typ von Eingabefeld auszugeben ist. Das setzen von Attributen könnte ich ja über eine Methode

PHP-Code:

  setAttr($name, $value)

lösen könnte. So könnte ich zum Beispiel einem Texfeld via

PHP-Code:

  setAttr('class', 'name left')

die CSS Klassen name und left zuordnen. Wie kann ich dann aber zum Beispiel bei Select-Menüs oder Checkboxen die verschiedenen Auswahlmöglichkeiten festlegen? Wäre nett, wenn ihr mir mit ein paar Anregungen oder Beispielen helfen könntet.
MfG, Andy

robo47 · 02.01.2009, 04:55

Das

PHP-Code:

  echo $form;

kannst du mit der methode __toString(); machen. Die wird aufgerufen wenn man versucht ein Objekt in einen String umzuwandeln oder es ausgeben will.

-> PHP: Magic Methods - Manual

#Avedo · 02.01.2009, 12:32

Guten Morgen!

Danke für deine Antwort.

Das ich das mit der Methode __toString machen kann, war mir schon klar. Mich würde allerdings interessieren, wie ich das am besten in die spezifischen Unterklassen für die Eingabefelder einbinden kann. Ich muss diese __toString Methode ja nicht nur in die Klasse Form sondern auch in die Unterklassen Form_Element_Text, etc. einbinden. Wie sollte das aussehen? Zudem ist die Frage, wie ich gestalterische Elemente, die ja von Formular zu Formular unterchiedlich sein können übergebe. Ein erster Ansatz war ja die Methode setAttr(), die die Tagattribute in ein Array schreibt, sodass sie in der __toString Methode verarbeitet werden können. Man könnte sich noch überlegen, ob an jede dieser Klassen noch ein Layout für sprintf() übergeben werden können soll. So könnte dem Formular zum Beispiel eine Formatierung in Form einer Tabelle oder so übergeben werden.

Des weiteren würde mich noch interessieren, ob es sinnvoll ist bei einem Validator, der einen Wertevergleich durchführtzuerst auf den richtigen Eingabetyp zu testen, oder sollte man einfach vorraussetzen, dass die entsprechenden Validatoren vorher genutzt worden sind? Oder gibt folgender Code einfach false zurück, wenn die Eingabe nicht numerisch ist?

PHP-Code:

  if( ($value < $max) && ($value > $min) )

Zudem wollte ich wissen, ob ich die Eingaben von $min und $max auch noch überprüfen soll. Oder soll ich davon ausgehen, dass der Programmierer (vorallem ich) weiß was er da tut?

MfG, Andy

//EDIT: Hätte noch eine Frage. Welche funktion verwende ich am besten beim Arbeiten mit RegExen? Habe es bisher immer so gemacht:

PHP-Code:

  if( ereg($regex, $input) )
    return true;
else 
    return false;

Ein paar Beispiele:

alphanumerische Eingaben

PHP-Code:

  ereg("[[:alnum:]]{".strlen($input)."}", $input)

Hexa-Code Eingaben

PHP-Code:

  ereg("[[:xdigit:]]{".strlen($input)."}", $input)

Datums Eingaben

PHP-Code:

  ereg("^([0]?[1-9]|[1|2][0-9]|[3][0|1])[./-]([0]?[1-9]|[1][0-2])[./-]([0-9]{4}|[0-9]{2})$", $input)

Sind diese so ok? Eine dazugehörige Klasse würde dann so aussehen:

PHP-Code:

  <?php
error_reporting(E_ALL | E_STRICT);

require_once('Validator.php');

/***
* Class Validator_Alnum
*
* The Validator_Alnum class 
* checks if a given input
* is alphanumeric.
*  
* @package Validator
* @subpackage Validator_Alnum
* @version 0.1
**/  
Class Validator_Alnum extends Validator
{
    /**
    * isValid() - Checks if input is alphanumeric
    *
    * @access public
    * @param Str $value
    * @return Boolean
    */
    public function isValid($value) 
    {
        // check if input is alphanumeric
        if( ereg("[[:alnum:]]{".strlen($value)."}", $value) ) 
        {
            $this->errors[] = 'Input must be alhanumeric.';
            return false;
        }
        
        return true;
    }
}
?>

Habe auch noch eine Klasse geschrieben, die beliebige Regex überprüfen soll. Jedoch weiß ich nicht, ob mir das sinnvoll gelungen ist. Was meint ihr.

PHP-Code:

  <?php
error_reporting(E_ALL | E_STRICT);

require_once('Validator.php');

/***
* Class Validator_Regex
*
* The Validator_Regex class 
* validates the given input
* with the aid of the given
* regex.
*  
* @package Validator
* @subpackage Validator_Regex
* @version 0.1
**/  
Class Validator_Regex extends Validator
{
    // protected class variables
    protected $regex = '';

    /**
    * Constructor - Sets protected class values
    *
    * @access public
    * @param Str $regex
    * @return NONE
    */
    public function __construct($regex) 
    {
        $this->regex = $regex;
    }
    
    /**
    * isValid() - Checks if input is alphanumeric
    *
    * @access public
    * @param Str $name
    * @param Str $value
    * @return Boolean
    */
    public function isValid($name, $value) 
    {
        // check if input is alphanumeric
        if( ereg($this->regex."{".strlen($value)."}", $value) ) 
        {
            $this->errors[] = "Invalid {$name}.";
            return false;
        }
        
        return true;
    }
}
?>

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
PHP: Formular Generator	Zergling-new	Tutorials	16	06.07.2010 03:14
Ergebnis von Formular in iFrame zeigen	PsychoEagle	HTML, Usability und Barrierefreiheit	2	22.07.2009 21:07
Problem mit Formular, Daten an datenbank senden	Aurec	PHP Tipps 2008	11	20.11.2008 18:17
Formular Pflichtfelder für Frontpage Formular erstellen	Caruso	JavaScript, Ajax und mehr	5	21.06.2008 09:17
Wert aus Textfeld in neues Formular übernehmen (kein Submit)	PsychoEagle	HTML, Usability und Barrierefreiheit	9	02.01.2007 14:51
formular includen	samspa5	PHP Tipps 2006	13	18.03.2006 11:37
[Erledigt] Formular in Formular		HTML, Usability und Barrierefreiheit	4	19.10.2005 12:18
[Erledigt] PHP Formular ~~wichtig~~		PHP Tipps 2005	27	29.05.2005 19:38
[Erledigt] Im Formular user abfragen?		Datenbanken	3	06.01.2005 10:51
[Erledigt] PHP Formular - Daten an nächsten Schritt weiter geben - Wie?		PHP Tipps 2005	14	02.01.2005 20:05
[Erledigt] Daten in einem Formular ausgeben und ändern		PHP Tipps 2004-2	7	07.12.2004 17:22
Wie implementiere ich mein Email Formular?		PHP Tipps 2004-2	2	14.11.2004 17:02
Brauche PHP Formular		Beitragsarchiv	9	16.08.2004 16:23
[Erledigt] Formular mit Daten werden auf anderer Seite zusammengefasst		PHP Tipps 2004	4	15.08.2004 15:44
[Erledigt] Daten in neuem Formular anzeigen		PHP Tipps 2004	6	21.07.2004 10:44

31.12.2008, 15:11
#Avedo Erfahrener Benutzer Registriert seit: 06.09.2008 Beiträge: 189	Eben eben und unnütze Zeitverschwendung ist nicht so schön. Habe das $errors Array noch in der abstrakten Klasse hinzugefügt. Danke. Mich würde noch interessieren, welche Validatoren ihr für den täglichen Gebrauch ihr für vorerst ausreichend haltet. Welche sollte ich also erstmal implementieren? Meine Idee dazu: Validator_Alnum (nur alphanumerische Zeichen) Validator_Alpha (nur alphabetische Zeichen) Validator_Between ( zwischen Minimum und Maximum) Validator_Date (YYYY-MM-DD) Validator_Num ( nur numerische Werte ) Validator_Int ( numerischer Wert ohne "." ) Validator_Float ( numerischer Wert mit "." ) Validator_Mail ( Email Addresse ) Validator_Greater ( größer als ) Validator_Less ( kleiner als ) Validator_Hex ( Hexacode ) Validator_Domain ( gültige Doamin ) Validator_Ip ( gültige IP-Addresse ) Validator_Regex ( stimmt mit Regex überein ) Validator_Notempty ( nicht leer) Validator_Inarray ( ist im array entahlten) Validator_Length ( Stringlänge zwischen Minimum und Maximum ) Habe mal ein Beispiel für eine solche Klasse geschrieben, wie es mir von dir gezeigt wurde. Ist ds so ok mit meinen Anpassungen? Bin mir nicht mehr sicher, ob Methoden und Eigenschaften die in abstrakten Klassen bereits implementiert wurden, in der erbenden Klasse erneut implementiert werden müssen. PHP-Code: <?php error_reporting(E_ALL \| E_STRICT); require_once('Validator.php'); /*** * Class Validator_Length * * The Validator_Length class * checks if the length of the * given String is between the given * maximm and minimum length. * * @package Validator * @subpackage Validator_Length * @version 0.1 / Class Validator_Length extends Validator { // protected class variables protected $min = 0; protected $max = 0; / * Constructor - Stes protected class values * * @access public * @param Int $min * @param Int $max * @return NONE / public function __construct($min = 3, $max = 10) { $this->min = $min; $this->max = $max; } /* * isValid() - Checks if inpout has a valid length * * @access public * @param Str $value * @return Boolean / public function isValid($value) { // check min if( strlen($value) < $this->min ) { $this->errors[] = sprintf('Input is too short, should be at least %s signs long.', $this->min); return false; } // check max if( strlen($value) < $this->max ) { $this->errors[] = sprintf('Input is too long, should be at least %s signs long.', $this->max); return false; } return true; } } ?> MfG, Andy __________________ I'm so tired of slitting the throats of people calling me a violent psychopath. Geändert von #Avedo (31.12.2008 um 16:05 Uhr).*


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

31.12.2008, 17:13
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.798 PHP-Kenntnisse: Fortgeschritten	Int macht wenn du Numeric hast für formulare keinen sinn __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework

01.01.2009, 13:55
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.425 PHP-Kenntnisse: Fortgeschritten	@#Avedo: der gleiche Validator mit unterschiedlich Optionen unterscheidet sich hier aber nicht, da der Hash mit get_class() erzeugt wird und get_class() ist immer gleich, wenn es sich um die gleiche Klasse handelt. Wenn du erreichen möchtest, dass der Hash auf Basis des Objekts erzeugt wird, musst du es entweder so: PHP-Code: `$hash = md5(serialize($validator));` oder so machen: PHP-Code: `$hash = md5(serialize(get_object_vars($validator)));` Eine Alternativ ohne Hashs ist diese (das ist wohl das, was du zuerst erreichen wolltest): PHP-Code: `$validatorAlreadySet = in_array($validator, $this->validators);` __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

01.01.2009, 15:39
#Avedo Erfahrener Benutzer Registriert seit: 06.09.2008 Beiträge: 189	Hallo! Ich wünsche euch allen ein frohes neues Jahr! Ich glaube ich werde wohl auf die erste Methode, also das erzeugen eines Hashes auf Basis des Objekts, zurückgreifen. PHP-Code: `$hash = md5(serialize($validator));` Ich habe boch eine Frage zu den Validator_ Klassen. Ist es sinnvoll bei einem Validator, der einen Wertevergleich durchführt, zum Beispiel mit Validator_Between, zuerst auf den richtigen Eingabetyp zu testen, oder sollte man einfach vorraussetzen, dass die entsprechenden Validatoren vorher genutzt worden sind? Wieso ist es eigentlich nicht sinnvoll zu testen, ob ein Eingabewert ein Integer ist, wenn man is_numeric() nutzt? Diese Funktion gibt auch dann true zurück, wenn der Eingabewert Hexadezimal ist. Von daher macht es schon Sinn. Habe im übrigen noch eine weitere Klasse geschrieben (Validator_Notempty) ist das in der Regel so ok? Und war die Validator_Length so ok? Erkundige mich nur bei eins zwei, keine Angst frage nun nicht bei jeder kleinen Klasse. Möchte nur das Prinzip verstehen. Prüft man auf alphanumerische Eingabe auch am betsen über Regex? Ja bestimmt oder? Was haltet ihr für die bewährtesten Mail und Domain Regex? Waren meine ok? PHP-Code: <?php error_reporting(E_ALL \| E_STRICT); require_once('Validator.php'); /*** * Class Validator_Notempty * * The Validator_Notempty class * checks if a given input * is not empty. * * @package Validator * @subpackage Validator_Notempty * @version 0.1 / Class Validator_Notempty extends Validator { / * isValid() - Checks if input is not empty * * @access public * @param Str $value * @return Boolean / public function isValid($value) { // check min if( empty($value) ) { $this->errors[] = 'Empty Input.'; return false; } return true; } } ?> Ich bedanke mich schonmal für eure Hilfen und Mühen. Seid spitze. MfG, Andy __________________ I'm so tired of slitting the throats of people calling me a violent psychopath*.

01.01.2009, 21:00
#Avedo Erfahrener Benutzer Registriert seit: 06.09.2008 Beiträge: 189	Naja in einem füheren Beitrag wurde mir gesagt, dass es nicht sinnvoll sein auf ein Integer-Input zu testen, wenn zuvor erfolgreich mittels is_numeric() getested wurde. Aber wie du richtig gesagt hast, gibt is_numeric() auf so zuemlich jedes Zahlensystem true zurück. Deine Art der Validierung von Integern gefällt mir sehr. Werde ich so auch machen. So ist es auch sehr einfach möglich auf Floats zu testen. Habe den ersten Validator so, wie von dir vorgeschlagen, geändert. Den zweiten - Validator_Notempty - habe ich nochmals komplett überarbeitet, weil du auf jeden Fall Recht hattest, dass es nicht sinnvoll ist, diese Validierung mittels empty() vorzunehmen. Eine Integereingabe 0 würde zum Beispiel einen Fehler erzeugen. Sehr schlecht, da dies doch mal vorkommt. Daher habe ich mir überlegt auf einen Leerstring, ein leeres Array oder NULL zu prüfen. Halte ich persönlich für sinnvoller. Zudem hatte ich vergessen, dass ja nicht immer, wenn ein Feld leer ist ein Fehler erzeugt werden soll. Daher sollte ich wohl doch noch einen zusätzlichen Parameter beim Instanzieren dieser Klasse übergeben, damit festgelegt werden kann, ob dieses Feld gesetzt werden muss oder nicht. Habe die Klasse daher nochmal überarbeitet. Das Porblem ist jetzt, dass ich nicht weiß, wie ich im Allgemeinen verhindern kann, dass leere Eingaben durch weitere Validatoren gehen. Ist eine Eingabe leer, muss aber nicht gesetzt sein, sollen ja auch keine weiteren Validatoren drüberlaufen. Ist die Eingabe leer soll aber gesetzt sein, so sollen ja aber trotzdem keine weiteren Validatoren drüberlaufen, macht ja keinen Sinn. Würde das ganze denn mit meinen Anpassungen richtig funktionieren, oder habe ich vielleicht etwas übersehen? PHP-Code: <?php error_reporting(E_ALL \| E_STRICT); require_once('Validator.php'); /*** * Class Validator_Notempty * * The Validator_Notempty class * checks if a given input * is not empty. * * @package Validator * @subpackage Validator_Notempty * @version 0.1 / Class Validator_Notempty extends Validator { // private class variables private $binding = true; / * Constructor - Stes protected class values * * @access public * @param Bool $binding * @return NONE / public function __construct($binding = true) { $this->binding = $binding; } /* * isValid() - Checks if input is not empty * * @access public * @param Str $value * @return Boolean / public function isValid($value) { // check min if( (($value == "") \|\| ($value = array()) \|\| ($value == NULL)) && ($binding === true) ) { $this->errors[] = 'Empty Input.'; return false; } if( binding === false ) return false; return true; } } ?> Zudem würde mich noch interessieren, ob es nicht sinnvoller wäre, anstatt protected doch lieber private in den Klassen für die eigentschaften zu nutzen, da man ja verschiedene Parameter angeben können soll. Wie hast du das mit der Fehlerbehandlung gemeint, Manko10. Verstehe ich leider nicht. Wie willst du die Klassen denn auf Exceptions umstellen? Hört sich für mich gut an, weiß aber nicht so richtig, wie ich diese dann so zentral verwalten und die Ausgabe Feld für Feld steueren soll. Zudem möchte ich ja alle Fehlermeldungen erhalten und nicht nur die erste für jedes Feld. MfG, Andy __________________ I'm so tired of slitting the throats of people calling me a violent psychopath. Geändert von #Avedo (01.01.2009 um 21:05 Uhr).*

01.01.2009, 22:22
#Avedo Erfahrener Benutzer Registriert seit: 06.09.2008 Beiträge: 189	Ah habs verstanden. Werds mir mal ansehen. Mich würde noch interessieren, wie ich es implementieren kann, dass ich durch die Ausgabe eines Objekts der Klasse Form via PHP-Code: `<?php echo $form; ?>` auch das Formular, dass zu diesem Objekt gehört ausgaben kann. Ich weiß nicht, wie ich festlegen soll, wann welcher Typ von Eingabefeld auszugeben ist. Das setzen von Attributen könnte ich ja über eine Methode PHP-Code: `setAttr($name, $value)` lösen könnte. So könnte ich zum Beispiel einem Texfeld via PHP-Code: `setAttr('class', 'name left')` die CSS Klassen name und left zuordnen. Wie kann ich dann aber zum Beispiel bei Select-Menüs oder Checkboxen die verschiedenen Auswahlmöglichkeiten festlegen? Wäre nett, wenn ihr mir mit ein paar Anregungen oder Beispielen helfen könntet. MfG, Andy __________________ I'm so tired of slitting the throats of people calling me a violent psychopath.

02.01.2009, 04:55
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.798 PHP-Kenntnisse: Fortgeschritten	Das PHP-Code: `echo $form;` kannst du mit der methode __toString(); machen. Die wird aufgerufen wenn man versucht ein Objekt in einen String umzuwandeln oder es ausgeben will. -> PHP: Magic Methods - Manual __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework

02.01.2009, 12:32
#Avedo Erfahrener Benutzer Registriert seit: 06.09.2008 Beiträge: 189	Guten Morgen! Danke für deine Antwort. Das ich das mit der Methode __toString machen kann, war mir schon klar. Mich würde allerdings interessieren, wie ich das am besten in die spezifischen Unterklassen für die Eingabefelder einbinden kann. Ich muss diese __toString Methode ja nicht nur in die Klasse Form sondern auch in die Unterklassen Form_Element_Text, etc. einbinden. Wie sollte das aussehen? Zudem ist die Frage, wie ich gestalterische Elemente, die ja von Formular zu Formular unterchiedlich sein können übergebe. Ein erster Ansatz war ja die Methode setAttr(), die die Tagattribute in ein Array schreibt, sodass sie in der __toString Methode verarbeitet werden können. Man könnte sich noch überlegen, ob an jede dieser Klassen noch ein Layout für sprintf() übergeben werden können soll. So könnte dem Formular zum Beispiel eine Formatierung in Form einer Tabelle oder so übergeben werden. Des weiteren würde mich noch interessieren, ob es sinnvoll ist bei einem Validator, der einen Wertevergleich durchführtzuerst auf den richtigen Eingabetyp zu testen, oder sollte man einfach vorraussetzen, dass die entsprechenden Validatoren vorher genutzt worden sind? Oder gibt folgender Code einfach false zurück, wenn die Eingabe nicht numerisch ist? PHP-Code: `if( ($value < $max) && ($value > $min) )` Zudem wollte ich wissen, ob ich die Eingaben von $min und $max auch noch überprüfen soll. Oder soll ich davon ausgehen, dass der Programmierer (vorallem ich) weiß was er da tut? MfG, Andy //EDIT: Hätte noch eine Frage. Welche funktion verwende ich am besten beim Arbeiten mit RegExen? Habe es bisher immer so gemacht: PHP-Code: `if( ereg($regex, $input) ) return true; else return false;` Ein paar Beispiele: alphanumerische Eingaben PHP-Code: `ereg("[[:alnum:]]{".strlen($input)."}", $input)` Hexa-Code Eingaben PHP-Code: `ereg("[[:xdigit:]]{".strlen($input)."}", $input)` Datums Eingaben PHP-Code: `ereg("^([0]?[1-9]\|[1\|2][0-9]\|[3][0\|1])[./-]([0]?[1-9]\|[1][0-2])[./-]([0-9]{4}\|[0-9]{2})$", $input)` Sind diese so ok? Eine dazugehörige Klasse würde dann so aussehen: PHP-Code: <?php error_reporting(E_ALL \| E_STRICT); require_once('Validator.php'); /*** * Class Validator_Alnum * * The Validator_Alnum class * checks if a given input * is alphanumeric. * * @package Validator * @subpackage Validator_Alnum * @version 0.1 / Class Validator_Alnum extends Validator { / * isValid() - Checks if input is alphanumeric * * @access public * @param Str $value * @return Boolean / public function isValid($value) { // check if input is alphanumeric if( ereg("[[:alnum:]]{".strlen($value)."}", $value) ) { $this->errors[] = 'Input must be alhanumeric.'; return false; } return true; } } ?> Habe auch noch eine Klasse geschrieben, die beliebige Regex überprüfen soll. Jedoch weiß ich nicht, ob mir das sinnvoll gelungen ist. Was meint ihr. PHP-Code: <?php error_reporting(E_ALL \| E_STRICT); require_once('Validator.php'); /** * Class Validator_Regex * * The Validator_Regex class * validates the given input * with the aid of the given * regex. * * @package Validator * @subpackage Validator_Regex * @version 0.1 / Class Validator_Regex extends Validator { // protected class variables protected $regex = ''; / * Constructor - Sets protected class values * * @access public * @param Str $regex * @return NONE / public function __construct($regex) { $this->regex = $regex; } /* * isValid() - Checks if input is alphanumeric * * @access public * @param Str $name * @param Str $value * @return Boolean / public function isValid($name, $value) { // check if input is alphanumeric if( ereg($this->regex."{".strlen($value)."}", $value) ) { $this->errors[] = "Invalid {$name}."; return false; } return true; } } ?> __________________ I'm so tired of slitting the throats of people calling me a violent psychopath. Geändert von #Avedo (02.01.2009 um 16:23 Uhr).*