Dropdownbox.selectedvalue переход на sql комментарий

Question

string ddorder = DropDownList2.SelectedValue; // column 
string ddtype = DropDownList3.SelectedValue; //asc or desc 
String str1 = "Select * from table1 order by("+ddorder+" "+ddtype+")"; 

// есть ошибка, связанная с ddtype, что я делаю неправильно?

SqlCommand cmd = new SqlCommand(str1, con); 
con.Open(); 
cmd.ExecuteNonQuery(); 
SqlDataAdapter da1 = new SqlDataAdapter(); 
da1.SelectCommand = cmd; 
DataSet ds1 = new DataSet(); 
da1.Fill(ds1, DropDownList2.SelectedValue); 
GridView2.DataSource = ds1; 
GridView2.DataBind(); 
con.Close(); 

Answer 1

Насколько я понимаю, вам не нужно использовать ( и ) в порядке п. It's syntax не используется для ( или ).

Например,

order by id desc 

будет работать, но

order by (id desc) 

не будет работать.

Кстати, использовать using statement распоряжаться вашим SqlConnection, SqlCommand и SqlDataAdapter автоматически вместо вызова метода Close вручную.

Также вам не нужна часть cmd.ExecuteNonQuery(); для инструкции SELECT. Это не нужно, так как это только выполните ваш запрос выбора. Он ничего не делает или не возвращает.

Несколько вещей больше;

• Измените свой table1 на что-то значимое.
• Не использовать SELECT *. It's quite bad.

Answer 2

Снимите скобку в "порядке" пункта:

String str1 = "Select * from table1 order by "+ddorder+" "+ddtype; 

Answer 3

Использование Dynamic Query:

Изменить здесь:

string ddorder = DropDownList2.SelectedValue; // column 
string ddtype = DropDownList3.SelectedValue; //asc or desc 
String str1 = "exec(Select * from table1 order by "+ddorder+" "+ddtype+")"; 

и

SqlCommand cmd = new SqlCommand(str1, con); 
con.Open(); 
cmd.ExecuteNonQuery(); 
SqlDataAdapter da1 = new SqlDataAdapter(); 
da1.SelectCommand = cmd; 
DataSet ds1 = new DataSet(); 
da1.Fill(ds1); 
GridView2.DataSource = ds1; 
GridView2.DataBind(); 
con.Close();