Удалить строку SQL и изменить ID

Question

Я создаю инструмент C#, который подключается к базе данных SQL для управления пользователями для другой программы, которую я создал. Я показываю Пользователей в ListBox, и я могу добавлять/удалять пользователей. После того, как я удалил некоторые профили, все стало странно, и я подумал, что это может иметь какое-то отношение к тому, как я удаляю строку из базы данных. Я использую Id, который автоматически увеличивается с каждым новым созданием пользователя. Это мой код для удаления из базы данных:

using (conn = new SqlConnection(connectionString)) 
using (SqlCommand command = new SqlCommand("DELETE FROM myDatabase WHERE Id = '" + (listBox1.SelectedIndex + 1) + "'", conn)) 
{ 
    conn.Open(); 
    command.ExecuteNonQuery(); 
} 

и это, как я загружаю пользователь в мой ListBox:

using (conn = new SqlConnection(connectionString)) 
using (SqlDataAdapter sqlAdapt = new SqlDataAdapter("SELECT * FROM myDatabase", conn)) 
{ 
    DataTable dataTable = new DataTable(); 
    sqlAdapt.Fill(dataTable); 
    listBox1.DisplayMember = "Name"; 
    listBox1.ValueMember = "Id"; 
    listBox1.DataSource = dataTable; 
} 

Как я могу удалить правильную строку из базы данных?

Answer 1

Вы должны использовать свойство SelectedValue найти свой идентификатор не SelectedIndex

if(listBox1.SelectedValue != null) 
{ 
    int userID = Convert.ToInt32(listBox1.SelectedValue); 
    using (conn = new SqlConnection(connectionString)) 
    using (SqlCommand command = new SqlCommand("DELETE FROM myDatabase WHERE Id = @uid", conn)) 
    { 

     conn.Open(); 
     command.Parameters.Add("@uid", MySqlDbType.Int32).Value = userID; 
     command.ExecuteNonQuery(); 
    } 
} 

Проблема SelectedIndex в том, что это значение от 0 до максимального числа элементов в ListBox. Это значение не имеет никакого отношения к идентификатору пользователя, который автоматически рассчитывается вашей базой данных. (После всего одного удаления и одного добавить эти значения не синхронизированы)

Обратите также внимание, что текст sql никогда не должен строиться с использованием конкатенации строк. Это хорошо известная проблема безопасности, которая называется Sql Injection.

Answer 2

"DELETE FROM myDatabase WHERE Id = '" + (listBox1.SelectedIndex + 1) + "'" 

Я не уверен в mySQL, но похоже, что вы передаете строку вместо int. Когда вы передаете параметр в виде числа, вы должны удалить «». так, это будет выглядеть так:

"DELETE FROM myDatabase WHERE Id = " + (listBox1.SelectedValue)