Автоматически задать новое свойство через setter?

Question

[NikSIk31]

как можно в laravel задать сеттер модели чтобы оно автоматом задавало новое значение коллекции, которого нету в БД?
например у меня есть автоматом изменение свойства через геттер:
я просто в моделе задал appends и геттер функцию

protected $appends = ['created_at'];

public function getCreatedAtAttribute($value)
    {
        return Date::parse($value)->format('d F в H:i');
    }

таким образом мне достаточно просто получить значения из БД и у меня будет спарсена дата. А как сделать тоже самое но чтобы было новое свойство?

Answer 1

[Григорий Васильков]

Намекну что лучше не привыкать учить модель делать то, что она не умеет, потому что она и без того в ларавеле несёт бешенный функционал, а написать свой класс в который класть модель на вход и в конструкторе заполнять данными из неё. И там это свойство сделать и другие модули опереть уже на свой класс, который ничего кроме взять-положить не умеет. Потому что такой лишенный функционала класс можно без проблем оставить навечно, т.к. он всегда будет выполнять задачу "дать-взять", в отличии от модели ларавеля которая умеет слишком много и отказ от неё будет означать мозговой штурм и попытки залепить получившиеся дыры скотчем

Но если прямо очень хочется модель то прям вот так и сделай

protected $appends = ['created_at2'];
getCreatedAt2Attribute() {}

если нужно чтобы сохранялось в базу - нужно записать в массив $this->attributes[ $name ] = $value; потом вызвать save();

Comments

[NikSIk31]

Последний вопрос в вопросе (как бы это дико не звучало) возможно ввел в заблуждение. Мне надо установить значение которого нет, т.е. установить - сеттер, а не геттером

[DevMan]

NikSIk31, это не имеет значения.

[Григорий Васильков]

NikSIk31, можешь в сеттере написать

setMyPropAttribute($value)
$this->attributes[ 'my_prop' ] = $value;

или в геттере

getMyPropAttribute()
$this->attributes[ 'my_prop' ] = 12345;

в attributes окажется новое значение которое будет записано в базу при вызове save(); и будет выдана ошибка, если его записать нельзя.

Где бы ты не написал с точки зрения проектирование - и то и то ошибка. Сеттеры и геттеры служат не для того, чтобы записывать что-то в базу.

Сеттерами иногда данные проверяют на то, чтобы они были определенной формы (смысл в том, чтобы когда другой программист вручную передает данные - вывалилась ошибка что тут так не задумывалось, ищи того кто это писал или думай). А геттерами - выводят например когда пилишь апишку и на базе одного поля делается второе - типа перевод на русский язык английского ключа перевода или конвертация формата даты - чтобы был и оригинал и второе конвертированное поле.

Но если геттеры или сеттеры меняют базу или делают куда-то запросы - это уже чудовищно кривая разработка. Но работать будет. Правда не долго. Пока ты не забудешь что у тебя есть такой странный механизм, который с какого-то хера меняет внешний источник втихаря

[NikSIk31]

Григорий Васильков, мне не для записи в БД оно надо

[Григорий Васильков]

NikSIk31, тогда тем более просто создай геттер и никуда ничего не сохраняй. из геттера верни значение. и его можно будет сделать appends. Имя геттера подскажет системе что как будто бы есть такое свойство и его можно будет вывести или обратиться к нему через стрелку.

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

[NikSIk31]

Григорий Васильков, мне они нужны для того чтобы сгенерить ссылку. У меня из бд получаю я slug + id + еще другое, надо добавить ссылку и отдать по апи, так сказать

[Григорий Васильков]

ну и отлично!

getUrlAttribute() : string
return $this->id . $this->slug . $this->drugoe;

просто следи чтобы не было public/protected $id, public/protected $slug и public/protected $drugoe, иначе обращение через $this-> внутри класса проигнорирует геттеры этих полей

только не привязывай к этому остальной код. потом пожалеешь. вывести в апи - ладно. но повесить на этом логику или зашить где-то в шаблонах - низя нини

[NikSIk31]

Григорий Васильков, а вот кстати насчет даты, я пытаюсь сделать конвертацию. Так вот в appends задал created_at , создал public function getCreatedAtAttribute и передал в него value , если вернуть value , то вернет текущее время (вообще вернет NULL, но у меня если в функцию его подставить, то будет текущее время), а если вернуть $this->created_at будет ошибка, мол такого нет свойства. (хотя если вернуть $this->updated_at , то все более чем норм, т.к. его нету в appends , как я понял ) Но меня updated не устраивает. Как быть? новое свойство что-ли создавать?

[Григорий Васильков]

а знач ошибся я, попробуй $this->getAttribute('created_at'), this попытался на уровне PHP получить свойство которого нет, ларавелевский код попросту не запускался

[NikSIk31]

Григорий Васильков, а как это "зашить в шаблонах?", я просто получаю уведомления в пагнации и мне надо им дату нормальную для воспрмятия сделать. + еще думаю для юзера модели сделать геттер " ссылка на профиль"

[Григорий Васильков]

Ну, я называю зашить - использовать код где бы то ни было кроме того места, где ты просто его вывел и не собираешься менять скорее всего никогда.

То есть не опирать другой код на обращение к этому месту. Случится что кроме базы данных у тебя будет например файл и нужно будет соединить данные из файла и базы, а тебя повсюду требуются модели которые только с базой работают. Будет долгий труд по вычищению старого кода и замены его на тупо класс который кроме дать-взять ничего не делает, чтобы на него опереть всё

[Григорий Васильков]

NikSIk31, как бы не хотелось вызывать модель и строить запрос прямо там, где он нужен - лучше эти вызовы обернуть в методы класса, и оттуда возвращать массив обьектов, но не самих моделей, а просто класса юзер какого-то.

то есть модель User у тебя тянет данные с базы, а просто User (говорят - сущность) у тебя держит свойства юзера. В неком классе (говорят - в репозитории) у тебя есть функции "получить то-то по том то", который возвращает массив вот этих простых сущностей или одну сущность. Соответственно добавится к данным временное хранилище (говорят - кеш), или добавится еще другой источник данных кроме базы, или потребуется выполнить поиск где-то ещё - у тебя осталось фиксированное название действия "получить тото по том то" и оно не изменится, т.к. меньше функционала уже не станет.

И ты этот метод уже вызываешь, а внутри метода можешь колхозить соединение с чем-то, кеширование и так далее. Важно в результат отдать массив Юзеров - не моделей, а "сущностей". Именно потому что они ничего кроме дать-взять не делают

Почему не просто массив массивов (говорят - коллекция, ларавель взял это слово чтобы именовать один из классов) - потому что в отличие от таблицы базы данных - этот массив нигде не описан, и рано поздно забудешь какие у него поля (поля элементов коллекции). А так он в файле лежит и там написано что и зачем и ты знаешь что в этот файл можно только дописать. А чтобы поменять - это нормальный пласт работы потому что на этот файл многие упираются

[Григорий Васильков]

NikSIk31, правило такое - чем меньше класс делает тем реже его будут переписывать, а значит его создал и забыл
а значит в идеале каждый класс выполняет одну задачу условную по проекту. понятно что даже одна задача может содержать в себе несколько задач попроще, но "дать-взять" годится под то чтобы быть одной задачей в данном случае. одну задачу с точки зрения того, как ты продумываешь проект. но пока код пишешь классы все равно будут выполнять много задач, так проще и быстрее написать. потом походу будешь делить. ведь когда класс начнет выполнять задачу "перекачать из А в Б информацию" захочется в него передать А и Б, а нужно будет передать в него "нечто умеющее получать" и "нечто умеющее отдавать" так как операции больше времени не изменятся, чем конкретные А и Б.

[NikSIk31]

Григорий Васильков, я не понимаю, с $this->getAttribute('created_at') проблема, что тупо белый экран остается. Хотя если написать $this->getAttribute('id') , то он сработает. (отличие только в том, что id нету в appends) . если добавить в appends то уже будет ошибка что

Call to undefined method App\Notification::getIdAttribute()

[Григорий Васильков]

1. хочешь получить свойство которого нет в базе get(свойство)Attribute, это же будет выполнено если модель новая и свойство еще ничем не заполнено. тут наблюдается некоторая странность мне кажется. надо пересмотреть. типа если поле пришло с базы - то get(свойство)Attribute помоему не выполнится, но я точно не помню
2. хочешь изнутри кода модели выполнить get(свойство)Attribute - getAttribute(свойство)
3. вне модели можно так $model->(свойство)
4. насчет appends - оно должно насколько я помню делать одно из трех выше описанных. но может нет. открывай исходник и пошагово смотри что за чем оно делает

Собственно код Ларавелевских моделей тем и ужасен, что есть в документации примеры как сделать простейшие вещи, а как только хочется чего-то особенного - начинаешь рыть код и дописывать его. И только через год когда все уляжется на свои места придет идея как сделать то же, что делал переписыванием - не переписывая. Когда в коде много неочевидных магических штук всегда приходится открывать исходный код. А значит вместо библиотеки получается монолитный кусок, который будешь учить, долбаться и мучаться.

Это и отличает фреймворки от библиотек. Библиотека умеет как правило меньше суперштук, весь код пишет разработчик и за него отвечает. А фреймворк творит какую-то магию под капотом, но как только что-то своё сделать - сопротивляется как только может, словно бы "нельзя". Ну и второй программист придет и тоже не поймет как так случилось, что он делает то, что не должен был.

В общем читай исходный код моделей ларавелины и постарайся побороть желание её дописать, мне чессговоря оочень хотелось. переписал. и потом пожалел.