S
да в целом типизация зло, ведь ты никогда не знаешь, что тебе придет в функцию, объект, строка или число. Да и по уму надо было сделать было 1 тип и все методы работы только с ним)
😂😂😂
Size: a a a
2016 July 02
N
а для простых данных нельзя указать вариацию..
VR
Вы чо тут насрали?
2016 July 03
🄽
Хм
🄽
А отсутствие интерфейсов, это нормально?
N
работает и ок
🄽
Пытался ознакомится с кодом со своего nexus 5, и первое что бросилось в глаза, так это передача $transport любой переменной без проверки типа .. а позже соответственно вызов методов от него. Эта переменная передается в yii::createObject, и от нее вызывают методы, которых может не быть. Такое в коде большого проекта отследить будет трудно
🄽
Хотябы несколько интерфейсов или уж абстрактных классов, на методы фабрики
🄽
Нет? Может я что-то упускаю?
2016 July 04
S
Эта переменная передается в yii::createObject, и от нее вызывают методы, которых может не быть. Такое в коде большого проекта отследить будет трудно
это как так вызываются методы которых может не быть? :)
🄽
Простой пример:
🄽
public function setTransport($transport)
{
$this->_transport = $transport;
}
public function getTransport()
{
if (!is_object($this->_transport)) {
$this->_transport = Yii::createObject($this->_transport);
}
return $this->_transport;
}
public function send($request)
{
return $this->getTransport()->send($request);
}
{
$this->_transport = $transport;
}
public function getTransport()
{
if (!is_object($this->_transport)) {
$this->_transport = Yii::createObject($this->_transport);
}
return $this->_transport;
}
public function send($request)
{
return $this->getTransport()->send($request);
}
🄽
В setter методе нет проверки по типу, поэтому туда может быть передан любая сущность , даже не класс. Если даже класс, то он может не обладать нужным набором атрибутов или свойств, необходимых для полноценной работы transport сущности.
🄽
Таким образом ошибки могут возникнуть на любом из этапов, если этот компонент не будет вызываться ток в 1 месте, а будет обернут во что либо динамическое, например внутри какой либо crm
🄽
Разработчик пользователь может не углядеть, каким требованиям должен удовлетворять его новый класс типа транспорт, так как нет строгово интерфейса, или абстрактного класса. И баги будут выдавливаться по ходу дела
🄽
В приведенной вырезке, у $transport может не быть send метода. Да глупость, но не требуют наличие и не проверяют, вплоть до места вызова..
🄽
И в этом духе все остальное. Этот компонент сделан расширяемый, но отсутствуют основные элементы SOLID
🄽
Если не прав, поправте
AB
Никита, я рекомендую тебе написать в гиттер с упоминанием samdark