0

Estoy desarrollando una platilla MVC en PHP para practicar y utilizo Index.php como controlador central.

Para elegir el Controlador y el método que se van a utilizar recojo los nombres de estos por parámetros GET en la URL y después instancio el controlador en el propio Index.php

El problema es que si instancio el controlador de esta manera:

$class = new IndexController();

el programa funciona perfectamente, pero si lo instancio de esta otra:

$controllerName = "IndexController";
$class = new $controllerName();

me lanza el siguiente error:

Uncaught Error: Class 'IndexController' not found

Os paso el código real para que lo veais: (Index.php)

<?php

require_once "vendor/autoload.php";

use Controllers\IndexController;
use Helpers\Request;

//$class = isset($_GET["class"]) ? new $_GET["class"] : new IndexController();

if (isset($_GET["class"])) {
    $class = new $_GET["class"](); // $class = "IndexController"; <-- NO FUNCIONA -->
}else{
    $class = new IndexController(); // <-- SI FUNCIONA -->
}

$function = isset($_GET["function"]) ? $_GET["function"] : "index";

if (count($_POST) > 0) {
    $request = new Request();

    $class->$function($request);
}elseif (count($_GET) > 0) {
    $request = new Request();

    $class->$function($request);
}else{
    $class->$function();
}

(IndexController.php):

<?php

namespace Controllers;

use Core\Controller;
use Models\Users;
use Helpers\Request;

class IndexController extends Controller
{

    public function __construct()
    {

    }


    public function index()
    {
        $user = new Users();

        $this->data["users"] = $user->getAll();
        
        $this->render("index");
    }


    public function prueba(Request $request)
    {
        $user = new Users();

        $this->data["users"] = $user->getById($request->get["id"]);

        $this->render("prueba");
    }
}

Si accedo a la URL:

http://checkinout.test?class=IndexController&function=prueba&id=1

Obtengo el error mencionado.

PD: utilizo el autoloader PSR-4 de composer.

10
  • Según el ejemplo 3 del Manual de PHP debería funcionar. ¿Intentaste con una variable en vez del GET? ¿Revisaste que el GET no tiene espacios en blanco o cosas por el estilo?
    – A. Cedano
    Commented el 13 may. 2021 a las 14:19
  • yo cuando hago cosas de ese estilo tengo que poner el new entre parentesis. Algo asi como $clase = (new $temp()) siendo $temp='IndexController' deberia funcionarte... Aunque no es directamente tu caso, en mi pregunta es.stackoverflow.com/questions/395799/… en el ultimo comentario me lo solucionan. Igual puede valerte.
    – Jakala
    Commented el 13 may. 2021 a las 14:47
  • @A.Cedano He intentado esa solución también sin éxito. Incluso si directamente hago; $var = "IndexController"; $class = new $var() también me da error. No encuentro solución alguna en internet. Commented el 13 may. 2021 a las 21:17
  • 1
    vale, Entonces puedes probar a concatenar en la variable el namespace? quiero decir, algo como esto: $temp = 'Controllers\'.$__GET['class']; $clase = (new $class()); .
    – Jakala
    Commented el 14 may. 2021 a las 7:35
  • 1
    creo que es mas un problema estructural. lo que sucede es que el index.php, que es la entrada, no usa namespace (porque se encarga de cargar el autoload), y no te va a encontrar de forma dinamica los controllers. Creo que la cuestion seria que hagas como otros frameworks, tu index.php sera el que cree una clase Kernel.php, que tendra su namespace, y ahi SI que puedes poner la logica de leer el __GET para dirigir a tu controller de forma dinamica.
    – Jakala
    Commented el 14 may. 2021 a las 9:58

0

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

Examina otras preguntas con la etiqueta o formula tu propia pregunta.