Какие подводные камни есть в том, чтобы просто подавить вывод «Trying to get property 'someProp' of non-object» через error_reporting?

Question

[Роми]

Легаси с кучей

PHP Notice - "Trying to get property 'someProp' of non-object"

по идее, если перенастроить error_reporting - все работает.

Но как-то мне такое решение не очень нравится :-)

Какие здесь есть подводные камни, и вообще best practice для подобных ситуаций?

Ставить везде условия - так это задолбаешься проверять объект там или NULL в реале - там этих ошибок может 100500 штук в коде, я пока не знаю.

Есть ли какая-то серебряная пуля для данной ситуации кроме error_reporting?

Answer 1

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

Подводные камни "отсутствует parent, нельзя сохранить child, полскрипта не сработало, половина сохранилась, теперь надо руками вычищать что корявое, а что нет"

=====

В новой пхп именно из этой проблемы (а точнее стонах по поводу неё) ввели оператор "?->" который вернет NULL если нету, но это про "сразу писать правильно" - а так тоже не бывает. Чаще всего надо "разобраться почему не работает", а не весь старый код перелопатить.

До этого ещё втиралось что NullObject Pattern нужно использовать - то есть любой объект может иметь __get()/__set() и если поля нет - вернет Null, или например если в свойстве должен быть обьект - то когда там NULL туда закидывается обьект с пустыми свойствами, чтоб все работало так же просто всегда NULL отдавало. Кое-кто взял на вооружение - например NullCache() который работает как кеш, но ничего не делает, хотя возвращает всё так же, как будто работает.

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

К примеру обьект Car может не иметь поля "fifth-wheel".
Делаем так (new CarDecorator(new Car()))->fifth_wheel где уже есть поле или она умеет возвращать NULL если поля нет.

Comments

[Роми]

Подводные камни "отсутствует parent, нельзя сохранить child, полскрипта не сработало, половина сохранилась, теперь надо руками вычищать что корявое, а что нет"

Ну, такое хеджируется)) транзакциями БД, допустим -)

[Роми]

...в целом, спасибо за развернутый ответ, но в данном конкретном случае соль в том, что если подавить ошибки - в целом все работает ))

вопрос в другом:

- Насколько критично выловить и пофиксить эти «Trying to get property 'someProp' of non-object» - или работает - и так сойдет пока что? Это же просто нотисы.

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

То что оно показывает говорит о том, что работает не так как должно, транзакция ролбэчится чаще чем нужно, что-то не долетает до базы.

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

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

А в их жизни ничего не изменится. Деньги то у них, это пролетарий ест из мусорок. А система она себя клонирует в малых объемах в любой отрасли жизни. И подобно тому как чиновник отсылает тебя нахер, так и бизнесмен тоже отсылает тебя туда же. А когда что-то не так - избавляется от тебя. Особенно тогда, когда ты признаешь эту проблему и не дай боже признаешься что не знаешь как её исправить.

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

Брага рекрутеров которая присосалась к нашим деньгам все равно это не оценит, они оценят только деньги, которые им платят.

Шо делать? А профсоюз. Но как если на вопрос "вуе не подходит" они отвечают "бред полный". То есть они готовы хвалить свою компанию потому, что их жизнь хороша и им хорошо платят. Не будет профсоюза. И улучшения ситуации тоже. А уволят сначала тех, кто делает по моральному кодексу и говорит правду. А другие переждут и продолжат уничтожать всё к чему прикасаются. Жизнь давно (начали в 63м Косыгин, потом в 91м переломный, 24 февраля - добивающий) сделали игрой, в которой нужно уничтожать, а не строить, ведь прибыли получаются из экспорта, а не из уровня жизни. Если продали - ну и заебис.

Этим долбанным странам не хватает соревнования "в какой стране меньше бедных", чтобы это вело к финансированию или допуску на увеличение вооруженных сил. Иначе а зачем вообще мы? Не, нужны те, кто (а точнее "то", что) работает, а значит - наполняет карман, а "мы" - это мусор.

[Роми]

Григорий Васильков, эпохально так))

Answer 2

[Максим Гришин]

Best practice - валидировать входящие данные. И да, если что-то хотя бы теоретически возвращает null или ещё какую-то чушь, всегда проверяете, вернулась ли чушь, иначе рано или поздно огребете серьезный баг, и хорошо ещё, если последствия будут всего лишь падением пхп. Серебряной пули, естественно, нет, но есть статические анализаторы, которые могут помочь найти места в коде, где валидации на чушь не хватает.