in reply to Coding for maintainability
The modification needed is that you aren't matching on a simple string. There's a few ways to improve this. One is to pass $family to every function and let them determine if they want to handle it. Then, the function returns either true (I handled it) or false (I don't deal with this value). Errors would be propagated with die and caught with an eval-block.my %dispatch = ( foo => sub { print "foo\n"; }, bar => \&bar, ); chomp( my $input = <> ); unless ( exists $dispatch{ $input } ) { die "I don't know what to do with '$input'\n"; } $dispatch{$input}->(); sub bar { print "bar\n"; }
Then, whenever you add something, you're working with a much smaller piece of the puzzle because the engine and the parts are separated.
|
---|
Replies are listed 'Best First'. | |
---|---|
Re^2: Coding for maintainability
by planetscape (Chancellor) on Feb 17, 2006 at 07:15 UTC | |
by monsterzero (Monk) on Feb 17, 2006 at 16:29 UTC |