Member 14855516 Ответов: 1

У меня возникли некоторые проблемы с моим кодом

Привет, у меня возникла ошибка с кодом обновления моей базы данных.Ошибка гласит: "необходимо объявить скалярную переменную @numara"
Вот код, с которым у меня была ошибка (этот код принадлежит моей форме.cs):
IslemVeritabani ekleme = new IslemVeritabani();<
ekleme.GuncelleIslem("update tbl_temeldb  set ogrenci_adi='" + ad.Text + "',ogrenci_soyadi='" + soyad.Text + "',ders1='" + dersler.SelectedItem + "',ders2='" + dersler.SelectedItem + "',ders3='" + dersler.SelectedItem + "',ders4='" + dersler.SelectedItem + "',ders5='" + dersler.SelectedItem + "',ilk_dersin_ogretmeni='" + ogretmenler.SelectedItem + "',ikinci_dersin_ogretmeni='" + ogretmenler2.SelectedItem + "',ucuncu_dersin_ogretmeni='" + ogretmenler3.SelectedItem + "',dorduncu_dersin_ogretmeni='" + ogretmenler4.SelectedItem + "',besinci_dersin_ogretmeni='" + ogretmenler5.SelectedItem + "',notu='" + int.Parse(not.Text) + "'where id=@numara");
           ekleme.komut.Parameters.AddWithValue("@numara", dataGridView1.CurrentRow.Cells[0].Value.ToString());
            listeleme();
            MessageBox.Show("Kayıt Başarıyla Güncellenmiştir.");


Вот код проблемы(компилятор показывает ошибку в этом разделе кода) :
class IslemVeritabani : TemelVeritabani
public void GuncelleIslem(string sorgu)
{
    Baglan();
    SqlCommand cmd = new SqlCommand(sorgu);
    cmd.Connection = baglanti;
    cmd.ExecuteNonQuery();
    baglanti.Close();
    baglanti.Dispose();
}


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

Я перепробовал все что знал но не смог решить эту проблему

1 Ответов

Рейтинг:
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;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

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

И параметр, который вы используете - @numara - должен быть добавлен до вызова ExecuteNonQuery, а не после - вот почему вы получаете ошибку.

Не пытайтесь создать метод для выполнения работы с БД - создайте соединение внутри БД. using блок, создайте объект команды внутри другого using блокируйте и всегда используйте параметризованные запросы. Попытка получить "универсальный" метод для работы с параметрами - это Пита, поэтому используйте встроенный код!

OriginalGriff

Нет, ты не знал.
Вы вызываете GuncelleIslem и передаете ему объединенную командную строку SQL, а он вызывает ExecuteNonQuery.

публичная пустота GuncelleIslem(строка sorgu)
{
Баглан();
SqlCommand cmd = новая команда SqlCommand(sorgu);
УМК.Подключение = baglanti;
УМК.Метод executenonquery();
baglanti.Закрывать();
baglanti.Располагать();
}

Когда он возвращается, вы добавляете значение параметра, а затем вызываете listeleme

Member 14855516

Привет спасибо, но я все еще не понимаю и чувствую себя дураком, не могли бы вы просто написать правильный код для меня, который нужно изменить, пожалуйста ?

OriginalGriff

Прочтите то, что я сказал выше: никогда не объединяйте строки для построения команды; всегда используйте параметризованные запросы. Полностью сбросить способ Guncellelslem, и встроенные в код с "помощью" блоков. Передайте все значения в качестве параметров

Кроме того, проверьте вводимые пользователем данные с помощью int.TryParse и т. д., и передайте преобразованное значение в качестве параметра только в том случае, если все они верны.

Member 14855516

Мне очень жаль, но я не знаю, как использовать блоки "using", я даже не знаю, является ли это кодом или нет.Не могли бы вы просто написать его для меня, если это не слишком долго.Я только понимаю, что мне нужно полностью удалить GuncelleIslem, вот и все.

Member 14855516

Я не использовал встроенный код, потому что мой учитель хотел, чтобы мы вызывали методы обновления из класса

OriginalGriff

Если вы не знаете, как использовать что-то, google это!
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/using-statement

Member 14855516

Хорошо, я поставил этот код перед командой обновления: ekleme.komut.Параметры.AddWithValue ("@numara", dataGridView1.CurrentRow.Ячейки[0].Значение.ToString()); теперь я получаю ошибку "ССЫЛКА на объект не установлена на экземпляр объекта"