Excel经DataTable/DataSet插入SQL表遇ExecuteNonQuery初始化错误求助

错误原因分析 #

• 初始化SqlCommand时，query1为空字符串，后续修改query1的值并未同步到cmd.CommandText，导致执行ExecuteNonQuery()时命令文本未初始化。
• 代码同时混用SqlCommand.ExecuteNonQuery()和SqlDataAdapter.Update()两种插入逻辑，造成执行流程混乱。
• 参数使用错误：AddWithValue的第二个参数应为参数值，而非SqlDbType枚举；参数名与SQL语句占位符不匹配，还存在拼写错误（@Emloyee ID应为@EmployeeID）。
• 循环内重复添加参数，会导致参数集合持续累积，引发后续执行异常。
• 未处理数据表中的空值（如First Check In或Last Check Out为空的情况），直接拼接SQL会触发语法错误。

修正后的代码实现 #

以下是优化后的VB.NET代码，采用参数化查询避免SQL注入，正确配置SqlDataAdapter的InsertCommand并处理空值：

' 使用Using语句自动释放资源，避免连接泄漏
Using conn As New SqlConnection("Server= DESKTOP-C1JI0OA\SQLEXPRESS; TrustServerCertificate=True; Encrypt=true; Trusted_Connection=True; Database = Fingerprint reader;")
    Dim da As New SqlDataAdapter()
    Dim ds As New DataSet()
    ds.Tables.Add(dt1)
    ds.Tables(0).TableName = "Table"
    da.TableMappings.Add("Table", "dbo.fingerprintslogs")

    ' 配置参数化InsertCommand，指定列名避免顺序依赖
    Dim insertCmd As New SqlCommand("INSERT INTO fingerprintslogs ([Employee ID], [Date], [Weekday], [First Check In], [Last Check Out]) " & _
                                   "VALUES (@EmployeeID, @Date, @Weekday, @FirstCheckIn, @LastCheckOut)", conn)
    
    ' 添加参数并指定数据类型，允许空值的参数开启AllowDBNull
    insertCmd.Parameters.Add("@EmployeeID", SqlDbType.Int)
    insertCmd.Parameters.Add("@Date", SqlDbType.Date)
    insertCmd.Parameters.Add("@Weekday", SqlDbType.NVarChar, 50)
    insertCmd.Parameters.Add("@FirstCheckIn", SqlDbType.Time).AllowDBNull = True
    insertCmd.Parameters.Add("@LastCheckOut", SqlDbType.Time).AllowDBNull = True

    ' 将参数与DataTable的列绑定，自动映射数据
    insertCmd.Parameters("@EmployeeID").SourceColumn = "Employee ID"
    insertCmd.Parameters("@Date").SourceColumn = "Date"
    insertCmd.Parameters("@Weekday").SourceColumn = "Weekday"
    insertCmd.Parameters("@FirstCheckIn").SourceColumn = "First Check In"
    insertCmd.Parameters("@LastCheckOut").SourceColumn = "Last Check Out"

    da.InsertCommand = insertCmd
    conn.Open()
    ' 一次性更新整个DataTable，无需手动逐行遍历
    da.Update(ds.Tables(0))
End Using ' 自动关闭连接并释放资源

关键优化点说明 #

• 用Using语句管理SqlConnection，确保连接自动关闭、资源及时释放，避免连接泄漏。
• 采用参数化查询，彻底规避SQL注入风险，同时解决数据类型匹配问题。
• 正确配置SqlDataAdapter的InsertCommand，通过参数绑定DataTable列，Update()方法会自动处理所有新增行，无需手动遍历。
• 为允许空值的参数设置AllowDBNull = True，适配数据表中的空值场景。
• 明确指定字符串参数的长度（如NVarChar(50)），提升数据库执行效率。

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