↵ Вернуться

Используйте логические переменные

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

if (
  elementIndex < 0 ||
  MAX_ELEMENTS < elementIndex ||
  elementIndex === lastElementIndex
) {
  // ...
}

Что тут написано?

Скорее всего рядовой разработчик пропустит эту строку и даже читать не будет. Хорошо, если автор оставил поясняющий комментарий.

Исправить ситуацию, можно, присвоив проверкам имена:

const finished = elementIndex < 0 || MAX_ELEMENTS < elementIndex;
const repeatedEntry = elementIndex === lastElementIndex;

if (finished || repeatedEntry) {
  // ...
}

Стало намного проще осознать назначение условного выражения.

↵ Вернуться