JavaScript: как был этот код написали профессионалы?

Question

[Shimpanze]

Добрый день!

Написал сниппет, который заменяет параграфы с тремя звездочками (типа разделители), на элемент <hr>.

Как бы этот код написали профессионалы?

// ищем все парараграфы
document.querySelectorAll('.entry-content p').forEach(function(entry) {
  // проверяем, подходит ли содержимое текущего параграфа в цикле
  // под наше условие (содержит ли только звездочки)
  if (entry.innerHTML == entry.innerHTML.match(/[\s]*\*[\s]*\*[\s]*\*[\s]*[\s]*/gi)) {
    // если да, вставляем после него элемент <hr>
    entry.insertAdjacentHTML('afterEnd', '<hr>');
    // и удаляем родительский элемент параграфа (как сделать умнее не придумал...)
    entry.closest("p").remove();
  }
});

Спасибо!

Comments

[longclaps]

Вот нафига тут класс поверх класса [\s], достаточно просто \s

Answer 1

[Сергей Соколов]

Я не про, сделал бы так:

function replaceStars(collection) {
  var i, el, re = /^(\s*\*\s*){3}$/;
  for(i = 0; i<collection.length; i++) {
    el = collection[i];
    if( el.innerHTML.match(re)) {
      el.insertAdjacentHTML('afterend', '<hr>');
      el.parentNode.removeChild(el);
    }
  }
}

replaceStars(document.querySelectorAll('.entry-content p'));

Answer 2

[xtabay]

Тот же верхний пример, только, немного более лаконично, на мой взгляд.

function replaceStars(collection) {
  const re = /^(\s*\*\s*){3}$/;

	collection.forEach((elem) => {
    if(elem.innerHTML.match(re)) {
      elem.insertAdjacentHTML('afterend', '<hr>');
      elem.parentNode.removeChild(elem);
    }
  });
}

replaceStars(document.querySelectorAll('.entry-content p'));