JS代码误给所有tr添加id而非首个匹配项，求排查修正

先帮你梳理下原代码的问题所在：

1. 你构造的goToweek是完整的<td>v2</td>字符串，还加了内层循环遍历这个字符串的每个字符——这会导致只要某个td的innerHTML里包含'<'、'v'、'2'这类单个字符，就会触发id设置，完全偏离了匹配v2的初衷。
2. 找到匹配元素后没有终止循环，所以后续所有符合（哪怕是误匹配）的tr都会被加上id。

下面是修正后的代码，能精准定位第一个包含v{result[1]}的td，给它的父tr添加id后立刻停止遍历：

function setAttributes() { 
  // 直接构造要匹配的目标文本，不需要包含td标签
  const targetText = "v" + result[1]; 
  const tds = document.body.getElementsByTagName("td");
  
  for (let i = 0; i < tds.length; i++) {
    // 检查当前td的内容是否包含目标文本
    if (tds[i].innerHTML.includes(targetText)) {
      const targetTr = tds[i].parentNode;
      targetTr.setAttribute("id", "goToThis");
      // 找到第一个匹配项后立刻终止循环，避免后续无效遍历
      break; 
    }
  }
}

关键修改说明： #

• 移除了错误的内层字符遍历，改用includes()方法直接检查td内容是否包含目标文本v2（当result[1]为2时），匹配逻辑更精准。
• 找到第一个符合条件的元素后用break跳出循环，确保只有首个匹配的tr会被添加id。
• 替换var为const/let，更贴合现代JavaScript规范，避免变量提升带来的潜在问题。

用你的示例HTML测试：

<tr> 
  <td>2016</td> 
  <td>v2</td> 
</tr>
<tr> 
  <td>2016</td> 
  <td>v4</td> 
</tr>

执行修正后的代码后，只有第一个tr会被加上id="goToThis"，完全符合你的需求。

内容的提问来源于stack exchange，提问作者wiaK