I think you may have some flawed logic in your "searchfile" subroutine. You have:

      foreach my $k ( keys %master ) {
         while ( my $line=<$foundfile> ) {
[download]

At the end of the first iteration through the foreach loop, you will have checked every line in the file. But every iteration of the foreach loop after that won't execute the while loop.

One option to is to open the file inside the foreach loop and then work through the while loop and then close the file in each iteration like this:

sub searchfile {
    my $file=shift;
    # print "open $file\n";

      foreach my $k ( keys %master ) {
         open my $foundfile, '<', $file or die "can't open $file";
         my $first=<$foundfile>;
         while ( my $line=<$foundfile> ) {
             if ($line =~ m/$k/ ) {
                 $master{$k}++ ;
                # print "$k found in $file\n";
             }
         }
         close $foundfile;         
      }
}
[download]

That's basically doing this. For every key in the hash, do the following: open the file, skip the first line, search the rest of the lines for the key (and do action if found) and close the file

Another option is to swap your foreach and while loops like this:

sub searchfile {
    my $file=shift;
    # print "open $file\n";
    
     open my $foundfile, '<', $file or die "can't open $file";
    
    my $first=<$foundfile>;
     
      while ( my $line=<$foundfile> ) {
          foreach my $k ( keys %master ){
             if ($line =~ m/$k/ ) {
                 $master{$k}++ ;
                # print "$k found in $file\n";
             }
         }
      }

    close $foundfile;         
}
[download]

This is basically doing the following: open the file, skip the first line, then for each line check to see if the matches any of the keys in the hash (and if it does, do some action) and then closes the file.

Not knowing more information about your files and their contents, I think I would personally opt to go the second route, which only opens each file once.

---

---

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! —

• How do I compose an effective node title?
• How do I post a question effectively?
• Markup in the Monastery

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:
	&		&
	<		&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.