2017-02-23 14 views
0

何かを見落としている可能性があります。私は型int(carRequired)の初期化されていない変数を使用しています。私はIF文の中でこの変数を利用しています(下記のコード)。しかし、変数がまだ使われていないという警告が出ます。はい、私はSQLを埋め込むための良い練習ではないことを認識していますが、私は今のところこのようにするように言われています。割り当てられた値が使用されていない

public bool UpdateDiscrepancyReport() 
    { 
     var errorStatus = false; 
     int carRequired ; 
     carRequired = cbxCarRequired.Checked == false ? 0 : 1; 
     var updateQuery = "my query string ";       
     dbf.OpenConnection(updateQuery); 
     bool updateStatus = dbf.OpenConnection(updateQuery); 
     if (updateStatus) 
     { 
      errorStatus = true; 
     } 
     else 
     { 
     MessageBox.Show(@"Error in updating DR" + [email protected]" ",@"Update Error", MessageBoxButtons.OK, MessageBoxIcon.Error); 
     } 
     return errorStatus; 
    } 
+6

いいえ、決して使用されません。それは割り当てられているが、その割り当ては役に立たない。三項式の結果を代入した後は変数を使用しないので、これらの2行を完全に削除してコードの機能には影響を与えません。 (最初の宣言と代入の後でerrorstatusの値に何も影響を与えないので、 'errorstatus'の宣言と単に' return false; 'を単に取り除くことができます) –

+0

*副作用* 'cbxCarRequired.Checked'からメソッドは' public bool UpdateDiscrepancyReport(){dbf.OpenConnection(updateQuery);}と等しくなります。偽を返します。 } ' –

答えて

1

変数はを使用ことはありません。 に割り当てられていますが、その割り当て後に変数が使用されることはありません。三項演算の結果を代入した後は何もしません。実際には変数はまったく必要ではなく、errorStatus(あなたが投稿した元のコード)はどちらも初期化された後に値を変更するコードで何もしないので、どちらもありません。

あなたが(前にいくつかのより多くの行を追加して、あなたの編集に)投稿したコードがあるため、コンパイラはcarRequiredに変数の宣言と代入が無意味であることを語っている

public bool UpdateDiscrepancyReport() 
    { 
     dbf.OpenConnection(updateQuery); 
     return false; 
    } 

の、完全に同等ですコード内の何にも影響しません。チェックボックスがチェックされているかどうかを判断することは、あなたのコードがその情報に基づいて何か異なったことをし、carRequiredの使用が割り当てられた値に基づいて何もしない場合に意味を持ちます。したがって、代入(および変数宣言)は役に立たない。

+0

私は人々がそれのいくつかの行を通過しないように短いコードを維持しようとしていました。しかし、それは常に良い考えではないと思います。回答が多すぎるとは思わないが、最初の投稿を修正して、そのコードブロックのすべてのコードを反映させる。 – TAdams79

+0

これは 'carRequired'が使用されていないという問題を変更しません。 :-) –

+0

多分私は今日のことを理解していない、私はまだ学んでいます。 「それはcarRequiredが使用されていないという問題を変えない」という意味ですか?チェックボックスがオンになっているかどうかを確認する必要があります。チェックされていない場合は、1または0の値を割り当てる必要があります。たぶん私は自分の質問に答えました。三項演算を使用して、私は既にチェックボックスがオンかどうかに応じてそれぞれ1または0を割り当てています。したがって、実際に変数を作成する必要はありません。 – TAdams79

関連する問題