割り当てられていないローカル変数の使用 - ifステートメント

Question

私は次のコードブロックを実行しています。コンパイラーは割り当てられていないローカル変数について不平を言っています。

while (rsData.Read()) 
{ 
    if (rsData["TYPE"] != DBNull.Value) 
     strType = rsData["TYPE"].ToString().Trim(); 


    if (strType == "01") 
    { 
     if (rsData["Text"] != DBNull.Value) 
      strwho = rsData["Text"].ToString(); 

     if ((strwho.Length < 10 || (strwho.IndexOf("NULL") > 1))) 
      strwho = ""; 
    } 
    else if (strType == "07") 
    { 
     if (rsData["Text"] != DBNull.Value) 
      strmetades = rsData["Text"].ToString(); 

     if ((strmetades.Length < 10 || (strmetades.IndexOf("NULL") > 1))) 
      strmetades = ""; 
    } 

「if（strType == "01"）」行のすべてに不満があり、何が起きているのかわかりません。私はこれにスイッチを使用することを考えましたが、それも同じ問題を抱えているようです。

アイデア？

Answer 1

文字列strTypeを宣言するときに、

string strType = null; 

詳細ような値を割り当てる必要があります：あなたはそれを使用する前に、ローカル変数に何らかの値を割り当てる必要がありますCompiler Error CS0165

Answer 2

を。 （あなたはstrTypeは、前の反復からその値を覚えてしたくない場合）、

var strType = ""; // or null 

それとも初期値を取得することを確認し、両方の時にリーダー：あなたは（ブロックしばらく前に）それを宣言する場所でそれを初期化することができますデータが含まれている、またはDbNullに

strType = rsData["TYPE"] == DBNull.Value ? "" : rsData["TYPE"].ToString().Trim(); 

Answer 3

がある場合であればの時点で変数がどのような値を持っていないなステートメントので、それは文句を言います。

だけstring strType = "";

Answer 4

このエラーは、以前にその変数を宣言していなかったことを意味します。 whileループの始めにこれらの変数を初期化するだけです。

例：

while (rsData.Read()) 
{ 
    string strType = string.Empty; 
    string strwho = string.Empty; // Do this if you have the same error for strwho 
    string strmetades = string.Empty; // Do this if you have the same error for strmetades 

    // Your other code comes here 
} 

あなたは少し異なるIF文あなたを注文した場合、あなたも変数に空の値の再割り当てを回避することができます。

Answer 5

この理由は、あなたがそれを使用前に、任意の値をするstrType変数を割り当てるされていないということです。 C＃コンパイラの規則によれば、はの変数をに割り当てなければなりません。の値を使用する前に値を割り当ててください。言い換えれば

、何は十分なはずは例えばたわごとのように、consditions前に空の文字列を代入です：

strType = srting.Empty; //at least one value is already assigned! 

while (rsData.Read()) 
{ 
    .... //your code here 
} 

なぜ、この？あいまいさを避け、明確なコード提示ではない。

このの詳細を、dierctlyエリックリペットから小さな記事読み：Why are local variables definitely assigned in unreachable statements?

Answer 6

を使用し、String.Emptyをいいことを。

string strType=String.Empty;