169 похожих чатов

Добрый вечер уважаемый чат! Подскажите пожалуйста корректно ли делать подобные

проверки таким образом (если данные с поля не пришли то ничего не делай)? Если да то где грамотно подобный код размещать? Спасибо!

10 ответов

12 просмотров

if (!empty($postData['tags'])) { // attach } будет достаточно, не нужны там доп переменные, и unset-ы. тем более ты проверил только наличие ключа в массиве, а не значение.

далее, обрати внимание что тебе приходит в сервис: $postData - ты уверен что это массив? у тебя нет типизации, это может быть строка или что угодно. далее. если ты пропишешь array, то будешь ли ты уверен в ключах, которые содердит массив, ты напрямую передаешь в create() (спасибо fillable что спасает немного, если указаны правильно там поля) лучше в такие сервисы передавать не массив, а DTO, или массив придется валидировать в сервисе.

на 19ой строке по хорошему необходимо передавать не какой-то абстрактный массив данных, а указать какие поля ты пишешь. Да, это может показаться лишней работой, но в таком случае будет хотя бы видно что и куда ты пишешь, а не просто какой-то $postData и ищи где он формируется, скорей всего ты так до формы дойдешь, где еще будет if-ы и в итоге без дебага не разберешься.

Pavel- Автор вопроса
PSYTRGLES
на 19ой строке по хорошему необходимо передавать н...

Мне пока сложновато но я разберусь над всем что написали. Спасибо!

Pavel- Автор вопроса
PSYTRGLES
if (!empty($postData['tags'])) { // attach } ...

А кстати я не могу туда сразу атач впихнуть потому что пивот требует айди поста которого на тот момент нету.

Pavel
А кстати я не могу туда сразу атач впихнуть потому...

Сразу сохраняешь модель, потом атачишь

Pavel- Автор вопроса

Тогда в таблицу с постами пойдут данные для пивота и она не пропустит. Или я вас не понял.

Pavel
Тогда в таблицу с постами пойдут данные для пивота...

А чего в таблицу пойдут данные от пивота, для пивота отдельная же таблица

Pavel
Тогда в таблицу с постами пойдут данные для пивота...

не должно. у модели есть fillable - он удалит все ключи, которые там не указаны

Похожие вопросы

Обсуждают сегодня

Ребят в СИ можно реализовать ООП?
Николай
33
~ 2m21s  nix shell github:nixos/nixpkgs#stack ~  stack ghc -- --version error: … while calling the 'derivationStrict' builtin at /builtin/derivation.nix:...
Rebuild your mind.
6
Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
Всем доброго вечера. Разрабатываю 32 раз. приложение в Delphi. Столкнулся с тем, что стандартный  TFilestream  не работает с большим файлом > 2 ГБайт (после вызова функции see...
Vadim Gl
16
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
добрый день. Подскажите, есть сайт на 1.4.7 и я хочу обновиться, особо ничего не меняя. мне выбирать версию 1.4.35 или третью ветку? и можно ли обновлять "как есть", или нужно...
Digital Cat
12
Кто кодит под Лазарем на винде, у вас аналогично VCL переопределяются CreateWnd и CreateParams для конкретных классов контролов и все заданные флаги влияют?
А Андрей
11
У меня задача: написать брокер сообщений. Очереди и потребители. Очереди поддерживают приоритеты. Очередь отдает сообщения, только обработчикам с соответствующими характеристи...
Aleksandr Filippov
2
народ, плиз хелп, всю голову сломал себе уже... разве может быть так, что GetProcAddress( GetModuleHandle( "kernel32.dll" ), "SetThreadDescription" ) вернёт ненулевое значение...
Iluha Companets
12
А, ты про текущую реализацию? Нет конечно, я бы сделал правильно - сейчас там гавнокод
Александр (Rouse_) Багель
6
Карта сайта