使用jQuery拼接表格两列生成SQL语句未达预期效果

我看了你的代码，问题出在这几个关键地方：

• 循环内重复清空输入框：你把$("#SQLstring").val("")放在了.each()循环里面，这意味着每循环一行，就会把之前拼接好的SQL字符串清空，最后自然只留下最后一行的内容。
• 错误的字符串处理逻辑：最后那行$('#SQLstring').contains('SELECT [undefined].[undefined],').remove();完全不符合jQuery语法，contains()是用来查找包含指定文本的DOM元素，而且你也不能直接移除输入框里的一段文本。
• 未处理末尾多余的逗号：循环拼接完成后，SQL语句末尾会多一个逗号，这会导致SQL语法错误。

下面是修复后的完整代码：

$(".ConvertSQL").click(function () {
  // 先清空输入框，放在循环外面只执行一次
  $("#SQLstring").val("");
  // 初始化SQL开头部分
  let query = "SELECT ";
  
  $(".TableQuery").children("tbody").children("tr").each(function (i) {
    const $this = $(this);
    const my_td = $this.children("td");
    const first_col = my_td.eq(0).html().trim();
    const second_col = my_td.eq(1).html().trim();
    
    // 跳过空行或者内容为undefined的无效行
    if (first_col && second_col && first_col !== "undefined" && second_col !== "undefined") {
      query += `[${first_col}].[${second_col}], `;
    }
  });
  
  // 去掉末尾多余的逗号和空格
  if (query.endsWith(", ")) {
    query = query.slice(0, -2);
  }
  
  // 把处理好的SQL写入输入框
  $("#SQLstring").val(query);
});

关键修改说明： #

• 把输入框清空操作移到循环之前，确保只清空一次，保留循环拼接的所有内容。
• 在循环外部初始化query变量，循环内只做拼接操作，避免反复读取输入框的值（既低效又容易出错）。
• 增加了对空值和"undefined"的判断，跳过无效行，避免生成[undefined].[undefined]这种无效片段。
• 最后处理掉末尾多余的逗号，让SQL语句符合语法规范。
• 使用模板字符串（` `）让拼接语法更简洁易读。

这样修改后，点击按钮就能得到你预期的拼接好的完整SQL语句了。

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