Подчеркните отрицание

13

Я просто писал оператор if с довольно длинными именами свойств и столкнулся с этой проблемой.

Допустим, у нас есть выражение if, подобное этому:

if(_someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue &&
   !_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant)
{
    //Do something
}

Второе свойство имеет логический тип, и нет смысла иметь такой тип

if(boleanValue == true)

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

readability

— Иван Крояч Карачич
источник

7

Извлеките условие в метод со значимым именем.

— Иоахим Зауэр

2

... или присвойте отрицательное значение переменной со значимым именем и используйте ее вместо условия if.

— scrwtp

+1 то, что сказал @JoachimSauer, если возможно, поместите этот метод на объект, который запрашивается .. В этом случае все условие может быть инкапсулировано с помощью метода on_someViewModelNameThatIsLong

— MattDavey

2

Обычно я часто окружаю отрицание пробелом с каждой стороны, чтобы сделать его более заметным. if( ! something)vsif(!something)

— Свиш

Если вы подчеркиваете отрицание, почему бы не использовать ... && model.Prop == false)? Лично я очень редко пользуюсь !, это слишком легко не заметить.

22

if(_someViewModelNameThatIsLong.NeedsMeToDoSomething(someValue))
{
    //Do something
}

И тогда в представлении модель объекта

public bool NeedsMeToDoSomething(string someValue)
{
    return AnotherPropertyINeedToCheck == someValue &&
        !ThisIsABooleanPropertyThatIsImportant;
}

(предполагается, что someValue является строкой и не известен объекту модели)

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

— прецизионный самописец
источник

1

+1, это хорошее применение принципа информационного эксперта .

— MattDavey

1

Стоит отметить, someValueчто в NeedsMeToDoSomethingзависимости от вашей ситуации может потребоваться указать параметр метода.

— MattDavey

@MattDavey: О, хорошая мысль.

— фунтовые

@pdr Возможно, вам следует создать свойство для этой модели представления с тем же логическим значением и повторно использовать этот флаг в других местах. Я уверен, что вы найдете это удобно.

— Раду Флореску

5

Поместите его в свой блок if перед оценкой менее важных условий. Мало того, что было бы легче читать без беспорядка других условий, но также и первое условие, которое будет читать программист. Объедините это с идеей, уже упомянутой @scrwtp, назначить переменную со значимым именем, и вы получите:

var isValid = !_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant;
if( isValid ) 
{
    if( _someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue ) 
    {
        //Do something
    }
}

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

— Нил
источник

2

Если вы используете C / C ++, то препроцессор может обеспечить читабельность.

#define NOT !

if(_someViewModelNameThatIsLong.AnotherPropertyINeedToCheck == someValue &&
    NOT _someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant)
{
    //Do something
}

— CWallach
источник

или, возможно ... функция не (х) {возврат! х; } // javascript

— Алекс

2

Операторы в C и C ++ Уже существуют стандартные синонимы для различных операторов.

— Джоэл

0

Я бы просто извлечь

`!_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant`

В методе, который возвращает это. Если вы назовете этот метод NotThisIsABooleanPropertyThatIsImportant, с вами все будет в порядке.

— MHR
источник