0

сообщество. Не пойму то ли баг, то ли фича.

// router.php
public function match()
    {
     // other code
        foreach ($this->routes AS $route) {
            if (true)) {
                // other code
                // Если все нормально вернет Route и механизм отработает
                // Но в лог попадет второй return 3;
                echo 2;
                return true;
            }
        }
        return 3;
    }
    // далее в index.php
    $router->run($router->match(), $di);

В error.log сыпятся ошибки

PHP Fatal error:  Uncaught TypeError: Argument 1 passed to system\\Router::run() must be an instance of system\\Route, boolean given, called in ... system\\Router->run(true, Object(system\\DependencyInjector))\n#1 {main} ...
PHP Fatal error:  Uncaught TypeError: Argument 1 passed to system\\Router::run() must be an instance of system\\Route, integer given, called in ... system\\Router->run(3, Object(system\\DependencyInjector))\n#1 {main} ...

То есть оба возврата выполняются и строка выводится. Есть мысли по этому поводу? Код вызывается только один раз, break не подействует. UPD: Если убрать строгий тип run(Route $route, Di $di) ошибка пропадает как и полагалось UPD: Банально не отрабатывается правило в htaccess на предмет favicon.ico . Запрос не находит картинку и сыплет ошибки. Вопрос решен

5
  • Разумеется, 2 возврата означают 2 вызова функции
    – vp_arth
    12 мая 2018 в 19:05
  • Так у меня match нигде не используется кроме старта, не могу отследить этого
    – ozornick
    12 мая 2018 в 19:11
  • а если $router->match() в другую какую функцию вставить как параметр, будет тоже 2 раза запускаться?
    – TigerTV.ru
    12 мая 2018 в 19:52
  • Если вы используете фреймворк какой, у вас могут быть подзапросы внутри приложения. Здесь явно два вызова index.php
    – korytoff
    12 мая 2018 в 20:12
  • Быстренько добросил код выше run(): function test(\system\Route $r){ debug($r); } test($router->match()); В логе ошибка в test(NULL)
    – ozornick
    13 мая 2018 в 15:51

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Посмотрите другие вопросы с метками или задайте свой вопрос.