Оглавление
V3077. Property setter / event accessor does not utilize its 'value' parameter.
Анализатор обнаружил потенциальную ошибку, связанную с тем, что в аксессорах свойств и событий не используется параметр 'value'.
Рассмотрим пример некорректного кода:
private bool _visible;
public bool IsVisible
{
get { return _visible; }
set { _visible = true; }
}Скорее всего, при задании нового значения свойству "IsVisible" планировали сохранить результат в переменную "_visible", но допустили ошибку, из-за чего изменение свойства никак не будет влиять на состояние объекта.
Корректный вариант кода:
public bool IsVisible
{
get { return _visible; }
set { _visible = value; }
}Анализатор может выдавать предупреждения на код следующего вида:
public bool Unsafe {
get { return (flags & Flags.Unsafe) != 0; }
set { flags |= Flags.Unsafe; }
}В этом случае метод 'set' используется для изменения состояния флага и в коде ошибки нет, но использование такого свойства может вводить в заблуждение разработчика, т.к. результат при присваиваниях "myobj.Unsafe = true" и "myobj.Unsafe = false" будет одинаковый.
Для сброса состояния внутренней переменной лучше воспользоваться функцией, а не свойством:
public bool Unsafe
{
get { return (flags & Flags.Unsafe) != 0; }
}
public void SetUnsafe()
{
flags |= Flags.Unsafe;
}Если от использования такого свойства нельзя отказаться, то можно пометить это место комментарием специального вида " //-V3077" и анализатор в дальнейшем не будет выдавать предупреждение для этого свойства:
public bool Unsafe {
get { return (flags & Flags.Unsafe) != 0; }
set { flags |= Flags.Unsafe; } //-V3077
}Все способы подавления ложных предупреждений описаны в документации.
Взгляните на примеры ошибок, обнаруженных с помощью диагностики V3077. |
Была ли полезна эта страница документации?
