Beefy Boxes and Bandwidth Generously Provided by pair Networks
Your skill will accomplish
what the force of many cannot
 
PerlMonks  

Re: Idiomize This - Cleanup/Transform

by Anonymous Monk
on Apr 16, 2012 at 02:10 UTC ( [id://965227] : note . print w/replies, xml ) Need Help??


in reply to Idiomize This - Cleanup/Transform

schwern would turn each if/else block into a function. See his slideshow about this skimmable code where he tackles refactoring WWW::Mechanize, and the shorter version of the skimmable code talk, in the longer version he subliminally introduces Method::Signatures

Replies are listed 'Best First'.
Re^2: Idiomize This - Cleanup/Transform
by johnr (Acolyte) on Apr 16, 2012 at 23:30 UTC
    Thanks for the presentations. The skimmable one especially helps. I will do some refactoring passes that combine suggestions. Further benefit of pulling out the conditions into hashes/arrays of hashes is that I can move those bits into my config file.