Ошибка в php коде.Что нужно изменить?

Question

[Данил Короткин]

Пытаюсь писать собственную CMS и возникла такая проблема:

Исходник:

class Router
{
    private $routes = [];
    private $dispatcher;
    private $host;

    /**
     * Router constructor.
     * @param $host
     */
    public function __construct($host)
    {
        $this->host = $host;
    }

    /**
     * @param $key
     * @param $pattern
     * @param $controller
     * @param string $method
     */
    public function add($key, $pattern, $controller, $method = 'GET')
    {
        $this->routes[$key] = [
            'pattern'    => $pattern,
            'controller' => $controller,
            'method'     => $method
        ];
    }

    public function dispatch($method, $uri)
    {
        return $this->getDispatcher()->dispatch($method, $uri);
    }

    /**
     * @return mixed
     */
    public function getDispatcher()
    {
        if($this->dispatcher == null)
        {
            $this->dispatcher = new UrlDispatcher();

            foreach ($this->routes as $route)
            {
                $this->dispatcher->register($route['method'], $route['pattern'], $route['controller']);
            }


        }

        return $this->dispatcher;
    }

Answer 1

[kpa6uu]

Comments

[Данил Короткин]

Легче не стало.Что именно нужно сделать?

[D3lphi]

KlausWinter: Легче станет после того, как в переводчик зайдете.

[Данил Короткин]

D3lphi: Я понимаю,что там слишком много аргументов,но как их может быть слишком много,если он один

[D3lphi]

МНОГО? FEW - мало, с английского. Вы не передаете ни одного аргумента в функцию.

[Данил Короткин]

KlausWinter: Я дурак *facepalm*

Answer 2

[novrm]

Разве трудно понять?
Где-то вы передаете в конструктор НОЛЬ переменных, а требуется лишь ОДНА.

Кстати - так чисто удовлетворить любопытство?
Вам некуда тратить свое драгоценно время, чем на переизобретение CMS?

Comments

[Данил Короткин]

Да,было трудно,еще новичок.
Таким образом пытаюсь набираться опыта.

[D3lphi]

Где-то вы передаете в конструктор ДВЕ переменных, а требуется лишь ОДНА.

Две? Может, воообще ничего не передает?

0 passed

[novrm]

D3lphi: ошибся - ноль... уже исправил

[novrm]

KlausWinter: Если вы воспринимаете советы...
... то писание своей CMS это не самый лучший способ набраться опыта.

[Данил Короткин]

novrm: Что можете посоветовать как альтернативу?

[novrm]

1. пишите сразу свой сайт в лоб как понимаете без CMS...
2. изучайте готовые CMS.
3. пишите блог на любом фреймворке - например Lavarel или Yii или Zend Framework или Symfony - в порядке сложности...

[Данил Короткин]

Спасибо за совет,буду пробовать)

[novrm]

KlausWinter: Кстати - если нужна будет помощь по постройке блога на Zend Framework 3 - могу бесплатно помочь: [e-mail]

[Сергей Мирченко]

novrm: Ну, тут я и с Вами не согласен (частично) и с автором вопроса.

К примеру, я делаю сейчас свой микро-фреймворк, для маленьких сайтов визиток, с примитивной системой управления контентом. Там будет примерно 15-20 базовых контроллеров, которые будут решать все задачи. Просто для таких целей "микро-фреймворк" Lumen от разработчиков Laravel не подходит, в нем куча ненужного. Других альтернатив я не знаю. Может они и есть, но без документации и с кучей ненужного.

Может Автор это и имел в виду, когда писал о своей "CMS". Если так, то он на правильном пути, и лучше чем реализовать что-то свое, на этапе обучения - придумать сложно.

[Данил Короткин]

Сергей Мирченко: Это, наверное, громко сказано "собственная CMS", по видео-урокам просто переписываю код и пытаюсь разобраться что к чему, дабы иметь хоть какое-то представление.

[Александр Шаповал]

Это, наверное, громко сказано "собственная CMS", по видео-урок просто переписываю код и пытаюсь разобраться что к чему, дабы иметь хоть какое-то представление.

В таком случае вставлю свои пять копеек, и порекомендую уже брать за привычку использовать новшества добавленные в PHP7, а именно типизацию передаваемых параметров в ф-ции.
Например вместо

/**
     * @return mixed
     */
    public function getDispatcher()
    {
    }

Пишите (работает в PHP7.1)

public function getDispatcher(): void
    {
    }

И комментарий return mixed не нужен, и ф-ция не вернет ничего неожиданного.
Аналогично с параметрами (работает в PHP7.0 и выше)

public function add(int $key, string $pattern, string $controller, string $method = 'GET')
    {
    }

[Данил Короткин]

Александр Шаповал: Спасибо большое за совет!