Проверка нулевых параметров в C#

Я согласен с Джоном, но я бы добавил к этому одну вещь.

мое отношение к тому, когда добавлять явные нулевые проверки, основано на этих предпосылках:

• должен быть способ для ваших модульных тестов выполнять каждый оператор в программе.
• throw отчетность заявления.
• следствием if это сообщении.
• таким образом, должен быть способ осуществлять throw на if (x == null) throw whatever;

если есть невозможно для выполнения этого оператора он не может быть протестирован и должен быть заменен на Debug.Assert(x != null);.

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

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

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

без явного if проверьте, это может быть очень трудно выяснить,что был null Если вы не владеете кодом.

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

эти if проверки не сделают ваш код заметно медленнее.

---

обратите внимание, что параметр к ArgumentNullException конструктор - это имя параметра, а не сообщение.
Ваш код должен быть

if (s == null) throw new ArgumentNullException("s");

Я написал фрагмент кода, чтобы сделать это проще:

<?xml version="1.0" encoding="utf-8" ?>
<CodeSnippets  xmlns="http://schemas.microsoft.com/VisualStudio/2005/CodeSnippet">
    <CodeSnippet Format="1.0.0">
        <Header>
            <Title>Check for null arguments</Title>
            <Shortcut>tna</Shortcut>
            <Description>Code snippet for throw new ArgumentNullException</Description>
            <Author>SLaks</Author>
            <SnippetTypes>
                <SnippetType>Expansion</SnippetType>
                <SnippetType>SurroundsWith</SnippetType>
            </SnippetTypes>
        </Header>
        <Snippet>
            <Declarations>
                <Literal>
                    <ID>Parameter</ID>
                    <ToolTip>Paremeter to check for null</ToolTip>
                    <Default>value</Default>
                </Literal>
            </Declarations>
            <Code Language="csharp"><![CDATA[if ($Parameter$ == null) throw new ArgumentNullException("$Parameter$");
        $end$]]>
            </Code>
        </Snippet>
    </CodeSnippet>
</CodeSnippets>

возможно, вы захотите взглянуть на Контракты Кода Если вам нужен более приятный способ, чтобы убедиться, что вы не получаете никаких нулевых объектов в качестве параметра.

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

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

это сохраняет некоторую отладку, когда вы нажмете это исключение.

ArgumentNullException явно указывает, что это был" s", который был null.

Если у вас нет этой проверки и пусть код blow op, вы получаете исключение NullReferenceException из какой-то неопознанной строки в этом методе. В сборке выпуска вы не получаете номера строк!

Я использую это уже год:

_ = s ?? throw new ArgumentNullException(nameof(s));

это oneliner, и отбросить (_) означает, что нет ненужного выделения.

int i = Age ?? 0;

вот пример:

if (age == null || age == 0)

или:

if (age.GetValueOrDefault(0) == 0)

или:

if ((age ?? 0) == 0)

или тройной:

int i = age.HasValue ? age.Value : 0;

Исходный Код:

void f(SomeType s)
{
  if (s == null)
  {
    throw new ArgumentNullException("s cannot be null.");
  }

  // Use s
}

переписать его в виде:

void f(SomeType s)
{
  if (s == null) throw new ArgumentNullException(nameof(s));
}