Question

在我的主表单中，我已实现此代码..

void SampleMethod(string name, string lastName, string database)
{
        SqlParameter sqlParam = new SqlParameter();
        sqlParam.ParameterName = "@name";
        sqlParam.Value = name;
        sqlParam.SqlDbType = SqlDbType.NVarChar;

        SqlParameter sqlParam1 = new SqlParameter();
        sqlParam1.ParameterName = "@lastName";
        sqlParam1.Value = lastName;
        sqlParam1.SqlDbType = SqlDbType.NVarChar;

        SqlParameter sqlParam2 = new SqlParameter();
        sqlParam2.ParameterName = "@database";
        sqlParam2.Value = database;
        sqlParam2.SqlDbType = SqlDbType.NVarChar;

        SampleClass sampleClass = new SampleClass(new DBConn(@serverName, tableName, userName, password));
        sampleClass.executeStoredProc(dataGridView1, "sp_sampleStoredProc", sqlParam, sqlParam1, sqlParam2);
}

在我的SampleClass中，我有这种方法。

public DataGridView executeStoredProc(DataGridView dtgrdView, string storedProcName, params SqlParameter[] parameters)
{
        try
        {
            DataTable dt = new DataTable();
            sqlDA = new SqlDataAdapter(storedProcName, sqlconn);
            sqlDA.SelectCommand.CommandType = CommandType.StoredProcedure;
            sqlDA.SelectCommand.CommandTimeout = 60;

            // Loop through passed parameters
            if (parameters != null && parameters.Length > 0)
            {
                foreach (var p in parameters)
                    sqlDA.SelectCommand.Parameters.Add(p);
            }

            sqlDA.Fill(dt);
            dtgrdView.DataSource = dt;
            sqlconn.Close();
        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.Message);
            sqlconn.Close();
        }

        return dtgrdView;
}

我想做的是避免多重

SqlParameter sqlParam = new SqlParameter()

在我的代码中，我已经为这个问题尝试了很多解决方案，但我没有得到正确的答案。我也尝试过对此进行研究，但我仍然无法得到正确答案。

请不要介意我的命名约定和其他代码，因为我故意更改其中许多代码:)谢谢。

Answer 1

作为解决方案的替代方案，请尝试使用现有的解决方案，使用Dapper（https://github.com/StackExchange/dapper-dot-net）。

如果你的存储过程或查询需要它，你仍然需要使用多个参数，但这很好地为你抽象，这肯定会减少代码量。

void SampleMethod(string name, string lastName, string database)
{
    using(var connection = new SqlConnection(MY_CONNECTION_STRING))
    {
        var resultListOfRows = connection.Query<ReturnObject>(MY_STORED_PROCEDURE, new { 
            name = name,
            lastName = lastName,
            database = database}, commandType: System.Data.CommandType.StoredProcedure);
    }
}

Answer 2

将数据库逻辑分开到一个地方（将sqladapter，sqlcommand等放在一个地方），然后将命令封装在命令中，如下所述，并且不需要单独声明sqlparameter，将其添加到参数列表中。 / p>

 cmdToExecute.Parameters.Add(new SqlParameter("@parameter", value));

看看下面的完整示例

    public DataTable ProdTypeSelectAll(string cultureCode)
    {
        SqlCommand cmdToExecute = new SqlCommand();
        cmdToExecute.CommandText = "dbo.[pk_ProdType_SelectAll]";
        cmdToExecute.CommandType = CommandType.StoredProcedure;
        DataTable toReturn = new DataTable("ProdType");
        SqlDataAdapter adapter = new SqlDataAdapter(cmdToExecute);
        cmdToExecute.Connection = _mainConnection;
        cmdToExecute.Parameters.Add(new SqlParameter("@CultureName", cultureCode));
        _mainConnection.Open();
        adapter.Fill(toReturn);
        return toReturn;
}

Answer 3

首先，最简单的选择是使用像Dapper这样的microORM，并检索强类型集合。 Gridview可以绑定到任何东西，包括强类型集合。所有这些代码都可能成为：

using(var con=new SqlConnection(myConnectionString))
{
    con.Open();
    var result= connection.Query<ResultClass>("sp_MySproc", 
                         new { Name= name, LastName= lastName,Database=database},
                         commandType: CommandType.StoredProcedure);
    return result;
}

即使使用原始ADO.NET，您也可以使用appropriate constructor 在一行中创建一个SqlParameter。例如，您可以使用以下命令创建新的nvarchar(n)参数：

var myParam=new SqlParameter("@name",SqlDbType.NVarchar,20);

或

var myParam=new SqlParameter("@name",SqlDbType.NVarchar,20){Value = name};

更好的想法是只创建一次SqlCommand对象并重用它。有了初始化的SqlCommand对象后，您只需设置一个新的连接并更改参数值，例如：

public void Init()
{
    _loadCustomers = new SqlCommand(...);
    _loadCustomers.Parameters.Add("@name",SqlDbType.NVarChar,20);
    ...
}

//In another method :
using(var con=new SqlConnection(myConnectionString)
{       
    _loadCustomers.Connection=con;
    _loadCustomers.Parameters["@name"].Value = myNameParam;
    con.Open();
    using(var reader=_load.ExecuteReader())
    {
    //...
    }
}

您可以使用SqlDataAdapter执行相同的操作，实际上，自.NET 1.0以来，Windows Forms和数据适配器的使用方式就是这样。

每次要填充网格时，不要创建新的网格，而是创建单个网格并通过在执行前设置连接和参数来重复使用它。您可以使用SqlDataAdapter(SqlCommand)构造函数使事情变得更清晰：

public void Init()
{
    _loadCustomers = new SqlCommand(...);
    _loadCustomers.Parameters.Add("@name",SqlDbType.NVarChar,20);
    ....
    _myGridAdapter = new SqlDataAdapter(_loadCustomers);
    ...
}

并称之为：

using(var con=new SqlConnection(myConnectionString))
{
    _myGridAdapter.SelectCommand.Connection=con;
    _myGridAdapter.SelectCommand.Parameters["@name"].Value =....;
    con.Open();

    var dt = new DataTable();
    _myGridAdapter.Fill(dt);
    dtgrdView.DataSource = dt;
    return dtgrdView;
}

Answer 4

您可以使用SqlParameter Constructor (String, Object)。替换：

sampleClass.executeStoredProc(dataGridView1,
        "sp_sampleStoredProc",
        sqlParam,
        sqlParam1,
        sqlParam2);

使用：

sampleClass.executeStoredProc(dataGridView1,
        "sp_sampleStoredProc",
        new SqlParameter("@name", (object)name),
        new SqlParameter("@lastName", (object)lastName),
        new SqlParameter("@database", (object)database));

但是，这样做不太可能使您的代码更快，但会降低可读性。

将多个SqlParameter传递给方法

4 个答案: