Ну, по-видимому, вы пытаетесь вставить значение в таблицу, и оно превышает длину столбца. Вы должны проверить данные, прежде чем пытаться выполнить вставку SQL.
Однако вы должны также использовать параметризованный запрос - в настоящее время ваш код уязвим для атаки SQL-инъекцией. Посетите сайт Bobby Tables для получения дополнительной информации. По сути, параметризованный SQL позволяет хранить код (SQL) отдельно от данных (параметров), делая ваш код более понятным и предотвращая атаки с использованием SQL-инъекций (при условии, что вы не также используете динамический SQL).
(Наконец, неясно, почему вы звоните ToString()
в результате вызова Substring
...)