Beefy Boxes and Bandwidth Generously Provided by pair Networks
"be consistent"

Re: Looping through numerous snmp strings....

by jwkrahn (Monsignor)
on Sep 19, 2012 at 19:51 UTC ( #994512=note: print w/replies, xml ) Need Help??

in reply to Looping through numerous snmp strings....

open( RTR, "$rtrlist" ) || die "Can't open $rtrlist file"; ... open( LOG, ">$workingdir/RESULT.TXT" )|| die "Can't open $workingdir/R +ESULT.TXT file"; ... $rtr = `$snmpget . $snmpro[0] $rtr` or print "hos +tname failed: $!\n"; ... $rtr = `$snmpget . $snmpro[1] $rtr` or print +"Host not responding to $snmpro[1]\n";

Consistency is always good.    You should stick with either ( ... ) || ... or  ... or ...

The use of $! with backticks is wrong.    You SHOULD be using $! to indicate errors from open and $? to indicate errors from backticks.

printf " Router\t\t Location\t\t\tContact\t\t Serial\n"; printf LOG " Router\t\t; Location\t\t\t;Contact\t\t ;Serial\n";

You are using printf when you should be using print.

foreach (<RTR>) { chomp( $rtr = "$_" );

foreach needs to read the entire file into memory first before it can loop through the lines.    The usual, more efficient method is to use a while loop if you only need one line at a time.

Why are you copying the value in $_ to a string when it is already a string?

Replies are listed 'Best First'.
Re^2: Looping through numerous snmp strings....
by timmytimay (Initiate) on Sep 20, 2012 at 10:28 UTC

    Thanks Monks - some great advice! I'll give it a go and let you know if I have any more queries.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://994512]
[GotToBTru]: I had to dig into the code to find out where it came from .. and it makes no sense
[GotToBTru]: so I emailed my contact asking if we could just stop sending it .. I'm afraid she is going to ask "what is that anyway?"
[MidLifeXis]: heh.
[MidLifeXis]: Most likely it is a code that some undocumented system, hidden behind layers of IT, deep in the bowels of the building under the machine room floor, reads that code to keep a presence switch from going off. :-b
[MidLifeXis]: I think I forgot "running on a farm of commodore 64, vic 20s, trs 80s, and apple ]|[e systems"
[GotToBTru]: oh I know what it is .. but it is a number only slightly useful to me and of no possible use to our customer

How do I use this? | Other CB clients
Other Users?
Others meditating upon the Monastery: (9)
As of 2017-01-20 19:06 GMT
Find Nodes?
    Voting Booth?
    Do you watch meteor showers?

    Results (176 votes). Check out past polls.