Метод соединения с forEach - лучшая практика? - PullRequest
0 голосов
/ 27 ноября 2018

Если это плохой вопрос, просто скажите мне, и я его удалю.У меня вопрос: Является ли прикрепленный код лучшим способом объединения строк текста с <hr> в качестве разделителя?

У меня есть таблица с различными списками.Для списков одной и той же «группы» (см. Массив) я хочу, чтобы они были разделены <hr>.Если в группе только 1, то я не хочу <hr>, потому что это выглядит плохо.Обычно я бы использовал метод array.join (), так как я уже использую forEach (), мне пришлось придумать обходной путь textContent & if / else.

Кажется, загрузка страницы занимает больше времени, чем раньшеЯ реализовал расширение <hr>.Это лучший способ кодирования или я должен оптимизировать?

<table border="1">
<tr>
  <td id="Group1"></td>
</tr>
<tr>
  <td id="Group2"></td>
</tr>
<tr>
  <td id="Group3"></td>
</tr>
</table>

<script>
var entry = [
["Group1","AAAAAAAAAAAAA"],
["Group2","BBBBBBBBBBBBB"],
["Group2","CCCCCCCCCCCCC"],
["Group3","DDDDDDDDDDDDD"],
["Group3","EEEEEEEEEEEEE"],
["Group3","FFFFFFFFFFFFF"],
]
  
  entry.forEach(
  function myFunction(item, index){
  var x = document.getElementById(item[0]);
  var y = document.getElementById(item[0]).textContent.length;
 
  if(x !== null) {
  	if(y < 1) {
    x.innerHTML += item[1];
    }
  else{
  x.innerHTML += '<hr>' + item[1];
  }
  
    
    }
 
  
});

</script>
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...