EVGD: Códigos Toscos

Assuntos Gerais
#927

Mas o tipo de dado a ser comparado é um int, impossibilitando retornar diretamente true ou false.

#928

Mas o tipo de dado a ser comparado é um int, impossibilitando retornar diretamente true ou false.[/quote]

Isso não importa,

vo.getAgencia() == 0

Retorna true ou false.

#929

Mas o tipo de dado a ser comparado é um int, impossibilitando retornar diretamente true ou false.[/quote]

Isso não importa,

vo.getAgencia() == 0

Retorna true ou false. [/quote]

:?:

Vc trocou o ternário por if/else…

Q diferença isso faz?

#930

eita, que dificuldade rs

private boolean isAgenciaVazia(VoCuentas vo) { return vo.getAgencia() == 0 ? true : false; }

é a mesma coisa que

private boolean isAgenciaVazia(VoCuentas vo) { return vo.getAgencia() == 0; }

vo.getAgencia() == 0 já retorna true ou false

#931

Mas o tipo de dado a ser comparado é um int, impossibilitando retornar diretamente true ou false.[/quote]

Isso não importa,

vo.getAgencia() == 0

Retorna true ou false. [/quote]

:?:

Vc trocou o ternário por if/else…

Q diferença isso faz?[/quote]

Pra que usar ternário se o objetivo é retornar true ou false?

expressaoLogica ? A se expressaoLogica for verdadeira : B se expressaoLogica for false;

A redundância no código era “retorne true se a espressao for true e retorne false se a expressao for false”. Está claro agora?

#932

[quote]Pra que usar ternário se o objetivo é retornar true ou false?
view plaincopy to clipboardprint?
expressaoLogica ? A se expressaoLogica for verdadeira : B se expressaoLogica for false;

A redundância no código era “retorne true se a espressao for true e retorne false se a expressao for false”. Está claro agora?[/quote]

O cara só quiz mostrar que sabia ternário mesmo, kkkk!!!Ficaria até um pouco mais claro o código!

#933

Caramba, não sabia que o código que tinha postado tinha gerado tantas dúvidas.

Mas como já disseram, é a redundância do código.

É como se fizesse:

if(x == 1){
    return true;
else {
    return false;
}

simplesmente basta fazer:

return x==1;
#934

Não tem como não postar essa:Date dataAgora = new Date(Calendar.getInstance().getTime().getTime());

#935

Código bem explicativo, não intrusivo…

#936

Nossa, e ainda por cima comentado :slight_smile:

#937

Achei esse trecho de código e ficamos brincando de jogo dos sete erros aqui no trabalho:

if (!flag.equals("")&&!flag.toString().equals("null") { // faz alguma coisa que dá erro }

#938

kkkkkk

if (msgResult.equals("Não foi possível excluir todos os registros selecionados.")){ msg = "Não foi possível excluir todos os registros selecionados."; }else if (msgResult.equals("Não foi possível excluir o registro.")){ msg = "Não foi possível excluir o registro."; }

#939

Melhor prevenir do que remediar:

if (tabelasCalculoList == null) tabelasCalculoList = new ArrayList<DataTableModel>(); else tabelasCalculoList = new ArrayList<DataTableModel>();

como o autor encontra-se em contrato de experiência, já foi encaminhado ao RH a recomendação pela não renovação do contrato de trabalho :stuck_out_tongue:

#940

Huahau maldade

#941

avaliem

ps: 80% das coisas aqui não fazem sentido

#942

public Boolean getChave() { if (selecao != null) { if (selecao.containsKey("chave")) { if (selecao.get("chave").equals("true")) { return Boolean.TRUE; } else { return Boolean.FALSE; } } return Boolean.FALSE; } return Boolean.FALSE; }

Bastava:

public Boolean getChave() { return selecao != null && "true".equals(selecao.get("chave")); }

Outro:

[code]public Boolean existeRestricao() {
String type = (String) selected.get(“type”);
String no = (String) selected.get(“no”);
if (“box”.equals(type) && “Instance”.equals(no)) {
return true;
}
if (type == null && “box”.equals(no)) {
return null;
}
return null;

}[/code]

Por que retornar null ao invés de false e pra que aquele último if?

Poderia ser:

public Boolean existeRestricao() { String type = (String) selected.get("type"); String no = (String) selected.get("no"); return "box".equals(type) && "Instance".equals(no); }

#943

Invertendo o equals, dava para dispensar até o cast:

public Boolean existeRestricao() { return "box".equals(selected.get("type")) && "Instance".equals(selected.get("no")); }

#944

É verdade, obrigado.
Se quiser manter as 3 linhas, dá pra trocar o tipo por Object ao invés de String.

Mais uma. Essa classe é das “boas”…

public String getNomeFuncao() { if (selected != null) { if (selected.containsKey("box")) { return null; } if (selected.containsKey("nome")) { return (String) selected.get("nome"); } return "-1"; } return "-1"; }
Parece que a pessoa acredita que esse map.containsKey() evita null pointer. Em muitos pontos tem esse if antes de usar o valor. Mas nesse caso a lógica ficaria diferente sem ele.

Obs: para não ser injusto, alguns nomes toscos foram edição minha.

#945

Já devem ter postado algo assim, mas gastei umas horas ajustando códigos de gente que não conhecia resource managers.

If System.Globalization.CultureInfo.CurrentCulture.Name = "pt-BR" Then
            strErrorMessage = "Por favor, tente mais tarde!"
        Else
            strErrorMessage = "Please, try again later!"
        End If
#946

Artigo muito apropriado no coding horror:
http://blog.codinghorror.com/new-programming-jargon/