(Consulta) modificar, ampliar, optimizar validación de formulario

[opzina]

Hola,

Me gustaría que me indiquen en que puedo trabajar para mejorar esta clase para validar un formulario con php:

Código PHP:

  
<?php
class Validator {
 
    var $errors; // Variable que almacena errores
 
   
    // Este método no evita  Inyecciones SQL
    // Usar con addslashes() (Este es un tema a tratar)
    function validateGeneral($theinput,$description = ''){
        if (trim($theinput)) {
            return true;
        }else{
            $this->errors[] = $description;
            return false;
        }
    }
    
    // Valida solo texto
    function validateTextOnly($theinput,$description = ''){
        $result = preg_match("/^[A-Za-z0-9-áéíóú\ ]+$/", $theinput );
        if ($result){
            return true;
        }else{
            $this->errors[] = $description;
            return false; 
        }
    }
 
    // Valida solo texto, no se permiten espacios
    function validateTextOnlyNoSpaces($theinput,$description = ''){
        $result = preg_match("/^[A-Za-z0-9]+$/", $theinput );
        if ($result){
            return true;
        }else{
            $this->errors[] = $description;
            return false; 
        }
    }
        
    // Valida correo electrónico
    function validateEmail($themail,$description = ''){
        $permitidos = "a-zA-Z0-9_\.\-]+@[a-zA-Z0-9\-]+\.[a-zA-Z0-9\-\.";
        $result = preg_match("/^[$permitidos]+$/", $themail);
        if ($result){
            return true;
        }else{
            $this->errors[] = $description;
            return false; 
        }
            
    }
    
    // Valida sólo números
    function validateNumber($theinput,$description = ''){
        if (is_numeric($theinput)) {
            return true; 
        }else{ 
            $this->errors[] = $description;
            return false;
        }
    }
    
    // Valida fechas
    function validateDate($thedate,$description = ''){
 
        if (strtotime($thedate) === -1 || $thedate == '') {
            $this->errors[] = $description;
            return false;
        }else{
            return true;
        }
    }
    
    // Compruebe si los errores se han encontrado
    function foundErrors() {
        if (count($this->errors) > 0){
            return true;
        }else{
            return false;
        }
    }
 
    // Devuelve una cadena que contiene una lista de errores encontrados,
    // Separado por un delimitador
    function listErrors($delim = ' '){
        return implode($delim,$this->errors);
    }
    
    // Agregar algo a la lista de errores manualmente
    function addError($description){
        $this->errors[] = $description;
    }    
        
}
?>   

[GatorV]

Tema movido desde PHP a Frameworks y PHP orientado a objetos

[masterpuppet]

Que tal opzina,

Deberias utilizar PHP 5, osea declarar la visibilidad, podrias ajustar un poco el código(a veces es mejor negar primero) y tambien podrias documentar correctamente, algo asi:

Código PHP:

class Validator
{
   /**
    * Variable que almacena errores
    * @var array
    */
    protected $_errors; 
   
   /**
    * Este método no evita  Inyecciones SQL
    * Usar con addslashes() (Este es un tema a tratar)
    * @param string $theInput
    * @param string $description
    * @return boolean
    */
    public function validateGeneral($theinput, $description = '')
    {
        if (!trim($theinput)) {
            $this->_errors[] = $description;
            return false;            
        }
        return true;
    }
 
    /**
     * Valida solo texto
     * @param string $theInput
     * @param string $description
     * @return boolean
     */   
    public function validateTextOnly($theinput, $description = '')
    {
        if (!preg_match("/^[A-Za-z0-9-áéíóú\ ]+$/", $theinput)) {
            $this->_errors[] = $description;
            return false;             
        }
        return true;
    }
 
    ...
}
 

Saludos.

[opzina]

Muchas Gracias,

Empiezo con lo que me indicas y voy mostrando y compartiendo.

Saludos.