# V3154. The 'a % b' expression always evaluates to 0.

Операция получения остатка от деления, выполненная на целочисленном типе, всегда возвращает 0. Это связано с тем, что справа от оператора находится значение 1 (-1). Скорее всего была допущена опечатка в правом операнде.

Частым паттерном данной ошибки является проверка делимости числа на 2 без остатка. Для этого необходимо поделить число по модулю и сравнить результат с 0 или 1. В этом месте легко сделать опечатку, так как раз ожидается значение 1, то кажется, что и поделить надо на 1. Пример:

    if ((x % 1) == 1)
    {
        ....
    }

Здесь вместо 2 написали 1 и теперь вне зависимости от значения 'x' выражение 'x % 1' всегда будет равно 0. Следовательно, условие всегда будет ложным.

Исправленный вариант:

    if ((x % 2) == 1)
    {
        ....
    }

Рассмотрим диагностику на реальном примере:

    const int SyncReportFrequency = 1;
    ....
    private void TimerOnElapsed(object sender, ElapsedEventArgs e)
    {
        if (_reportId % SyncReportFrequency == 0)
        {
            WriteSyncReport();
        }
    }

Переменная 'SyncReportFrequency ' является константой равной единице. Поэтому при любом значении переменной '_reportId' условие в операторе 'if' всегда будет истинным.

Возможно, в данном фрагменте кода разработчик совершил опечатку или условие является излишним. Пример исправленного варианта кода:

    const int SyncReportFrequency = 2;
    ....
    private void TimerOnElapsed(object sender, ElapsedEventArgs e)
    {
        if (_reportId % SyncReportFrequency == 0)
        {
            WriteSyncReport();
        }
    }

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

CWE-682