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

Question

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

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

#laravel #programming #russian

0

06.08.2023

10 ответов

2 просмотра

PSYTRGLES

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

0

9 месяцев назад

PSYTRGLES

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

0

9 месяцев назад

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

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

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

0

9 месяцев назад

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

PSYTRGLES
if (!empty($postData['tags'])) { // attach } ...

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

0

9 месяцев назад

Nikita

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

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

0

9 месяцев назад

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

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

0

9 месяцев назад

Nikita

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

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

0

9 месяцев назад

PSYTRGLES

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

так этот if после create()

0

9 месяцев назад

PSYTRGLES

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

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

0

9 месяцев назад

PSYTRGLES · Accepted Answer

PSYTRGLES

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

0

9 месяцев назад

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

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

10 ответов

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