两个Items集合均实现IList

因此，您不必传递ComboBox / ListBox，而只需传递comboBox1.Items或listBox1.Items到您的方法即可。

 private void UpdateBox (IList items,string select,string from,string order = "")

...

 items.Clear();
 while (dataReader.Read())
 {
     items.Add(dataReader[select]);
 }

您可以使用@Holger答案。

对于速度性能，您可以使用重构后的值：

private void PopulateItems(List<IList> listItems,string column,string table,string orderByColumn = "")
{
  using ( var connection = new SqlConnection(connectionString) )
  {
    connection.Open();
    if ( orderByColumn == "" )
      orderByColumn = column;
    string sql = $"SELECT {column} FROM {table} ORDER BY {orderByColumn}";
    using ( var command = new SqlCommand(sql,connection) )
    {
      var reader = command.ExecuteReader();
      foreach ( var items in listItems )
        items.Clear();
      while ( reader.Read() )
        foreach ( var items in listItems )
          items.Add(reader[column]);
    }
  }
}

用法：

var listItemsToPopulate = new List<IList>()
{
  comboBox1.Items,listBox1.Items
};

PopulateLists(listItemsToPopulate,"","");

我将重组此代码。您正在尝试在1个函数中做太多事情。

通过操纵控件从DB中拆分数据。现在，GetDbItems函数是独立的-可以轻松移至单独的类，而不必直接绑定到UI。

类似这样的东西：

IList<string> items = GetDbItems("a","b","c");
if (items.Any())
{
    control.Items.Clear();
    control.Items.Add(items);
}

private IList<string> GetDbItems(string select,string order = "")
{
        var result = new List<string>();
        using (SqlConnection conn = new SqlConnection(connectionString))
        {
            conn.Open();
            if (order == "") order = select;
            using (SqlCommand command = new SqlCommand("SELECT " + select +
                " FROM " + from + " ORDER BY " + order,conn))
            {
                SqlDataReader dataReader = command.ExecuteReader();
                while (dataReader.Read())
                {
                    result.Add(dataReader[select]);
                }
            }
        }
        return result;
    }

• 这只是在编辑框中输入的内容-我不能保证它会被编译。我只是想给您一个开始。