[Rio-pm] Comparação de arquivos

Tiago Peczenyj tiago.peczenyj em gmail.com
Sábado Novembro 24 12:15:36 PST 2012


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
-------------- Próxima Parte ----------
Um anexo em HTML foi limpo...
URL: <http://mail.pm.org/pipermail/rio-pm/attachments/20121124/cad3e326/attachment.html>


Mais detalhes sobre a lista de discussão Rio-pm