use Perl

• API (Score:1)

  by izut (6269) on 2008.01.15 1:51 (#60238) Journal

  Hi Adriano,
  
  I was reading WWW::CPAN API, and I have one suggestion to the API: change the method name fetch_distmeta() to get_meta().
  
  This reflects the fact that you're dealing with CPAN, and the word 'meta' is used by modules on CPAN, so you can avoid the word 'dist' there - CPAN deals with software distribution. And fetch IMHO seems to be too related with the medium (and it is longer than get).
  
  For example:
  
  
my $cpan = WWW::CPAN->new;
my $meta = $cpan->get_meta({ dist => 'WWW::CPAN' });

  
  Is more clear and less polluted than:
  
  
my $cpan = WWW::CPAN->new;
my $meta = $cpan->fetch_distmeta({ dist => "WWW::CPAN" });

  
  Other possibility would have a WWW::CPAN::Dist class, which is a stub for actually retrieving data from CPAN, like:
  
  
my $cpan = WWW::CPAN->new;

for my $d (qw( WWW::CPAN )) {
        my $meta = $cpan->dist('WWW::CPAN')->get_meta; ...
}

  
  What do you think?

  --
  Igor Sutton

  Reply to This

  • Re: (Score:1)

    by ferreira (5993)

    Those sound like very elegant ideas and I will think harder about them in the next days. The reason I went with "fetch_distmeta" was to anticipate the occurrence of other meta data beyond CPAN distributions, like authors. And "fetch" was chosen instead of "get" because the latter reminded me only simple accessor rather than a real action to retrieve some data. But I am thinking about it. Thanks for your feedback, Igor.