Автозагрузка классов PHP, как сделать?

Question

[Алексей Веснин]

Всем привет. Пытаюсь сделать, автозагрузку классов для админки из папки admin, для остального из папки controllers.

Но что-то не получается. Что я делаю:

Маршрутизатор уже подключен $lib_path , далее делаю проверку, если layout == admin, тогда путь один, если нет, то другой. Код автозагрузки:

<?php

require_once (ROOT.DS.'config'.DS.'config.php');

function __autoload($class_name){
    $lib_path = ROOT.DS.'lib'.DS.strtolower($class_name).'.class.php';

    $model_path = ROOT.DS.'models'.DS.strtolower($class_name).'.php';

    if (file_exists($lib_path)){
        require_once ($lib_path);
    } 

  // Возвращает маршрут фронт или админки( default или admin)
    $layout = new Router($uri);
    $layout = $layout->getRoute();
    if ($layout == 'admin'){
        $controllers_path = ROOT.DS.'controllers/admin'.DS.str_replace('controller', '', strtolower($class_name)).'.controller.php';
    } else {
        $controllers_path = ROOT.DS.'controllers'.DS.str_replace('controller', '', strtolower($class_name)).'.controller.php';
    }

if (file_exists($controllers_path)){
        require_once ($controllers_path);
    } elseif (file_exists($model_path)){
        require_once ($model_path);
    } else {
      
       throw new Exception('Failed to include class' . $class_name);
    }


}

Получаю ошибку:

prntscr.com/h7o35w

Не могу понять, почему он пытается загрузить Class 'Router' , вместо того, чтобы выполнить условие.

Класс Router :

<?php

class Router {

    protected $uri;

    protected $controller;

    protected $action;

    protected $params;

    protected $route;

    protected $method_prefix;


    public function __construct($uri) {
        $this->uri = urldecode(trim($uri, '/'));

        // Получаем значения по умолчанию
        $routes = Config::get('routes');
        $this->route = Config::get('default_route');
        $this->method_prefix = isset($routes[$this->route]) ? $routes[$this->route] : '';
        $this->controller = Config::get('default_controller');
        $this->action = Config::get('default_action');
        // Разбиваем запрос на строку
        $uri_parts = explode('?', $this->uri);
        // Получаем путь ввида /controller/action/param1/param2/.../.../...
        $path = $uri_parts[0];

        $path_parts = explode('/', $path);

        // Проверяем есть ли в массиве $routes значение из текущей строки запроса
        if (in_array(strtolower(current($path_parts)), array_keys($routes))){
            // Если значение есть, записываем в переменную $this->route значение из текущего пути запрса
            $this->route = strtolower(current($path_parts));
            Config::set('aroute','strtolower(current($path_parts))');
            $this->method_prefix = isset($routes[$this->route]) ? $routes[$this->route] : '';
            array_shift($path_parts);
        }

        // Получаем контроллер - следующий элемент в массиве
        if (current($path_parts)) {
            $this->controller = strtolower(current($path_parts));
            array_shift($path_parts);
        }

        // Получаем вызываемый метод
        if (current($path_parts)){
            $this->action = strtolower(current($path_parts));
            array_shift($path_parts);
        }

        // Получаем параметры - оставшаяся часть запроса
        $this->params = $path_parts;

      //  echo "<pre>"; print_r($routes[$this->route]);
    }

    public function getUri()
    {
        return $this->uri;
    }

    public function getController()
    {
        return $this->controller;
    }

    public function getAction()
    {
        return $this->action;
    }

    public function getParams()
    {
        return $this->params;
    }

    public function getRoute()
    {
        return $this->route;
    }

    public function getMethodPrefix()
    {
        return $this->method_prefix;
    }

    public static function redirect($location){
        header("Location: $location");
    }



}

Что я делаю не так?

Answer 1

[Arman]

Видно что подключили config, но не видно где подключили класс Router или он в этом же файле? или выше где-то инициализируется?

Comments

[Алексей Веснин]

Неправильно написал сразу немного, правильно так наверное:

<?php
// Выполняет автозагрузку классов


require_once (ROOT.DS.'config'.DS.'config.php');

function __autoload($class_name){
    $lib_path = ROOT.DS.'lib'.DS.strtolower($class_name).'.class.php';








    $model_path = ROOT.DS.'models'.DS.strtolower($class_name).'.php';

    if (file_exists($lib_path)){
        require_once ($lib_path);
        
    }


    // Возвращает маршрут фронт или админки default или admin
    $layout = new Router($uri);
    $layout = $layout->getRoute();
    if ($layout == 'admin'){
        $controllers_path = ROOT.DS.'controllers/admin'.DS.str_replace('controller', '', strtolower($class_name)).'.controller.php';
    } else {
        $controllers_path = ROOT.DS.'controllers'.DS.str_replace('controller', '', strtolower($class_name)).'.controller.php';
    }




    if (file_exists($controllers_path)){
        require_once ($controllers_path);
    } elseif (file_exists($model_path)){
        require_once ($model_path);
    } else {
      //  header( "Location: /404/index.html", true);
       throw new Exception('Failed to include class' . $class_name);
    }


}

но тогда возникает другая ошибка:

prntscr.com/h7o6uw

[Алексей Уколов]

Алексей Веснин, ну что же тут непонятного - у вас Config::get('routes') возвращает null, разберитесь, почему это происходит.

Answer 2

[Алексей Уколов]

Не могу понять, почему он пытается загрузить Class 'Router' , вместо того, чтобы выполнить условие.

Потому что вы ему об этом говорите при помощи new Router($uri).

Comments

[Алексей Веснин]

А если у меня getRoute() не статический метод, то я же не могу его вызвать так:

Route::getRoute()
как тогда правильно?

[Алексей Уколов]

Правильно сначала подключить файл класса, а потом уже использовать. Если бы метод был статическим, ошибка бы никуда не делась.

[Алексей Веснин]

Алексей Уколов, так он вот здесь подключается, я сразу не правильно написал в шапке поста:

if (file_exists($lib_path)){
        require_once ($lib_path);
        
    }

[Anton Mashletov]

Алексей Веснин, автолоад внутри автолоада, возможно, не будет работать. Подключайте роутер с помощью require.