0
 public void MyExecuteNonQuery(string SqlText)
    {
        SqlConnection cn; // экземпляр класса типа SqlConnection
        SqlCommand cmd;

        // выделение памяти с инициализацией строки соединения с базой данных
        cn = new SqlConnection(ConnStr);
        cn.Open(); // открыть источник данных
        cmd = cn.CreateCommand(); // задать SQL-команду
        cmd.CommandText = SqlText; // задать командную строку
        cmd.ExecuteNonQuery(); // выполнить SQL-команду
        cn.Close(); // закрыть источник данных
    }
    private void FillSource()
    {
        string SqlText = "SELECT * FROM [Books]";
        SqlDataAdapter da = new SqlDataAdapter(SqlText, ConnStr);
        DataSet ds = new DataSet();
        da.Fill(ds, "[Books]");
        dataGridView.DataSource = ds.Tables["[Books]"].DefaultView;
    }
    private void metroTile1_Click(object sender, EventArgs e)
    {
        string SqlText = "INSERT INTO [Books] ([Id],[Name],[Aftor],[Ganr],[Location]) VALUES (1, 'Name-01','Aftor-01','Ganr-01','Location-01')";
        frmAddEditBooks f = new frmAddEditBooks(); // создать экземпляр окна

        if (f.ShowDialog() == DialogResult.OK)
        {
            // сформировать SQL-строку
            SqlText = "INSERT INTO [Books] ([Name],[Aftor],[Ganr],[Location]) VALUES (";
            SqlText = SqlText + "\'" + f.txtName.Text + "\', ";
            SqlText = SqlText + "\'" + f.txtAftor.Text + "\', ";
            SqlText = SqlText + "\'" + f.cboJanr.Text + "\', ";
            SqlText = SqlText + "\'" + f.txtLocation.Text + "\')";

            // выполнить SQL-команду
            MyExecuteNonQuery(SqlText);
            // отобразить таблицу Source
            FillSource();
        }
    }

выдает ошибку System.Data.SqlClient.SqlException: "String or binary data would be truncated. The statement has been terminated."

1
  • Посмотрите такой пример, как лучше писать работу с БД.
    – Bulson
    10 дек 2019 в 18:43

1 ответ 1

2

Ошибка

System.Data.SqlClient.SqlException: "String or binary data would be truncated. The statement has been terminated."

говорит о том, что данные, которые вы передаете в параметрах Insert имеют большую длину, чем задана для колонки в БД.

Не в даваясь в качество кода приведенного фрагмента, запредельную уязвимость для SQL-инъекций и прочее, вариантов у вас два:

  1. (предпочтительный вариант) Посмотреть какой размер имеют колонки в базе и предварительно обрезать данные на клиенте. Например так:

    ... + f.txtName.Text.Substring(0,<размер колонки>) + ...
    
  2. (нежелательный вариант) Оставить код как есть, а в базе задать колонкам максимальный размер - varchar(max)|nvarchar(max), ну или совсем устаревший вариант - тип text.


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

1
  • Больше Спасибо!!
    – mur0wilja
    11 дек 2019 в 5:48

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.