[Rio-pm] Comparação de arquivos

[Tiago Peczenyj]

ordene por severidade e se guie. se não me engano vc tem uns 9 nivels de
"verbosidade" do perlcritic e no mais avantajado (la ele) tem até exemplos.
vale a pena.

2012/11/24 Aureliano Guedes <guedes_1000 em hotmail.com>

>  Tem muitas coisas que posso desconsiderar desse perlcritc, ne?!
>
> agora em
>
> Regular expression without "/x" flag at line 49, column 18. <http://perlcritic.com/tmp/ovXTXZfUGR.html#49>  See page 236 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>
>
> fiquei sem entender, pra que esse maldito /x, não me parece que va mudar
> muita coisa.
>
> Mas legal, se jogar no nivel  parece ajudar muito... vou ler o PBP
>
> Hard tabs used at line 15, column 12. <http://perlcritic.com/tmp/ovXTXZfUGR.html#15>  See page 20 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Hard tabs used at line 17, column 17. <http://perlcritic.com/tmp/ovXTXZfUGR.html#17>  See page 20 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Close filehandles as soon as possible after opening them at line 18, column 2. <http://perlcritic.com/tmp/ovXTXZfUGR.html#18>  See page 209 of PBP.  Severity: 4 <http://perlcritic.com/pod/Perl/Critic/Policy/InputOutput/RequireBriefOpen.html>Ambiguously named variable "record" at line 20, column 2. <http://perlcritic.com/tmp/ovXTXZfUGR.html#20>  See page 48 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/NamingConventions/ProhibitAmbiguousNames.html>Regular expression without "/x" flag at line 23, column 18. <http://perlcritic.com/tmp/ovXTXZfUGR.html#23>  See page 236 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Regular expression without "/x" flag at line 29, column 20. <http://perlcritic.com/tmp/ovXTXZfUGR.html#29>  See page 236 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Hard tabs used at line 41, column 13. <http://perlcritic.com/tmp/ovXTXZfUGR.html#41>  See page 20 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Hard tabs used at line 43, column 17. <http://perlcritic.com/tmp/ovXTXZfUGR.html#43>  See page 20 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Close filehandles as soon as possible after opening them at line 44, column 2. <http://perlcritic.com/tmp/ovXTXZfUGR.html#44>  See page 209 of PBP.  Severity: 4 <http://perlcritic.com/pod/Perl/Critic/Policy/InputOutput/RequireBriefOpen.html>Ambiguously named variable "record" at line 46, column 2. <http://perlcritic.com/tmp/ovXTXZfUGR.html#46>  See page 48 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/NamingConventions/ProhibitAmbiguousNames.html>Regular expression without "/x" flag at line 49, column 18. <http://perlcritic.com/tmp/ovXTXZfUGR.html#49>  See page 236 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Regular expression without "/x" flag at line 56, column 25. <http://perlcritic.com/tmp/ovXTXZfUGR.html#56>  See page 236 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Hard tabs used at line 58, column 4. <http://perlcritic.com/tmp/ovXTXZfUGR.html#58>  See page 20 of PBP.  Severity: 3 <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>
>
>
>
> ------------------------------
> From: tiago.peczenyj em gmail.com
> Date: Sat, 24 Nov 2012 18:15:36 -0200
>
> To: rio-pm em pm.org
> Subject: Re: [Rio-pm] Comparação de arquivos
>
> use o nivel 3 (mediano)
>
> 2012/11/24 Aureliano Guedes <guedes_1000 em hotmail.com>
>
>  Opa, obrigado então por toda ajuda.
>
> No mais, so o log do perlcritc.
>
> http://pastebin.com/esYuR04J
>
> tenso, ne?!
>
> > Date: Sat, 24 Nov 2012 18:01:13 -0200
>
> > From: breno em rio.pm.org
> > To: rio-pm em pm.org
> > Subject: Re: [Rio-pm] Comparação de arquivos
> >
> > 2012/11/24 Aureliano Guedes <guedes_1000 em hotmail.com>:
> > > Breno, que ser meu orientador... hahaha...
> > >
> > > Enfim, ficou assim http://pastebin.com/zrpLGwQs
> > >
> > > Ha algo que você mudaria???
> > >
> >
> > Sempre há :)
> >
> > Particularmente, não gosto dessa sua chamada a main() no final do
> > arquivo, pq esconde a lógica e dá pouca confiança a quem vai olhar o
> > seu código (instruções podem aparecer a qualquer momento fora de uma
> > sub, o figura tem que passar por todo o seu script até saber que vc só
> > está chamando main()). Eu trocaria a ordem, deixando assim:
> >
> > --------------------8<--------------------
> > use strict;
> > use warnings;
> > use autodie;
> >
> > main();
> > exit;
> >
> > # declaracoes de sub aqui embaixo
> > -------------------->8--------------------
> >
> > Assim vc deixa claro o (começo do) fluxo do seu programa e mostra ao
> > leitor que não haverão surpresas no meio das subs.
> >
> > No mais, experimente usar o Perl::Critic (dá até pra usar online via
> > perlcritic.com) e veja se ele reclama de alguma coisa.
> >
> > []s
> >
> > -b
> > _______________________________________________
> > Rio-pm mailing list
> > Rio-pm em pm.org
> > http://mail.pm.org/mailman/listinfo/rio-pm
>
> _______________________________________________
> Rio-pm mailing list
> Rio-pm em pm.org
> http://mail.pm.org/mailman/listinfo/rio-pm
>
>
>
>
> --
> Tiago B. Peczenyj
> Linux User #405772
>
> http://pacman.blog.br
>
> _______________________________________________ Rio-pm mailing list
> Rio-pm em pm.org http://mail.pm.org/mailman/listinfo/rio-pm
>
> _______________________________________________
> Rio-pm mailing list
> Rio-pm em pm.org
> http://mail.pm.org/mailman/listinfo/rio-pm
>



-- 
Tiago B. Peczenyj
Linux User #405772

http://pacman.blog.br
-------------- Próxima Parte ----------
Um anexo em HTML foi limpo...
URL: <http://mail.pm.org/pipermail/rio-pm/attachments/20121124/9128f6e1/attachment.html>