Member 13796859 Ответов: 2

Проблема с командой обновления в winform с базой данных

создано insert update delete в winform с базой данных в базе данных sql server
проблема в том, что когда я выбираю мои обновленные данные и нажимаю на кнопку Обновить, он показывает мне это недопустимое имя столбца?
может ли кто нибудь плз помочь мне в решении этой проблемы

Что я уже пробовал:

private void updatebtn_Click(object sender, EventArgs e)
      {
          {
              try
              {
                  if (comboBox1.Text=="")
                  {
                      MessageBox.Show("enter army number to update");
                  }
                  else
                  {
                      SqlCommand cmdupdate = new SqlCommand("Update insupdel SET name='" + textBox1.Text + "',rank='" + comboBox2.SelectedItem + "' ,unit='" + comboBox3.SelectedItem + "', subunit='" + comboBox4.SelectedItem + "', medicalcategory='" + comboBox5.SelectedItem + "', category='" + comboBox6.SelectedItem+ "',datetimepicker1='"+dateTimePicker1.Text+"'  where armynumber=" + comboBox1.SelectedItem + "", con);
                      con.Open();
                      cmdupdate.CommandType = CommandType.Text;
                      cmdupdate.ExecuteNonQuery();
                      MessageBox.Show("Data Updated");
                  }
              }
              catch (Exception ex)
              {
                  MessageBox.Show(ex.Message);
              }
              finally
              {
                  if (con.State == ConnectionState.Open)
                  {
                      con.Close();
                  }
              }
          }
      }

Richard Deeming

Все, что вы хотели знать о SQL-инъекции (но боялись спросить) | Трой Хант[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? | Обмен Стеками Информационной Безопасности[^]
Шпаргалка по параметризации запросов | OWASP[^]

2 Ответов

Рейтинг:
1

Dave Kreskowiak

Google для "атаки SQL-инъекций", чтобы узнать, почему то, что вы делаете, так плохо.

Затем Google для "параметризованных запросов C# SQL", чтобы узнать, как это исправить, и, вероятно, решить вашу проблему в то же время.

Рейтинг:
1

OriginalGriff

Только не это! Никогда не объединяйте строки для построения команды SQL. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Скорее всего, когда вы исправите это на протяжении всего вашего приложения, проблема также исчезнет.