Нужно ли заморачиваться проверками, если рефлексия сама бросает нужные исключения?

@DefenseMinister · Регистрация: 17.02.2021

Студворк — интернет-сервис помощи студентам

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
if (!array_key_exists($id, $this->definitions)) {
 
    if (class_exists($id)) {
 
        $reflection = new ReflectionClass($id);
 
        $args = [];
 
        if (($constructor = $reflection->getConstructor())) {
 
            foreach ($constructor->getParameters() as $parameter) {
 
//                if (!$parameter->hasType()) {
//                    throw new Error("Не указан тип параметра $parameter->name");
//                }
 
                if ($parameter->getType()->isBuiltin()) {
//                    if (!$parameter->isDefaultValueAvailable()) {
//                        throw new Error("Не указано значение по умолчанию параметра $parameter->name");
//                    }
 
                    $args[] = $parameter->getDefaultValue();
                } else {
                    $args[] = $this->get($parameter->getType()->getName());
                }
            }
        }
 
        return $this->results[$id] = $reflection->newInstanceArgs($args);
    }
 
    throw new Error("Параметр $id не найден");
}

Я закомментировал проверки: дело в том что при этих ошибках выбрасываются исключения ReflectionException, мне ловить свои проверки или ловить ReflectionException?

То есть не лишние ли эти проверки?

@ke1evra · 15.09.2021, 16:51

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

@Fart83 · 22.09.2021, 17:31

Сообщение от ke1evra

проверки всегда лишние, т.к. это избыточное выполнение кода

кто вам такой тезис подсказал? в каком из принципов "хорошего кода"?

Сообщение от ke1evra

но без них скрипт может сломаться наглухо.

и с ними может "наглухо" сломаться, да так, что не поймешь что к чему.

имхо, с такими убеждениями легко скатиться в БК.

@sad67man · 23.09.2021, 02:29

DefenseMinister, Если вы не видите в них смысла и вам достаточно ReflectionException, то в данном случае их можно и опустить. Ведь исключения тем и удобны, что освобождают нас от лишних проверок. Хотя лишних проверок не бывает)

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

Добавлено через 5 минут
Поэтому если уже выкидываете свою ошибку, то обязательно указывайте класс, который не удалось создать.

DefinitionException 'Entry \app\Users\UserServices cannot be resolved: Parameter $settings has no value defined'

@DefenseMinister · 23.09.2021, 04:58 **[ТС]**

Сообщение от sad67man

кто-то увидит некий ReflectionException, то может не сразу догадаться что не так

согласен

Новые блоги и статьи Все статьи Все блоги /
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом

@ke1evra 51 / 38 / 13 Регистрация: 20.01.2019 Сообщений: 115
	15.09.2021, 16:51
	проверки всегда лишние, т.к. это избыточное выполнение кода, но без них скрипт может сломаться наглухо. поэтому думай сам - если это для личного пользования и ты всегда можешь отдебажить, то какая разница откуда ексепшн прилетит? а вот если это либа "для людей" и хочется по красоте сделать, чтоб другому кодеру было легче разобраться, то нужны. 1

@sad67man 2603 / 1507 / 689 Регистрация: 23.08.2015 Сообщений: 3,823
	23.09.2021, 02:29
	Сообщение было отмечено DefenseMinister как решение Решение DefenseMinister, Если вы не видите в них смысла и вам достаточно ReflectionException, то в данном случае их можно и опустить. Ведь исключения тем и удобны, что освобождают нас от лишних проверок. Хотя лишних проверок не бывает) Другое дело, что рефлексия может где-то еще использоваться, а вам бы хотелось указать, что исключение идет именно из контейнера зависимостей. Код пишется для других программистов в том числе. Если кто-то увидит некий ReflectionException, то может не сразу догадаться что не так и как это исправить. Добавлено через 5 минут Поэтому если уже выкидываете свою ошибку, то обязательно указывайте класс, который не удалось создать. DefinitionException 'Entry \app\Users\UserServices cannot be resolved: Parameter $settings has no value defined' 1

Нужно ли заморачиваться проверками, если рефлексия сама бросает нужные исключения?

Решение