in reply to Refactor method calls or not?

This might be crazy, but you could use a closure to generate the required subs and then load them into a hash. Something like this, perhaps:
# Define appropriate "methods" my @dispatch_methods = qw [ company financialDiary ]; # Routine to generate an anonymous sub using closure sub dispatch_add { my ($what) = @_; return sub { my ( $self, $data ) = @_; my $table = $what; my $data = $self->_generic_insert( $data, $table ); $self->{ _dbh }->commit if ! $self->{ _error }; return $data; }; } # Build the dispatch table, trying to minimize error sub new { # Usual stuff... my $self = bless ... # : $self->{dispatch} = { map { $_ => dispatch_add{$_} } @dispatch_methods }; }
Then you could call them along the lines of:
Of course, this might be completely nuts.

Don't get me wrong. AUTOLOAD is fun, too. A hybrid approach would merely interface with the internal {dispatch} structure and call the appropriate function. This way they are pre-cached, and pre-validated. If there is no defined dispatch, AUTOLOAD can warn/die/carp appropriately.