Sql命令参数未执行

Question

当尝试使用 VB.NET、SQL 命令和参数对 SQL Server 表执行 DELETE 查询时，我遇到了一个奇怪的问题。

我有以下代码：

Try    
    sqlCommand.Transaction = transaction1
    sqlCommand.Connection = conn
    sqlCommand.CommandText = sqlQuery
    sqlCommand.Parameters.Add("@userID", SqlDbType.Int).Value = Convert.ToInt32(userID)
    sqlCommand.Parameters.Add("@groupID", SqlDbType.Int).Value = Convert.ToInt32(groupID)
    ''#Delete the user from the group.
    MessageBox.Show("User: " + Convert.ToString(userID) + "    Group: " + Convert.ToString(groupID))
    MessageBox.Show("Param, UserID: " + sqlCommand.Parameters.Item(0).Value.ToString)
    MessageBox.Show("Param, GroupID: " + sqlCommand.Parameters.Item(1).Value.ToString)
    return_deleteUser = sqlCommand.ExecuteNonQuery()
Catch ex As Exception
    transaction1.Rollback()
    Dim hr As Integer = Marshal.GetHRForException(ex)
    MsgBox("Removal of user from group has failed: " + ex.Message() & hr)
End Try

执行以下 SQL 查询：

Dim sqlQuery As String = "DELETE FROM MHGROUP.GROUPMEMS WHERE USERNUM =@userID AND GROUPNUM =@groupID"

我的问题是，当代码执行时，根本没有报告错误。我已运行 SQL Profiler，但查询未出现在跟踪列表中。我添加的三个消息框都返回正确的值，如果我要使用这些值对表执行 SQL 查询，查询就会成功。 userID和groupID都是3位整数。

任何人都可以建议为什么代码没有按预期工作，或者我可以使用任何进一步的调试来逐步执行代码？理想情况下，我希望看到已完成的 SQL 查询以及已完成的参数，但我还没有找到如何执行此操作。

编辑： 我稍后在代码中添加以下内容来检查执行是否全部成功处理：

If return_insertEvent > 0 And return_updateUser > 0 And return_nextSID > 0 And return_deleteUser > 0 Then
    MessageBox.Show("Success")
    return_removeADGroup = RemoveUserFromGroup(userID, groupName)
    MessageBox.Show("Remove FS User from AD Group: " + return_removeADGroup)
    transaction1.Commit()
    transaction2.Commit()
    transaction3.Commit()
    transaction4.Commit()
    returnResult = 1
Else
    transaction1.Rollback()
    transaction2.Rollback()
    transaction3.Rollback()
    transaction4.Rollback()
    returnResult = 0
End If

如果您需要任何进一步的信息，请随时与我联系。

Answer 1

您缺少 Transaction.Commit

Update 来响应添加到问题中的附加信息：

为什么您有 4 笔交易？由于它们的提交和回滚都是一起执行的，因此您只需要一个事务。我建议您使用 TransactionScope

您可以分配ADO.NET Command 对象的当前事务：

ADO.NET 和 System.Transactions

ADO.NET 2.0 中的事务处理

Answer 2

我可能猜测您的调用过程具有向后的 userid 和 groupid 值。如果 DELETE 没有找到匹配的记录，它将成功完成，但不会执行任何操作。我建议将删除包装在存储过程中。然后您可以添加代码来测试参数值是否正确通过。

Create Procedure UserDelete
@userid int, @groupID int
As
BEGIN
    Select @userid as UID, @groupID as GID INTO TESTTABLE;
    DELETE FROM MHGROUP.GROUPMEMS WHERE USERNUM =@userID AND GROUPNUM =@groupID;
END

运行您的代码，然后检查 TETTABLE 的内容。

FWIW：我不喜欢尝试在一行中获取整个参数声明。对我来说发生的事情太多了。我喜欢这个...

Dim pUID as New Parameter("@userid", SqlDbType.Int)
pUID.Value = userid
cmd.Parameters.Add(pUID)

Answer 3

经过一段时间的调试和 SQL 跟踪，我发现数据库所属的愚蠢应用程序以不同的方式对待组成员，组驻留在管理数据库中，但该组的用户成员身份驻留在另一个数据库中。

感谢上面所有提供时间和想法来协助编写代码的人。我已按照建议更改了代码，仅使用两个事务和两个连接（1 个用于管理和子数据库）。现在代码好多了并且更容易阅读。

再次感谢，

马特