search

freezemage0 / bitrix.snippets

0 stars 0 forks source link

feature/24: описана работа с Entity, добавлен README.md #25

Open freezemage0 opened 3 years ago

freezemage0 commented 3 years ago

Proof of Concept for #24.

freezemage0 commented 3 years ago

Я несколько не согласен с тем, что InvalidArgumentException здесь уместно. Во-первых, оно скорее предназначено для проверки типа, а не для проверки корректности его содержимого.

Документация PHP явно утверждает, что:

Exception thrown if an argument does not match with the expected value.

Это касается не только типизации, но и содержимого в принципе.

freezemage0 commented 3 years ago

@Juliaaan1 Хотелось бы ещё конкретно узнать твое мнение о самом сниппете, его практичности при выполнении конкретных задач.

Juliaaan1 commented 3 years ago

Это касается не только типизации, но и содержимого в принципе. Есть ощущение, что мои представления об исключениях расходятся с представлениями авторов языка. Я в любом случае считаю, что это должно быть непроверяемое исключение.

Оставляю на твое усмотрение, если считаешь, что это исключение здесь корректно - я не буду против.

Хотелось бы ещё конкретно узнать твое мнение о самом сниппете, его практичности при выполнении конкретных задач.

Считаю, что сниппет будет "хорошо практически применяться". Чем меньше статики - тем лучше, сам же знаешь. Мне нравится эта идеология в целом.