@bedolazhka

Какой тип тут нужно вернуть?

Пациент
<?php


namespace core;


class Router
{
    private static array $routes;
    private static array $route;

    private static function upperCamelCase($name): string
    {
        return str_replace(' ', '', ucwords(str_replace('-', ' ', $name)));
    }

    private static function removeQueryString($url)
    {
        if ($url) {
            $params = explode('&', $url, 2);
            if (strpos($params[0], '=') === false) {
                return rtrim($params[0], '/');
            }
        }
        return false;
    }

    private static function matchRoute($url): bool 
    {
        foreach (self::$routes as $pattern => $route) {
            if (preg_match("#{$pattern}#i", $url, $matches)) {
                foreach ($matches as $k => $v) {
                    if (is_string($k)) {
                        $route[$k] = $v;
                    }
                }
                if (!isset($route['action'])) {
                    $route['action'] = 'index';
                }
                self::$route['controller'] = self::upperCamelCase($route['controller']);
                self::$route['action'] = lcfirst(self::upperCamelCase($route['action']));
                return true;
            }
        }
        return false;
    }

    public static function add($regex, $route = []): void 
    {
        self::$routes[$regex] = $route;
    }

    public static function dispatch($url): void 
    {
        if (self::matchRoute(self::removeQueryString($url))) {
            $controller = 'app\controllers\\' . self::$route['controller'] . 'Controller';
            if (class_exists($controller)) {
                $cObj = new $controller(self::$route);
                $action = self::$route['action'] . 'Action';
                if (method_exists($cObj, $action)) {
                    $cObj->$action();
                    $cObj->getView();
                } elseif ($_SERVER['REMOTE_ADDR'] == '127.0.0.1') {
                    echo "<code>Метод {$controller}::{$action} не найден</code>";
                } else {
                    http_response_code(404);
                    require_once WWW . '/404.html';
                    die();
                }
            } elseif ($_SERVER['REMOTE_ADDR'] == '127.0.0.1') {
                echo "<code>Контроллер {$controller} не найден</code>";
            } else {
                http_response_code(404);
                require_once WWW . '/404.html';
                die();
            }
        } else {
            http_response_code(404);
            require_once WWW . '/404.html';
            die();
        }
    }
}

Какой тип нужно указать для метода removeQueryString?
  • Вопрос задан
  • 144 просмотра
Решения вопроса 1
syamskoy
@syamskoy
private static function removeQueryString(string $url): string
{
    $params = explode('&', $url, 2);
    if (strpos($params[0], '=') === false) {
        return rtrim($params[0], '/');
    }
    throw new WrongParamException(); // или какая-то другая логика, например возвращать строку по умолчанию
}


У вас не должна функция возвращать разные типы. В случае неправильного параметра кидайте экспешен, а сверху уже перехватывайте.
Ответ написан
Пригласить эксперта
Ответы на вопрос 2
@anna_makeenko
Ничего не нужно указывать, если у вас не PHP 8 конечно :D

https://wiki.php.net/rfc/union_types_v2

В PHP 8 можно будет указывать bool|string
Ответ написан
Комментировать
@Mylistryx
Сейчас (PHP ~7.3.0) в phpDoc:
return bool|string
а указывать, что вернет ф-ия (метод) ненадо
P.S. PHP 7.4 - дали не выход, что можно указать интерфейс, а ГРАБЛИ!
не бегите вперед паровоза! Все успеется.
Ответ написан
Комментировать
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы