イベントを介して変数を渡すC＃

Question

これはルーキーのように見える場合は申し訳ありません：3 ...私はC＃で多くの経験がありませんが、C＃でコーディングするのが本当に好きです。言語。

私は、ボタンクリックで得られた値でデータベースに挿入できるように、ボタンクリックで変数を渡そうとしています。私はいくつかのボタンを別の文字列に、製品の値をリサイクルする自然言語のアプローチでこれはそうでしょう：

私は特定の "製品"をクリックすると、変数の製品は、これは私が "数量"ボタンをクリックすると、それは私にこれらの値を持つDBへの挿入を行うプログラムに続いて、数量番号を与えるだろう。しかし、私はクリックしたボタンに応じてこの値をリサイクルする方法には疑問があります。要するに

は、私が持っているコードは以下の通りであるデータベースに挿入された後

private String product; 
    private int quantity; 
    private char prodchosen; 
    private char quantitychosen; 

    private void Product_Click(object sender, EventArgs e) 
    { 
     prodchosen = 'Y'; 
     product = "Beer"; 
    } 


    private void Product2_Click(object sender, EventArgs e) 
    { 
     prodchosen = 'Y'; 
     product = "Juice"; 
    } 

    public void bttn1_Click(object sender, EventArgs e) 
    { 
     quantitychosen = 'Y'; 
     quantity = 1; 
     if (prodchosen == 'Y' && quanitychosen == 'Y') 
     { 
     MySqlConnection mcon = new MySqlConnection("server=localhost;database=***;username=root;password=***"); 
     mcon.Open(); 
     string insert = "INSERT INTO accounts product, quantity VALUES" + product + " , " + quantity; 


     MySqlCommand cmd = new MySqlCommand(insert, mcon); 
     MessageBox.Show("Product was succesfully added"); 
    } 
    else { 
     MessageBox.Show("Quantity and product not recognized"); 
    } 

、これらの値はのGridViewに示しされ、SELECT文のようなコンボボックスselectedIndexの値によって区切られます：

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e) 
     { 
     clientchosen = 'Y'; 
     client_id = comboBox1.SelectedIndex; 

     MySqlConnection mcon = new MySqlConnection("server=localhost;database=***;username=root;password=***"); 
     mcon.Open(); 

     char open = 'Y'; 

     int selectedIndex = comboBox1.SelectedIndex; 
     Object selectedItem = comboBox1.SelectedItem; 

     string select2 = "SELECT * FROM account where client_id = " + selectedIndex + " and open = " + open; 

     MySqlCommand cmd2 = new MySqlCommand(select2, mcon); 

     object result = cmd.ExecuteNonQuery(); 

    } 

これまでのところ、私はメッセージボックスを表示しないようにしても、成功のインサートを作ることができていない、私はずっとこの問題についてあなたが持っている任意のコメントを感謝します。

Answer 1

は、私は、データベースに値を挿入し、これが道である。この

SqlConnection dbconnect = new SqlConnection("server=localhost;database=***;username=root;password=***"); 
SqlCommand cmd; 

    try 
    { 
     cmd = new SqlCommand("INSERT INTO accounts(product, quantity) VALUES(@prod, @quan)", dbconnect); 
     cmd.Parameters.AddWithValue("@prod", Your_Text_Box.Text); 
     cmd.Parameters.AddWithValue("@quan", Your_Text_Box.Text); 
     dbconnect.Open(); 
    } 

で試してみてください。うまく動作します。

Answer 2

あなたのSQLは正しくないと思います。列と値の句にはカッコが必要で、文字列の値には引用符が必要です。これを試してみてください：

string insert = "INSERT INTO accounts(product, quantity) " + 
       "VALUES('" + product + "' , " + quantity + ")"; 

Answer 3

他の回答を読んだ後でさらに詳しく説明します。パラメータ化されたクエリは素晴らしいアイデアです。なぜそうなのか見てみましょう。

VALUESの後にスペースがないため、SQLは機能しません。それ以降の製品パラメータは認識されず、挿入物は機能しません。

コマンドを実行することはありません。

cmd.ExecuteNonQuery(); 

あなたが好きな場合は、上記のステートメントからの戻り値を取得すること自由に感じて、それを実行する - それが影響を受けた行数を返します。