Есть ли недоступный код в этом фрагменте? Я так не думаю, но Решарпер говорит мне иначе

Question

У меня есть следующий метод, с которым я столкнулся в обзоре кода. Внутри петли Решарпер говорит мне, что if (narrativefound == false) неверно, потому что narrativeFound всегда верно. Я не думаю, что это так, потому что для того, чтобы установить narrativeFound в true, он должен сначала пройти сравнение условной строки, так как же это может быть всегда? Я что-то пропустил? Это ошибка в Resharper или в нашем коде?

public Chassis GetChassisForElcomp(SPPA.Domain.ChassisData.Chassis asMaintained, SPPA.Domain.ChassisData.Chassis newChassis)
{
    Chassis c = asMaintained;
    List<Narrative> newNarrativeList = new List<Narrative>();

    foreach (Narrative newNarrative in newChassis.Narratives)
    {
         bool narrativefound = false; 

         foreach (Narrative orig in asMaintained.Narratives)
         {
                if (string.Compare(orig.PCode, newNarrative.PCode) ==0 )
                {
                          narrativefound = true;
                          if (newNarrative.NarrativeValue.Trim().Length != 0)
                          {
                             orig.NarrativeValue = newNarrative.NarrativeValue;
                             newNarrativeList.Add(orig);                            
                          }
                          break;
                }
                if (narrativefound == false)
                {
                     newNarrativeList.Add(newNarrative); 
                }
         }
    }

    c.SalesCodes = newChassis.SalesCodes;
    c.Narratives = newNarrativeList;
    return c; 
}

Answer 1

Переменная narrativefound никогда не будет истинной, когда элемент управления достигнет этого оператора:

narrativefound = true;
// ...
break;  // This causes control to break out of the loop.

Я думаю, что Решарпер пытается сказать вам, что условие narrativefound == false всегда будет верным.

Answer 2

Это ошибка в вашем коде.

foreach (Narrative newNarrative in newChassis.Narratives) 
{ 
     bool narrativefound = false;  

     foreach (Narrative orig in asMaintained.Narratives) 
     { 
            if (string.Compare(orig.PCode, newNarrative.PCode) ==0 ) 
            { 
                      narrativefound = true; 
                      if (newNarrative.NarrativeValue.Trim().Length != 0) 
                      { 
                         orig.NarrativeValue = newNarrative.NarrativeValue; 
                         newNarrativeList.Add(orig);                             
                      } 
// narrativeFound == true, but now we exit the for loop
                      break; 
            } 
// narrativeFound is always false here.  The test is redundant
            if (narrativefound == false) 
            { 
                 newNarrativeList.Add(newNarrative);  
            } 
     } 
} 

Answer 3

Вам не нужна переменная narrativeFound. В области, в которой вы устанавливаете значение true, вы выходите из цикла foreach. Если вы не установите его в true, вы не сломаете, и вы добавите newNarrative к newNarrativeList.

Итак, это можно переписать как

foreach (Narrative newNarrative in newChassis.Narratives)
{
     foreach (Narrative orig in asMaintained.Narratives)
     {
            if (string.Compare(orig.PCode, newNarrative.PCode) == 0)
            {
                      if (newNarrative.NarrativeValue.Trim().Length != 0)
                      {
                         orig.NarrativeValue = newNarrative.NarrativeValue;
                         newNarrativeList.Add(orig);                            
                      }
                      break;
            }

            newNarrativeList.Add(newNarrative);                 
     }
}

Answer 4

R # является правильным, потому что если вы повествуете, что повествование найдено истиной, вы выходите из foreach сразу после того, как вы его установили.

Answer 5

Я полагаю, это говорит вам о том, что если для параметра narriativefound установлено значение true, то цикл for завершается (break;). Таким образом, если вычисляется if (narriativefound == false), оно всегда будет иметь значение false.