Hiall,
here is my code it works fine this way:-

If ListBox1.SelectedIndex > 0 Then

Dim dr As SqlDataReader

Dim cmd As New SqlCommand

Dim sql As String = "select name,price,[open] from lotterysale where Name = '" & ListBox1.SelectedValue & "'"

cmd.CommandText = sql

cmd.CommandType = CommandType.Text

cmd.Connection = con

dr = cmd.ExecuteReader

dr.Read()

If dr.HasRows Then

TxtName.Text = dr(0)

Txtprice.Text = dr(1)

Txtopen.Text = dr(2)

End If

dr.Close()

cmd.Dispose()

End If



But if i removeIf ListBox1.SelectedIndex > 0 It gives me error:-
Operator '&' is not defined for string "select name,price,[open] from lo" and type 'DataRowView'.

I am just stuck with it please help!

Well why would you want to remove that line? If nothing is selected, then the SelectedIndex will be -1 and the SelectedValue will be undefined. You should always check theSelectedIndex, but you should actually be checking that it's >= 0.

---

~~Bonnie Berent [C# MVP]

Hi Schauhan,

try to separate the string assignment from the declaration of the sql identifier.
Anyway, I'd suggest to use the StringBuilder class to concatenate strings.



regards,
franking

Well why would you want to remove that line? If nothing is selected, then the SelectedIndex will be -1 and the SelectedValue will be undefined. You should always check theSelectedIndex, but you should actually be checking that it's >= 0.

---

~~Bonnie Berent [C# MVP]

Hi Schauhan,

Firstly, if you'd like to remove the "if"condition line of code, you should remove the "End If" at the end as well.

Secondly, I don't think you should remove the "if" conditionfrom your code because you need to check whether there's an item selected in the ListBox before you use the ListBox's SelectedValue property.

Sincerely,
Linda Liu

A repeat of what I just said, more or less. =0(

---

~~Bonnie Berent [C# MVP]