how to insert a date n time in database

Question

1.00/5 (1 vote)

See more:

it is giving me a syntax error

C#

con.Open();
OleDbCommand cmd =
    new OleDbCommand(
        "insert into coupen(voucher,RS,Date) values('" + _voucher + "','" + TxtAmount.Text + "',GetDate())", con);
int result = cmd.ExecuteNonQuery();
con.Close();

Posted 19-Mar-13 21:53pm

Amirsalgar1

Updated 19-Mar-13 22:48pm

v3

Add a Solution

Comments

Shubh Agrahari 20-Mar-13 4:07am

what is data type of date column in db...?

Amirsalgar1 20-Mar-13 4:42am

date and time

Amirsalgar1 20-Mar-13 4:45am

in my access database in table coupon there are three columns voucher, Rs,date

here voucher, Rs are working properly , i want to insert current date and time while clicking

4 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)

Kuthuparakkal · Accepted Answer · 2013-03-19T22:03:00

Solution 1

Try this:

C#

con.Open();
OleDbCommand cmd =
    new OleDbCommand(
        "insert into coupen(voucher,RS,[Date]) values('" + _password + "','" + TxtAmount.Text + "',"+ DateTime.Now.Date + ")", con);
int result = cmd.ExecuteNonQuery();
con.Close();

Posted 19-Mar-13 22:03pm

Kuthuparakkal

Updated 19-Mar-13 23:21pm

v2

Comments

Amirsalgar1 20-Mar-13 4:41am

nahh dude it still giving me a same error

Syntax error in INSERT INTO statement.

Amirsalgar1 20-Mar-13 4:46am

in my access database in table coupon there are three columns voucher, Rs,date here voucher, Rs are working properly , i want to insert current date and time while clicking
date and time is data type of date column in db

Kuthuparakkal 20-Mar-13 5:22am

Try now, it could be the column name match with DataType name, so enclose those columns in [] brackets. Modified solution too...

Amirsalgar1 21-Mar-13 4:36am

will you please join me via team viewer ? my ID 175 735 453 pwd 4281

Nandakishore G N · Accepted Answer · 2013-03-19T23:07:00

getdate() is the sql function it will not execute using oledb. And, one more point is you cannot send getdate() from codebehind using ado.net.

change the query,

C#

con.Open();
OleDbCommand cmd =
    new OleDbCommand(
        "insert into coupen([voucher],[RS],[Date]) values('" + _password + "','" + TxtAmount.Text + "','"+ System.DateTime.Now + "')", con);
int result = cmd.ExecuteNonQuery();
con.Close();

try this.
if error occurs paste the error occuring.

_Amy · Accepted Answer · 2013-03-19T23:13:00

Solution 4

Try this:

C#

con.Open();
OleDbCommand cmd = new OleDbCommand("INSERT INTO COUPEN(voucher,RS,Date) VALUES ('" + _password + "','" + txt.Text + "','" + DateTime.Now + "')", con);
int result = cmd.ExecuteNonQuery();
con.Close();

Try to crosscheck your table/column name once.

--Amit

Posted 19-Mar-13 23:13pm

_Amy

Updated 19-Mar-13 23:24pm

v2

phil.o · Accepted Answer · 2013-03-19T23:19:00

It is a really bad habit to build your SQL queries by concatenating string fields like you do.
Always do it with parameterized queries.
Thus :

con.Open();
OleDbCommand cmd = new OleDbCommand("insert into coupen(voucher,RS,Date) values(@voucher, @amount, @now)", con);
cmd.Parameters.AddWithValue("voucher", _voucher);
cmd.Parameters.AddWithValue("amount", TxtAmount.Text);
cmd.Parameters.AddWithValue("now", DateTime.Now);
int result = cmd.ExecuteNonQuery();
con.Close();

This way your code won't be prone to SQL injection attacks, and it's a little bit readable.

Hope this helps.