Your get_name and set_name functions have now become object methods. This means that the first parameter they receive is the object reference. You need to deal with that before looking for the parameters you expect. A quick fix would be to do something like this:

sub get_name {
  my $self = shift;
  if (defined ($name)) {
    return $name;
  } else {
    return "error";
  }
}

sub put_name {
  my $self = shift;
  $name = "$_[0]";

}
[download]

But you have a bunch of other problems. Your $name variable should become an entry in the object hash ($self->{name}) otherwise it will be shared between all instances of your object.

Good places to learn about this stuff are the perlboot and perltoot manual pages and Damian Conway's most excellent book Object Oriented Perl.

"Perl makes the fun jobs fun
and the boring jobs bearable" - me