Podkreśl negację

13

Właśnie pisałem oświadczenie if z dość długimi nazwami nieruchomości i napotkałem ten problem.

Powiedzmy, że mamy takie wyrażenie if:

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

Druga właściwość jest typu boolowskiego i nie ma sensu mieć podobnego do stetement

if(boleanValue == true)

Czy istnieje lepszy sposób na podkreślenie negacji niż postawienie jej !na pierwszym planie . Wydaje mi się, że można to łatwo nadzorować podczas czytania kodu i może potencjalnie powodować problemy z debugowaniem

readability

— Ivan Crojach Karačić
źródło

7

Wyodrębnij warunek do metody o znaczącej nazwie.

— Joachim Sauer

2

... lub przypisz zanegowaną wartość do zmiennej o znaczącej nazwie i użyj jej w warunku if.

— scrwtp

+1 to, co powiedział @JoachimSauer, jeśli to możliwe, umieść tę metodę na obiekcie, do którego jest wysyłane zapytanie. W takim przypadku cały warunek może być enkapsulowany za pomocą metody_someViewModelNameThatIsLong

— MattDavey

2

Ogólnie rzeczą, którą często robię, jest otoczenie negacji spacją po każdej stronie, aby była bardziej widoczna. if( ! something)vsif(!something)

— Svish

Jeśli podkreślasz negację, dlaczego nie użyć ... && model.Prop == false)? Osobiście bardzo rzadko używam !, zbyt łatwo to przeoczyć.

22

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

A następnie w obiekcie modelu widoku

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

(zakładając, że someValue jest łańcuchem i nie jest znany obiektowi modelu)

To nie tylko podkreśla! operatora, ale ogólnie czyni go bardziej czytelnym. Teraz w metodzie wywoływania widzę jeden warunek, który powinien mieć dobrą nazwę, aby opisać warunek w kontekście obiektu wywołującego. A w obiekcie modelowym widzę, co to oznacza w kontekście obiektu modelowego.

— pdr
źródło

1

+1, jest to dobre zastosowanie zasady eksperta informacji .

— MattDavey,

1

Warto zauważyć, że someValuemoże być konieczny parametr NeedsMeToDoSomethingmetody, w zależności od twojej sytuacji.

— MattDavey,

@MattDavey: Oh, dobra uwaga.

— pdr

@pdr Może powinieneś zrobić właściwość dla tego modelu widoku z tym samym logicznym i ponownie użyć tej flagi w większej liczbie miejsc. Jestem pewien, że ci się przyda.

— radu florescu

5

Włóż go do własnego bloku if przed oceną mniej ważnych warunków. Nie tylko łatwiej byłoby czytać bez bałaganu w innych warunkach, ale jest to również pierwszy warunek, który programista odczyta. Połącz to z pomysłem wspomnianym już przez @scrwtp, aby przypisać do zmiennej o znaczącej nazwie, a otrzymasz:

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

Jeśli programujesz w języku kompilatora, w większości przypadków są one zagnieżdżane, jeśli bloki łączą się na końcu, o ile nie wstawisz kodu między zewnętrznym if i wewnętrznym if, więc nie powinno to wpływać na wydajność w tych skrzynie

— Neil
źródło

2

Jeśli używasz C / C ++, preprocesor może zapewnić czytelność.

#define NOT !

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

— CWallach
źródło

a może ... funkcja not (x) {return! x; } // javascript

— Alex

2

Operatory w C i C ++ Istnieją już standardowe synonimy dla różnych operatorów.

— Joel

0

Po prostu wyodrębnię

`!_someViewModelNameThatIsLong.ThisIsABooleanPropertyThatIsImportant`

W metodzie, która zwraca to. Jeśli nazwiesz tę metodę NotThisIsABooleanPropertyThatIsImportant, wszystko powinno być w porządku.

— mhr
źródło