[SP-pm] Sugestão para melhorar código

["Flávio R. Lopes"]

Olá Breno!
Cara, tem certeza que você está respondendo nesta thread ?
Ainda não escrevi nenhum módulo (quem dera!) e muito menos é meu 
aniverário... :)

Em 01-10-2010 20:54, breno escreveu:
> Que bom que agradou! Qq dúvida, crítica ou sugestão, vc conhece o autor :-)
>
> []s
>
> -b (garu)
>
> ps: ver que recomendaram um módulo seu, sem qq influência direta sua,
> e no *dia* do seu aniversário.... puts, não tem preço! \o/
>
>
> 2010/9/29 "Flávio R. Lopes"<flavio.lopes em links.inf.br>:
>    
>> Olá Solli !
>> Putz...maravilhoso para trabalhar com scripts via linha de comando!!
>> Estarei olhando este módulo também.
>>
>> Obrigado pela dica!
>>
>> Abraço,
>> Flávio
>>
>> Em 29-09-2010 22:22, Solli Honorio escreveu:
>>
>> Cara,
>>
>> Dê uma olhada no App::Rad
>> (http://search.cpan.org/~garu/App-Rad-1.04/lib/App/Rad.pm).
>>
>> Solli M. Honorio
>>
>> Em 29 de setembro de 2010 20:12, "Flávio R. Lopes"
>> <flavio.lopes em links.inf.br>  escreveu:
>>      
>>> Obrigado Nelson.
>>> Estarei estudando suas sugestões sim e vou ver no CPAN como funciona o
>>> Getopt::Long, pois até então não conhecia este módulo.
>>>
>>> abraço,
>>> Flávio
>>>
>>>
>>>
>>>
>>> Em 29-09-2010 19:33, Nelson Ferraz escreveu:
>>>        
>>>> 2010/9/29 "Flávio R. Lopes"<flavio.lopes em links.inf.br>:
>>>>
>>>>          
>>>>> Olá pessoal.
>>>>>
>>>>> Fiz um programa que gera um backup dos diretórios informados pelo
>>>>> usuário e
>>>>> ainda criptografa o arquivo gerado.
>>>>> Gostaria que se possível vocês dessem uma olhada no código e me dessem
>>>>> sugestões para enxugar mais ou mesmo otimizá-lo.
>>>>> Quando digo "enxugar" e "otimizar" gostaria de sugestões, pois na
>>>>> verdade
>>>>> vocês poderiam escrever de outra forma que eu não tenha enxergado, para
>>>>> que
>>>>> eu possa ver outras formas de se escrever.
>>>>>
>>>>> Aqui segue ele: http://gist.github.com/603556
>>>>>
>>>>> Acho que a maior parte das funcionalidades para qual escrevi ele
>>>>> estariam
>>>>> prontas. O que ainda farei é dar a opção (STDIN) para que o usuário
>>>>> possa
>>>>> informar a senha, diretórios de origem, diretório de destino, nome do
>>>>> backup
>>>>> e tipo de compressão sem ter que ficar editando o código.
>>>>>
>>>>>            
>>>> Flavio,
>>>>
>>>> Parabéns pelo código, parece muito bom.
>>>>
>>>> Algumas recomendações:
>>>>
>>>> 1) Escreva a interface do seu código em ingles. "directory" ao invés
>>>> de "diretorio", "target" ao invés de "destino".
>>>>
>>>> 2) Use o Getopt::Long para ler as opções de linha de comando. Alguma
>>>> coisa como (obs: não testei o código a seguir):
>>>>
>>>> use Getopt::Long;
>>>>
>>>> my %options;
>>>>
>>>> my $result = GetOptions(
>>>>      \%options,
>>>>      'directory|dir|d=s',
>>>>      'target|t=s',
>>>>      'prefix|p=s',
>>>> );
>>>>
>>>> A partir daí você deve checar e usar as variáveis $opt{directory},
>>>> $opt{target} e $opt{prefix}. Adicione mais opções de linha de comando
>>>> se necessário.
>>>>
>>>> 3) Escreva a documentação do seu código em formato POD, após o __END__:
>>>>
>>>> __END__
>>>>
>>>> =head1 NAME
>>>>
>>>>      backup.pl - brief description of your script
>>>>
>>>> =head1 SYNOPSIS
>>>>
>>>>      backup.pl [options]
>>>>
>>>>      Where options are:
>>>>
>>>>          --directory
>>>>          --target
>>>>          --prefix
>>>>
>>>> =head1 DESCRIPTION
>>>>
>>>> A more complete description of your module. (Purpose, etc)
>>>>
>>>> =head1 AUTHOR
>>>>
>>>>      Flávio R. Lopes<flavio.lopes em links.inf.br>
>>>> _______________________________________________
>>>> SaoPaulo-pm mailing list
>>>> SaoPaulo-pm em pm.org
>>>> http://mail.pm.org/mailman/listinfo/saopaulo-pm
>>>>
>>>>
>>>>          
>>> _______________________________________________
>>> SaoPaulo-pm mailing list
>>> SaoPaulo-pm em pm.org
>>> http://mail.pm.org/mailman/listinfo/saopaulo-pm
>>>
>>>        
>>
>>
>> --
>> "o animal satisfeito dorme". - Guimarães Rosa
>>
>> _______________________________________________
>> SaoPaulo-pm mailing list
>> SaoPaulo-pm em pm.org
>> http://mail.pm.org/mailman/listinfo/saopaulo-pm
>>
>> _______________________________________________
>> SaoPaulo-pm mailing list
>> SaoPaulo-pm em pm.org
>> http://mail.pm.org/mailman/listinfo/saopaulo-pm
>>
>>      
> _______________________________________________
> SaoPaulo-pm mailing list
> SaoPaulo-pm em pm.org
> http://mail.pm.org/mailman/listinfo/saopaulo-pm
>
>