Недавно работаю на yii2, поэтому ещё очень слабо знаю его особенности. Вот понадобилось написать компонент. Прошу помощи у гуру, что можно улучшить и как вообще правильно писать компоненты для yii2? Приветствуется любая критика, даже самая незначительная.
Код компонента: https://github.com/nplander/yii2-robokassa-compone...
Заранее спасибо вам за потраченное время.
Насколько вижу вам не обязательно наследовать yii\base\Component, достаточно yii\base\Object. Component добавляет в объект поведения(Behavior) и события(Event). У вас же вроде используется лишь "магическая" конфигурация, которая есть в Object.
Также все параметры компонента я разместил бы в config/main.php или в main-local.php, и обращался бы через Yii::$app->robokassa->... но это субъективно.
А если разместить параметры в main.php, наследование от yii\base\Object подойдёт? Или же если я его в конфиге объявляю компонентом, то и наследовать от компонента надо?
Lander: Там без разницы. Вы наследуете определенный класс фреймворка лишь чтобы получить его возможности. Если вам не нужны Behavior и Event, то не за чем наследовать Component.
К примеру можно сделать Bootstrap-компонент, который будет выполняться при каждом запросе страниц. Для этого его нужно прописать в секции components конфигурации, указать его в секции bootstrap, применить интерфейс BootstrapInterface и реализовать публичный метод bootstrap().
Lander: Я глянул лишь поверхностно, ничего проблемного не увидел. Заметил, что вы подражаете поведению ActiveRecord, что мне кажется лишним. Эти load() и validate() меня немного сбили, я сперва подучал что Robokassa у вас это сущность. Улучшать есть что, однозначно. Смотрите на Github примеры реализации других платежных систем, сравнивайте со своей.
Средний
