Can You Help To Explain This Cod What Dose It Mean And What It Do ?

Question

0.00/5 (No votes)

See more: (untagged)

C#

private int checkDay( int testDay )
{
      int daysperMonth [] = {0,31,28,31,30,31,30,31,31,30,31,30,31};
       // check if day in rang for month
      if (testDay >0 && testDay <= daysperMonth[month])
         return testDay ;
         // check for leap year

      if (month ==2 && testDay ==24 &&(year %400 ==0 || (year %4 ==0 && year %100!=0) ) )
         return testDay ;
         System.out.printf ("Invalid day (%d) set to 1 ", testDay);
         return 1 ; // maintain object n consistent state
     } // end method checkDay

Posted 9-Feb-14 5:57am

Nima Khanmohammadi

Updated 9-Feb-14 6:09am

CHill60

v2

Add a Solution

Comments

Richard MacCutchan 9-Feb-14 12:19pm

It is trying to check whether a date is valid or not. But it won't work for February 29th.

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)

OriginalGriff · Answer 1 · 2014-02-09T06:17:00

Solution 1

It looks at the date information provided and decides if the date is valid.

However, it doesn't work. No month has only 24 days, even in a leap year...
And it returns poor information: a bool valid/invalid check would be a lot more sensible.

Posted 9-Feb-14 6:17am

OriginalGriff

CPallini · Answer 2 · 2014-02-09T06:25:00

This code is checking if the given day is in the calendar (that is its number is between current month's minimum and maximum number).

The following line

Quote:
if (month ==2 && testDay ==24 &&(year %400 ==0 || (year %4 ==0 && year %100!=0) ) )

Is wrong: it should be

C++

if (month ==2 && testDay ==29 && (year %400 ==0 || (year %4 ==0 && year %100!=0) ) )

It checks if 29th of february is a valid day, that is if current year is a leap year.
You know, a year is leap if it is exactly divisible by 4 but it isn't a century year (e.g. 1900), unless it is also divisible by 400 (e.g. 2000).

By the way, the indentation is misleading.

[update]
As noted by OriginalGriff (I have overlooked it)

Quote:
return 1 ; // maintain object n consistent state

is plain madness: if you detect an invalid input, you have to signal it.
[/update]