O impossível acontece aqui

249
Thanks!
An error occurred!
if( preco == null && preco > 100.0 ) {  
  // ...  
}

São Tomé

246
Thanks!
An error occurred!

Vai que as leis do universo mudem.

if (true==true) {  
           (...)
}

A verdade sempre aparece

243
Thanks!
An error occurred!
if(true){  
    total = total + 1;  
}

deveria ser:

total = total + 1;

Time que está ganhando …

251
Thanks!
An error occurred!
setCodEmpresa(getCodEmpresa());  
setNomeCliente(getNomeCliente());

Só Jesus Salva

240
Thanks!
An error occurred!

Repare no “;” após o curto-circuíto do if.

 
    if(a<5);  
        salvarLinha();

Deveria ser:

 
    if(a<5)
    {
        salvarLinha();
    }

Else desnecessário

237
Thanks!
An error occurred!

O comentário é real!

 if (idade == novaIdade) {  
         (...)
    } else {  
        //faz nada  
    }

Deveria ser:

 if (idade == novaIdade) {  
         (...)
    }

Assinatura de função

259
Thanks!
An error occurred!

Existe um motivo de você utilizar gets e sets…

    public void salvar(Cliente c){  
       boolean isPagante = c.getPago() != 0;  
       salvarTudo(c.getId(), c.getIdade(), c.getUF(),isPagante , ....(19 parâmetros));
    }  
 
    private void salvarTudo(... 23 parâmetros...){  
                      (...)
    }

Quando puder, complique!

231
Thanks!
An error occurred!

<PENSAMENTO OGRO>

Quanto mais complexo, menor a chance de te mandarem embora para colocarem aquele estagiário que se acha o bom no seu lugar. Vamos incentivar o uso do código abaixo ^^

</PENSAMENTO OGRO>

 public static Timestamp obterTimestampPorDataDDMMYYYY(String stData) {  
        SimpleDateFormat sdf = new SimpleDateFormat("dd/MM/yyyy");  
        String strDate;  
        Timestamp dataTimestamp;  
 
        try {  
            if (new SimpleDateFormat("dd/MM/yyyy").format(sdf.parse(stData)).equals(stData)) {  
                strDate = new SimpleDateFormat("dd-MM-yyyy 00:00:00").format(sdf.parse(stData));  
                dataTimestamp = new Timestamp(new SimpleDateFormat("dd-MM-yyyy HH:mm:ss").parse(strDate).getTime());  
            } else {  
                dataTimestamp = null;  
            }  
 
        } catch (ParseException e) {  
            dataTimestamp = null;  
        } catch (Exception e) {  
            dataTimestamp = null;  
        }  
 
        return dataTimestamp;  
    }

Poderia ser:

        public static Timestamp obterTimestampPorDataDDMMYYYY(String stData) throws ParseException {  
 
	          Date date = new SimpleDateFormat("dd-MM-yyyy").parse(stData);
	          return new Timestamp(date.getTime());
	}

Funfa

224
Thanks!
An error occurred!

Eu sou da opinião que um código bom é um código sem comentários. Pois de tão legível, não precisa de comentários. Comentários devem ser exceção, a regra deve ser a clareza. Código comentado para mim é um crime, pois se está sem uso deve ser removido, não comentado. Para voltar o código se precisar, se usa um controle de versão e não comentários. Pior ainda é ficar fazendo comentários com gírias, mais antiprofissional, impossível =)

     //legenda.calculateHeightsFast(); e legenda.getTotalHeight() nao funfa

Treinando switch

219
Thanks!
An error occurred!

Treinando switch:

 
    switch (valor1) {  
        case 1:  
            codigo1();  
            break;  
        case 2:  
            codigo1();  
            break;  
        case 3:  
            codigo1();  
            break;  
        case 4:  
            codigo1();  
            break;  
        case 5:  
            codigo1();  
            break;  
        default:  
            codigo1();  
            break;  
    }

Deveria ser:

codigo1();