Должен ли я добавить быстрый возврат до forEach? - PullRequest
0 голосов
/ 17 февраля 2020

Есть ли какое-либо преимущество в проверке длины массива перед использованием forEach ?

Примите во внимание следующее:

const foo = () => {
  const elements = [...document.querySelectorAll(".selector")];

  elements.forEach(element => {
    element.style.height = `${element._someProperty}px`;
  });
};

Это то, что у меня есть в моем проекте. В некоторых случаях массив elements будет пустым, потому что существование таких элементов основано на вводе пользователем.

Мой вопрос:

Должен ли я добавить что-то вроде

if (!elements.length) return;

перед вызовом метода forEach следующим образом:

const foo = () => {
  const elements = [...document.querySelectorAll(".selector")];

  if (!elements.length) return;

  elements.forEach(element => {
    element.style.height = `${element._someProperty}px`;
  });
};

Или это можно считать микрооптимизацией?

...