How do I solve this line

Question

0.00/5 (No votes)

See more:

C#

cmd.Parameters.AddWithValue("@dateFrom",DateTime.Now.ToString("dd/MM/yyyy"));

//storing as date in my db

//error is Conversion failed when converting date and/or time from character string.

Posted 30-Jan-14 22:28pm

Member 10548723

Updated 30-Jan-14 22:44pm

ravikhoda

v2

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Vedat Ozan Oner · Answer 1 · 2014-01-30T22:33:00

Solution 1

don't convert datetime.now to string. just add it as it is.

C#

cmd.Parameters.AddWithValue("@dateFrom",DateTime.Now)

Posted 30-Jan-14 22:33pm

Vedat Ozan Oner

Comments

Member 10548723 31-Jan-14 4:46am

but datetime.now is getting the current date of mine, not the selected date.

Vedat Ozan Oner 31-Jan-14 4:52am

:) yes, of course, it is your datetime. you have coded that. i can't know what you are trying to do. i just gave the solution for the given problem which is "//error is Conversion failed when converting date and/or time from character string."

Karthik_Mahalingam · Answer 2 · 2014-01-30T23:37:00

Solution 2

Try this

C#

cmd.Parameters.AddWithValue("@dateFrom" , Convert.ToDateTime( DateTime.Now.ToShortDateString()));

Posted 30-Jan-14 23:37pm

Karthik_Mahalingam

Comments

phil.o 31-Jan-14 10:04am

Well: the DB expects a DateTime.
You then take a DateTime (Now), convert it to string, and use the Convert function to convert it back to DateTime.
I'm sorry, but I cannot agree with you on that.

Karthik_Mahalingam 31-Jan-14 10:06am

it will save in the DB as time current date with time stamp as 12:00:AM.. which will be easy for querying..
else it will save with the current TIme stamp..
hop you understood ??

phil.o 31-Jan-14 10:09am

Then use DateTime.Today. Much more clean, and does not involve two opposite conversions in the process.
It may 'work', but, I'm sorry, it's ugly :(

Karthik_Mahalingam 31-Jan-14 10:32am

If the sql column is of type date your logic will work. What if datetime column