Semicolon on same line as if, for or while statement

Точка с запятой на той же линии приводит к пустому телу if, for или while оператор

Описание

Этот дефект возникает, когда точка с запятой в той же линии, что и последняя лексема if, for или while оператор приводит к пустому телу.

Проверка делает исключение для случая, когда if оператор немедленно сопровождается else оператор:

if(condition);
else {
  ...
}

Риск

Точка с запятой, следующая за оператором if, for или while, часто указывает на ошибку программирования. Ложная точка с запятой изменяет поток выполнения и приводит к непреднамеренным результатам.

Зафиксировать

Если вам нужно пустое тело для if, for или while оператор, оберните точку с запятой в блок и поместите блок в новую линию, чтобы явным образом указать ваш намерение:

if(condition)
   {;}
В противном случае удалите ложную точку с запятой.

Примеры

расширить все

int credentialsOK(void);

void login () {
    int loggedIn = 0;
    if(credentialsOK());
      loggedIn = 1;
}

В этом примере ложная точка с запятой приводит к пустой if тело. Назначение loggedIn=1 всегда выполняется. Однако поручение, вероятно, должно было выполняться только при условии.

Коррекция - Удалите ложную точку с запятой

Если точка с запятой была непреднамеренной, удалите точку с запятой.

int credentialsOK(void);

void login () {
    int loggedIn = 0;
    if(credentialsOK())
      loggedIn = 1;
}

Информация о результатах

Группа: Хорошая практика
Язык: C | C++
По умолчанию: Off
Синтаксис командной строки : SEMICOLON_CTRL_STMT_SAME_LINE
Влияние: Низкое
Введенный в R2020a