EVGD: Códigos Toscos

André Fonseca wrote:String yes_no = rs.getString("foo"); if (yes_no != null) { if ( yes_no.equals("Y") || yes_no.equals("YES") || yes_no.equals("Yes") || yes_no.equals("y") || yes_no.equals("yes") ) ret = true; }

bacana ein!

JavaChild Membro desde: 04/06/2009 09:43:58 Mensagens: 109 Offline

André Fonseca wrote:String yes_no = rs.getString("foo"); if (yes_no != null) { if ( yes_no.equals("Y") || yes_no.equals("YES") || yes_no.equals("Yes") || yes_no.equals("y") || yes_no.equals("yes") ) ret = true; }

Só faltou comparar caso seja não

[risos]

André Fonseca wrote:String yes_no = rs.getString("foo"); if (yes_no != null) { if ( yes_no.equals("Y") || yes_no.equals("YES") || yes_no.equals("Yes") || yes_no.equals("y") || yes_no.equals("yes") ) ret = true; }

Alguem avisa para cidadao que existe equalsIgnoreCase()

Ele esqueceu de testar todas as possíveis:

yes
Yes
yEs
yeS
YES
YEs
YeS
yES

haaahhahaha

Adoro a criatividade na hora de dar nomes.

DAOConsultaConsultaAplicacao

É o pattern Pleonasmo Vicioso.

Eu já vi:
DaoConsultaDaoConsulta

Ja vi um método que se chamava.

boolean podeConsultar(String sql)

Nesse método verificava se havia algum dado que pude-se ser resgatado e retornava um boolean.

Eu ein...

JavaEvangelist Membro desde: 01/12/2006 13:26:07 Mensagens: 381 Offline

Já levei bronca de "senior" (o kra tinha 4 anos na empresa e só trabalhou la, e com um framework proprio td feito com JavaSE, Swing) pq eu usei operador ternário em uma condicao simples como no exemplo:

ao invez de:

A justificativa dele nem foi por questão de legibilidade e sim de performance hahahaha ele dizia que operador ternario leva o dobro do tempo de um if normal... so se for o dobro do tempo pra ele (programador) entender o que está sendo feito

JavaEvangelist Membro desde: 14/02/2007 04:23:55 Mensagens: 305 Offline

andrerios wrote:Já levei bronca de "senior" (o kra tinha 4 anos na empresa e só trabalhou la, e com um framework proprio td feito com JavaSE, Swing) pq eu usei operador ternário em uma condicao simples como no exemplo:

<condicaoQueNaoLembro> ? true : false;

ao invez de:

if(<condicaoQueNaoLembro>) { return true; } else { return false; }

A justificativa dele nem foi por questão de legibilidade e sim de performance hahahaha ele dizia que operador ternario leva o dobro do tempo de um if normal... so se for o dobro do tempo pra ele (programador) entender o que está sendo feito

de todo o jeito, acho que o melhor era fazer

Você podia agrada-lo:

JavaEvangelist Membro desde: 01/12/2006 13:26:07 Mensagens: 381 Offline

dm_thiago wrote:

andrerios wrote:Já levei bronca de "senior" (o kra tinha 4 anos na empresa e só trabalhou la, e com um framework proprio td feito com JavaSE, Swing) pq eu usei operador ternário em uma condicao simples como no exemplo:

<condicaoQueNaoLembro> ? true : false;

ao invez de:

if(<condicaoQueNaoLembro>) { return true; } else { return false; }

A justificativa dele nem foi por questão de legibilidade e sim de performance hahahaha ele dizia que operador ternario leva o dobro do tempo de um if normal... so se for o dobro do tempo pra ele (programador) entender o que está sendo feito

de todo o jeito, acho que o melhor era fazer
return condicaoQueNaoLembro

Nao retornava true/false direto senao com certeza usaria essa ideia... foi so um exemplo

JavaEvangelist Membro desde: 01/12/2006 13:26:07 Mensagens: 381 Offline

ViniGodoy wrote:Você podia agrada-lo:

List<Boolean> condicoes = new ArrayList<Boolean>(); condicoes.add(true); condicoes.add(false); int index = condicaoQueNaoLembro ? (18720 % 10) : (192812701 % 100); return condicoes.get(index);

Ai de duas uma, ou ele se demitia ou se matava hahahaha

This message was edited 1 time. Last update was at 15/12/2009 15:38:36

imagina isso entao:

HAHA

ViniGodoy wrote:Você podia agrada-lo:

List<Boolean> condicoes = new ArrayList<Boolean>(); condicoes.add(true); condicoes.add(false); int index = condicaoQueNaoLembro ? (18720 % 10) : (192812701 % 100); return condicoes.get(index);

Demorei pra entender que só o que importava é o 18720 e 192812701.
Levei um susto quando vi.
Pensei que fosse outro jogo binario que nem um que voce postou uma vez para pegar os maiores numeros.

JonathanSSantos wrote:imagina isso entao:

public boolean continuaValidacao(){ boolean continuaValidando = true; List<Usuario> usuarios = DAO.getAllUsuarios(); for (Usuario user : usuarios){ continuaValidando = (user != null ? true : false); } return continuaValidando; }

HAHA

kkk, não acredito que alguem possa implementar algo assim...

Esqueceu de pedir a alternativa: Qual a pior?
código inútil? operador ternário inútil? falta de um teste para parar o for?

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