Re: Iterating though an array of objects

[moritz]

in reply to Iterating though an array of objects

You should really format your code better (one indetion level per opneing brace, see perlstyle), it is very hard to read as is. Maybe the problem becomes visible then?

One obvious problem is:

    sub scores {
        my ( $self, @scores )= @_;
        ...
    }

# and later
$node->scores(['3','177','12',]);

The scores method expcets a flat list, but you pass an array reference to it.

Also dubious:

eval{person->new();} or die ($@);

Why catch errors with eval if all you do is rethrowing them?

[chromatic]

Why catch errors with eval if all you do is rethrowing them?

What if the constructor returned a false value? (It doesn't, but it could.) Then again, the exception message would be awful.