Как решить проблему конверта DateTime при селекте из Gridview?

Question

[Therapyx]

Такая проблемка. Есть GridView, кнопка и текстовое поле. Задача: вывести все записи в GridView из базы по заданной дате в тестовом поле.

Код по нажатию кнопки:
            DateTime datetime = Convert.ToDateTime(TextBox1.Text);
            SqlCommand cmd = new SqlCommand("Select * from Purchase where P_Date = '" + datetime + "'", cn);
            SqlDataAdapter sda = new SqlDataAdapter(cmd);
            DataTable dt = new DataTable();
            sda.Fill(dt);
            gvPurchase.DataSourceID = null;
            gvPurchase.DataSource = dt;
            gvPurchase.DataBind();

для теста еще создал лейбл, если ввожу в TextBox1 дату, то лейбл показывает "17.07.2015 00:00:00"

А вот если же пытаюсь делать селект из базы с SqlCommand, то получаю следующую ошибку:

Conversion failed when converting date and/or time from character string.
возникает на: Line 100: sda.Fill(dt);

Очень не люблю эти танцы с бубном из-за DateTim'ов. Как решить эту проблему в моем случае? И если у кого-нибудь есть хорошая статья по поводу таких проблем, тоже буду рад ссылочке )

---(Ответы в комментах)---

Answer 1

[Дмитрий Ковальский]

Устал уже если честно видеть подобный код. Ваши танцы с бубнами связаны с тем что вы не имеете представления о грамотной работе с базой данных. Конкатенация SQL-запроса в обработчике события в code-behind - это решение человека, работающего с технологией минут 20. Решений - гора. 1) Использование ORM(использование Entity Framework с подходом DB First избавляет от приведения типов и возможности сделать инъекцию) 2) Использование хранимых процедур. 3) использование параметризованных запросов. Парадокс в том что если вы в запросе используете, например, хранимую процедуру, то вы можете использовать код вида cmd.Parameters.AddIDateTime(sName, val) - и приводить типы не надо.

Comments

[Therapyx]

Про инжекшен я знаю, для всех тестов я делаю для начала так, после чего переделываю конечный вариант в процедуру(не спрашивайте зачем, это экономит мне время).
а по типу такого ----
if (stxt_date.Length > 0)
cmd.Parameters.Add("@p_date", SqlDbType.DateTime).Value = Convert.ToDateTime(stxt_date);
так и делаю, но это на апдейт же. Мне всего-лишь надо сделать селект из гридвью по заданной дате.

[Дмитрий Ковальский]

Therapyx: Ради костыля попробуйте datetime.toString("yyyy-MM-dd HH:mm")

[Therapyx]

Дмитрий Ковальский: Во, спасибо) Понял где я "дотанцевался". Да уж, вот на таких мелочах больше всего времени теряю. Называется - приходилось учится методом тыка.

[Therapyx]

Дмитрий Ковальский: datetime.ToString("yyyy-MM-dd") так тоже пашет )

[Дмитрий Ковальский]

Therapyx: Ну я написал полностью так как не знаю актуально ли время.

[Therapyx]

Дмитрий Ковальский: я так понял. Создаю переменную datetime, конвертирую текст из текстбокса в дейт. На этом этапе datetime имеет формат точно такой же как и написан в текст боксе. Т.е. 17.07.2015. После чего я пытался сделать селект из точно такого же формата, притом, что в SQL формат по дефолту стоит yyyy-mm-dd и из-за этого выскакивала постоянно ошибка.
Но вот тут недопонимание. А почему когда я в текстбокс пишу изначально форматом yyyy-mm-dd, то в datetime сохраняется все-равно как dd.mm.yyyy?

[Дмитрий Ковальский]

Therapyx: Я не специалист по реализации dateTime но внутри могут храниться вообще только тики после некого момента времени. То о чем вы говорите - настройки локали интерфейса на вашей машине. dd.mm.yyyy - Российский стандарт представления даты. В США бы вывелось иначе. Вам нужно в базу подать в строгом формате, который понимает БД. Потому и приводим к выше описанному виду

Answer 2

[Артур Полозов]

При необходимости использовать такой подход, приводи дату к формату ISO = 'yyyy-mm-ddThh:mm:ss'.
в скулевом конверте - 121.

попробуй так:
Convert.ToDateTime(TextBox1.Text).ToString("yyyy-MM-dd hh:mm:ss"), или Convert.ToDateTime(TextBox1.Text).ToString("yyyy-MM-dd") - если нужен только день.

upd. не заметил, что ответили выше. сори.

Comments

[Therapyx]

Ошибка выскакивает, если делаю так))
DateTime datetime = Convert.ToDateTime(TextBox1.Text).ToString("yyyy-mm-dd");

[Артур Полозов]

Therapyx: возможно yyyy-MM-dd надо.

[Therapyx]

Артур Полозов: все равно ошибка) пишет cannot implicitly convert type 'string' to 'System.DateTime'
А так можно чтоли? ) Получается ты создаешь тип datetime, конвертируешь стрингговый текстбокс в дейттайм и тут же конвертируешь его снова в стринг, который пытается попасть в DateTime datetime. Я думаю ошибка где-то тут) хотя могу ошибаться

[Артур Полозов]

Therapyx:
да, не посмотрел на тип, предполагал вот так:
SqlCommand cmd = new SqlCommand("Select * from Purchase where P_Date = '" + Convert.ToDateTime(TextBox1.Text).ToString("yyyy-MM-dd")+ "'", cn);

[Therapyx]

Артур Полозов: А, так то вообще шикарно :)