The fact that a program consists of instantiating a single object and calling a single method on it, is not necessarily an indication of the complexity of the program as a whole.
Sorry, but that is simply fatuous pseud-OO.
You pull in nearly 300 modules to do nothing more than: system '/usr/sbin/httpd';
That's not just fatuous, it is moronic.
I truly hope that doesn't offend you, because that is not my intention -- all said and done, it is just the opinion of some obscure Englishman on the net -- but it is my opinion.
You are gaining exactly nothing from your use of OO here; and exactly nothing from your dependency on, and the considerable overheads of, Moose.
It's not even validating your parameter for you. It will happily accept 1234567890 as a string -- as all Perl will.
If App::Reprove consisted of (something like):
package App::Reprove; use App::Prove; use LWP::Simpe; require Exporter; our @ISA = qw[ Exporter ]; our @EXPORT = qw[ doit ]; sub doit { my %arg = @_; ## maybe a little validation my $file1 = get delete $args{ file1 }; my $file2 = get delete $args{ file2 }; ## do whatver you do with the file contents... App::Prove->new( %args )->run } 1;
It would:
By any semi-competent perl (or C or D or Ruby or Java or Python or maybe even PHP) programmer, anywhere.
As is, you've got a module that requires users to install a huge crap-load of dependencies only to find -- for 10% of them at least -- that it doesn't work. And probably never will unless 1 or more of the 3 or 4 guys that understand what the **** is going on inside Moose decide to step in and help you out.
I'm pretty damn good at working my way through new modules and working out what is wrong, but with Moose in the picture, I can't even get started.
Quite frankly, I cannot see any merit in your module over using App::Prove myself. But then, I can see no merit in App::Prove over using the prove command directly either. But at least I can look inside the latter and see the value-add -- no matter how tenuous I might think it is -- and can understand the code enough that I could fix any problems that arose. With yours, NADA. Not a hope in hell.
In reply to Re^6: Should I upload it on CPAN?
by BrowserUk
in thread Should I upload it on CPAN?
by llancet
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |