Почему Resharper делает следующую рекомендацию?

Question

Мой код ...

public static void AssertNotNull<T>(string name, T val) {
    if (val == null)
        throw new ArgumentNullException(String.Format("{0} must not be null", name));
}

Решарпер рекомендует ...

public static void AssertNotNull<T>(string name, T val) {
    if (Equals(val, default(T)))
        throw new ArgumentNullException(String.Format("{0} must not be null", name));
}

Answer 1

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

Answer 2

Я второй ответ Берадо, но добавил бы, что вы можете предотвратить это, добавив следующее ограничение:

public static void AssertNotNull<T>(string name, T val) where T : class

Answer 3

Это, очевидно, не , что вы хотите в этом случае, но он просто пытается быть полезным, убедившись, что вы не вносите ошибку, забывая, что ссылочные типы могут использоваться для T. Как сказал @ Michael Meadows , вы, вероятно, хотите добавить ограничение class к T.

Answer 4

Эти два метода не эквивалентны. Первый разрешает AssertNotNull ("foo", 0), а второй бросает. Я думаю, что Решарпер слишком усердствует в этом деле.

Answer 5

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

public static void AssertNotNull<T>(string name, T val)
    where T : class
    {
        if (val == null)
            throw new ArgumentNullException(String.Format("{0} must not be null", name));
    }

Answer 6

Я полагаю, потому что T может быть не ссылочным типом.