C#: Znajdowanie błędów: if (false)

Question

Właśnie znalazłeś błąd w moim programie (po pewnej ilości debugowania i łzawienie moje włosy)

bool first = true; 

foreach (RdAbstractNode node in listNodes) 
{ 
    if (!first) 
    { 
     // do stuff (does not change first) 
    } 
    // do more stuff (does not change first) 
} 

Jak widać first jest zawsze true - nigdy zmienione. Więc if(!first) jest w zasadzie if(false).
Kompilator nie wygenerował ostrzeżenia, mimo że jest ustawiony na poziom 4 (najwyższy poziom).

Jak mogę znaleźć podobne błędy if(false)?

używam VS 2010, .Net 4.0 kompilator, projekt ustawienie .NET 2.0

Answer 1

Z Resharper dostaniesz ostrzeżenia jak ten

Answer 2

nie wiem z funkcją kompilatora, które pomogą Ci - trzeba testów jednostkowych.

Answer 3

Kompilator będzie ostrzegać oczywistych rzeczy jak nieosiągalnego kodu w oparciu o stałą wartość w instrukcji if - przykład "if (false)". Jednakże, ponieważ zmienna może zostać zmieniona, nie zrobi tego w tym przypadku niestety.

Answer 4

Zgadzam się, że Resharper jest dobrym narzędziem do posiadania, ale chciałbym dodać, że myślę, że to demonstruje powód, aby NIE używać tej konstrukcji.

Alternatywą może być:

EDIT:

//.net 4.0 
    var firstNode = listNodes.First(); 
     //do something special for first. 

    foreach(var node in listNodes.Skip(1)) 
    { 
     //do something for the rest. 
    } 

//.net 2.0 
using (IEnumerator<RdAbstractNode> enumerator = listNodes.GetEnumerator();) 
{ 
    if (!enumerator.MoveNext()) 
     return; 

    RdAbstractNode first = enumerator.Current; 
    //do something with first 
    while(enumerator.MoveNext()) 
    { 
     RdAbstractNode currentNode = enumerator.Current; 
     //do something with the node. 
    } 
} 

Często znajdujemy się potrzeby mieć jakąś specjalną obsługę dla pierwszego elementu, czasami lepiej jest zmienić logikę i uczynić cały jednolity procesor. Jeśli nie możemy, zawsze wolę wyraźne oddzielenie obsługi specjalnych przypadków od reszty elementów, zamiast zagnieżdżać je.