web-dev-qa-db-ja.com

SQL Serverテーブルから行を削除する

ボタンイベントを使用して、SQL Serverデータベーステーブルから行全体を単に削除しようとしています。これまでのところ、私の試みはどれも成功していません。これは私がやろうとしていることです：

public static void deleteRow(string table, string columnName, string IDNumber)
{
    try
    {
    using (SqlConnection con = new SqlConnection(Global.connectionString))
    {
         con.Open();
         using (SqlCommand command = new SqlCommand("DELETE FROM " + table + " WHERE " + columnName + " = " + IDNumber, con))
         {
               command.ExecuteNonQuery();
         }
         con.Close();
    }
    }
    catch (SystemException ex)
       {
       MessageBox.Show(string.Format("An error occurred: {0}", ex.Message));
       }
    }
}

エラーが発生し続ける：

タイプSystem.Data.SqlClient.SqlExceptionの最初のチャンス例外がSystem.Data.dllで発生しましたエラーが発生しました：オペランドタイプの衝突：テキストはintと互換性がありません

テーブルのすべての列はTEXTタイプです。タイプstringの関数引数を列と比較して一致を見つけることができないのはなぜですか？ （そして、行を削除しますか？）

c#sql-serverdelete-row
6
ikathegreat

すべての列名はTEXT型であると述べたので、IDNumberを囲む単一引用符を使用して、IDNumberをテキストとして使用する必要があります。

    public static void deleteRow(string table, string columnName, string IDNumber)
    {
    try
    {
    using (SqlConnection con = new SqlConnection(Global.connectionString))
    {
         con.Open();
         using (SqlCommand command = new SqlCommand("DELETE FROM " + table + " WHERE " + columnName + " = '" + IDNumber+"'", con))
         {
               command.ExecuteNonQuery();
         }
         con.Close();
    }
    }
    catch (SystemException ex)
       {
       MessageBox.Show(string.Format("An error occurred: {0}", ex.Message));
       }
    }
 }
17
user1082916

IDNumberはintではなくstringにする必要があります。または、実際にstringである場合は、引用符を追加します。

さらに良いのは、パラメーターを使用することです。

6
Jon B

パラメータで試す

.....................
.....................

    using (SqlCommand command = new SqlCommand("DELETE FROM " + table + " WHERE " + columnName + " = " + @IDNumber, con))
             {
                   command.Paramter.Add("@IDNumber",IDNumber)
                   command.ExecuteNonQuery();
             }

.....................
.....................

ステートメントを使用して接続を閉じる必要はありません

3
Ali

"columnName"タイプをTEXTからVARCHAR(MAX). TEXTに変更できます。列は"="では使用できません。
このトピックを参照

1
lmatt

IDNumberは文字列のようです。単一引用符で囲む必要があります。

"DELETE FROM " + table + " WHERE " + columnName + " = '" + IDNumber + "'"
1
sam yi
private void DeleteProductButton_Click(object sender, EventArgs e)
{


    string ProductID = deleteProductButton.Text;
    if (string.IsNullOrEmpty(ProductID))
    {
        MessageBox.Show("Please enter valid ProductID");
        deleteProductButton.Focus();
    }
    try
    {
        string SelectDelete = "Delete from Products where ProductID=" + deleteProductButton.Text;
        SqlCommand command = new SqlCommand(SelectDelete, Conn);
        command.CommandType = CommandType.Text;
        command.CommandTimeout = 15;

        DialogResult comfirmDelete = MessageBox.Show("Are you sure you want to delete this record?");
        if (comfirmDelete == DialogResult.No)
        {
            return;
        }
    }
    catch (Exception Ex)
    {
        MessageBox.Show(Ex.Message);

    }
}
0
Barbi Angel Tupaz
private void button4_Click(object sender, EventArgs e)
{
    String st = "DELETE FROM supplier WHERE supplier_id =" + textBox1.Text;

    SqlCommand sqlcom = new SqlCommand(st, myConnection);
    try
    {
        sqlcom.ExecuteNonQuery();
        MessageBox.Show("delete successful");
    }
    catch (SqlException ex)
    {
        MessageBox.Show(ex.Message);
    }
}



private void button6_Click(object sender, EventArgs e)
{
    String st = "SELECT * FROM suppliers";

    SqlCommand sqlcom = new SqlCommand(st, myConnection);
    try
    {
        sqlcom.ExecuteNonQuery();
        SqlDataReader reader = sqlcom.ExecuteReader();
        DataTable datatable = new DataTable();
        datatable.Load(reader);
        dataGridView1.DataSource = datatable;
    }
    catch (SqlException ex)
    {
        MessageBox.Show(ex.Message);
    }
}
0
robin

MySql Wampを使用している場合。このコードは機能します。

string con="SERVER=localhost; user id=root; password=; database=dbname";
public void delete()
{
try
{
MySqlConnection connect = new MySqlConnection(con);
MySqlDataAdapter da = new MySqlDataAdapter();
connect.Open();
da.DeleteCommand = new MySqlCommand("DELETE FROM table WHERE ID='" + ID.Text + "'", connect);
da.DeleteCommand.ExecuteNonQuery();

MessageBox.Show("Successfully Deleted");
}
catch(Exception e)
{
MessageBox.Show(e.Message);
}
}
0
Miguel