Telerik UI for UWP

BUG OF THE MONTH | An unconditional break

V3020 An unconditional ‘break’ within a loop. NodePool.cs 189

public IEnumerable<KeyValuePair<int, List<T>>>
                GetUnfrozenDisplayedElements()
{
  foreach (var item in this.generatedContainers)
  {
    foreach (var pair in item.Value)
    {
      if (!pair.IsFrozen)
      {
        yield return item;
      }
      break;
    }
  }
}

The break statement is not part of the if statement. It will execute no matter what value is stored in pair.IsFrozen, so the foreach loop will iterate only once.

Please click here to see more bugs from this project.