Что можно улучшить в коде компонента?

Question

[Lander]

Недавно работаю на yii2, поэтому ещё очень слабо знаю его особенности. Вот понадобилось написать компонент. Прошу помощи у гуру, что можно улучшить и как вообще правильно писать компоненты для yii2? Приветствуется любая критика, даже самая незначительная.
Код компонента: https://github.com/nplander/yii2-robokassa-compone...
Заранее спасибо вам за потраченное время.

Answer 1

[Виталий Хоменко]

Насколько вижу вам не обязательно наследовать yii\base\Component, достаточно yii\base\Object. Component добавляет в объект поведения(Behavior) и события(Event). У вас же вроде используется лишь "магическая" конфигурация, которая есть в Object.

Также все параметры компонента я разместил бы в config/main.php или в main-local.php, и обращался бы через Yii::$app->robokassa->... но это субъективно.

Comments

[Lander]

А если разместить параметры в main.php, наследование от yii\base\Object подойдёт? Или же если я его в конфиге объявляю компонентом, то и наследовать от компонента надо?

[Виталий Хоменко]

Lander: Там без разницы. Вы наследуете определенный класс фреймворка лишь чтобы получить его возможности. Если вам не нужны Behavior и Event, то не за чем наследовать Component.

К примеру можно сделать Bootstrap-компонент, который будет выполняться при каждом запросе страниц. Для этого его нужно прописать в секции components конфигурации, указать его в секции bootstrap, применить интерфейс BootstrapInterface и реализовать публичный метод bootstrap().

Вот пример: конфиг c2n.me/3m4Cort и компонент c2n.me/3m4CB3y

[Lander]

Виталий Хоменко: Оу! Спасибо большое за разъяснения! А по стилю, структуре и прочему есть замечания? Не верю, что больше нечего улучшить...

[Виталий Хоменко]

Lander: Я глянул лишь поверхностно, ничего проблемного не увидел. Заметил, что вы подражаете поведению ActiveRecord, что мне кажется лишним. Эти load() и validate() меня немного сбили, я сперва подучал что Robokassa у вас это сущность. Улучшать есть что, однозначно. Смотрите на Github примеры реализации других платежных систем, сравнивайте со своей.

[Виталий Хоменко]

Lander: Если не сложно, отметьте решением ответ.