Почему Notice: Undefined variable при подключении файла через require. хотя переменная точно есть?

Question

[Павел Клочков]

Здравствуйте!
Сделал файл со списком классов, там простой массив:

$a = array (
    'Route'   => '/core/system/autoload/',
    'handler' => '/core/system/config/'
);

Другой файл подключается к нему и перебирает массив

require_once ('Classes.php');

    foreach ($a as $b => $c) {
        echo $b . '->' . $c . '<br>';
    }

Браузер при этом выдаёт следующее:

Route->/core/system/autoload/
handler->/core/system/config/

Notice: Undefined variable: a in /home/fenix/WWW/pv.site/core/system/autoload/Autoloader.php on line 19

Warning: Invalid argument supplied for foreach() in /home/fenix/WWW/pv.site/core/system/autoload/Autoloader.php on line 19

Никак не могу понять в чём ошибка. Ведь массив виден, обрабатывается корректно, Но при этом показывает ошибки.

Comments

[Дмитрий]

Добрый вечер.
Именование переменных на высоте!!!
$a as $b => $c

[Mylistryx]

Я не буду спрашивать зачем нужна такая загадочная конструкция, но для поиска ошибки неплохо было бы увидеть /home/fenix/WWW/pv.site/core/system/autoload/Autoloader.php вот этот файл. Ну хотяб строки 1-20.

[Павел Клочков]

Mylistryx, ошибка была в require_once

[Павел Клочков]

Дмитрий, да у меня есть дурацкая привычка когда пишу что-то новое для себя (то есть почти всё) сначала давать переменным такие имена. И только поняв, что всё работает как мне хочется начинаю придумывать им вменяемые имена.

Answer 1

[FanatPHP]

Между выводом $a и 19-й строкой происходит что-то ужасное.
Мы не знаем, что

Comments

[Павел Клочков]

Ничего ужасного до 19-й строки нет. только комментарии с моими хотелками к этой функции.
Что-то ужасное происходит после.

function pvAutoloader($className)
{

    require_once (__DIR__ . '/../config/systemClasses.php');
    
    foreach ($a as $b => $c) {

        echo $b . '->' . $c . '<br>';
        
        if ($className == $b) {
           $classFullName  = '\\' . $c . '\\' . $b;
        }
    }

    $classFile = './' . str_replace('\\', '/', $classFullName) . '.class.php';
    
    
    if (file_exists($classFile)) {
        require_once $classFile;
    } else {
        Handler::engineError('Not File', $classFile);

        function exception_handler( $exception )
        {
            
            echo "<b>Ошибка:</b> <br>" , $exception->getMessage();
            $exception->getMessage();
        }
        set_exception_handler('exception_handler');
        throw new Exception('Невозможно загрузить класс:<b> { ' . $className . ' } </b>');
    }
}
spl_autoload_register('pvAutoloader');

При попытке вместо функции exception_handler запустить класс Handler.

Класса Route по нужному адресу нет (специально переименовал, чтобы проверить как будет показываться ошибка и понять как её исправить.

Пока понял, что всё нужно переделать.
Получается, что ошибка возникает когда

[FanatPHP]

Я был прав.
Весь этот код - ужас, летящий на крыльях ночи.
Никогда не пиши require_once в функции.

[Павел Клочков]

Благодарю за ответ и за чувство юмора!
Ошибку понял, поставил require - работает как задумал.
Что код ужасен, понимаю сам, но не всегда понимаю почему. Раньше было ещё хуже. Надеюсь постепенно научиться писать хороший и рабочий код.

Answer 2

[Vladimir Protsenko]

return [
    'Route'   => '/core/system/autoload/',
    'handler' => '/core/system/config/'
];

$a = require_once ('Classes.php');

    foreach ($a as $b => $c) {
        echo $b . '->' . $c . '<br>';
    }

Впрочем, как правильно заметили, что-то там происходит )