Select запрос через с#, в чем проблема?

Question

[Therapyx]

string Query;
            try
            {              
                if (!string.IsNullOrEmpty(txtSelectByDate1.Text) && !string.IsNullOrEmpty(txtSelectByDate2.Text))                
                     Query = "Select * from Purchase where P_Date >= '"+Convert.ToDateTime(txtSelectByDate1.Text).ToString("yyyy-MM-dd") + 
                     "' and P_Date <= '" + Convert.ToDateTime(txtSelectByDate2.Text).ToString("yyyy-MM-dd") + "'";
                SqlCommand cmd = new SqlCommand(Query, cn);
                SqlDataAdapter sda = new SqlDataAdapter(cmd);
                DataTable dt = new DataTable();
                sda.Fill(dt);
                gvPurchase.DataSourceID = null;
                gvPurchase.DataSource = dt;
            }

В этой строке на Query выходит ошибка с описанием "Use of unassigned local variable 'Query'"
SqlCommand cmd = new SqlCommand(Query, cn);
Если же убрать эту проверку, то все работает прекрасно.

if (!string.IsNullOrEmpty(txtSelectByDate1.Text) && !string.IsNullOrEmpty(txtSelectByDate2.Text))

Почему так выходит? Или какой способ проверки двух текст боксов на наличие информации подошел бы в этом случае?

Answer 1

[ar4ebaldello]

Так зачем вы выполняете запрос, если не инициализируете query?

var dateFromString = txtSelectByDate1.Text;
var dateToString = txtSelectByDate2.Text;

if (!string.IsNullOrEmpty(dateFromString) && !string.IsNullOrEmpty(dateToString))                
{
    var dateFrom = Convert.ToDateTime(dateFromString).ToString("yyyy-MM-dd");
    var dateTo = Convert.ToDateTime(dateToString).ToString("yyyy-MM-dd");
    
    var query = string.Format("Select * from Purchase where P_Date >= '{0}' and P_Date <= '{1}'", dateFrom, dateTo);
    
    var command = new SqlCommand(query, cn);
    var adapter = new SqlDataAdapter(command);
    var table = new DataTable();
    adapter.Fill(table);
    
    gvPurchase.DataSourceID = null;
    gvPurchase.DataSource = table;
}

Comments

[Therapyx]

Понял уже свою ошибку)) Надо в скобки не только Query брать, а и все остальное...) п.с. спасибо за красоту хД Но тут загвоздка такая... Мне таких if проверок надо штуки 3, если только сам string Query изменять, то это еще смотрится ок... Если же дублировать "весь" код 3-4 раза, то получается вообще каша :(

[Therapyx]

Интересно просто даже, почему именно 1 Query нельзя захватить в скобки If'a

[Станислав Макаров]

Therapyx
> почему именно 1 Query нельзя захватить в скобки If'a
потому что компилятору непонятно, что делать с этой переменной и какое значение туда поместить, чтобы выполнить нижеследующий код. Вы-то сами что там ожидали бы увидеть? В случае НЕвыполнения условия в if

> дублировать "весь" код 3-4 раза
зачем? вам надо, чтобы ВСЕ поля были заполнены, или чтобы в зависимости от заполненных полей выполнялся разный запрос?

[Therapyx]

Станислав Макаров: Надо сделать проверки наличия текста в полях. Если 2 поля заполнены, то идет селект между ними(датами), если же заполнено только первое поле, то идет селект только по этой дате, если же заполнено только второе поле, то соответственно именно по этой дате.

[Станислав Макаров]

Therapyx сделайте проще: один запрос с двумя фильтрами (кстати, сделайте его prepared statement, вручную собирать запрос без особых причин - плохая практика) по обеим датам, но если какая-то из дат не заполнена, ставьте ей дефолтовое значение условной бесконечности. Напр., для даты From - значение 01-01-0001, для даты To - 31-12-9999:
var dateFrom = string.IsNullOrEmpty(dateFromString) ? new DateTime(1, 1, 1) : Convert.ToDateTime(dateFromString).ToString("yyyy-MM-dd");
var dateTo = string.IsNullOrEmpty(dateToString) ? new DateTime(9999, 12, 31) : Convert.ToDateTime(dateToString).ToString("yyyy-MM-dd");