Member 13522006 Ответов: 2

Я написал код команды insert для вставки записи в таблицу с помощью SQL server 2008


Imports System.Data.SqlClient
Imports System.Data.SqlClient.SqlCommand
Imports System.IO
Public Class MainWindow
 Dim con As New SqlConnection
    Dim cmd As New SqlCommand
<pre> Private Sub Insert_Click(sender As Object, e As RoutedEventArgs) Handles Insert.Click

        Try
            con.ConnectionString = "Data Source=MARVELL-PC;Initial Catalog=Management;User ID=abcd"
            con.Open()
            cmd.Connection = con
            cmd.CommandText = "insert into student(First Name,Middle Name,Last Nameq,Father Name,Mother Name,Occupation,DOB,Mobile No,Email-id,Address,Nationality)values('" & txtbx_student_name.Text & "','" & txtbx_student_mname.Text & "','" & txtbx_student_lname.Text & "','" & txtbx_student_ftname.Text & "','" & txtbx_student_mtname.Text & "','" & txtbx_student_occp.Text & "','" & txtbx_student_dob.Text & "','" & txtbx_student_mb.Text & "','" & txtbx_student_emid.Text & "','" & txtbx_student_addr.Text & "','" & txtbx_student_nat.Text & "')"
            cmd.ExecuteNonQuery()
            MessageBox.Show("Data Inserted Sucessfully")
        Catch ex As Exception
            MessageBox.Show("Error while inserting a record in a table..." & ex.Message, "Insert Records")
        Finally
            con.Close()
        End Try
    End Sub

What I have tried:

It gives the error message
"Error while inserting a record in a table, Login failed for the user abcd"

Member 13522006

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

2 Ответов

Рейтинг:
2

OriginalGriff

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

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

2) Никогда не используйте жесткие строки подключения кода - если их нужно изменить (а они обычно отличаются для разработки и производства или у вас есть серьезные проблемы, возникающие позже), то вы должны изменить их везде. Используйте конфигурационный файл и извлеките оттуда свою строку.

3) строка подключения неправильная. Вот на что он жалуется - он не может войти в ваш пользователь "abcd" как пользователь SQL этой базы данных с пустым паролем.
Попробовать это: Простое создание строки подключения SQL[^]


Рейтинг:
2

Patrice T

Цитата:
"Ошибка при вставке записи в таблицу не удалось войти в систему для пользователя abcd"

cmd.CommandText = "insert into student(First Name,Middle Name,Last Nameq,Father Name,Mother Name,Occupation,DOB,Mobile No,Email-id,Address,Nationality)values('" & txtbx_student_name.Text & "','" & txtbx_student_mname.Text & "','" & txtbx_student_lname.Text & "','" & txtbx_student_ftname.Text & "','" & txtbx_student_mtname.Text & "','" & txtbx_student_occp.Text & "','" & txtbx_student_dob.Text & "','" & txtbx_student_mb.Text & "','" & txtbx_student_emid.Text & "','" & txtbx_student_addr.Text & "','" & txtbx_student_nat.Text & "')"

Не решение вашего вопроса, но у вас есть еще одна проблема.
Никогда не создавайте SQL-запрос путем объединения строк. Рано или поздно вы сделаете это с помощью пользовательских вводов, и это откроет дверь к уязвимости под названием "SQL-инъекция", она опасна для вашей базы данных и подвержена ошибкам.
Одна кавычка в имени - и ваша программа выйдет из строя. Если пользователь вводит имя, например "Брайан О'Коннер", может привести к сбою вашего приложения, это уязвимость SQL-инъекции, и сбой-это наименьшая из проблем, вредоносный пользовательский ввод, и он продвигается к командам SQL со всеми учетными данными.
SQL-инъекция - Википедия[^]
SQL-инъекция[^]
Атаки SQL-инъекций на примере[^]
PHP: SQL-инъекция - руководство пользователя[^]
Шпаргалка по предотвращению инъекций SQL - OWASP[^]