Правильно обработать переменную ссылочного типа на null внутри метода

@Serg34 · Регистрация: 20.09.2014

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

Добрый день!
вопрос вроде простой, но никак не пойму, как правильнее сделать.
Простая ситуация:

C#
1
2
3
4
public void DoSomething(object obj)
{
//
}

obj == null, недопустимо, это означает, что есть ошибка в алгоритме, поэтому if (obj != null) или if (obj == null) return; не подходят,
Но и вызывать исключение тоже считаю бессмысленным - NullReferenceException сам сработает и еще мое какое-либо исключение считаю лишним.
Игнорировать тоже неправильно - анализатор ругается.
Так что лучше написать внутри метода?

@Woldemar89 · 16.02.2016, 17:01

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

obj == null, недопустимо, то есть ошибка в алгоритме, поэтому if (obj != null) не подходит,

Расшифруйте.

@quantumbox · 16.02.2016, 17:02

Дай ей дефолтное значение.
И, все таки, не понятно, почему obj == null - недопустимо и if (obj != null) не подходит.

@valera_21 · 16.02.2016, 17:07

имхо: кидать все же ArgumentNullException(либо какое либо свое).
раз таки алгоритм, то такого быть не должно, и подразумевается, что если как-то обрабатывать - результат работы алгоритма будет все равно неправильным

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

@Serg34 · 16.02.2016, 19:07 **[ТС]**

Woldemar89,quantumbox, я имел ввиду, что если аргумент равен null, то это ошибка разработчика и обходить ее нельзя, надо об этом явно сообщить. Выполнять метод в этом случае нельзя.
valera_21, тоже об этом думал, но как-то некрасиво. Хотя если название аргумента передать в конструктор ArgumentNullException,

C#
1
if (obj == null) throw new ArgumentNullException("obj");

то конечно, ошибка становится более информативнее, нежели NullReferenceException.
Наверно, правила Microsoft это и имеют ввиду, когда ругаются. Спасибо

Добавлено через 1 минуту

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

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

В том то все и дело, что NullReferenceException тоже можно отловить

@Woldemar89 · 16.02.2016, 19:09

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

я имел ввиду, что если аргумент равен null, то это ошибка разработчика и обходить ее нельзя

Выбрасывайте ArgumentNullException.

@Serg34 · 16.02.2016, 20:13 **[ТС]**

Woldemar89, да, спасибо

@kolorotur · 16.02.2016, 20:30

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

если аргумент равен null, то это ошибка разработчика и обходить ее нельзя

На случай, если где-то выше по стеку идет отлов исключений, то на всякий случай добавьте еще Debug.Assert. В релиз-версии он работать не будет, но во время разработки/тестирования поможет быстро опознать проблему.

@Serg34 · 16.02.2016, 21:05 **[ТС]**

kolorotur, Спасибо, буду знать

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .

@quantumbox 8 / 8 / 3 Регистрация: 24.02.2015 Сообщений: 60
	16.02.2016, 17:02
	Дай ей дефолтное значение. И, все таки, не понятно, почему obj == null - недопустимо и if (obj != null) не подходит. 0

@valera_21 484 / 439 / 123 Регистрация: 05.01.2010 Сообщений: 1,848
	16.02.2016, 17:07
	имхо: кидать все же ArgumentNullException(либо какое либо свое). раз таки алгоритм, то такого быть не должно, и подразумевается, что если как-то обрабатывать - результат работы алгоритма будет все равно неправильным з.ы. можно потом на уровнях выше в слое работы с UI отловить, чтобы приложение не падало 1

@Serg34 100 / 100 / 33 Регистрация: 20.09.2014 Сообщений: 457 Записей в блоге: 3
	16.02.2016, 20:13 [ТС]
	Woldemar89, да, спасибо 0

@Serg34 100 / 100 / 33 Регистрация: 20.09.2014 Сообщений: 457 Записей в блоге: 3
	16.02.2016, 21:05 [ТС]
	kolorotur, Спасибо, буду знать 0