Примеры ошибок, обнаруженных с помощью диагностики V3031


V3031. An excessive check can be simplified. The operator '||' operator is surrounded by opposite expressions 'x' and '!x'.


SharpDevelop

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. DockablePane.cs 549


internal override ManagedContent RemoveContent(int index)
{
  ....
  if (((dockableContent == null)
      || (dockableContent != null &&
          dockableContent.SavedStateAndPosition == null)
      || (dockableContent != null &&
          dockableContent.SavedStateAndPosition.ContainerPane
          != this))
      && Items.Count == 0)
  ....
}

There is no error here, but the code can be simplified. "dockableContent != null" checks are not necessary here.


Xamarin.Forms

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. Xamarin.Forms.Platform.iOS.Classic ContextActionCell.cs 102


public override void LayoutSubviews()
{
  ....
  if (_scroller == null || (_scroller != null &&
                            _scroller.Frame == Bounds))
    return;
  ....
}

There is no error here, but the code can be simplified.


FlashDevelop

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. DockContentHandler.cs 540


internal void SetDockState(....)
{
  ....
  if ((Pane != oldPane) || (Pane == oldPane
    && oldDockState != oldPane.DockState))
  {
    RefreshDockPane(Pane);
  }
  ....
}

There is no error here, however, the conditions Pane != oldPane and Pane == oldPane are mutually exclusive, so this expression can be simplified


Mono

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. mcs-net_4_x ILGenerator.cs 456


public void Emit(OpCode opc)
{
  Debug.Assert(opc != OpCodes.Ret || (
               opc == OpCodes.Ret && stackHeight <= 1));
  ....
}

There is no error here, but the code can be simplified.


Mono

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. System.Windows.Forms-net_4_x ContainerControl.cs 506


public bool Validate (bool checkAutoValidate)
{
  if ((checkAutoValidate &&
      (AutoValidate != AutoValidate.Disable)) ||
      !checkAutoValidate)
    return Validate ();

  return true;
}

There is no error here, but the code can be simplified.


Orchard CMS

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions '!ssl' and 'ssl && CdnSupportsSsl'. ResourceDefinition.cs 190


public string ResolveUrl(....) {
  ....
  if (!ssl || (ssl && CdnSupportsSsl)) {
    ....
  }
}

Telerik UI for UWP

V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. SelectedItemCollection.cs 77


private bool CanInsertItem(object item)
{
  return    this.suspendLevel == 0
         && this.AllowSelect
         && ((!this.AllowMultipleSelect && this.Count == 0)
             || this.AllowMultipleSelect);
}

There is no error here, but the code can be simplified.

Similar errors can be found in some other places:

  • V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. SelectedItemCollection.cs 93
  • V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions. StackVirtualizationStrategy.cs 49
  • V3031 An excessive check can be simplified. The '||' operator is surrounded by opposite expressions 'state == null' and 'state != null'. LocalFieldDescriptionsProviderBase.cs 24


Найденные ошибки

Проверено проектов
367
Собрано ошибок
13 552

А ты совершаешь ошибки в коде?

Проверь с помощью
PVS-Studio

Статический анализ
кода для C, C++, C#
и Java

goto PVS-Studio;