Beefy Boxes and Bandwidth Generously Provided by pair Networks vroom
Keep It Simple, Stupid
 
PerlMonks  

Re: Perl::Tidy formatting question

by vendion (Scribe)
on Nov 22, 2011 at 16:13 UTC ( #939481=note: print w/ replies, xml ) Need Help??


in reply to Perl::Tidy formatting question

I also have a question on Perl::Tidy's formatting, one complaint I have is the new line for else statements. So a if/else statement that looks like this:

if ( $foo eq 'bar' ) { # Do something } else { # Do something else }
Which in my option is perfectly readable becomes this once ran through Perl::Tidy:
if ( $foo eq 'bar' ) { # Do something } else { # Do something else }
I don't quite get the reason for breaking the else statement in such a way and if there is a way to tell it not to do such, I may have over looked it when reading through the man for it, please let me know. Also if there is some really good reason why Perl::Tidy would default to doing this that is considered best practice and doesn't cause Perl::Critic on its most brutal setting to complain about it please enlighten me.


Comment on Re: Perl::Tidy formatting question
Select or Download Code
Re^2: Perl::Tidy formatting question
by toolic (Chancellor) on Nov 22, 2011 at 16:58 UTC
    if there is a way to tell it not to do such, I may have over looked it when reading through the man for it, please let me know
    You want "cuddled else" (-ce):
    perltidy -ce script.pl
    The Perl Best Practices book recommends: "Don't cuddle an else". You need to buy the book if you want to know the reason. Perhaps perltidy adheres to this as its default behavior.

      I do have the PBP I guess I have not run across that before, what is even weirder is Perl::Critic doesn't say anything under Brutal mode and that basically "throws the book at you". Anyways thanks for the pointer and I'll have to search for the section of the PBP later today for why that is recommended.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://939481]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others musing on the Monastery: (9)
As of 2014-04-18 10:21 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    April first is:







    Results (466 votes), past polls