problema validacion formulario en poo

[rufus]

que tengo el siguiente codigo de validacion

Código PHP:

  <?php 
if (isset($_POST['submit'])) 
{
    $mudanza = new Mudanza();
    $mudanza->nombre =$_POST["nombre"];
    $mudanza->telefono =$_POST["telefono"];
    $mudanza->email =$_POST["email"];
    $mudanza->montar =$_POST["montar"];
    $mudanza->embalar =$_POST["embalar"];
    $mudanza->ascensor =$_POST["ascensor"];
    $mudanza->distancia =$_POST["distancia"];
    $mudanza->metros =$_POST["metros"];
    $mudanza->fecha =$_POST["fecha"];
    $mudanza->direccion1 =$_POST["direccion1"];
    $mudanza->poblacion1 =$_POST["poblacion1"];
    $mudanza->provincia1 =$_POST["provincia1"];
    $mudanza->direccion2 =$_POST["direccion2"];
    $mudanza->poblacion2 =$_POST["poblacion2"];
    $mudanza->provincia2 =$_POST["provincia2"];
    $mudanza->salon =$_POST["salon"];
    $mudanza->cocina =$_POST["cocina"];
    $mudanza->dormitorio1 =$_POST["dormitorio1"];
    $mudanza->dormitorio2 =$_POST["dormitorio3"];
    $mudanza->dormitorio4 =$_POST["dormitorio4"];
    $mudanza->otrosobjetos =$_POST["otrosobjetos"];
    $error = "";
    if(empty($mudanza->nombre))
    {
        $error .= "Nombre <br>";
    }
    
    if(empty($mudanza->telefono))
    {
        $error .= "telefono <br>";
    }
    
    if(empty($mudanza->email))
    {
        $error .= "email <br>";
    }
    
    if(empty($error))
    {
        echo "por favor rellene los siguientes campos:<br>".$error;
        
    }
    else
        {
            if($mudanza->crear())
        
            $mensaje = "el archivo se ha subido";
        }
    }
    else
    {
        $mensaje = "existen los siguientes errores : <br />";
        $mensaje .= join("<br/>", $mudanza->errores);
        
    }
 
?>   

pero no me funciona no valida los campos obligatorios no se si es por que no e llamado bien a las variables = propiedad

alguien me puede ayudar?

[miktrv]

Porque no validas antes los datos de pasárselos a tu objeto?

[loluchis]

estaria bien que hagas algo asi, es mi punto de vista cuidado:

Código PHP:

  class Mudanza {
 
protected $datos;
public $errores = false;
 
public function __construct(array $datos = array()) {
    $this->datos = $datos;
    $this->validar();
}
 
public function validar() {
    foreach($this->datos as $key => $value) {
        switch($key) {
             case 'nombre_campo_obligatorio1':
             case 'nombre_campo_obligatorio2':
             case 'nombre_campo_obligatorio3':
             case 'nombre_campo_obligatorioX':
                 if(empty($value))
                     $this->errores .= $key . '<br />';
        }
    }
    return $this->errores;
}
 
} 
   

con esto podrias hacer algo asi luego:

Código PHP:

  if(isset($_POST['submit']) {
 
$mudanza = new Mudanza($_POST);
 
if($mudanza->errores) {
    echo 'errores: ' . $mudanza->errores;
}
else {
    echo 'correcto';
}
 
} 
   

[miktrv]

A mi particularmente no me gusta que la clase me devuelva una cadena de errores,

Un saludo!