Risk-Aware AI Code Review for Business-Critical Systems
LiveReview LiveReview Explore LiveReview

Perl::Critic::Policy::InputOutput::RequireBracedFileHandleWithPrint - Write "print {$FH} $foo, $bar;"

Affiliation

       This Policy is part of the core Perl::Critic distribution.

Author

       Jeffrey Ryan Thalhammer <jeff@imaginative-software.com>

Configuration

       This Policy is not configurable except for the standard options.

Description

       The "print" and "printf" functions have a unique syntax that supports an optional file handle argument.
       Conway suggests wrapping this argument in braces to make it visually stand out from the other arguments.
       When you put braces around any of the special package-level file handles like "STDOUT", "STDERR", and
       "DATA", you must add the '*' sigil or else it won't compile under use strict 'subs'.

         print $FH   "Mary had a little lamb\n";  #not ok
         print {$FH} "Mary had a little lamb\n";  #ok

         print   STDERR   $foo, $bar, $baz;  #not ok
         print  {STDERR}  $foo, $bar, $baz;  #won't compile under 'strict'
         print {*STDERR}  $foo, $bar, $baz;  #perfect!

Name

       Perl::Critic::Policy::InputOutput::RequireBracedFileHandleWithPrint - Write "print {$FH} $foo, $bar;"
       instead of "print $FH $foo, $bar;".

See Also