Depends on what you want it to do. You need to be aware that:

• It won't work if it's in a module that you're importing elsewhere (because you end up changing Module::foo, which is irrelevant to the ModuleUser::foo that was imported).

  You can change that with

    no strict 'refs';
    my $pkg = caller;
    *{ $pkg . "::foo" } = sub { ... }; # the final foo
  [download]

  but only if that's what you want ("call once per user" instead of "call once globally").
• There's no easy way to reset the trigger after it's fired. For your application, this is probably what you want, but you could imagine scenarios where it is not (eg mod_perl).

ikegami's lexical variable version gives "call once globally" semantics, without allowing resets (which you can trivially add by introducing another subroutine that captures the same lexical.) Same for the global variable, except resetting can be done directly.

Personally, I like the idiom

sub foo {
  our $CALL_COUNT;
  call_me_only_once() unless $CALL_COUNT++;
  ...do stuff...
}
[download]

But I'll admit I always wonder whether the code is going to get run 4.3 billion times and redo the init code...

---

---

• 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 How to display code and escape characters are good places to start.