CheckedListBox DataSourceが突然動作しない

Question

私はこのコードを使ってアイテムを取得するCheckedListBoxを持っています。

public static void GetDisplayValueList(object clk, string[] kv, string tableName) 
{ 
    using (SqlConnection conn = new SqlConnection(connectionString)) 
    { 
     try 
     { 
      string list = ""; 
      foreach (string item in kv) 
       list += item + ","; 
      string query = "SELECT " + list.Substring(0, list.Length - 1) + " FROM [dbo].[" + tableName + "]"; 
      SqlDataAdapter da = new SqlDataAdapter(query, conn); 
      DataSet ds = new DataSet(); 
      da.Fill(ds); 
      ((CheckedListBox)clk).DataSource = ds.Tables[0]; 
      ((CheckedListBox)clk).DisplayMember = "Description"; 
      ((CheckedListBox)clk).ValueMember = "Id"; 
     } 
     catch (Exception ex) 
     { 
      MessageBox.Show("An error has occurred: " + ex.Message, "Error"); 
     } 
    } 
} 

突然、それはもう動作し、次のエラーを与えていない：

Exception thrown: 'System.NullReferenceException' in System.Windows.Forms.dll

が、私はこれを修正するために何ができるかのDataTableににCheckedListBoxをバインドする方法を他の方法があります？ ありがとうございます。

Answer 1

私は答えを提供するとは思わないが、いくつかのコードラインを改善するのに役立つだろう。

任意の区切り文字で区切ら配列から文字列を取得する：文字列は、Alとビット混乱アップとなるであろう場合

string list = string.Join(",", kv); 

+演算子：速度を向上させる

string query = string.Format("SELECT {0} FROM [dbo].[{1}]", list, tableName); 

およびパフォーマンス、一度だけキャストしよう：

var clb = (CheckedListBox)clk; 
clb.DataSource = ds.Tables[0]; 
clb.DisplayMember = "Description"; 
clb.ValueMember = "Id"; 

Answer 2

答えに関するコメントあなたはあなたのコールスタックの上に行く必要があると思う。

元のエラーは((CheckedListBox)clk).DataSource = ds.Tables[0];です。これはclkがnullであることを意味し、キャスト後にはまだnullであるので、CheckedListBoxにキャストしてから使用することはできません。 （ds.Tables[0]がヌルであることを意味するわけではありません。データソースをnullに設定してデータを消去することができます）。

その後、@ DDD Softの回答から変更があった場合、そのclkアウト

var clb = (CheckedListBox)clk;  
clb.DataSource = ds.Tables[0]; // <- error here 

このさらなるポイントがnullの場合、鋳造時に、clbはその後もnullです。

あなたはclk