.\" Automatically generated by Pod::Man 4.14 (Pod::Simple 3.40) .\" .\" 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 .. .\" Set up some character translations and predefined strings. \*(-- will .\" give an unbreakable dash, \*(PI will give pi, \*(L" will give a left .\" double quote, and \*(R" will give a right double quote. \*(C+ will .\" give a nicer C++. Capital omega is used to do unbreakable dashes and .\" therefore won't be available. \*(C` and \*(C' expand to `' in nroff, .\" nothing in troff, for use with C<>. .tr \(*W- .ds C+ C\v'-.1v'\h'-1p'\s-2+\h'-1p'+\s0\v'.1v'\h'-1p' .ie n \{\ . ds -- \(*W- . ds PI pi . if (\n(.H=4u)&(1m=24u) .ds -- \(*W\h'-12u'\(*W\h'-12u'-\" diablo 10 pitch . if (\n(.H=4u)&(1m=20u) .ds -- \(*W\h'-12u'\(*W\h'-8u'-\" diablo 12 pitch . ds L" "" . ds R" "" . ds C` "" . ds C' "" 'br\} .el\{\ . ds -- \|\(em\| . ds PI \(*p . ds L" `` . ds R" '' . 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::CodeLayout::ProhibitIfIfSameLine 3pm" .TH Perl::Critic::Policy::CodeLayout::ProhibitIfIfSameLine 3pm "2021-02-28" "perl v5.32.1" "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::CodeLayout::ProhibitIfIfSameLine \- don't put if after if on same line .SH "DESCRIPTION" .IX Header "DESCRIPTION" This policy is part of the \f(CW\*(C`Perl::Critic::Pulp\*(C'\fR add-on. It asks you to not to write an \f(CW\*(C`if\*(C'\fR statement on the same line as a preceding \f(CW\*(C`if\*(C'\fR. .PP .Vb 5 \& if ($x) { \& ... \& } if ($y) { # bad \& ... \& } \& \& if ($x) { \& ... \& } elsif ($y) { # was "elsif" intended ? \& ... \& } .Ve .PP The idea is that an \f(CW\*(C`if\*(C'\fR in the layout of an \f(CW\*(C`elsif\*(C'\fR may be either a mistake or will be confusing to a human reader. On that basis this policy is under the \*(L"bugs\*(R" theme and medium severity (see \*(L"\s-1POLICY THEMES\*(R"\s0 in Perl::Critic). .SS "Unless" .IX Subsection "Unless" An \f(CW\*(C`unless...if\*(C'\fR is treated the same. Perl allows \f(CW\*(C`unless ... elsif\*(C'\fR and so the same potential confusion with an \f(CW\*(C`elsif\*(C'\fR layout arises. .PP .Vb 5 \& unless ($x) { \& ... \& } if ($y) { # bad \& ... \& } \& \& unless ($x) { \& ... \& } elsif ($y) { # maybe meant to be "elsif" like this ? \& ... \& } .Ve .PP Whether \f(CW\*(C`unless ... elsif\*(C'\fR is a good idea at all is another matter. Sometimes it suits a combination of conditions. .SS "Statement Modifiers" .IX Subsection "Statement Modifiers" This policy only applies to a statement followed by a statement. An \f(CW\*(C`if\*(C'\fR as a statement modifier is not affected. It's usual to put that on the same line as the statement it modifies. .PP .Vb 3 \& do { \& ... \& } if ($x); # ok, statement modifier .Ve .SS "All One Line" .IX Subsection "All One Line" Two \f(CW\*(C`if\*(C'\fR statements written on the same line will trigger the policy. .PP .Vb 1 \& if(1){one;} if(2){two;} # bad .Ve .PP Perhaps there could be an exception or option when both statements are entirely on the one line, or some such, for code which is trying to be compact. .SS "Disabling" .IX Subsection "Disabling" As always if you don't care about this then you can disable \&\f(CW\*(C`ProhibitIfIfSameLine\*(C'\fR from your \fI.perlcriticrc\fR (see \&\*(L"\s-1CONFIGURATION\*(R"\s0 in Perl::Critic), .PP .Vb 1 \& [\-CodeLayout::ProhibitIfIfSameLine] .Ve .SH "SEE ALSO" .IX Header "SEE ALSO" Perl::Critic::Pulp, Perl::Critic .PP Perl::Critic::Policy::ControlStructures::ProhibitCascadingIfElse, Perl::Critic::Policy::ControlStructures::ProhibitUnlessBlocks .SH "HOME PAGE" .IX Header "HOME PAGE" .SH "COPYRIGHT" .IX Header "COPYRIGHT" Copyright 2013, 2014, 2015, 2016, 2017, 2019, 2021 Kevin Ryde .PP Perl-Critic-Pulp is free software; you can redistribute it and/or modify it under the terms of the \s-1GNU\s0 General Public License as published by the Free Software Foundation; either version 3, or (at your option) any later version. .PP Perl-Critic-Pulp is distributed in the hope that it will be useful, but \&\s-1WITHOUT ANY WARRANTY\s0; without even the implied warranty of \s-1MERCHANTABILITY\s0 or \s-1FITNESS FOR A PARTICULAR PURPOSE.\s0 See the \s-1GNU\s0 General Public License for more details. .PP You should have received a copy of the \s-1GNU\s0 General Public License along with Perl-Critic-Pulp. If not, see .