EVGD: Códigos Toscos

Certa fez, disseram para um determinado pogueiro que conheço, que colocar números mágicos no código era ruim, que o certo era definir constantes e acessar sempre as constantes, para deixar a manutenção no código mais fácil.

O resultado disso foi mais ou menos isso:

JWizard Membro desde: 23/02/2007 15:52:55 Mensagens: 2034 Offline

mais uma contribuição

java.lang.Exception mesmo

This message was edited 1 time. Last update was at 03/07/2009 08:31:39

E esse caso?! Sem comentários...

quem nunca fez uma pog que atire a primeira pedra^^

Jose111 wrote:quem nunca fez uma pog que atire a primeira pedra^^

Eu já fiz, assumo. Inclusive postei uma de minha própria autoria várias páginas atrás.

victorwss wrote:Certa fez, disseram para um determinado pogueiro que conheço, que colocar números mágicos no código era ruim, que o certo era definir constantes e acessar sempre as constantes, para deixar a manutenção no código mais fácil.

O resultado disso foi mais ou menos isso:
public static final int UM = 1; public static final int DOIS = 2; public static final int TRES = 3; public static final int QUATRO = 4; public static final int CINCO = 5; ...if (sessao.getUsuario().getIsso().fazAquilo().metodoBizarro() == QUARENTA_E_TRES) { ... }

Putz nem me fala o sistema que eu estou mexendo atual, tem trocentas constantes pra lá e pra cá.. mas cá entre nós a única alternativa é os enums ou colocar tudo isso em properties? pra mim no fundo parece tudo igual só muda a cara da coisa..( cara de gambiarra )

This message was edited 1 time. Last update was at 03/07/2009 21:16:35

albertongai wrote:

victorwss wrote:Certa fez, disseram para um determinado pogueiro que conheço, que colocar números mágicos no código era ruim, que o certo era definir constantes e acessar sempre as constantes, para deixar a manutenção no código mais fácil.

O resultado disso foi mais ou menos isso:
public static final int UM = 1; public static final int DOIS = 2; public static final int TRES = 3; public static final int QUATRO = 4; public static final int CINCO = 5; ...if (sessao.getUsuario().getIsso().fazAquilo().metodoBizarro() == QUARENTA_E_TRES) { ... }

Putz nem me fala o sistema que eu estou mexendo atual, tem trocentas constantes pra lá e pra cá.. mas cá entre nós a única alternativa é os enums ou colocar tudo isso em properties? pra mim no fundo parece tudo igual só muda a cara da coisa..( cara de gambiarra )

Tenho um amigo que diz que prefere uma classe com umonte de statics do que enum... Não desse tipo dos números que o Victor passou, mas enfim...

peczenyj wrote:

paulog wrote:Enquanto isso na minha empresa...
for (int i = 0; i < 100; ++i) { try { recurso.close(); } catch (Throwable e) {} }
Alguém queria ter certeza que o recurso iria ser fechado.
Se eu colocar o for até 50, será que melhora a performance?

É para fechar muito bem fechado. Ao menos não tem um sleep no meio.

Ahahahahaha...
Basicamente, já falta um break depois do close(). Se o uso do for fosse "válido", então que pelo menos estivesse correto.

Um colega meu deu uma alternativa:

Assim é possível determinar a "performance" do for mudando o valor de FRACTION. Prontamente respondi, falando que deveríamos até disponibilizar para o usuário algum controle visual para mudar esse valor, como se fosse uma resistência de um circuito elétrico.
Hahahahaha... Poderia ler o valor de alguma propriedade também...

JavaEvangelist Membro desde: 21/11/2004 16:40:00 Mensagens: 424 Offline

JavaScript + Scriptlet + Java = Aberrações

atentem para o detalhe de se criar uma variável javasricot no meio do scriptlet

Oh SHIT..

Debugger Membro desde: 13/05/2009 12:16:08 Mensagens: 64 Offline

estou bestificada olhando para este código. O método não faz nada de útil, só aumenta o tempo de processamento.

Tosco, não? Além do mais as principais variáveis são estáticas, todas as classes estão acessando-as, alterando-as. Os botões são estáticos, criados num canto, adicionados em outro, capturado evento em outra classe. Nada de encapsulamento, péssimas praticas de OO, difícil entendimento e manutenção.

Vontade de pegar o infeliz que fez isso e...

GUJ Expert Membro desde: 10/07/2006 10:37:23 Mensagens: 3161 Offline

quem não lembra da velha gambi para fazer muitas vezes sqls dinamicos?

luistiagos wrote:quem não lembra da velha gambi para fazer muitas vezes sqls dinamicos?

where 1 = 1

Eu usava isso para fazer SQL Injection. XD

JavaTeenager Membro desde: 09/03/2005 09:49:33 Mensagens: 182 Offline

Eu gosto dos comentários e os nomes dos métodos misturando português com inglês..
E o uso do FIXME e sem falar do números mágicos.
E pior aí vc clica com o direito e vai em referências no projeto abre uma lista com 325 ocorrências.

GUJ Ranger Membro desde: 13/01/2005 12:31:27 Mensagens: 854 Offline

Avisa a esse sujeito sobre o método replaceAll.

Notícias, artigos e o maior fórum brasileiro sobre Java