Open Time

Hier je posais une question — de néophyte un peu éclairé, mais néophyte tout de même — au sujet d’une optimisation, ou plutôt d’une ré-écriture d’une petite fonction javascript.

Le code d’origine était celui-ci :

window.addEventListener('load', () => {
  const images = document.getElementsByTagName('img');
  for (let i = 0; i < images.length; i++) {
    if (images[i].alt !== '') {
      images[i].classList.add('a42-ac-replace-img');
    }
  }
});

Et le code final est celui-là :

window.addEventListener('load', () => {
  const images = document.querySelectorAll('img:not([alt=""])');
  images.forEach((image) => image.classList.add('a42-ac-replace-img'));
});

Je pense qu’on pourrait même aller plus loin en évitant de déclarer la constante images et faire comme-ci :

window.addEventListener('load', () => {
  document.querySelectorAll('img:not([alt=""])').forEach((image) => image.classList.add('a42-ac-replace-img'));
});

La question est : « Est-ce que ça reste lisible (compréhensible facilement) ? »