in reply to Producing Calender
JBUK wrote:
I am trying to produce a calender, which I have managed to do using perl modules but I want to avoid using any for this particular task.
You have a lot of duplicated code and it happens to give an incorrect answer for February. Code like the following is why I would use a proper module:
sub days_in_month { my ( $month, $year ) = @_; my @days_in_month = qw/ 31 28 31 30 31 30 31 31 30 31 30 31 /; if ( $month == 1 ) { return 29 if is_leap_year( $year ); } return $days_in_month[ $month ]; } sub is_leap_year { my $year = shift; return ( 0 == $year % 4 and 0 != $year % 100 or 0 == $year % 400 ) + ? 1 : 0; }
&days_in_month will accurately return the number of days in a given month for and a given year, but it does assume that the month range is zero to eleven. It should be easy to modify, but it shows how apparently simple tasks can quickly become complex. However, I didn't check to see if I have a valid year or month. If other's use your module, they will use it wrong and you'll have to account for that, too :)
Cheers,
Ovid
Join the Perlmonks Setiathome Group or just click on the the link and check out our stats.
|
|---|