...all i need it to say is Car Record Not Found once not multiple times...
You have the following:
...
foreach my $carroadnameverify2 (@lines) {
if ( $carroadnameverify2 =~ /$carroadnameverify/ ) {
my $format = " %-13s %-15s %-12s %0s";
printf( "$format", split /\:/, "$carroadnameverify2" );
}
else {
printf("CAR RECORD NOT FOUND");
}
} # END FILE LOOP
...
Indeed, if you have 10 non-matches within the foreach loop, "CAR RECORD NOT FOUND" will be printed 10 times. It seems, however, that you may be looking for something like the following:
...
my $found = 0;
foreach my $carroadnameverify2 (@lines) {
if ( $carroadnameverify2 =~ /$carroadnameverify/ ) {
my $format = " %-13s %-15s %-12s %0s";
printf( "$format", split /\:/, "$carroadnameverify2" );
$found = 1;
}
} # END FILE LOOP
printf("CAR RECORD NOT FOUND") if !$found;
...
This just uses $found as a flag; if it's not set (i.e., if the item wasn't found in the array), the "CAR RECORD NOT FOUND" message is printed only once.
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.