SQL Server Вставить запрос

Question

Я задаюсь вопросом, почему я получаю исключение на SqlDataReader reader2 = theCommandInsert.ExecuteReader();

на это код, я использую:

var conString = ConfigurationManager.ConnectionStrings["LocalSqlServer"]; 
string strConnString = conString.ConnectionString; 

SqlConnection dbConnection = new SqlConnection(strConnString); 

dynamic queryString = ("INSERT INTO FOLDERS (Name) VALUES ('" + txtBoxFolderLabel.Text + "') "); 

int param = CheckBoxList2.SelectedIndex; 
param = param + 1; 

dynamic queryStringInsert = ("INSERT INTO GROUPS_FOLDERS (Folder_Id, Group_Id) VALUES(IDENT_CURRENT('Folders') , " + param + ")"); 
SqlCommand theCommand = new SqlCommand(queryString, dbConnection); 
SqlCommand theCommandInsert = new SqlCommand(queryStringInsert, dbConnection); 

//Connection opening and executing 
if (string.IsNullOrEmpty(txtBoxFolderLabel.Text) | CheckBoxList2.SelectedIndex.ToString() == null) 
{ 
    Response.Write("Empty fields !"); 
} 
else if (Functions.IsNumeric(txtBoxFolderLabel.Text)) 
{ 
    Response.Write("No numerics !"); 
} 
else 
{ 
    dbConnection.Open(); 
    SqlDataReader reader = theCommand.ExecuteReader(); 

    string folderName = txtBoxFolderLabel.Text; 

    // Create folder 
    System.IO.Directory.CreateDirectory("C://inetpub//wwwroot//Files//" + folderName); 

    dbConnection.Close(); 

    dbConnection.Open(); 
    SqlDataReader reader2 = theCommandInsert.ExecuteReader(); 
    dbConnection.Close(); 

Это исключение, которое я получаю:

Заявление INSERT противоречило ограничениям FOREIGN KEY «FK_GROUPS_FOLDERS_Groups». Конфликт произошел в базе данных "9B15719DF48C3E2301D7F965674A6F93_VISUAL STUDIO 2010 \ Projects \ CLIENTPORTAL \ APPLICATIONUI \ ВЕБСАЙТОВ \ CLIENTPORTAL \ App_Data \ DATAUI.MDF", стол "dbo.Groups", столбец 'Id'.

Я проверить в базе данных, нет нет Id дублированный или что-то ..

Не могли бы вы мне помочь, пожалуйста?

Cheers.

Answer 1

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

Вы также настраиваете себя, чтобы позволить кому-либо выполнить атаку SQL Injection на вашем сайте.

string queryString = ("INSERT INTO FOLDERS (Name) VALUES ('" + txtBoxFolderLabel.Text + "') "); 

Неплохая идея, вы должны использовать параметры Sql.

string queryString = ("INSERT INTO FOLDERS (Name) VALUES (@folder) "); 

Вы можете определить параметр после создания команды

theCommand.Parameters.AddWithValue("@folder", txtBoxFolderLabel.Text); 

Что касается конкретной ошибки, похоже, у вас есть внешний ключ, определенный на этой таблице, и вы не отвечают требованиям норм внешний ключ

Answer 2

Это не проблема с дублирующимся идентификатором; строке, которую вы пытаетесь вставить в Group_Folders, предоставляется GroupId, который не существует в базе данных. Похоже, что

int param = CheckBoxList2.SelectedIndex; 
param = param + 1; 

на самом деле не правильно генерирует желаемый GroupId. Попробуйте оценить значение param и убедитесь, что это действительно то, что вы хотите.

Answer 3

У вас есть риск атаки SQL-инъекций. Вам нужно использовать параметры для передачи значений из текстового поля, а не только конкатенации строк.

Есть ли причина, по которой ваши строки запросов являются динамическими типами?

Проблема связана с нарушением внешнего ключа, а не с первичным ключом или нарушением уникальности. Кажется, что ваш внешний ключ FK_GROUPS_FOLDERS_Groups не позволяет дублировать одно из двух полей, которые вы вставляете.

Answer 4

Речь идет не о дублированном идентификаторе, а об ограничении внешнего ключа. Порекомендовал бы вам посмотреть таблицу GROUPS_FOLDERS, и вы, вероятно, увидите ограничение FK на Group_Id, указывающее на таблицу ваших групп. Убедитесь, что значение, которое вы пытаетесь вставить для param, является существующим значением в Groups.Id

Answer 5

1. Вы пытаетесь вставить значение в базу данных, которая не существует в родительской таблице. Он приходит из-за внешним ключ ограничения на столе 2.

Пожалуйста, используйте параметризованный запрос, чтобы избежать инъекций SQL & Используйте параметров, чтобы связать свои ценности (это сделает ваш код более читаемым & безопасно)