SQL插入查询未执行而未在C＃中引发任何异常

Question

我为论坛论坛网站编写了一个插入函数，用于在数据库中插入新的讨论线程。 功能如下：

public int createDiscuss(Discussion dis)
    {
        query = "insert into [MyForums].[dbo].[Discussions](desid, text, date, time, replyto, uid, isPrivate) values(@id, @text, @date, @time, @replyto, @uid, @pvp)";
        string did=getDesID();
        if (did == null)
            return -1;
        try
        {
            com = new SqlCommand(query, con);
            com.Parameters.AddWithValue("@id", did); 
            com.Parameters.AddWithValue("@text", dis.gettext()); 
            com.Parameters.AddWithValue("@date", SqlDateTime.Parse(DateTime.Now.ToString())); 
            com.Parameters.AddWithValue("@time", SqlDateTime.Parse(DateTime.Now.ToString()));
            com.Parameters.AddWithValue("@replyto", dis.getreplyto());
            com.Parameters.AddWithValue("@uid", dis.getuid());
            if (dis.IsPrivate() == false)
            { com.Parameters.AddWithValue("@pvp", 1); }
            else
            { com.Parameters.AddWithValue("@pvp", 2);  }
            con.Open();
            int r=com.ExecuteNonQuery();
            if (r <= 0)
            {
                return -1;
            }
            con.Close();

        }
        catch (Exception)
        {
            return -1;
        }
        return 0;
    }

如果遇到错误或异常，此函数将返回-1，并由调用函数进行检查，然后调用函数显示错误，则该函数将返回-1。 问题在于此插入功能未执行查询。 当我检查值是否正确传递给AddWithValue函数作为参数时，我发现每个值都按asp.net页上给出的值传递。 但是当控制权转到com.ExecuteNonQuery（）函数时。 它返回-1，这是错误。 谁能告诉我我的代码有什么问题吗？ 或如何检查com.ExecuteNonQuery（）为什么不返回任何受影响的行？

---

此异常显示

 A first chance exception of type 'System.Threading.ThreadAbortException' occurred in mscorlib.dll An exception of type 'System.Threading.ThreadAbortException' occurred in mscorlib.dll but was not handled in user code 

Answer 1

话虽如此，SQL Profiler是您的朋友。 从探查器中获取生成的sql并在SQL mgmt studio中运行它。 我有99％的信心，答案将很明显。 如果不是，请使用从事件探查器生成的sql更新您的答案，然后从那里获取答案。

如果您需要有关SQL Profiler的一些指导，简单的Google搜索会带来很多选择： SQL Profiler搜索

Answer 2

• 您应该在finally块中关闭连接。 您在con.Close();之前遇到的任何异常con.Close(); 将导致您的连接保持打开状态。 这可能是这里的错误（取决于con的范围）

• 您不应吞下该异常，而应将其记录下来。 然后，您可以正确检查错误。 如果您没有日志记录，请获取日志记录。 但是与此同时，请查看使用调试器捕获的异常。

• 您还应该捕获更多特定的异常，以便可以分辨出连接错误和应用程序中的长期错误之间的区别。 检查您正在调用的方法可能抛出哪些异常。 例如，con.Open（）可以抛出以下内容：

SqlException

InvalidOperationException

因此，首先捕获那些Exception最后捕获Exception 。

根据异常的类型，您可以返回不同的错误代码和/或以不同的严重性级别记录日志。

---

另外，找出一种方法（使用性能分析或仅复制+粘贴）以对数据库运行该查询，并查看会发生什么情况。 例如，您确定这部分是正确的：

        com.Parameters.AddWithValue("@date",
            SqlDateTime.Parse(DateTime.Now.ToString())); 
        com.Parameters.AddWithValue("@time",
            SqlDateTime.Parse(DateTime.Now.ToString()));

@date和@time相同。 数据库值是否也相同，并且列都需要字符串吗？

Answer 3

您需要进行一些基本的调试。 设置代码如下：

catch(Exception ex)
{
  return -1;
}

在返回时设置一个断点，并查看异常是什么。

Answer 4

您可能未达到int r=com.ExecuteNonQuery(); 码。 可能是con.Open();上的错误con.Open(); 方法。

您能否在捕获中抛出一个新异常并向我们显示实际错误？

Answer 5

当前代码的一个挑战是在几种不同的情况下返回“ -1”：

• 发生异常
• getDesID的结果为null
• INSERT语句无法插入行

一种简单的检查方法是从每个故障点返回不同的数字（例如-2或-3）。 这样一来，您就可以确定发生错误的位置，而无需太多更改代码或附加调试器。

在我看来，该问题不太可能是您的SQL INSERT语句-如果该语句本身无效或违反了数据库约束，则SQL Server将向您引发一个异常，该异常将由您的Exception处理程序捕获。