PHP OO (Consulta) Propiedad tipo array y recorrerlo

[opzina]

Hola a todos.

Declare una propiedad como array:

Código PHP:

var $tipo = array('jpg', 'gif', 'png');
 

y los metodos:

Código PHP:

// Obtengo la extension del archivo.
    private function extension($file) {
        
        $this->ext .= substr(strrchr($file, '.'), 1);
       
        return $this->ext;
    }
    
    // Compruebo qye el archivo tiene una extension permitida.
    function comprobar($file) {
        
        foreach ($this->allows as $allow) {
            $this->extension .= $allow;
        }
            
        if ($this->extension == $this->extension($file)) {
                return  $this->correcto;
            } 
            else {
                return $this->error;
            }       
    }
 

Cuando hago el foreach, solo me devuelve el primer valor del arreglo.
A que se debe este comportamiento? Debo corregir el acceso de las propiedades y/o metodos? ¿O cual es el error que estoy cometiendo?

Saludos.

[abimaelrc]

Haz un print_r a la variable $this->allows y mira lo que tiene.

[Eleazan]

No debería ser $this->tipo en vez de $this->allows ?

[opzina]

Iniciado por abimaelrc

Haz un print_r a la variable $this->allows y mira lo que tiene.

Me trae el array...

No debería ser $this->tipo en vez de $this->allows ?

Si, me equivoque cdo lo publique en el foro. Es lo mismo.

[Eleazan]

De todas formas no entiendo una cosa (un par, pero bueno...)

Cuando haces

Código PHP:

  foreach ($this->allows as $allow) {
            $this->extension .= $allow;
        }  
   

Estás guardando todas las... ¿extensiones permitidas? en un string.. q sería algo así 'jpgpnggif' por ejemplo...

Y luego haces

Código PHP:

  if ($this->extension == $this->extension($file)) {
                return  $this->correcto;
            }  
   

Que serñia algo así como
if( 'jpgpnggif' == 'jpg')

Nunca sería "true" si tienes puesta más de una extensión. ¿No?

[opzina]

Verdad, estoy revisando lo que tu dices....

Creo que lo que hay que hacer es buscar dentro de ese array y comparar cada valor y cuando lo encuentra salir del bucle...

Estoy intentando esto, pero creo que estoy con el mismo problema...

Código PHP:

function comprobar($file) {
        
        $this->extension = false;
        foreach ($this->allows as $allow) {
            
            if ($allow == $this->extension($file)) {
                $this->extension = true;
                break;
            }       
        }
            return $this->extension;
    }
 

[Eleazan]

Bueno, yo he probado esto:

Código PHP:

  class Test {
    private $allows = array('jpg','png','gif');
    
    
    
    private function extension($file) {
       
        return substr(strrchr($file, '.'), 1);
       
        
    }
    
    public function comprobar($file) {
           $extension_a_comprobar = $this->extension($file); //Pq voy a buscar la extension 300 veces? ;)
            $result = false;
            foreach ($this->allows as $allow) {
               
                if ($allow == $extension_a_comprobar) {
                    //Sí q está permitida
                    $result = true;
                }      
            }
                return $result;
        }
}

$test = new Test;
if($test->comprobar('test.jpg')) echo 'test.jpg aceptado';
if($test->comprobar('test.jpeg')) echo 'test.jpeg aceptado';  
   

Y me funciona bien así ;)

[opzina]

Ahora di con esta soluciòn:

Código PHP:

if (in_array($this->extension($file), $this->allows)) {
            return $this->correcto;
        } else {
            return $this->error;
        }
 

[Webstudio]

Chicos, están complicando el método... es más simple.
Solamente hagan:

Código PHP:

  public function comprobar($file) 
{ 
    $ext = $this->extension($file); 
    return in_array($ext, $this->allows); 
}