I second stvn.

Since you know which fields are permitted when the class compiles you can generate them and get rid of AUTOLOAD. AUTOLOAD is really only needed when you do not know which methods the class will need to handle.

my @fields = qw/
  name
  age
  peers
/;

for my $name (@fields) {
  my $code = sub {
    my $self = shift;
    $self->{$name} = $_[0] if @_;
    return $self->{$name};
  };

  no strict 'refs';
  *$name = $code;
}
[download]

Now your class acts exactly like it would have acted if you manually wrote the methods, and you pay no run-time penalties.

Note how simple the logic of the code is compared to AUTOLOAD. And the AUTOLOAD implementation will get more complicated still. Consider what happens when you

• do $obj->can('age'),
• refactor and create a superclass that also uses AUTOLOAD, or
• use multiple inheritance where another class also uses AUTOLOAD.
• (Or forget to do return if $name eq 'DESTROY';.)

A proper implementation of AUTOLOAD requires an awful lot of thinking, and still it does not buy you anything, rather the opposite, if you already know the method names.

lodin

---

---

• 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.