Как игнорировать или пропускать варнинги при вставке значений из одного массива в другой?

Question

[Little_Junior]

Я получаю API ответ объектом из file_get_contents('php://input')
И чтобы было удобно с ним работать, переношу данные в мною созданный массив:

$array = [
  'id' => $data->{'id'},
  'name' => $data->{'name'},
  ...
];

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

Answer 1

[humoured]

Нужно проверять каждое значение перед использованием.
Смотрите в сторону конструкций empty($var), isset($arr['index']), $myVar = $input[123] ?? null и прочих

Comments

[Little_Junior]

Понял, спасибо, так и думал. Еще интересно, если массив довольно объемный, то такая проверка влияет на производительность сильно или можно глаза на это закрыть?

[humoured]

Лучше сосредоточиться на логике приложения. Такая проверка даст гораздо больше пользы, а её влияние на производительность менее чем ничтожна.

[Сергей delphinpro]

Little_Junior,

Еще интересно, если массив довольно объемный, то такая проверка влияет на производительность сильно или можно глаза на это закрыть?

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

Хорошо, конечно писать сразу оптимизированный код, но это не всем дано.

Answer 2

[rPman]

если тебе не надо это контролировать, то игнорировать
@$data->{'id'}

p.s. вторым параметром json_decode пропиши true, тогда результатом будет не object а array в нужном тебе виде, да и работать можно будет с ним $data['id']

Comments

[Little_Junior]

Ну просто, в конечном итоге я делаю unset($data) и работаю уже со своим массивом, где ключи его подстроены правильно под меня. А уже из него делаю проверку на то, существует ли значение определенного элемента или нет.

[Little_Junior]

Насчет json_decode($data, true) что-то не так. Там объект объемный $data->data->...->...->...
Такое в массив не получается засунуть, он не может его вывести $data[data][...][...][...]

[rPman]

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

[humoured]

Little_Junior, игнорирование ошибок через @ — плохая практика. В PHP раньше было это актуально, но сегодня так делать не стоит: язык развивается, способов проверки данных и обработки исключений достаточно.
Ошибки нужно обрабатывать. Тем более, когда речь идёт про работу с пользовательскими данными.

[rPman]

конечно нужно понимать что делаешь и зачем, думать надо до.

но и делать лишнюю работу тоже не надо!

[Сергей delphinpro]

собачку рекомендовать - это полный пц...

чем это не подходит-то?

$data->id ?? null

[rPman]

короче
твой вариант подойдет если надо не null выдавать в качестве дефолта, а так это то же самое

[Alexey Marchenko]

Подавлять ошибки через @ не надо! Это очень плохо!

[rPman]

неправильная формулировка!

если в моей задаче НЕ ВАЖНО, установлено ли значение или нет, я могу спокойно запрограммировать логику игнорирования его отсутствия, и сделать это могу кучей способов, и if isset поставить, и относительно новый ?? оператор использовать и в т.ч. могу использовать @, если его поведение мне подходит!

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

Answer 3

[Сергей Соколов]

Начиная с PHP v.7.0 можно использовать Null Coalescing operator ??

$field1 = 'Habr';
$field2 = 'QnA';
$array = [
  'id' => $data->xxx->yyy->id ?? null,
  'name' => $data->{'name'} ?? 'untitled',
  'site' => $data->$field1->$field2 ?? 'поиск';
];

если запрошенного поля нет, ошибку не бросает, а берет значение по умолчанию, которое после ??

Comments

[Сергей delphinpro]

Чем
$data->{'name'}
отличается от
$data->name
?

[rPman]

наверное тем что если вместо name будет что то типа $-@ то она сработает, а в ассоциативный массив почему то не хочет переводить