EVGD: Códigos Toscos

Assuntos Gerais
#205

[quote=danieldestro][quote=ramilani12][quote=danieldestro]
…
Como o cara garante NULL com um campo usando long (primitivo)???
[/quote]

Ai cara testa se eh igual a null caso seja atribui 0 ou -1 … :slight_smile: [/quote]

ZERO não é NULL… o BD vai procurar por algum registro com ID=0 (ou -1) e vai dar pau de integridade referêncial. BOOOM![/quote]
.
Tipo no momento de recuperar o campo do Banco de Dados , mesmo assim ao tentar recuperar ele ira atribuir 0 caso campo no B.D esteja null.
Obvio que ao tentar inserir a integridade dos dados vai pro lixo.

#206

LOGO… :wink:

#207

Acabei de pegar esse, e o pior, FUI EU QUE FIZ.

listCompany.removeAll(list); listCompany.addAll(list);
Acho que eu estava bêbado.

#208

Hehe, já aconteceu comigo. Você olha para o código e pergunta “Quem foi o babaca que escreveu esse lixo?”. Mas, poucos segundos depois você descobre que foi você mesmo o babaca que escreveu aquele lixo.

#209

[quote=spycall]Acabei de pegar esse, e o pior, FUI EU QUE FIZ.

listCompany.removeAll(list); listCompany.addAll(list);
Acho que eu estava bêbado.[/quote]

Um refresh? Q legal!!

#210

Patterns bem aplicados:

enviaEmailDAO.enviar( to, from, subject, text );
#211

Acho que ainda estou para ver algo tão genial quando a comparar no método “equals” um objeto de um tipo com outro completamente diferente… e esperar que isso funcione…

MinhaEntidade aux = new MinhaEntidade(); // seta a entidade com os valores... if (!aux.equals("")) { //WTF?? test.add(aux); }

Talvez o cara estava querendo testar se a entidade estava vazia?! vai saber!
O melhor é que o nome das variáveis estava exatamente desse jeito aux e test… ao ver esse código não deu nem vontade de rir … deu vontade de chorar…

#212

Adoro este tipo de redundância… será que o cara quis garantir que o valor foi REALMENTE BEM transformado?

retorno.setDataVigencia( DateUtil.parseData( DateUtil.formataData( DateUtil.parseData(retorno.getDataVigenciaFormatada()) ) ) );

O que um caboclo deste pensa?

#213

essa acabei de pegar… hehe

[code]if (codigo == null) {
faz uma coisa
faz outra coisa
}

SIM, EXATAMENTE NA LINHA DE BAIXO.

if (codigo == null) {
faz outra outra coisa
}[/code]

A variavel código deve pensar… “mas que cara chato! não sou null, porra!”

aheuaheuhuea

#214

peguei uma pérola não no código mas sim no comentário

list.add(resumoROAvisoSinistroDTO);
//list = sumResumoRO(list, resumoROAvisoSinistroDTO); ROGER Esse cara não faz sentido para mim

bem louco! haha… dá vontade de colocar mais uma linha, no estilo “Não faz sentindo pra mim também…”

hahaha…

#215

[quote=danieldestro]Patterns bem aplicados:

enviaEmailDAO.enviar( to, from, subject, text );

Se o cara só tem martelo na mão, tudo que ele vê na frente é prego…

A mentalidade dele é: “se não é uma tela ou é um DTO ou é um DAO”

#216

Daniel: Cara, porque você fez um DAO para mandar e-mail?
Criatura: Porque ele acessa o BD para enviar o e-mail (via UTL do Oracle).
Daniel: Não faz sentido ser um DAO.
Criatura: Mas ele acessa o BD.
Daniel: Nem tudo o que acessa o BD é necessariamente um DAO. E outra, porque não fazer via Java. Com o BD você está colocando mais uma etapa, o que pode causar mais suscetibilidade a falhas.
Criatura: Ok, vou mudar.

#217

Daniel… vc não explicou direito pra ele… ele vai criar um DTO agora… :?

#218

Olá Daniel, qual o seu nome?

verificarAlteracaoForm(document.forms['manterInfoGeraisProdutoForm'].name)
#219

Porque será que as consultas ficam lentas???

if ( query.getResultList().size() > 0 ) { return (AlgumaEntidade) query.getResultList().get(0); }

#220

Não tenho o código original de uma tosqueira que vi a um tempo, mas era mais ou menos isso:

String sql1 = "SELECT CAMPO1 FROM TABELA";
String sql2 = "SELECT CAMPO2 FROM TABELA";
String sql3 = "SELECT CAMPO3 FROM TABELA";

Vector vector = new Vector(); // O cara nunca tinha ouvido falar em generics.
QualquerCoisaVO vo;

PreparedStatement ps1 = conexao.prepareStatement(sql1);
ResultSet rs1 = ps1.executeQuery();

while (rs1.next()) {
    vector.add(new QualquerCoisaVO());
}

PreparedStatement ps2 = conexao.prepareStatement(sql1);
ResultSet rs2 = ps2.executeQuery();

for (int i = 0; i < vector.size(); i++) {
    rs2.next();
    vector.get(i).setCampo1(rs2.getString(1));
}

PreparedStatement ps3 = conexao.prepareStatement(sql2);
ResultSet rs3 = ps3.executeQuery();

for (int i = 0; i < vector.size(); i++) {
    rs3.next();
    vector.get(i).setCampo2(rs3.getString(1));
}

PreparedStatement ps4 = conexao.prepareStatement(sql3);
ResultSet rs4 = ps4.executeQuery();

for (int i = 0; i < vector.size(); i++) {
    rs4.next();
    vector.get(i).setCampo3(rs4.getString(1));
}

return vector;
#222

[quote=danieldestro]Quando o programador estudou no Mobral:

[code]package um.pacote.esception;

public class … {}[/code][/quote]

Hehehehe

Por aqui, já vi o método montaCabecario().

#223

[quote=Schuenemann][quote=danieldestro]Quando o programador estudou no Mobral:

[code]package um.pacote.esception;

public class … {}[/code][/quote]

Hehehehe

Por aqui, já vi o método montaCabecario().[/quote]

ESCEPTION ?!?!? montaCabecario?!?! Mas que #%$#%$#%#!

Por aqui eu já vi uma variável isReimpresao e impresaoNota… Inclusive nos comentários…

#224

[quote=Thiagosc][quote=giulianocosta]hhehehe…

Essa é classica:

if(isAlgumaCoisa == true){
//sfkljsdklfjklsdfkl
}[/quote]

Acho que isso se encaixa mais no “estilo de código” do que código burro. O “true” é desnecesssário mas não faz mal algum.

Seria o mesmo que implicar com indentação do código de alguém ou algo parecido, tipo o indivídio que usa tab tirando uma onda de quem usa espaço, e o que usa a formatação automática da IDE tirando sarro dos dois.[/quote]

Concordo, eu não acho que isso seja um erro de código ou erro lógico. Acredito que é até proveitoso usar isso, pois ao menos pra mim, todas essas besteirinhas como essa tem alguma utilidade. E sinceramente, muitas dessas besteirinhas deixam o código mais lógico e legivel.

#225

[quote=Sergio Figueras][quote=Thiagosc][quote=giulianocosta]hhehehe…

Essa é classica:

if(isAlgumaCoisa == true){
//sfkljsdklfjklsdfkl
}[/quote]

Acho que isso se encaixa mais no “estilo de código” do que código burro. O “true” é desnecesssário mas não faz mal algum.

Seria o mesmo que implicar com indentação do código de alguém ou algo parecido, tipo o indivídio que usa tab tirando uma onda de quem usa espaço, e o que usa a formatação automática da IDE tirando sarro dos dois.[/quote]

Concordo, eu não acho que isso seja um erro de código ou erro lógico. Acredito que é até proveitoso usar isso, pois ao menos pra mim, todas essas besteirinhas como essa tem alguma utilidade. E sinceramente, muitas dessas besteirinhas deixam o código mais lógico e legivel. [/quote]

Não acho isso proveitoso… Se a variável já é booleana, essa parte é desnecessária… acho mais legível sem comparar com true