Beefy Boxes and Bandwidth Generously Provided by pair Networks
Syntactic Confectionery Delight

comment on

( #3333=superdoc: print w/replies, xml ) Need Help??
More actually. On my machine:
qwurx [shmem] ~ > perl -le 'use CGI::Ajax; print "$_ => $INC{$_}" for +sort keys %INC' CGI/ => /usr/lib/perl5/site_perl/5.8.8/CGI/ => /usr/lib/perl5/5.8.8/ Class/ => /usr/lib/perl5/site_perl/5.8.8/Class/ Data/ => /usr/lib/perl5/5.8.8/i386-linux-thread-multi/Data/Du => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/i386-linux-thread-multi/XSLoader.p +m => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ warnings/ => /usr/lib/perl5/5.8.8/warnings/

Class::Accessor is included out of laziness - not having to code setters/getters for objects.

As we all know, laziness is one of the virtues of a perl programmer, so that seems OK. But it isn't, because it conflicts with hubris, which beats laziness for Module authors. You don't want anybody to ever say something bad about your module, do you?

Data::Dumper is a leftover from development. It isn't used anywhere in that module. And that one pulls in:

qwurx [shmem] ~ > perl -le 'use Data::Dumper; print "$_ => $INC{$_}" f +or sort keys %INC' => /usr/lib/perl5/5.8.8/ Data/ => /usr/lib/perl5/5.8.8/i386-linux-thread-multi/Data/Du => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/i386-linux-thread-multi/XSLoader.p +m => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ warnings/ => /usr/lib/perl5/5.8.8/warnings/

Then there's the ubiquitous 'vars' module, which does

qwurx [shmem] ~ > perl -le 'use vars qw($foo); print "$_ => $INC{$_}" +for sort keys %INC' => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ warnings/ => /usr/lib/perl5/5.8.8/warnings/

A little bit of (crude *) cleanup to the header of CGI::Ajax

--- /usr/lib/perl5/site_perl/5.8.8/CGI/ 2007-02-01 00:35:44.00 +0000000 +0100 +++ CGI/ 2007-09-16 16:18:47.000000000 +0200 @@ -1,17 +1,32 @@ package CGI::Ajax; -use strict; -use Data::Dumper; -use base qw(Class::Accessor); -use overload '""' => 'show_javascript'; # for building web pages, +so - # you can just say: print +$pjx BEGIN { - use vars qw ($VERSION @ISA @METHODS); + our ($VERSION, @ISA, @METHODS); @METHODS = qw(url_list coderef_list DEBUG JSDEBUG html js_encode_function cgi_header_extra); - CGI::Ajax->mk_accessors(@METHODS); - + for my $field (@METHODS) { + *{'CGI::Ajax::'.$field} = sub { + my $self = shift @_; + if (@_) { + return $self->set($field, @_); + } else { + return $self->get($field); + } + }; + }; + for (qw(set get)) { + *{'CGI::Ajax::'.$_} = sub { + my $self = shift @_; + if (@_ == 1) { + return $$self{$_[0]}; + } elsif (@_ > 1) { + return $self->{$_[0]} = $_[1]; + } else { + $self->_croak('Wrong number of arguments received'); + } + }; + } $VERSION = .701; }
and all dependencies are eliminated
qwurx [shmem] ~ > perl -I. -le 'use CGI::Ajax(); print "$_ => $INC{$_} +" for sort keys %INC' CGI/ => CGI/

at the expense of not being able to just print $pjx but having to say print $pjx->show_javascript.

qwurx [shmem] ~ > perl -le 'use overload "+" => \&add; print "$_ => $ +INC{$_}" for sort keys %INC' => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ => /usr/lib/perl5/5.8.8/ warnings/ => /usr/lib/perl5/5.8.8/warnings/

Of course the code using CGI::Ajax might pull in Exporter, scrict, warnings, vars et al anyways - but it is not CGI::Ajax's business to do so if it can do without.


OTOH, if the code that uses CGI::Ajax uses all those modules anyways, it is better to have them imported once instead of compiling duplicated code. So the cleanest thing would be adding a conditional in a BEGIN block, and pulling in Class::Accessor et cetera only if those namespaces are already present, because the cost of linking in already loaded modules is close to nil.

But then, this would depend on module loading order... not doing so for this particular module would just add 15 lines of code (which doesn't say much about perl's internal waste thereof... :-)

*)I just dumped the methods mk_accessors() generates and stuck them into the BEGIN block. Works.

update: corrected bug in set/get constructor


_($_=" "x(1<<5)."?\n".q·/)Oo.  G°\        /
                              /\_¯/(q    /
----------------------------  \__(m.====·.(_("always off the crowd"))."·
");sub _{s./.($e="'Itrs `mnsgdq Gdbj O`qkdq")=~y/"-y/#-z/;$e.e && print}

In reply to Re^3: AJAX popup windows - an example by shmem
in thread AJAX popup windows - an example by snowhare

Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post; it's "PerlMonks-approved HTML":

  • Are you posting in the right place? Check out Where do I post X? to know for sure.
  • Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
    <code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
  • Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
  • Want more info? How to link or or How to display code and escape characters are good places to start.
Log In?

What's my password?
Create A New User
Domain Nodelet?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (3)
As of 2021-09-20 03:18 GMT
Find Nodes?
    Voting Booth?

    No recent polls found