OriginalGriff
Тебе не нужен гипс: DateTime.Now
уже возвращает DateTime
значение, поэтому приведение является избыточным.
Сообщение об ошибке относится к символьной строке:
Conversion failed when converting date and/or time from Character string.
Это означает, что вы передаете значение datetime в БД строк конкатенация :
string sql = "UPDATE MyTable SET Updated_At='" + _updated_at + "'";
И это то, что вызывает вашу проблему, потому что строка, которую вы передаете, неправильно интерпретируется SQL, и вы должны быть рады, что это так, потому что есть вероятность, что она работала бы два дня назад, но хранила неправильные значения в вашей БД.
Никогда не объединяйте строки для построения команды 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;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.
Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
Исправьте это во всем вашем приложении, и проблема, которую вы заметили, исчезнет в то же самое время.