in reply to Re: Date::Manip Help
in thread Date::Manip Help

I tried the following, but all the dates in the table are coming the same
$date=ParseDate("today"); my @city=("C","K","D"); my ($j,$fut); $fut=ParseDate("in 90 days"); for ($j=0;$j<3;$j++) { my $i; for ($i=1;$i<90;$i++) { # $date=ParseDate("in 1 day"); $date= UnixDate( $date => '%Y-%m-%d' ); print "Date: $date\n"; # $rows=$dbh->do("INSERT INTO appCancel (appt_date,status, no_c +ancel, city) # VALUES ('$date','U','0', '$city[ +$j]')") || # die "Couldn't insert record : $DBI::errstr"; $date=ParseDate("in 1 day"); print "Date: $date\n"; } }
the number of records being generated is 267? and not 270

Replies are listed 'Best First'.
Re: Re: Re: Date::Manip Help
by sacked (Hermit) on May 11, 2004 at 13:50 UTC
    Please take a look at the documentation for Date::Manip, as I suggested in my earlier post. You need the DateCalc function to calculate ($somedate + 1 day).

    Regarding your code above, in your original post, it wasn't clear that you wanted to enter every date from tomorrow to 90 days from now. If that's the case, you need to create a new date object for each day:
    #!/usr/bin/perl # get current Date from MySql use warnings; use strict; use Date::Manip; + my @city=("C","K","D"); my $j; for ($j=0;$j<3;$j++) { my $date= ParseDate("today"); + my $i; + # either start from 0, or use <= for ($i=1;$i<=90;$i++) { $date= DateCalc($date, "+ 1 day"); $date= UnixDate($date, '%Y-%m-%d'); print "Date: $date, city: $city[$j]\n"; } }
    The reason you were only getting 267 records is because your inner loop only executed 89 times:
    for ($i=1;$i<90;$i++)
    You either need to initialize $i to 0 or change the conditional expression to $i <= 90.

    If you'd like to speed up the code, you should reverse the loop structure, with the date calculation in the outer loop (so each date is only calculated once rather than three times). Also note that the code below uses Perl-style loops rather than C-style:
    #!/usr/bin/perl # get current Date from MySql use warnings; use strict; use Date::Manip; + my @city=("C","K","D"); + my $date= ParseDate("today"); + # using a Perl-style (not C-style) loop helps prevent # the error seen in the earlier loop construct for my $i ( 1 .. 90 ) { $date= DateCalc($date, "+ 1 day"); $date= UnixDate($date, '%Y-%m-%d'); + for my $j ( 0 .. 2 ) { print "Date: $date, city: $city[$j]\n"; } }
    Hope this helps,

    --sacked
      Thanx
      this worked. I had trying to break my head over this for the past few days

      Regards
      Agyeya Gupta