.\" -*- mode: troff; coding: utf-8 -*- .\" Automatically generated by Pod::Man 5.01 (Pod::Simple 3.43) .\" .\" Standard preamble: .\" ======================================================================== .de Sp \" Vertical space (when we can't use .PP) .if t .sp .5v .if n .sp .. .de Vb \" Begin verbatim text .ft CW .nf .ne \\$1 .. .de Ve \" End verbatim text .ft R .fi .. .\" \*(C` and \*(C' are quotes in nroff, nothing in troff, for use with C<>. .ie n \{\ . ds C` "" . ds C' "" 'br\} .el\{\ . ds C` . ds C' 'br\} .\" .\" Escape single quotes in literal strings from groff's Unicode transform. .ie \n(.g .ds Aq \(aq .el .ds Aq ' .\" .\" If the F register is >0, we'll generate index entries on stderr for .\" titles (.TH), headers (.SH), subsections (.SS), items (.Ip), and index .\" entries marked with X<> in POD. Of course, you'll have to process the .\" output yourself in some meaningful fashion. .\" .\" Avoid warning from groff about undefined register 'F'. .de IX .. .nr rF 0 .if \n(.g .if rF .nr rF 1 .if (\n(rF:(\n(.g==0)) \{\ . if \nF \{\ . de IX . tm Index:\\$1\t\\n%\t"\\$2" .. . if !\nF==2 \{\ . nr % 0 . nr F 2 . \} . \} .\} .rr rF .\" ======================================================================== .\" .IX Title "Perl::Critic::Policy::InputOutput::ProhibitBacktickOperators 3pm" .TH Perl::Critic::Policy::InputOutput::ProhibitBacktickOperators 3pm 2023-07-26 "perl v5.38.0" "User Contributed Perl Documentation" .\" For nroff, turn off justification. Always turn off hyphenation; it makes .\" way too many mistakes in technical documents. .if n .ad l .nh .SH NAME Perl::Critic::Policy::InputOutput::ProhibitBacktickOperators \- Discourage stuff like "@files = \`ls $directory\`". .SH AFFILIATION .IX Header "AFFILIATION" This Policy is part of the core Perl::Critic distribution. .SH DESCRIPTION .IX Header "DESCRIPTION" Backticks are super-convenient, especially for CGI programs, but I find that they make a lot of noise by filling up STDERR with messages when they fail. I think its better to use IPC::Open3 to trap all the output and let the application decide what to do with it. .PP .Vb 2 \& use IPC::Open3 \*(Aqopen3\*(Aq; \& $SIG{CHLD} = \*(AqIGNORE\*(Aq; \& \& @output = \`some_command\`; #not ok \& \& my ($writer, $reader, $err); \& open3($writer, $reader, $err, \*(Aqsome_command\*(Aq); #ok; \& @output = <$reader>; #Output here \& @errors = <$err>; #Errors here, instead of the console .Ve .SH CONFIGURATION .IX Header "CONFIGURATION" Alternatively, if you do want to use backticks, you can restrict checks to void contexts by adding the following to your \&\fI.perlcriticrc\fR file: .PP .Vb 2 \& [InputOutput::ProhibitBacktickOperators] \& only_in_void_context = 1 .Ve .PP The purpose of backticks is to capture the output of an external command. Use of them in a void context is likely a bug. If the output isn't actually required, \f(CWsystem()\fR should be used. Otherwise assign the result to a variable. .PP .Vb 3 \& \`some_command\`; #not ok \& $output = \`some_command\`; #ok \& @output = \`some_command\`; #ok .Ve .SH NOTES .IX Header "NOTES" This policy also prohibits the generalized form of backticks seen as \&\f(CW\*(C`qx{}\*(C'\fR. .PP See perlipc for more discussion on using \f(CWwait()\fR instead of \f(CW\*(C`$SIG{CHLD} = \*(AqIGNORE\*(Aq\*(C'\fR. .PP You might consider using the \f(CWcapture()\fR function from the IPC::System::Simple module for a safer way of doing what backticks do, especially on Windows. The module also has a safe wrapper around \f(CWsystem()\fR. .SH AUTHOR .IX Header "AUTHOR" Jeffrey Ryan Thalhammer .SH COPYRIGHT .IX Header "COPYRIGHT" Copyright (c) 2005\-2011 Imaginative Software Systems. All rights reserved. .PP This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module.