EVGD: Códigos Toscos

Quem sacaneia quem? Depende do ponto de vista. Eu posso estar sendo sacaneado, já que pagamos por este tipo de código produzido pelos fornecedores.

Eu estaria sacaneando se informasse nomes, mas até nome de classes eu acabo mudando pra não escancarar.

O objetivo é didático, ou seja, ensinar através do mau exemplo (“não faça assim”), e não sacanear alguém.

Pense nisso!

Foi só uma brincadeira. Pelos códigos que você cola, está sendo até bonzinho :mrgreen:

Parece letra de música infantil.

Nunhum nunhum nunhum,
Nunhum nunhum nhum!
Nunhum nunhum nunhum,
Nunhuuuuuuuuuuuuuuuuuummmmm!!!..

:roll:

[quote=danieldestro]Porque diabos colocar um return deste de um finally?

public int nomeDoMetodo() { try { // ... } catc( Exception e ) { //... trata } finally { filePost.releaseConnection(); return (status == HttpStatus.SC_OK); } }[/quote]
Provavelmente o intelijegue colocou return no try, no catch e no finally ao invés de pensar em um jeito melhor…

Que tal usar NullPointerException para controle de fluxo?

try {
    // Faz alguma coisa qualquer aqui.
} catch (NullPointerException e) {
    // Faz uma outra coisa aqui, que não é tratar o erro.
}

[quote=victorwss]Que tal usar NullPointerException para controle de fluxo?

try { // Faz alguma coisa qualquer aqui. } catch (NullPointerException e) { // Faz uma outra coisa aqui, que não é tratar o erro. } [/quote]

Parece com uma discussão que houve aqui no GUJ sobre o que era melhor: usar instanceof ou capturar o ClassCastException… :mrgreen:

[quote=Schuenemann][quote=victorwss]Que tal usar NullPointerException para controle de fluxo?

try { // Faz alguma coisa qualquer aqui. } catch (NullPointerException e) { // Faz uma outra coisa aqui, que não é tratar o erro. } [/quote]

Parece com uma discussão que houve aqui no GUJ sobre o que era melhor: usar instanceof ou capturar o ClassCastException… :mrgreen:[/quote]
Bom… desde que haja uniformidade no desenvolvimento de forma que as pessoas da equipe sigam um mesmo padrão…

Porque é horrível ter que ficar desvendando as mais diferentes lógicas em pontos diferentes de um sistema grande…

1. NumberFormat nf = NumberFormat.getInstance(); 2. nf.setMinimumIntegerDigits(9); 3. System.out.println(nf.format(40).replace(".", ""));

A pog do dia. rsrsrs

bah aqui na empresa tive que arrumar uma porrada de codigos que passavam o HttpRequest direto para a camada de negocios…

Cara, nem me fale. Já tive a infelicidade de fazer esse trabalho.
Os outros cagam e você tem que limpar.
No meu caso, tinha HttpServletRequest e HttpServletResponse indo até o DAO. O HttpServletRequest também era usado como um grande repositório de variáveis públicas, mesmo que não tivessem nada a ver com a requisição ou com qualquer outra coisa. Não há porque usar atributos, variáveis locais ou parâmetros se tudo que você precisa é de um HttpServletRequest!

Uma variação de Black Cat in a Dark Room Pattern. :lol:

Hehe, peguei uma vez um caso que ao invés de declarar variáveis para cada coisa separada, havia um único Object[] bem grande e em cada posição era colocada uma coisa diferente. Preguiça de declarar variáveis.

Uma variação de Black Cat in a Dark Room Pattern. :lol: [/quote]

Sim. Inclusive alguns dos padrões dessa página fui eu que escrevi. :smiley: Basta olhar o histórico.

Alguém, por favor, avisa o cara que, somar zero a um número não altera o resultado.

IF V_TOTAL = 0 THEN V_SQ_RGIST := NVL( V_SQ_RGIST,0 ) + V_N_HUM ; ELSE V_SQ_RGIST := NVL( V_SQ_RGIST,0 ) + V_N_HUM + NVL( V_TOTAL, 0 ); END IF;

[quote=Fox McCloud][quote=Schuenemann][quote=victorwss]Que tal usar NullPointerException para controle de fluxo?

try { // Faz alguma coisa qualquer aqui. } catch (NullPointerException e) { // Faz uma outra coisa aqui, que não é tratar o erro. } [/quote]

Parece com uma discussão que houve aqui no GUJ sobre o que era melhor: usar instanceof ou capturar o ClassCastException… :mrgreen:[/quote]
Bom… desde que haja uniformidade no desenvolvimento de forma que as pessoas da equipe sigam um mesmo padrão…

Porque é horrível ter que ficar desvendando as mais diferentes lógicas em pontos diferentes de um sistema grande…[/quote]
Não existe motivo para capturar o NullPointerException ou o ClassCastException nesses casos.

Nossa, essa daqui foi demais.

  if (valor.equals(null)) {
    //faz qualquer coisa.
  }

:lol: :lol:

[quote=nbluis]Nossa, essa daqui foi demais.

  if (valor.equals(null)) {
    //faz qualquer coisa.
  }

:lol: :lol: [/quote]

:shock:

Um dia veremos isso:

pubLIc KlAXX ExXxeMPlU ExXxteNdxXx OTrAclaXXi{ pRIvaTI int iDADI; // kONStrUtoR pUbLIc exXxemPLU(){ ThixXx......iDaDi = 0;} PuBLIC gETIdaDI(){ rETUrN THixXx......iDaDI; } PuBLic SeTIdaDI(iNT idAdI){ ThixXx......IDAdi = idadI; } }

qro morre…

Fonte: http://www.coisinha.com.br/miguxeitor/

Emo Code Convention???

[quote=peczenyj]Um dia veremos isso:

pubLIc KlAXX ExXxeMPlU ExXxteNdxXx OTrAclaXXi{ pRIvaTI int iDADI; // kONStrUtoR pUbLIc exXxemPLU(){ ThixXx......iDaDi = 0;} PuBLIC gETIdaDI(){ rETUrN THixXx......iDaDI; } PuBLic SeTIdaDI(iNT idAdI){ ThixXx......IDAdi = idadI; } }

qro morre…[/quote]

uaheuaheuaheuhaeuhauehauehauehauehuaehuaeh, boa peczenyj

Att
Ricoldi