No logic errors stand out immediately, but you're doing way too much work. For example,
$xf = 0;
$sthf->execute;
while (@dataf = $sthf->fetchrow_array())
{
$filiais[$xf] = $dataf[0];
$xf++;
}
is generally better written as:
while (my ($col) = $sthf->fetchrow_array())
{
push @filiais, $col;
}
Similarly,
for ($m=1; $m<=12; $m++)#Make an array of months
{
$months[$m] = $m;
}
foreach $months(@months)
{
would be shorter, clearer, and prone to one less boundary condition as:
foreach my $month (1 ... 12)
{
The biggest issue you have is what appears to be a complete reliance on package globals. If you forget to reset one to zero at the start of the sub, you'll have values bleading over across invocations -- exactly what you describe. Change all of these variables into lexicals within your sub and I suspect your error will disappear.
Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
Read Where should I post X? if you're not absolutely sure you're posting in the right place.
Please read these before you post! —
Posts may use any of the Perl Monks Approved HTML tags:
- a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
| |
For: |
|
Use: |
| & | | & |
| < | | < |
| > | | > |
| [ | | [ |
| ] | | ] |
Link using PerlMonks shortcuts! What shortcuts can I use for linking?
See Writeup Formatting Tips and other pages linked from there for more info.