You don't indicate what the filestamp() subroutine is doing, but it if it doesn't contain an eval, then the datestamps do not contain what you think they contain. By putting single quotes around $logfile when you pass it to filestamp(), you have prevented it from being interpolated, so instead of a logfile name, you are actually just passing that subroutine the word 'logfile' preceded by a dollar sign.

Also, you have a fundamental flaw in your calculation that determines if the log is from yesterday. If today is the first day of the month, then subtracting one from the date string won't get you yesterday, use something like Date::Calc instead.

And that inner 'foreach($logfile) {', while not causing problems for you, doesn't actually do anything.

We're not surrounded, we're in a target-rich environment!

In reply to Re: If logic fails foreach mistake by jasonk
in thread If logic fails foreach mistake by dbrock

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • 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:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.