search

heavenstudio / pag_seguro

A Ruby Client to deal with PagSeguro's API
http://heavenstudio.com.br
71 stars 38 forks source link

Improve code readability #11

Closed efmiglioranza closed 11 years ago

efmiglioranza commented 11 years ago

Buenas, fiz algumas melhorias estéticas no código. Diminuí o tamanho de algumas linhas para facilitar a leitura. Mesmo com monitores maiores, eu geralmente tenho 2-3 arquivos abertos na mesma tela e as vezes dificulta ter que ficar rolando cada um deles pros lados, então tentei manter um padrão de até 80 chars/linha. Removi também os espaços em branco adicionais no código e nos specs.

kurko commented 11 years ago

Está muito bom para mim, @efmiglioranza. Poderia dar apenas uma olhada no comentário acima?

efmiglioranza commented 11 years ago

Eu tinha apenas quebrado a linha, mas analisando o caso, realmente \A e \z é melhor. http://www.ruby-doc.org/docs/ProgrammingRuby/html/language.html#UL

ps: utilizei \z ao invés do \Z para ele considerar toda a string, sem parar em algum possível \n.

teonimesic commented 11 years ago

Eu discordo de \a \z ao invés de ^ e $. Talvez seja porque estudei PERL antes de Ruby ( e a sintaxe de Ruby veio de PERL ), mas acho ^ e $ mais claros como início e término do match. O que acham? Se o \a \z fizer alguma diferença em termos de código, seria ter legal algum teste que verifique o comportamento errado de funcionamento antes de fazer a alteração.

teonimesic commented 11 years ago

@efmiglioranza te dei acesso de commit ao repositório, então pode fazer os merges diretos ao código :D. Só peço que sempre mantenha a cobertura de testes em caso de adições de código.

efmiglioranza commented 11 years ago

@teonimesic é que em Ruby eles tem semânticas diferentes. Exemplos:

^ Matches the beginning of a line. $ Matches the end of a line. \A Matches the beginning of the string. \z Matches the end of the string. \Z Matches the end of the string unless the string ends with a \n'', in which case it matches just before the\n''.

logo:

com /^...$/ a entrada "16.52\nqualquercoisa" é válida com /\A...\z/ a entrada "16.52\nqualquercoisa" não é válida

fonte: http://www.ruby-doc.org/docs/ProgrammingRuby/html/language.html#UL

vou fazer um teste pra esse caso :)

efmiglioranza commented 11 years ago

@teonimesic valeu pelo acesso :D bom, adicionei os testes, es estiver de acordo eu faço o merge, abraço!

teonimesic commented 11 years ago

Estou de acordo sim! Pode mandar bala!