Как оптимизировать код?

Question

[Юрий Иванов]

Добрый день, помогите оптимизировать код
у меня вот несколько таких блоков, в которых регион повторяется и проверка, возможно как-то проще и читабельнее сделать

let linkAll = [...d.querySelectorAll('.item_row')];
      linkAll.forEach((i) => {
        let region = i.querySelector('.item_region > a').textContent.trim()
        if(region == 'Минск') {
          let linkhref = (('https://zooby.by/'+i.getAttribute('href')));
          data.push({link: linkhref});
        }
      });
      let nameAll = [...d.querySelectorAll('.item_row')];
      nameAll.forEach((i) => {
        let region = i.querySelector('.item_region > a').textContent.trim()
        let title = i.querySelector('.title').textContent;
        console.log(title);
        if(region == 'Минск') {
          let nameText = title;
          data.push({name: nameText});
        }
      });

Comments

[WbICHA]

[...d.querySelectorAll('.item_row')];

Для чего ты это делаешь?

[Юрий Иванов]

WbICHA, Убрал;)

[Юрий Иванов]

Александр, Нет, я это знаю, там просто были другие эксперименты, которые остались за кадром и я забыл убрать спред, тс рак, но это он знает!

Answer 1

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

Можно постараться не повторять себя (DRY = don't repeat yourself).

const data = [];

document.querySelectorAll('.item_row').forEach(row => {
  const region = row.querySelector('.item_region > a')?.textContent?.trim();
  if (region !== 'Минск') return;

  const name = row.querySelector('.title')?.textContent;
  const link = 'https://zooby.by/' + row.getAttribute('href');

  data.push({ name, link });
});

Comments

[Юрий Иванов]

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

[Юрий Иванов]

Сейчас переписал весь код под ваше решение, какая же красота, ля , еще раз спасибо!