Java Selenium拼接HTML表格字符串时循环未正确分行输出TR标签

异常根因分析 #

你的字符串拼接逻辑和println逐行输出的逻辑存在本质的状态残留差异，具体有两个关键错误：

1. 行内容变量未在每轮循环重置，导致历史内容累积
   代码中用于存储td内容的rows变量定义在循环外部，不会在每遍历一个新的<tr>元素时自动清空。第一轮循环拼接完第一行的所有<td>标签后，进入第二轮循环时，rows中仍然保留上一轮的td内容，直接在上次的字符串后追加新的td内容，最终会把多行的td全部堆在一起。
2. 最终结果变量使用赋值而非追加，导致历史内容被覆盖
   代码中对scrape的赋值语句为scrape = rowHeader + rows + rowFooter+"\n\n"，用的是直接赋值符=，意味着每轮循环都会把scrape中存储的历史内容完全替换成本轮拼接的字符串，之前拼好的行内容会直接丢失，最终仅能留存最后一轮循环生成的内容。

而println逐行输出的写法不存在这两个问题：每次调用println都是即时把当前标签内容输出到控制台，不需要跨循环保留字符串状态，每轮循环的<tr>开头、td内容、</tr>结尾都是独立即时输出，不会残留上一轮的拼接内容，因此格式正常。

修正方案 #

按照需求调整变量的初始化和拼接逻辑：

• 每轮外层循环（对应一个表格行）时，先初始化空的临时变量存储当前行的td内容，避免历史残留
• 拼接完整表格内容时，使用追加逻辑而非直接覆盖

修正后的可正常运行代码示例：

// 在外层循环外初始化最终存储所有表格内容的变量
StringBuilder scrape = new StringBuilder();
for(WebElement trElement : tr_collection) {
    List<WebElement> td_collection = trElement.findElements(By.xpath("td"));
    // 每轮循环初始化空的临时变量，存储当前行的td内容
    StringBuilder currentRowTdContent = new StringBuilder();
    for(WebElement tdElement : td_collection) {
        currentRowTdContent.append("<td>").append(tdElement.getText()).append("</td>");
    }
    // 将当前行完整的<tr>结构追加到最终结果中，避免覆盖历史内容
    scrape.append("<tr>")
          .append(currentRowTdContent)
          .append("</tr>\r\n\n");
    row_num++;
}
// 循环结束后即可从scrape变量获取完整格式的表格HTML
String fullTableHtml = scrape.toString();

补充优化提示：如果需要遍历的表格行数较多，不建议使用String类型的直接+=做字符串拼接——Java中String为不可变对象，频繁拼接会生成大量临时实例，造成不必要的性能损耗，使用StringBuilder做字符串拼接的效率会高很多。

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