C# ListView多行选中更新SQL Server仅生效首行问题求助

你这段代码的问题其实很直观——在foreach循环遍历选中项的时候，你每次都固定取listView1.SelectedItems[0]，也就是不管循环到哪一行，都只拿第一个选中的项来更新数据库，这就导致哪怕选了N行，最终只有第一行被处理了。

核心修正思路 #

把循环里固定访问SelectedItems[0]的逻辑，改成用当前循环的变量itemRow来获取子项内容。因为foreach已经帮你遍历每一个选中的ListViewItem了，直接用这个变量就能拿到当前行的数据。

修正后的完整代码示例 #

private void updateinline() 
{
    foreach (ListViewItem itemRow in this.listView1.SelectedItems) 
    {
        // 改用当前循环的itemRow获取子项，而不是一直取第一个选中项
        string fnumber1 = itemRow.SubItems[0].Text.ToString();
        string fname1 = itemRow.SubItems[1].Text.ToString(); // 这里根据你实际的列索引调整
        
        // 数据库更新逻辑，务必使用参数化SQL避免注入风险
        string updateSql = "UPDATE YourTable SET fname = @FName WHERE fnumber = @FNumber";
        
        using (SqlConnection conn = new SqlConnection("你的数据库连接字符串"))
        {
            try
            {
                conn.Open();
                using (SqlCommand cmd = new SqlCommand(updateSql, conn))
                {
                    // 添加参数，避免SQL注入和语法错误
                    cmd.Parameters.AddWithValue("@FNumber", fnumber1);
                    cmd.Parameters.AddWithValue("@FName", fname1);
                    cmd.ExecuteNonQuery();
                }
            }
            catch (Exception ex)
            {
                // 添加异常处理，给用户友好提示同时方便排查问题
                MessageBox.Show($"更新失败：{ex.Message}");
            }
        }
    }
}

额外优化建议 #

• 强制使用参数化SQL：如果原来的代码是直接拼接SQL字符串，一定要改成参数化方式，这不仅能避免SQL注入的安全风险，还能解决特殊字符导致的SQL语法错误。
• 批量更新提升性能：如果经常需要批量更新大量行，可以把所有要更新的数据收集起来，通过一次批量SQL操作完成，比循环执行单条更新效率高很多。
• 异常处理不能省：添加try-catch块捕获数据库操作的异常，既可以给用户友好的操作反馈，也方便后续排查问题。

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