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

Question

[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?

Answer 1

[Sergei Iamskoi]

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

У вас не должна функция возвращать разные типы. В случае неправильного параметра кидайте экспешен, а сверху уже перехватывайте.

Comments

[bedolazhka]

как вариант тоже годится, спс

[anna_makeenko]

bedolazhka, это не ответ на вопрос. Вы спрашиваете про А, вам отвечают про Б. Вы пишете что сойдет и отмечаете решением? Мда...

[bedolazhka]

anna_makeenko, да, потому что как вариант, сам сделал так

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

[anna_makeenko]

bedolazhka, Вы показали код и спросили:

Какой тип нужно указать для метода removeQueryString?

Но отметили решением ответ о том, как переписать функцию, чтобы она возвращала string потому что:

как вариант тоже годится, спс

Логика.

[bedolazhka]

anna_makeenko,

throw new WrongParamException(); // или какая-то другая логика, например возвращать строку по умолчанию

private static function removeQueryString($url): string
    {
        if ($url) {
            $params = explode('&', $url, 2);
            if (strpos($params[0], '=') === false) {
                return rtrim($params[0], '/');
            }
        }
        return ''; // вернём пустую строку по умолчанию
    }

мне в ответ поместить? специально для Вас

я ответом иногда помечаю даже ответы левые, но где в комментариях тот, кто ответил, выводит ответ окончательный

[anna_makeenko]

bedolazhka, у Вас все ок?

Вы в своем вопросе привели код:

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

и спросили:

Какой тип нужно указать для метода removeQueryString?

Вам ответили:

bool|string

Но Вы отметили решением ответ, в котором автор разжевал как правильно нужно писать методы.

Если не можете сформулировать вопрос - это не значит что нужно отмечать решением ответы, которые не являются ответом на вопрос.

[bedolazhka]

Вам ответили: bool|string

нет такого в ответе

ответ, в котором автор разжевал как правильно нужно писать методы

автор ответа такого не разжевал, я сделал, как мне нужно

[anna_makeenko]

bedolazhka,

нет такого в ответе

Глаза раскройте.

автор ответа такого не разжевал, я сделал, как мне нужно

Не обязательно рассказывать про свою самовлюбленность.

[bedolazhka]

anna_makeenko, предыдущий мой коммент

[anna_makeenko]

bedolazhka, рукалицо

[bedolazhka]

anna_makeenko, предыдущий мой коммент

[anna_makeenko]

bedolazhka, да уже понятно, что ты дебил, не продолжай.

Answer 2

[anna_makeenko]

Ничего не нужно указывать, если у вас не PHP 8 конечно :D

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

В PHP 8 можно будет указывать bool|string

Answer 3

[Mylistryx]

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