Код: Выделить всё
public function getFullName()
{
return $this->last_name . ' ' . $this->first_name . ' ' . $this->middle_name;
}
Код: Выделить всё
class UserViewHelper
{
public static function getFullName($model)
{
return $model->last_name . ' ' . $model->first_name . ' ' . $model->middle_name;
}
}
это визуальное представление атрибутовlynicidn писал(а):в моделе ар и оставлять, это простая конкатенация значений колонок, это слой бд
перенести все во вью слой. Например виджет.Bio man писал(а): Тут говорят, что в view слой, а как не говорят.
Создать класс со статическими методами?
для меня тоже, тем не менее никто не ограничивает использовать и для простых вещей. Хотя лично я бы обошелся хелпером.lynicidn писал(а):для меня виджет это более сложная логика с хтмл или же со яваскриптами
Код: Выделить всё
class Vehicle extends ActiveRecord
{
use VehicleTrait;
const TYPE_CAR = 'car';
const TYPE_MOTORCYCLE = 'motorcycle';
const STATUS_ACTIVE = '1';
const STATUS_INACTIVE = '0';
// В модели храниться код, который непосредственно связан с AR, более никакой дополнительной логики
// Условия к запросам я люблю выносить в Query класс
}
Код: Выделить всё
trait VehicleTrait
{
use PictureTrait;
/**
* @return string
*/
public function getVehicleName()
{
return implode(' ', [$this->carBrand->name, $this->carModel->name, $this->name]);
}
/**
* @param null $size
* @return string
*/
public function getVehicleImage($size = null)
{
$picSize = is_null($size) ? self::$car_168x99 : $size;
if (is_null($this->photos)) {
return 'default.png';
}
return Html::encode($this->getPictureUrl($this->photos['0'], $picSize));
}
/**
* @return bool
*/
public function hasPictures()
{
return !is_null($this->photos);
}
}
Код: Выделить всё
class Car extends Vehicle implements FeedProviderInterface, ChannelProviderInterface
{
const TYPE = self::TYPE_CAR;
public function init()
{
parent::init();
$this->setAttribute('vehicle_type', self::TYPE_CAR);
}
public static function find()
{
return new VehicleQuery(get_called_class(), ['type' => self::TYPE]);
}
public function rules()
{
$rules = parent::rules();
return $rules;
}
/**
* @return string
*/
public function getType()
{
return $this->vehicle_type;
}
/**
* @return int
*/
public function getTypeId()
{
return $this->id;
}
/**
* @return int
*/
public function getOwnerId()
{
return $this->user_id;
}
/**
* @return int
*/
public function getLanguageId()
{
return $this->language_id;
}
/**
* @return string
*/
public function getChannelModel()
{
return Channel::MODEL_CAR;
}
/**
* @return int
*/
public function getChannelKey()
{
return $this->getTypeId();
}
}
$car = new Car();
echo $car->getFullName(); // вызов метода trait'а который подключен в родительском классе
Код: Выделить всё
public function fields()
{
return [
'id',
'user_id',
'name',
'text',
'photos' => function () {
// Какой-то код
// В несколько строчек
// С использованием оператора сравнения
},
'photo' => function () {
// Какой-то код
// В несколько строчек
// С использованием оператора сравнения
},
'fullName' => function () {
return implode(' ', [$this->carBrand->name, $this->carModel->name, $this->name]);
},
}
Код: Выделить всё
public function fields()
{
return [
'id',
'user_id',
'name',
'text',
'photos' => function (Vehicle $model) {
return $model->photos();
},
'photo' => function (Vehicle $model) {
return $model->mainPhoto();
},
'fullName' => function (Vehicle $model) {
return $model->fullName();
},
}
Ну зачем же так жестко ? Лучше бы описали или аргументировали, показали пример как лучше. У меня например нет представления, я использую Yii в качестве REST фреймворка, зачем мне, например widget ?zelenin писал(а):только учтите, что добавляя в модель метод типа getFullName, вы используете модель в качестве хелпера для view, и в аду (а вы точно там будете) вам придется несладко.
модель - это бизнес-слой. Отображение имени в одном формате в api и еще в трех форматах в представлении не относится к бизнесу.maxsupermaxyii писал(а):Ну зачем же так жестко ? Лучше бы описали или аргументировали, показали пример как лучше. У меня например нет представления, я использую Yii в качестве REST фреймворка, зачем мне, например widget ?zelenin писал(а):только учтите, что добавляя в модель метод типа getFullName, вы используете модель в качестве хелпера для view, и в аду (а вы точно там будете) вам придется несладко.
Код: Выделить всё
public function fields()
{
return [
// field name is the same as the attribute name
'id',
// field name is "email", the corresponding attribute name is "email_address"
'email' => 'email_address',
// field name is "name", its value is defined by a PHP callback
'name' => function ($model) {
return $model->first_name . ' ' . $model->last_name;
},
];
}
Код: Выделить всё
'name' => function ($model) {
return $model->first_name . ' ' . $model->last_name;
},
Я согласен с этим правилом, если мы рендерим представление на сервере, работаем непосредственно с представлением. Имеет смысл использовать виджеты, а трейты перенести в widget с модели.zelenin писал(а):Модель не должна заботиться о том, как она будет выглядеть в представлении или выхлопе апи.
Код: Выделить всё
class HelloWidget extends Widget
{
use HelpfulTrait
public $data;
public $view = 'something';
public function run()
{
return $this->render($this->view, ['data'=>$this->data])
}
}
// Родительское представление в котором вызывается widget
<?= HelloWidget::widget(['data' => $this->dataFromController]) ?>
Код: Выделить всё
trait HelpfulTrait {
public function doSomething() {
return $this->name . ' ' . $this->secondName;
}
}
Код: Выделить всё
// Представление something.php которое используется в widget
<p>SomeViewHelper::doSomething($this->data)</p>
Код: Выделить всё
// Представление something.php которое используется в widget
<p>$this->doSomething()</p>
вообще речи нет от трейтах или виджетах. трейт это способ реюзать код, а не какая-то отдельная сущность, поэтому правильность или неправильность обьсуждаем в контексте объекта куда этот трейт внедряется (т.е. модель).maxsupermaxyii писал(а):Я согласен с этим правилом, если мы рендерим представление на сервере, работаем непосредственно с представлением. Имеет смысл использовать виджеты, а трейты перенести в widget с модели.zelenin писал(а):Модель не должна заботиться о том, как она будет выглядеть в представлении или выхлопе апи.
если мы говорим об отвественностях, то трейт ею не обладает, но может добавить таковую объекту, в который внедряется. сам по себе трейт ничто.maxsupermaxyii писал(а):По сути, trait от хелпера который вы напишите сами отличается только названием и тем, что его можно внедрить в класс и использовать в контексте этого класса
причем тут трейты? я могу еще раз повторить: о методе из трейта нельзя рассуждать вне контекста объекта, в который внедряется. если трейт добавляет в модель метод getFullName, то я говорю об этом методе как о методе модели, утверждая, что модель не должна заниматься собственным отображением в различных представлениях (html/json).maxsupermaxyii писал(а):Код становится чище и понятней.
Если мы говорим о разработке REST и использование Yii2 для этой задачи, как фреймворка, я не вижу каких-то органичней, для того, что бы использовать трейты непосредственно в моделях в контексте Yii2, если это не большой энтерпрайз проект.
мы не видим тут модели - мы видим json-представление. что было между моделями и json бог весть. очевидно, что цепочка построена примерно так: UserModel -> UserApiAssembler -> UserApiDTO. И именно UserApiDTO мы и видим. В yii в качестве ассемблера выступает сериалайзер, который с помощью fields, преобразует в модель в dto. Но вводя в модель fields, мы заставляем модель еще думать и о своем отображении в апи, что неверно.maxsupermaxyii писал(а):Если мы вернемся к методу getFullName и использование этого метода в REST то можно обратиться за примером к Facebook Graph API, при вызове
/me?fields=first_name,last_name,middle_name,name
мы получим ответ вида
{
"first_name": "FirstName",
"last_name": "LastName",
"name": "FirstName LastName",
"id": "{id}"
}
Мы видим, что модель есть некоторым представлением данных.
