Usar elementos de arraycomo parametros de funcion

[Atomycko]

Hola.

Lo que intento es usar elementos de un array y pasarlos como parametros a una funcion, actualmente lo unico que se me ocurre es usar eval, pero por mi experiencia se que este metodo no es muy confiable en cuanto a seguridad.
Espero poder explicarme mejor con este ejemplo de como lo hago con eval.

Código:

// Nombre de la funcion
$fnc = 'mi_funcion';

// Array el cual contiene los parametros a ser pasados a la funcion
$ary = array('a', 'b', 'c');

// Se comprueba la existencia de la funcion a llamar
if(function_exists($fnc)){
    // Si el array de parametros esta vacio solo se llama a la funcion
    if(empty($ary)){
        $fnc();
    }
    else{
        // Aqui esta lo rudo...
        @eval("$fnc(\$ary['" . implode("'],\$ary['", array_keys($ary)) . "']);");
    }
}

Bueno, espero que me haya explicado y que puedan ayudarme, de antemano gracias.

Saludos!

[pateketrueke]

Jamás uses eval(), eso es lo peor que puedes hacer.

En tu caso debes usar call_user_func_array(), consulta el manual.

[Atomycko]

Saludos pateketrueke!

Gracias por el dato de esta funcion, solo que en realidad lo que intento hacer es cargar clases desde otra clase, aun que si me funciono usando call_user_func_array, solo que ahora tengo que asegurarme que las clases tengan los valores por defecto en los parametros del constructor, pero por lo menos me libre del eval.

El codigo quedo mas o menos asi.

Código PHP:

  class Mi_Clase_Principal{
    var $clases = array();
    function clase($clase, $parametros = array(), $nueva = false){
        if(isset($this->clases[$clase]) && !$nueva){
            return $this->clases[$clase];
        }
        if(!class_exists($clase)){
            $archivo = DIRECTORIO_DE_ARCHIVOS_DE_CLASES . 'clase_' . $clase . '.php';
            if(!file_exists($archivo)){
                exit("El archivo de la clase '$clase' no existe");
            }
            include($archivo);
        }
        if(class_exists($clase)){
            $instancia = new $clase;
            if(!empty($parametros) && method_exists($instancia, '__construct')){
                call_user_func_array(array($instancia, '__construct'), $parametros);
            }
            $this->clases[$clase] = $instancia;
            return $this->clases[$clase];
        }
        exit("La clase '$clase' no existe");
    }
} 
   

Aunque por el momento parece estar solucionado, se aceptan opiniones.

Saludos!

[hhs]

Por lo que veo, estas haciendo una clase que te ayude a cargar e instanciar otras clases, pero creo que te estas haciendo un lío ya que puedes usar la función spl_autoload_register que precisamente te permite cargar clases cuando haces referencia a ellas, te dejo un ejemplo:

Código PHP:

<?php
//Ojo este ejemplo es muy sencillo y es solo para explicar
spl_autoload_register(function($class){
    $file = sprintf('%s/lib/%s.php',__DIR__, $class);
 
    if(file_exists($file)) {
        require_once($file);
    } else {
        throw new Exception("Cant load file: $file");
    }
    
});
 
//Instanciar clase qie se encuentra en la carpeta lib/
try{
    $class = new A();
    $class->display();
}catch (Exception $e) {
    echo $e->getMessage();
}
 

Cuando tu intentas crear una nueva instancia spl_autoload_register intenta buscar un archivo que tiene el nombre de la clase, de fallar lanza un excepción. En nuestro ejemplo intenta instanciar la clase A que tiene como archivo A.php.

Con esto tu puedes simplificar tu clase a esto:

Código PHP:

class Mi_Clase_Principal{
    
    private $clases = array();
    
    function clase($clase, $parametros = array()){
        if(isset($this->clases[$clase]) && in_array($clase, $this->clases[$clase])){
            return $this->clases[$clase];
        }
 
        if(class_exists($clase)){
            $instancia = new $clase;
            if(!empty($parametros) && method_exists($instancia, '__construct')){
                call_user_func_array(array($instancia, '__construct'), $parametros);
            }
            $this->clases[$clase] = $instancia;
            return $this->clases[$clase];
        }
        exit("La clase '$clase' no existe");
    }
}
 

El cuerpo de la segunda condición no requiere que compruebes si existe el constructor si agregaste parametros por defecto en la clase que estas queriendo instanciar, así que puedes hacer algo como lo siguiente

Código PHP:

class Mi_Clase_Principal{
 
    private $clases = array();
 
    function clase($clase, $parametros = array()){
 
        if(isset($this->clases[$clase]) && in_array($clase, $this->clases[$clase])){
            return $this->clases[$clase];
        }
 
        if(class_exists($clase)){
            $instancia = (empty($parametros)) ? new $clase : new $clase(...$parametros);
 
            $this->clases[$clase] = $instancia;
 
            return $this->clases[$clase];
        }
 
        throw new Exception("La clase $clase no existe");
    }
}
 

Si mueves tu clase a un directorio lib como en nuestro ejemplo y creas una clase para probar; en mi ejemplo mi clase es de esta forma:

Código PHP:

//archivo A.php en lib/
class A {
    
    private $x;
    
    private $y;
 
    public function __construct($x = 'hola', $y = 'mundo')
    {
        $this->x = $x;
        $this->y = $y;
    }
    
    public function display()
    {
        echo "$this->x $this->y";
    }
}
 

Y juntamos todo, nuestro ejemplo puede quedar algo como esto

Código PHP:

spl_autoload_register(function($class){
    $file = sprintf('%s/lib/%s.php',__DIR__, $class);
 
    if(file_exists($file)) {
        require_once($file);
    } else {
        throw new Exception("Cant load file: $file");
    }
    
});
 
//Instanciar clase qie se encuentra en la carpeta lib/
try{
    $cp = new Mi_Clase_Principal();
 
    $instancias = $cp->clase('A', [1, 2]);
 
    print_r($instancias->display());
 
}catch (Exception $e) {
 
    echo $e->getMessage();
 
}
 

Espero que el ejemplo te haya ayudado, lo siguiente es que ahora hagas tu propia implementación de autoloading y que mejores la clase Mi_Clase_Principal.