[quote]Os nomes de alguns métodos e objetos foram modificados, para preservar o programador e a empresa, senão, fica fácil saber de quem é a káca!
OBS: A lógica dos algoritmos foram preservadas, por questão de lógica! hã … quanto aos comentários, foram adicionados para sacanear mesmo…
DIVIRTA-SE …
[/quote]
ModuloImpl[] modulos;
Object obj = cacheBy.get(indice);
if (obj != null){
modulos = (ModuloImpl[]) obj;
}
else{
modulos = null;
}
//2a forma de se fazer (após a alteração)
ModuloImpl[] modulos = (ModuloImpl[]) cacheBy.get(indice);
//Acho que o cara acabou de aprender a usar expressão condicional ternária...
boolean x = (isValid(obj) ? true : false);
//2a forma de se fazer (até o Papai Noel sabe)
boolean x = isValid(obj);
/**
* Pelo menos assim o cara garante que quando a variável str for nula,
* o código não continuará a executar... é NullPointerException garantido !!!
*/
if(str.equals(null)) {
// ...
}
//2a forma de se fazer (que mané, porque não fez assim)
if(str == null) {
// ...
}
if(obj == null) {
return false;
} else {
return true;
}
//2a forma de se fazer (é verdade, nem todo mundo sabe que o resultado de uma expressão booleana será TRUE | FALSE)
return (obj == null);
try {
beginTransaction();
salvarDadosA();
salvarDadosB(cod);
} catch (Exception e){
//Exception silenciada
} finally {
try{
/*
* Para quê o cara iniciou uma transação se ele sempre irá fazer o commit,
* independente do resultado.
*/
commitTransaction();
} catch (Exception e){
//Exception silenciada
}
}
//Sem comentários...
//Digo, vou fazer um comentário sim... que cara mais poético hein??
//Hehehehehehe...
try {
transaction.rollbackTransaction();
} catch (Exception f) { //reparem o nome do objeto Exception "f"
//Exception silenciada
}
//Esse aqui também é interessante...
try {
query.execute();
//Pra quê esta exception aqui no bloco catch ??
} catch (SQLException sql) {
throw new QueryException(sql);
} catch (Exception e) {
throw new QueryException(e);
} // só faltou incluir o bloco "Throwable" aí, eu matava o cara que fez isso!
//Será que gera exception se for alterado o conteúdo da variável de null para null ??
if (objXYZ != null) {
objXYZ = null;
}
//Aaahhh bom... agora eu entendi... ou não... hehehe
objXYZ = new ObjetoXYZ();
//Legal é o comentário abaixo... hehehe
updateStatus(statusInformado); //acho que esta linha não precisa existir.
/**
* Vou fazer o comentário depois para ficar mais interessante, antes, analisem o código!
* depois leiam o comentário.
*/
public void doView(Request request, Response response) throws Exception, IOException {
....
//Crio a string de retorno.
String strRetorno = "";
....
}
// O detalhes principal é que o método retorna void, e
// esta variável strRetorno não é utilizada em nenhum
// ponto do método. E pra ajudar ainda mais, esta classe
// possuia mais de 2.000 linhas de código, e o único lugar
// onde havia um comentário era nesta bendita linha...
// hehehe...
int valor = -1;
valor = Integer.valueOf(param).intValue();
//2a forma de se fazer (será que o cara não conhece a API??)
int valor = Integer.parseInt(param);
/*
* esse aqui, se alguém conseguir saber o que se passava na kbc do
* pobre coitado, mereçe um prêmio!!!! hehehe
*/
String strCod = request.getParameter("codSelect");
if (strCod != null && (strCod = strCod).length() > 0) {
...
}
//Acho que o cara quer derrubar a VM...
private void fecharModulo() {
fecharModulo();
}
/**
* aonde ele aprendeu isso?
* @see java.io.FileFilter#accept(File)
*/
public boolean accept(File file) {
if (filename.getName().substring((filename.getName().length() - 4), filename.getName().length()).compareTo(EXTENSAO) == 1) {
return true;
} else {
return false;
}
}
/**
* Que tal fazer assim?
* @see java.io.FileFilter#accept(File)
*/
public boolean accept(File file) {
String filename = file.getName();
return filename.endsWith(EXTENSAO)
}
...
if (rs.getString("TIPO_CAMPO").equals("NR")){
ps.setString(1, fvp.getValue());
} else {
ps.setString(1, fvp.getValue());
}
...
"Até um programador de Pascal faria desse jeito!"
...
ps.setString(1, fvp.getValue());
...
//código comentado...
function validaData(campo) {
var passou = true;
...
if (isDataValida(campo.value)) {
//É bom ter certeza de que o valor da variável é true... hehehe
passou = true;
}
//Se o método isDataValida() retornar false, passou será alterado para false...
//E de quebra o código dentro do else{...} nunca será executado.
else if (passou = false) {//ao invés de usar "==" o cara usou "="
alert('Data inválida.');
//Essa alteração abaixo na variável passou nunca será utilizada...
passou = false;
return false;
}
...
//É bom ter certeza que o interpretador não vai errar na hora de avaliar um booleano...
if (passou == true) {
if (!isDataEmIntervalo(campo.value)) {
alert('Data inválida.');
//Essa alteração abaixo na variável passou nunca será utilizada...
passou = false;
return false;
}
}
/*
* Condição de controle de xunxo, quando tudo mais falhar neste código atrapalhado...
*/
if(!passou){
alert('Data inválida.');
//Essa alteração abaixo na variável passou nunca será utilizada...
passou = false;
return false;
}
return true;
}