Cant get hour and minute of the day

Question

I am trying to get the current time:

date = "(" + DateTime.Today.Year.ToString() + "-" + 
DateTime.Today.Month.ToString() + "-" + DateTime.Today.Day.ToString() + " " + 
"(" + DateTime.Today.Hour.ToString() + ":" + DateTime.Today.Minute.ToString() 
+")" + ")";

This should get a date like:

(2013-2-1 (13:01))

But it give me:

(2013-2-1 (0:0))

How can i fix this?

Use `DateTime.Now`, `DateTime.Today` only gives you the date I think. — antonijn, Feb 01 '13 at 11:03
Wow! 3 answers within 26 seconds of each other - 4 if you count the comment above. :-) — Colin Mackay, Feb 01 '13 at 11:04
Please use `date = DateTime.Now.ToString(@"(yyyy\-MM\-dd (HH\:mm))")` instead of that. — GSerg, Feb 01 '13 at 11:05
Also, not sure why people are downvoting the question. It doesn't seem a bad question to me. It shows a lack of understanding about how DateTimes work, but then all questions show a lack of understanding about something, which is why the question was asked. — Colin Mackay, Feb 01 '13 at 11:06

score 9 · Accepted Answer · answered Feb 01 '13 at 11:03

You're using DateTime.Today which is documented as:

An object that is set to today's date, with the time component set to 00:00:00.

So yes, if you find the Hour and Minute components, they will be 0...

If you want the current time of day, use DateTime.Now instead. Note that both Today and Now use the system-local time zone - you need to be sure that that really is what you want to use. (It's probably fine for a local client app, but not for a web app.)

score 5 · Answer 2 · answered Feb 01 '13 at 11:08

Instead of all the concatenation you are using at the moment, use the DateTime object's ToString() method...

string date = DateTime.Now.ToString("(yyyy-M-d (hh:mm))");

Also, as pointed out by many others, note the use of DateTime.Now instead of DateTime.Today

score 2 · Answer 3 · answered Feb 01 '13 at 11:03

2

Use DateTime.Now instead of DateTime.Today

The DateTime.Today has its time part set to 00:00:00
The DateTime.Now gets the current date and time for the local computer

answered Feb 01 '13 at 11:03

Steve

213,761
22
232
286

score 2 · Answer 4 · answered Feb 01 '13 at 11:04

2

You have DateTime.Today which truncates the time element away.

Use DateTime.Now instead which gives you the full date and time.

answered Feb 01 '13 at 11:04

Colin Mackay

18,736
7
61
88

score 2 · Answer 5 · edited May 23 '17 at 12:27

2

Use DateTime.Now instead of DateTime.Today. DateTime.Today doesn't include the time, only the date. See also: Difference between System.DateTime.Now and System.DateTime.Today

edited May 23 '17 at 12:27

Community

1
1

answered Feb 01 '13 at 11:05

antonijn

5,702
2
26
33

score 2 · Answer 6 · answered Feb 01 '13 at 11:06

2

Use NodaTime if its a web application. Use DateTime.Now for local date and time.

So it should be:

date = "(" + DateTime.Now.Year.ToString() + "-" + DateTime.Now.Month.ToString() + "-" + DateTime.Now.Day.ToString() + " " + "(" + DateTime.Now.Hour.ToString() + ":" + DateTime.Now.Minute.ToString() +")" + ")";

answered Feb 01 '13 at 11:06

Aniket Inge

25,375
5
50
78

Calling `DateTime.Now` multiple times is never a good thing. And Noda Time is awesome, but it is not limited to web applications. This code doesn't demonstrate Noda Time anyway, so I'm not sure what you were getting at. – Matt Johnson-Pint Jul 24 '13 at 00:33

score 2 · Answer 7 · answered Feb 01 '13 at 11:11

2

Please try this shorter code for current date and time as per your format

var date = DateTime.Now.ToString("(yyyy-MM-dd(HH:mm))");

answered Feb 01 '13 at 11:11

Cris

12,799
5
35
50

score 2 · Answer 8 · edited Feb 01 '13 at 11:32

2

Use the following code instead of concatenation. Its much more optimized then the concatenation work you have done.

string date = DateTime.Now.ToString("(yyyy-M-d (hh:mm))");

Also DateTime.Today does not have time element in it.

edited Feb 01 '13 at 11:32

Tom

26,212
21
100
111

answered Feb 01 '13 at 11:15

user1771046

21
1

score 1 · Answer 9 · answered Feb 01 '13 at 12:06

1

DateTime.Today will return u 2013-2-1 00:00:00 .. It gives the time sure but the timestamp is set to 00:00:00.. that is the reason why u r not getting an exception n still getting 00:00:00. Use DateTime.Now for current timestamp.

answered Feb 01 '13 at 12:06

Raghavan

637
3
12

Cant get hour and minute of the day

9 Answers9