Convert problem!

[miss YY]

int mm =int.Parse(dateTimePicker1.Value.ToString("dd")); int nn =int.Parse(dateTimePicker1.Value.ToString("dd")); SqlCommand SCD = new SqlCommand("select * from table_inf where time>=mm and time<=nn", Conn); i use data Sql2000,Field time's Property is datetime what's wrong with it? how can i change it thank you everyone!!

[Luc Pattyn]

you only get questions now,l for you to answer and deduce reasonable conclusions yourself: what is the difference between mm and nn??? what type is dateTimePicker1.Value? why would you turn it into a string and then parse that string? that type has a lot of interesting Properties. Start reading the documentation on that type! and what is the type of your database table's "time" field? are mm and nn in your SQL of the same type as time? :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]

---

I only read code that is properly indented, and rendered in a non-proportional font; hint: use PRE tags in forum messages

---

[N a v a n e e t h]

miss YY wrote:

how can i change it

What are you trying to do here? Both mm and nn variables will have same value and your query may not return any values.

miss YY wrote:

i use data Sql2000,Field time's Property is datetime

Then you need to compare against a DateTime not just an integer variable that has date.

DateTime mm = dateTimePicker1.Value;
DateTime nn = dateTimePicker2.Value;
SqlCommand SCD = new SqlCommand("select * from table_inf where time >= @mm and time <= @nn", Conn);
SCD.Parameters.AddWithValue("@mm", mm);
SCD.Parameters.AddWithValue("@nn", nn);

Best wishes, Navaneeth

[dan sh]

1. You are comparing a date and integer. Compare dates with dates. You should use your date time picker's value right away. 2. See you select command:

miss YY wrote:

select * from table_inf where time>=mm and time<=nn

Entire thing is a string. How is it supposed to get the values in mm and nn? You should read about SQLParameters class and use it. Also, from the ToString overload you are using, it shows that you are just checking the day part of the date. Do you really want to do that way? If you want to check that way, you should use day(time) instead on time in your SQL and then compare. But be sure that this will ONLY compare the day part and not the month and year. Edit: Missed what Luc spotted. Both mm and nn will have same values. So, why not just check equality with one value instead of two?

50-50-90 rule: Anytime I have a 50-50 chance of getting something right, there's a 90% probability I'll get it wrong...!!

modified on Sunday, November 29, 2009 10:29 PM

[PIEBALDconsult]

Yes, but why use local variables at all?