V7020. Suspicious code formatting. Curly brackets may be missing.

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

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

if (a == 1)
  b = c;
  d = b;    // <=

В этом примере инструкция d = b; выполнится вне зависимости от условия, поскольку при отсутствии блока после управляющей конструкции её телом будет только первое выражение. При этом форматирование указывает на то, что выполнение d = b; ожидалось только при истинности условия a == 1.

Согласно Google JavaScript Style Guide фигурные скобки рекомендуется использовать для всех управляющих конструкций, даже если их тело содержит всего одну операцию. Это позволит не допустить путаницы из-за кода, вводящего в заблуждение.

Варианты исправленного кода:

// Option N1
if (a == 1) {
  b = c;
}
d = b; 

// Option N2
if (a == 1) {
  b = c;
  d = b;
}

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

if (a == 1) b = c; d = b;

В этом случае операция d = b; также выполнится вне зависимости от условия.

Варианты исправления:

if (a == 1) { b = c; d = b; }

if (a == 1) { b = c; }
d = b;

if (a == 1) b = c;
d = b;

Примечание. В качестве управляющих конструкций анализатор рассматривает if, for и while.

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

Была ли полезна эта страница документации?

Ваше сообщение отправлено

Мы ответим вам на:

Если вы так и не получили ответ, пожалуйста, проверьте, отфильтровано ли письмо в одну из следующих стандартных папок:

• Промоакции
• Оповещения
• Спам

Напишите, пожалуйста, на какой вопрос вы не смогли найти ответ, и наши разработчики ответят вам лично по почте.

Даю согласие на обработку своих персональных данных в соответствии с Политикой обработки персональных данных