EVGD: Códigos Toscos

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.

GUJ Expert Membro desde: 18/02/2008 16:17:53 Mensagens: 3002 Offline

Feijão wrote:Para mim uma verdadeira obra de arte:

public int qdadePorPessoaPorEventoPorSessao(Timestamp sessaoDate) { sessaoDate = updateDataEventoIfUniqueEvento(sessaoDate); int count = 0; try { for (SessaoPessoa sessaoPessoa : EventoBusiness.getInstance().getSessaoPessoaByEvento(new Integer(codigoEvento))){ if (sessaoPessoa.getPk().getCodPessoa().intValue()== new Integer(codigoAssinatura).intValue() && sessaoPessoa.getPk().getCodEvento().compareTo(new Integer(codigoEvento))==0 && sessaoPessoa.getPk().getDatSessao().compareTo(sessaoDate) ==0){ count++; } } return new Integer(count); } catch(Exception e){ e.printStackTrace(); } return new Integer(0); }

Cada getPk é uma consulta ao banco?

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

É irritante ver código com esses new Integer().

Schuenemann wrote:É irritante ver código com esses new Integer().

Você não viu o código nojento que eu estava trabalhando esses dias. Devia ter pelo menos uns dez mil new Integer() espalhados.
Aliás, dava para postar o projeto inteiro neste tópico.

Código "doado" por um amigo que já não aguenta mais encontrar tosquices todos os dias:

function vaiParaOProximoCampo(campoAtual) {

if (campoAtual.name == 'codPessoa') {
        if (campoAtual.value.length == 9) {
            document.oForm.emailPessoa.focus();
        }
    } else if (campoAtual.name == 'diaNascimentoPessoa') {
        if (campoAtual.value.length == 2) {
            document.oForm.mesNascimentoPessoa.focus();
        }
    } else if (campoAtual.name == 'mesNascimentoPessoa') {
        if (campoAtual.value.length == 2) {
            document.oForm.anoNascimentoPessoa.focus();
        }
    } else if (campoAtual.name == 'anoNascimentoPessoa') {
        if (campoAtual.value.length == 4) {
            document.oForm.cpfPessoa.focus();
        }
    } else if (campoAtual.name == 'cpfPessoa') {
        if (campoAtual.value.length == 11) {
            document.oForm.telefoneResidencialDDD.focus();
        }
    } else if (campoAtual.name == 'cnpj') {
        if (campoAtual.value.length == 14) {
            document.oForm.telefoneResidencialDDD.focus();
        }
    } else if (campoAtual.name == 'telefoneResidencialDDD') {
        if (campoAtual.value.length == 2) {
            document.oForm.telefoneResidencial.focus();
        }
    } else if (campoAtual.name == 'telefoneResidencial') {
        if (campoAtual.value.length == 8) {
            document.oForm.telefoneComercialDDD.focus();
        }
    } else if (campoAtual.name == 'telefoneComercialDDD') {
        if (campoAtual.value.length == 2) {
            document.oForm.telefoneComercial.focus();
        }
    } else if (campoAtual.name == 'telefoneComercial') {
        if (campoAtual.value.length == 8) {
            document.oForm.telefoneCelularDDD.focus();
        }
    } else if (campoAtual.name == 'telefoneCelularDDD') {
        if (campoAtual.value.length == 2) {
            document.oForm.telefoneCelular.focus();
        }
    } else if (campoAtual.name == 'telefoneCelular') {
        if (campoAtual.value.length == 8) {
            document.oForm.senhaPessoa.focus();
        }
    }

GUJ Expert Membro desde: 13/11/2008 12:10:18 Mensagens: 3928 Online

thingol wrote:

danieldestro wrote:Código "doado" por um amigo que já não aguenta mais encontrar tosquices todos os dias:

function vaiParaOProximoCampo(campoAtual) { ...

Ainda se funcionasse...

O cara deve ter confundido com pascal. rs

Moderador Membro desde: 29/07/2004 16:10:13 Mensagens: 17543 Offline

juliocbq wrote:

thingol wrote:

danieldestro wrote:Código "doado" por um amigo que já não aguenta mais encontrar tosquices todos os dias:

function vaiParaOProximoCampo(campoAtual) { ...

Ainda se funcionasse...

O cara deve ter confundido com pascal. rs

Dessa vez foi tosquice minha. O código é JavaScript, não Java, portanto dá para comparar strings com "==". Mesmo assim esse código parece um pouco tosco.

Vi um que fiquei espanidiotizado.

O cara abaixo tinha medo de lançar uma Exception pra cima então decediu tentar denovo.

hAUhAU

Entusiasta Java Membro desde: 26/05/2009 11:32:44 Mensagens: 20 Offline

vi alguem na facul fazer:

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

Vale PL/SQL?

Gente, esses dias peguei um:

GUJ Master Membro desde: 26/03/2006 18:54:30 Mensagens: 1861 Offline

Essa é fresquinha, tirei agora aqui do fórum

Para quê usar o StringBuilder então, usa logo concatenação de strings

sugestão:

This message was edited 2 times. Last update was at 11/06/2009 07:51:48

E o que vocês acham disso? o cara usa constantes como variáveis globais. e isso se repete infinitas vezes pelo sistema... em que eu estou dando manutenção

p.s.: esse sai do mesmo lugar daonde o Destro tirou vááááááááááááários dos que ele mesmo já colocou aqui e que amigos ainda passam para ele...

edit 1: Alterei nomes de variáveis e classes.

Tenho que compartilhar com o mundo, pois obras de arte foram feitas para isso.
Se este método funciona não sei, mas os nomes são maravilhosos. Show de bola. Parabéns campeão!

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