comment on

You should get out of the habit of unnecessarily quoting variables - it is bad habit. Here are a couple of corrections:

my %snmp_options = (
   hostname    => $args->{h},
   retries     => $retries,
   # etc
);

# and 
$snmp_options{username} = $snmp_username;
[download]

Sure, using quotes works for those examples, but it is unnecessary and will one day lead to something like:

package My::CGI;

use strict;
use CGI;

sub new {
   my $class = shift;
   my $foo   = 'bar';
   my $q     = CGI->new();
   my $self = {
      foo => "$foo",
      cgi => "$q",
   };
   return bless $self, $class;
}

package main;

use strict;

my $q = My::CGI->new();

print $q->{foo}, "\n";
print $q->{cgi}->a({href=>'http://foo.bar'},'some link'),"\n";

__END__
yields:
bar
Can't locate object method "a" via package "CGI=HASH(0x80f8a20)"
at ./foo.pl line 26.
[download]

Whoops! What happened? You literalized a reference. Try this code out, then go back and remove the unnecessary quotes inside the $self hash ref.

jeffa

L-LL-L--L-LL-L--L-LL-L--
-R--R-RR-R--R-RR-R--R-RR
B--B--B--B--B--B--B--B--
H---H---H---H---H---H---
(the triplet paradiddle with high-hat)

In reply to (jeffa) 3Re: using return to create a module object by jeffa
in thread using return to create a module object by c

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.


Pathologically Eclectic Rubbish Lister
	PerlMonks