# V517. Potential logical error. The 'if (A) {...} else if (A) {...}' pattern was detected.

Анализатор обнаружил потенциально возможную ошибку в конструкции, состоящей из условных операторов.

Рассмотрим пример:

    if (a == 1)
      Foo1();
    else if (a == 2)
      Foo2();
    else if (a == 1)
      Foo3();

В данном примере функции 'Foo3()' никогда не получит управления. Вероятно, мы имеем дело с логической ошибкой и корректный код должен выглядеть так:

    if (a == 1)
      Foo1();
    else if (a == 2)
      Foo2();
    else if (a == 3)
      Foo3()

На практике подобная ошибка может выглядеть следующим образом:

    if (radius < THRESH * 5)
      *yOut = THRESH * 10 / radius;
    else if (radius < THRESH * 5)
      *yOut = -3.0f / (THRESH * 5.0f) * (radius - THRESH * 5.0f) + 3.0f;
    else
       *yOut = 0.0f;

Трудно сказать, как должно выглядеть корректное условие сравнения, но наличие в коде ошибки очевидно.

Данная диагностика классифицируется как:

CWE-570 CERT-MSC01-C

Взгляните на примеры ошибок, обнаруженных с помощью диагностики V517.