<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
Opa, obrigado então por toda ajuda.<br><br>No mais, so o log do perlcritc.<br><br>http://pastebin.com/esYuR04J<br><br>tenso, ne?!<br><br><div><div id="SkyDrivePlaceholder"></div>> Date: Sat, 24 Nov 2012 18:01:13 -0200<br>> From: breno@rio.pm.org<br>> To: rio-pm@pm.org<br>> Subject: Re: [Rio-pm] Comparação de arquivos<br>> <br>> 2012/11/24 Aureliano Guedes <guedes_1000@hotmail.com>:<br>> > Breno, que ser meu orientador... hahaha...<br>> ><br>> > Enfim, ficou assim http://pastebin.com/zrpLGwQs<br>> ><br>> > Ha algo que você mudaria???<br>> ><br>> <br>> Sempre há :)<br>> <br>> Particularmente, não gosto dessa sua chamada a main() no final do<br>> arquivo, pq esconde a lógica e dá pouca confiança a quem vai olhar o<br>> seu código (instruções podem aparecer a qualquer momento fora de uma<br>> sub, o figura tem que passar por todo o seu script até saber que vc só<br>> está chamando main()). Eu trocaria a ordem, deixando assim:<br>> <br>> --------------------8<--------------------<br>> use strict;<br>> use warnings;<br>> use autodie;<br>> <br>> main();<br>> exit;<br>> <br>> # declaracoes de sub aqui embaixo<br>> -------------------->8--------------------<br>> <br>> Assim vc deixa claro o (começo do) fluxo do seu programa e mostra ao<br>> leitor que não haverão surpresas no meio das subs.<br>> <br>> No mais, experimente usar o Perl::Critic (dá até pra usar online via<br>> perlcritic.com) e veja se ele reclama de alguma coisa.<br>> <br>> []s<br>> <br>> -b<br>> _______________________________________________<br>> Rio-pm mailing list<br>> Rio-pm@pm.org<br>> http://mail.pm.org/mailman/listinfo/rio-pm<br></div>                                           </div></body>
</html>