"割り当てられていない変数の使用"エラー

私はこのWebサイトをASP.NETで開発中で、C＃を使用しています。私はエラーを取得しています：割り当てられていない変数usnの使用。データベースも空ではありません。私のコードは次のとおりです。"割り当てられていない変数の使用"エラー

protected void Button1_Click(object sender, EventArgs e) 
{ 

    SqlConnection cn = new SqlConnection(); 
    SqlCommand cm = new SqlCommand(); 
    SqlDataReader dr; 
    cn.ConnectionString = @"Data Source=.\SQLEXPRESS;AttachDbFilename=C:\Users\Vijaylaxmi\Desktop\TrainReserveold\App_Data\Database.mdf;Integrated Security=True;User Instance=True"; 
    cn.Open(); 
    cm.Connection = cn; 
    String usn; 
    cm.CommandText = "Select UserName from User where UserName='" + TextBox1.Text + "'"; 
    dr = cm.ExecuteReader(); 
    while (dr.Read()) 
    { 
     usn = dr.GetString(0); 
    } 
    if (String.Compare(usn, TextBox1.Text) != 0) 
    { 
     Response.Write("Invalid user name... try again"); 
     TextBox1.Text = ""; 
     TextBox2.Text = ""; 
     TextBox1.Focus(); 
    } 
    Response.Write("user valid now"); 
}

出典

2011-12-21 VijayLaxmi

'using'ステートメントで' SqlConnection'、 'SqlCommand'と' SqlDataReader'の作成をラップする必要があります。 – Oded

あなたは失礼です。できるだけ速やかに？私はこれのために支払われていません、あなたは知っています。そして、これで私は答えにすべての関心を失いました。 – Oded

また、接続とコマンドを作成する方法を書き直すことになります。あなたが持っているものをフォローするのが簡単になる例を投稿します。 – MethodMan

は、私はまた、アプリケーションの種類に応じて、web.configファイルまたはapp.configをからの私のSQL connectiong文字列を読んでいました

string usn = string.empty; then go from there 
//create a Stored Procedure and put your Select Statement in there.. to avoid Sql Injection 
cmd.CommandText = "name of your stored proc"; 
cmd.CommandType = System.Data.CommandType.StoredProcedure;

としてトップをUSNの文字列をASSINGます走っている。

出典

2011-12-21 17:09:25 MethodMan

DBに他のユーザがいなくても空のユーザ名はおそらく無効なので、おそらくNULLを割り当てるほうがよいでしょう。 –

Emptyはちょっとイニシャライザです。私は本当に、彼がこのようなものを使ってリファクタリングすることを推奨し、次にコマンドオブジェクトを使用して作成/割り当てます（SqlConnection sqlConnSqlConnection =新しいSqlConnection（strConnectionString））{} – MethodMan

すべてのコードはひどく匂いがする。しかし、私はローマが一日で建てられなかったと思う。 MOREバグを導入する理由はありません:) –

変更あなたのcm.CommandText =「ユーザーからどこのUserName = 私はここを参照してください

cm.CommandText = string.Format("Select UserName from User where UserName= '{0}'",Textbox1.Text);

出典

2011-12-21 17:13:50 MethodMan

これはSQLインジェクションを防ぐものではありません。 –

はい、いつでもストアドプロシージャでsqlコマンドを提供するように変更できます。私は気付いていますが、良い点です。 – MethodMan

ストアドプロシージャはインジェクションを防止しません。コマンドのパラメータは次のとおりです。 –

いくつかの問題にユーザー名を選択して、あなたの質問に特定の応答では、あなたがこの置き換えたい：。

dr = cm.ExecuteReader(); 
while(dr.Read()) 
{ 
    usn = dr.GetString(0); 
}

をこれで

：

usn = cm.ExecuteScalar().ToString();

はDBNulのために確認してください最初に、ちょうどの場合。

さらに一般的には、
とします。a）生の入力を使用する代わりに、SQLをパラメータ化します（または、ストアドプロシージャを使用することをお勧めします）。これにより、SQLインジェクション攻撃から保護されます。
b）コードに直接接続文字列を含めないでください。それを設定ファイルに入れてください。最も確かにそれをインターネットに掲載しないでください。

出典

2011-12-21 17:14:33 AllenG

"割り当てられていない変数の使用"エラー

答えて

関連する問題