bensoccer94 Ответов: 2

Проблема C# в visual studio.

Я использую visual studio 2019 и пытаюсь вставить запись в локальную базу данных, созданную в visual studio. во время этого процесса я получаю эту ошибку

System.Data.SqlClient.SqlException: 'преобразование не удалось при преобразовании значения varchar 'string' в тип данных int.'

он говорит мне, что эта строка кода-неправильный cmd.Метод executenonquery();
но я тоже не понимаю что нужно менять

это мой код.
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Data.SqlClient;

namespace EmployeeDataBaseProject
{
    public partial class Form1 : Form
    {

        SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename=C:\Users\Owner\Documents\Employee.mdf;Integrated Security=True;Connect Timeout=30");
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void button2_Click(object sender, EventArgs e)
        {
            con.Open();
            SqlCommand cmd = new SqlCommand("INSERT INTO Employee_Table VALUES('" + textBox1.Text + "',' " + textBox2.Text + "','" + textBox3.Text + "','" + textBox4.Text + "','" + comboBox1.Text + "')", con);
            cmd.ExecuteNonQuery();
            MessageBox.Show("Inserted Data Successfully");
            con.Close();




        }
        

    }
}


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

Я не понимаю, как это исправить

Sinisa Hajnal

1) никогда не объединяйте пользовательский ввод (google SQL Injection)
2) проверьте, что все данные содержат правильный тип данных (это помогло бы, если бы у нас были имена столбцов или полезные имена элементов управления txtEmploymentDate или txtSalary или что-то еще)

F-ES Sitecore

Вы помещаете свои значения в кавычки, что указывает на то, что они являются текстовыми значениями, однако одно из полей в вашей таблице ожидает int. Как уже говорили другие, переключитесь на использование параметризованных запросов, это сделает код более легким для отслеживания и более надежным. Для поля, которое нуждается в int, вам нужно будет сначала преобразовать текстовое значение в int vlue с помощью int.Метод tryparse

Richard Deeming

В дополнение к этому SQL-инъекция[^] уязвимость в вашем коде, вы не указали столбцы для INSERT заявление.

Всегда указывайте имена столбцов, в которые вы вставляете:

INSERT INTO Employee_Table (Column1, Column2, Column3, Column4, Column5) VALUES (@V1, @V2, @V3, @V4, @V5)

(Очевидно, что вам нужно будет использовать правильные имена столбцов для вашей таблицы.)

2 Ответов

Рейтинг:
2

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

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

И никогда не доверяйте пользователям вводить номера. Проверьте их входные данные, преобразовав входные данные целочисленных значений в целые числа с помощью TryParse, а затем передайте преобразованное значение через параметризованный запрос:
int value;
if (!int.TryParse(MyTextbox.Text, out value))
   {
   ... report problem to user ...
   return;
   }

Рейтинг:
2

MadMyche

Перво - наперво- ваш код восприимчив к SQL-инъекция.
NEVER EVER объедините команды SQL и переменные для формирования запроса.

Рекомендуемый способ использования переменных и команд заключается в использовании Объект sqlparameter[^] класс для добавления переменных в объект команды.

Примененный к вашему коду даст нам нечто похожее на это

private void button2_Click(object sender, EventArgs e)
{
	string Tx1 = textbox1.Text;
	string Tx2 = textbox2.Text;
	string Tx3 = textbox3.Text;
	string CB1 = comboBox1.Text;
	
	con.Open();
	string qry = "INSERT INTO Employee_Table VALUES (@Tx1, @Tx2, @Tx3, @CB1)";
	SqlCommand cmd = new SqlCommand(qry, con);

	cmd.Parameters.AddWithValue("@Tx1", Tx1);
	cmd.Parameters.AddWithValue("@Tx2", Tx2);
	cmd.Parameters.AddWithValue("@Tx3", Tx3);
	cmd.Parameters.AddWithValue("@CB1", CB1);

	cmd.ExecuteNonQuery();

	MessageBox.Show("Inserted Data Successfully");

	con.Close();
}
Итак, теперь, когда мы позаботились об этом, давайте посмотрим на вашу ошибку
Системы.Данных.Sqlclient как.Sqlexception в
'Ошибка преобразования строки В тип varchar значение '' в тип данных int'.
Это довольно понятно - вы пытаетесь поместить строковое значение в поле базы данных, которое определено так, чтобы принимать только целые числа.

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

Что бы я сделал, так это установил точка прерывания ВКЛ сразу после того, как 4 переменные будут заполнены при открытии скрипта, и запустите это в режиме отладки. Когда вы дойдете до этой точки, проверьте, какие значения назначены вашим текстовым и флажкам; и сравните это с типами данных в базе данных.

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

Ссылка:
Класс SqlParameter (System.Data.SqlClient) | Microsoft Docs[^]