Question

I have a DateTime object that I’d like to check and see if it falls within the last 24 hours.

I did something like this but its wrong:

myDateTime > DateTime.Now.AddHours(-24) && myDateTime < DateTime.Now

where did I go wrong?

Answer 1

There is nothing wrong with the code that you posted, so whatever you did wrong is somewhere else in the code.

I only see two minor flaws in the code, but they only affect corner cases:

You should avoid getting the DateTime.Now property repeatedly in the code. Its value changes, so you may get inconsistent results in some cases when the values changes from one use to the next.

To get a time interval you would usually pair one inclusive and one exclusive operator, like > and <=, or >= and <. That way you can check for intervals next to each other, like 0 – 24 hours and 24 – 28 hours, without getting a gap or an overlap.

DateTime now = DateTime.Now;
if (myDateTime > now.AddHours(-24) && myDateTime <= now)