in reply to Array size issue

if($line =~ m/ on $ipaddress/)
    {
        if (scalar(@matches) <= 2){
            push(@matches, $line);
        }        
        else {
            exit(0);
        }
        foreach $line (@matches){
            print "$line\n";
        }
    }

[download]
This is where your trouble is. You probably meant to write it thusly: 
if($line =~ m/ on $ipaddress/)
    {
        if (scalar(@matches) <= 2){
            push(@matches, $line);
        }        
        else {
            foreach $line (@matches){
                  print "$line\n";
            }
            exit(0);
        }
        
    }

[download]

Peter L. Berghold -- Unix Professional
Peter -at- Berghold -dot- Net; AOL IM redcowdawg Yahoo IM: blue_cowdawg

Replies are listed 'Best First'.
Re^2: Array size issue
by hmb104 (Sexton) on Oct 15, 2012 at 18:17 UTC

    That is not what I want. I want to save the line to the array if it matches the expression. I don't want to print anything this is just for debugging to make sure my array is containing the proper lines.

[reply]
          That is not what I want.

      Sorry.... could have fooled me considering the logic in your original post.

      If I break your code down into pseudo-code it reads like this:

        open a pipe to the tail -f command for a logfile (unnamed)
        iterate over the pipe
          if less than 3 files are in array
            if the line matches a (undetermined) IP address
              push the line into an array
            otherwise
              exit the program
          Iterate over the contents of the array
            print line

      Part of the art of getting good answers on Perl Monks is being concise about your questions and not leaving what you are after up to the interpretation of the reader.

      Once again I will attempt to extrapolate what you are after. You code will work better if you don't put the print lines in and change the exit for last.

      Peter L. Berghold -- Unix Professional
      Peter -at- Berghold -dot- Net; AOL IM redcowdawg Yahoo IM: blue_cowdawg
[reply]

        I apologize for not being clear and I do appreciate all the valuable input I get from this community.

        Without the print statement I will not be able to debug my code. I want to make sure my array is carrying the proper line/data before attempting to process it. This is what I got after reading all the comments. I think I'm missing a small piece and I can't figure it out. 

        open my $tailf, "tail -f $logFile |" or die;
while (<$tailf>) {
    chomp;
    my $line = $_;
    if($line =~ m/ on $ipaddress/)
    {
        if (scalar(@matches) <= 2){
            push(@matches, $line);
        }        
        else {
            last;
        }
    }
}

foreach $line (@matches){
     print "$line\n";
}
        
close $tailf;

        [download]
[reply]
[d/l]

In Section Seekers of Perl Wisdom