有没有更好的方法来填充此自动完成？ C＃

Question

我有一个文本框，该文本框从SQL Server数据库中的值自动完成。我还创建了一个非常简单的存储过程：

存储过程代码

我的代码是这样的：

public AutoCompleteStringCollection AutoCompleteFlight(TextBox flight)
{
    using (SqlConnection connection = new SqlConnection(ConnectionLoader.ConnectionString("Threshold")))
    {
        AutoCompleteStringCollection flightCollection = new AutoCompleteStringCollection();

        connection.Open();

        SqlCommand flights = new SqlCommand("AutoComplete_Flight", connection);
        flights.CommandType = CommandType.StoredProcedure; 

        SqlDataReader readFlights = flights.ExecuteReader();

        while (readFlights.Read())
        {
            flightCollection.Add(readFlights["Flight_Number"].ToString());
        }

        return flight.AutoCompleteCustomSource = flightCollection;
    }
}

由于它是如此简单的查询，是否有一个存储过程的意义？还是我做错了，因为它仍然必须使用数据读取器并将其插入集合中。

在存储过程之前，我以前的代码是：

using (SqlConnection connection = new SqlConnection(ConnectionLoader.ConnectionString("Threshold")))
{
    AutoCompleteStringCollection flightCollection = new AutoCompleteStringCollection();

    connection.Open();

    SqlCommand flights = new SqlCommand("SELECT DISTINCT Flight_Number FROM Ramp_Board", connection);

    SqlDataReader readFlights = flights.ExecuteReader();

    while (readFlights.Read())
    {
        flightCollection.Add(readFlights["Flight_Number"].ToString());
    }

    return flight.AutoCompleteCustomSource = flightCollection;
}

第二部分是更好的代码，还是它们都错了，并且有更好的方法可以做到这一点？

Answer 1

“更好的方式”有点不确定。

如果您是否正在寻找存储过程的性能答案，那么我不确定这与小的数据集和简单的查询很重要。当有复杂的操作可以执行可以来回限制服务器或限制返回的数据量时，存储过程会发光。在您的情况下，服务器端的工作方式是相同的，并且返回的数据量也相同。 @niel指出，可以在不更改部署的代码的情况下更新服务器端。这是存储过程的另一个有用功能，但是您可能不需要这种情况。

如果您正在寻找替代代码答案，则可以使用DataAdapter而不是DataReader。该网站上有许多文章谈论了两者的性能，大多数文章都同意它们或多或少是相同的。唯一的例外是，如果您不打算阅读所有行。就您而言，您正在阅读整张桌子，因此它们实际上是相同的。

    SqlCommand sqlCmd = new SqlCommand("SELECT * FROM SomeTable", connection);

    SqlDataAdapter sqlDA= new SqlDataAdapter();
    sqlDA.SelectCommand = sqlCmd;
    DataTable table = new DataTable();

    // Fill table from SQL using the command and connection
    sqlDA.Fill(table);
    
    // Fill autoComplete from table
    autoComplete.AddRange(table.AsEnumerable().Select(dr => dr["ColumnName"].ToString()).ToArray());

如果您决定使用这种LINQ语句，则最好设置该列以不允许nulls，或添加一个过滤nulls的位置。我不确定如何或是否自动化策略处理空。