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

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

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

10 ответов

33 просмотра

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 - он удалит все ключи, которые там не указаны

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

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

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта