# V3115\. It is not recommended to throw exceptions from 'Equals\(object obj\)' method\.

Анализатор обнаружил, что в переопределенном методе 'Equals\(object obj\)' возможно возникновение исключения\.

Рассмотрим пример\. Пусть у нас имеется метод следующего вида:

```cpp
public override bool Equals(object obj)
{
  return obj.GetType() == this.GetType();
}
```

В том случае, если аргумент 'obj' будет нулевым \(null\) \- это повлечет за собой исключение 'NullReferenceException'\. Вероятнее всего программист не предусмотрел подобную ситуацию при реализации данного метода\. Для корректной работы можно использовать проверку на 'null':

```cpp
public override bool Equals(object obj)
{
  if (obj == null)
    return false;

  return obj.GetType() == this.GetType();
}
```

Другой пример плохой практики при реализации метода 'Equals\(object obj\)' \- явное выбрасывание исключения из этого метода\. Рассмотрим следующий пример:

```cpp
public override bool Equals(object obj)
{
    if (obj == null)
        throw new InvalidOperationException("Invalid argument.");

    return obj == this;
}
```

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

Если один из объектов не соответствует условиям сравнения, лучше будет вернуть 'false':

```cpp
public override bool Equals(object obj)
{
    if (obj == null)
        return false;

    return obj == this;
}
```

![V3115_ru/image1.png](https://import.viva64.com/docx/blog/V3115_ru/image1.png)

Вы нашли спрятанный ссееккрреетт и, возможно, успеете забрать один из ппррииззоовв\. [Подробнее](https://pvs-studio.ru/ru/rules/)\.