Question

我正在使用visual studios 2010来创建一个带有数据库的c＃web应用程序。我的目标是让default.aspx调用一个c＃类，它运行一个存储过程，从表中选择一个条目并返回它。这是代码：

'The stored procedure.  I want it to send back the name it gets from doing
'the query to the c# class.
ALTER PROCEDURE getName (@id int)
AS
BEGIN
SET NOCOUNT ON;

--   
SELECT name FROM tableA where id = @id;

END 
Return
//Here's the c# class I'm using.
public class student
{
    public string name;
    public int id;

    public student()
    { }

    public String doQuery(int id)
    {
        SqlConnection conn = null;

        try
        {
             conn = new SqlConnection("Server =(local); Database = Database1.mdf;
   Integrated Security = SSPI");
            conn.Open();
            SqlCommand cmd = new SqlCommand("getName", conn);
            cmd.CommandType = CommandType.StoredProcedure;
            SqlParameter param = new SqlParameter("@id", SqlDbType.Int);
            param.Direction = ParameterDirection.Input;
            param.Value = id;
            cmd.Parameters.Add(param);
            //This is some code from when I tryed return value
            //SqlParameter reVal = cmd.Parameters.Add("@name", SqlDbType.VarChar);
            //reVal.Direction = ParameterDirection.ReturnValue;

            //before using ExecuteScalar I tried ExcuteNonQuery with the commented    
            //out code
            name = (string)cmd.ExecuteScalar();

            //name = (String)cmd.Parameters["@name"].Value;

            conn.Close();
        }

        catch(Exception)
        {}
        return name;
    }
}

运行我的程序不会返回错误，它不会在名称中放置任何值。我在sql过程中选择的名称到我的c＃类中的name变量中我缺少什么。我希望我能清楚地传达我的问题。

edit1：我没有在catch中放任何东西因为没有决定使用什么来查看它已经出错了。当它没有通过尝试时我改变了它以使name =“error”，这就是我得到的，这就是我得到的。我也尝试在sql server management中运行“exec getName 5，otherstuff”。我有点不清楚在运行exec getName时使用什么作为第二个参数，因为第二个参数假设只是输出但似乎仍然需要运行它。它只是说命令成功执行但不显示id为5的名称

Answer 1

问题在于您的连接字符串：除非您有奇怪的命名约定，否则您指定数据库文件名而不是数据库本身的名称。

尝试更改连接字符串的这一部分：Database = Database1.mdf;为Database = Database1;。

如果您对连接字符串中有效或无效的内容感到困惑，您可以始终使用SqlConnectionStringBuilder，它将在您设置正确的属性后为您创建适当的连接字符串。

您还可以使用SqlConnection.ConnectionString文档中指定的属性列表作为包含示例的参考。

最后，我强烈推荐以下最佳做法：

1）使用带有连接和命令的块来确保它们被正确关闭和处理。

2）不要将名称直接指定给ExecuteScalar的结果，以防它返回为DBNull.Value

3）除非您在代码中记录了为什么这样做，否则永远不要忽略异常。

以下是所有上述建议的快速重写：

        try
        {
            using (var conn = new SqlConnection("Server =(local); Database = Database1; Integrated Security = SSPI"))
            {
                conn.Open();
                using (var cmd = new SqlCommand("getName", conn))
                {
                    cmd.CommandType = CommandType.StoredProcedure;
                    var param = new SqlParameter("@id", SqlDbType.Int);
                    param.Direction = ParameterDirection.Input;
                    param.Value = id;
                    cmd.Parameters.Add(param);

                    var oResult = cmd.ExecuteScalar();
                    if ((oResult != null) && (oResult != DBNull.Value))
                    {
                        name = (string)oResult;
                    }
                }
                conn.Close();
            }
        }

        catch (Exception)
        { 
            //  Do something with the exception here, don't just ignore it
        }

Answer 2

我建议对{em> SQL 语句使用async / await模式。幸运的是，它不需要太多的重构。

看看这是否适合你：

public async Task<string> QueryGetNameAsync(int id)
{
  using (var dbConn = new SqlConnection("..."))
  using (var command = new SqlCommand("getName", dbConn))
  {
    try
    {
      command.CommandType = CommandType.StoredProcedure;
      command.Parameters.AddWithValue("@id", id);

      await dbConn.OpenAsync();

      var result = await command.ExecuteScalarAsync();
      dbConn.Close();

      var name = result as string;          
      return name;
    }
    catch (Exception ex)
    {
      // Handle exception here.
    }
  }
}

你可以用以下内容来称呼它：

private async void DoLookup_Clicked(object sender, EventArgs e)
{
   var id = int.Parse(idText.Text);
   var name = await QueryGetNameAsync(id);
}

或者，可以在 SQL 中使用OUTPUT参数，但您必须将存储过程调整为以下内容：

ALTER PROCEDURE getName
(
  @id int, 
  @name varchar(100) OUTPUT
)
AS
BEGIN
SET NOCOUNT ON;

SELECT @name = name FROM tableA where id = @id;

END

然后您的 C＃函数将类似于：

public async Task<string> QueryGetNameAsync(int id)
{
  using (var dbConn = new SqlConnection("..."))
  using (var command = new SqlCommand("getName", dbConn))
  {
    try
    {
      command.CommandType = CommandType.StoredProcedure;
      command.Parameters.AddWithValue("@id", id);
      command.Parameters.Add("@name", SqlDbType.VarChar, 100);
      command.Parameters["@name"].Direction = ParameterDirection.Output;

      await dbConn.OpenAsync();
      await command.ExecuteNonQueryAsync();
      dbConn.Close();

      var name = command.Parameters["@name"].Value as string;          
      return name;
    }
    catch (Exception ex)
    {
      // Handle exception here.
    }
  }
}

使用C＃方法从存储过程中获取返回值

2 个答案: