EVGD: Códigos Toscos

fmeyer wrote:http://flickr.com/photos/fmcamargo/187561305/
essa me doeu o utero quando vi
amigos do governo ... tstststs

Faltou:

Eu dei manutenção em um framework web made-in-house no mês passado e olha o que eu encontrei num ponto do código:

O engraçado é que em nenhuma parte do programa inteiro essas variáveis eram usadas.

Bom, fora 3 bugs de prender conexão que eu achei, que derrubava a comunicação com o BD 3 vezes por semana.

Esse aqui é velho na empresa, todo mundo tem medo de fazer um refactor, aliás ninguém nem sabe o que é ou pra quê serve esse atributo.

rmarin wrote:Esse aqui é velho na empresa, todo mundo tem medo de fazer um refactor, aliás ninguém nem sabe o que é ou pra quê serve esse atributo.

Na falta de um bom nome, não use nenhum...

danieldestro wrote:

rmarin wrote:Esse aqui é velho na empresa, todo mundo tem medo de fazer um refactor, aliás ninguém nem sabe o que é ou pra quê serve esse atributo.

Na falta de um bom nome, não use nenhum...

, vendo por esse lado...

Olha esse.

Ah, pensaram que acabou? Esse foi somente o adiciona, tem o altera tampém. Claro que tem!

Ainda bem que o Altera recebe os mesmos 46 parametros, né?
Ah, e esse é só o maior, tem uma outra classe que tem dois métodos que recebem 44 Strings, no mesmo estilo, s1, s2, etc.
...
Eu juro por tudo que é mais sagrado que está em ambiente de produção.

Ele já ouviu falar em OOP?

GUJ Master Membro desde: 16/12/2004 17:09:19 Mensagens: 1787 Offline

Do TheDailyWTF, reparem no change log

/* adds up all the column values for a specific row from an excel file.
 * 20050823 - BSR - v1.0 - Adds only columns a to z.
 * 20050909 - BSR - v2.0 - Added support for adding columns aa to zz.
 */
public long addUpAllExcelColumns(
  int a,  int b,  int c,  int d,  int e,  int f,  int g,  int h,  
  int i,  int j,  int k,  int l,  int m,  int n,  int o,  int p,  
  int q,  int r,  int s,  int t,  int u,  int v,  int w,  int x,  
  int y,  int z,
  
  int aa, int ab, int ac, int ad, int ae, int af, int ag, int ah, 
  int ai, int aj, int ak, int al, int am, int an, int ao, int ap, 
  int aq, int ar, int _as, int at, int au, int av, int aw, int ax, 
  int ay, int az,

int ba, int bb, int bc, int bd, int be, int bf, int bg, int bh, 
  int bi, int bj, int bk, int bl, int bm, int bn, int bo, int bp,
  int bq, int br, int bs, int bt, int bu, int bv, int bw, int bx, 
  int by, int bz,
  
  int ca, int cb, int cc, int cd, int ce, int cf, int cg, int ch,
  int ci, int cj, int ck, int cl, int cm, int cn, int co, int cp,
  int cq, int cr, int cs, int ct, int cu, int cv, int cw, int cx,
  int cy, int cz,
  
  int da, int db, int dc, int dd, int de, int df, int dg, int dh,
  int di, int dj, int dk, int dl, int dm, int dn, int _do, int dp,
  int dq, int dr, int ds, int dt, int du, int dv, int dw, int dx,
  int dy, int dz,

/* Snip 100 lines */
  
  int ya, int yb, int yc, int yd, int ye, int yf, int yg, int yh,
  int yi, int yj, int yk, int yl, int ym, int yn, int yo, int yp,
  int yq, int yr, int ys, int yt, int yu, int yv, int yw, int yx,
  int yy, int yz,

int za, int zb, int zc, int zd, int ze, int zf, int zg, int zh,
  int zi, int zj, int zk, int zl, int zm, int zn, int zo, int zp,
  int zq, int zr, int zs, int zt, int zu, int zv, int zw, int zx,
  int zy, int zz)
{
    return 
      a+b+c+d+e+f+g+h+i+j+k+l+m+n+o+p+q+r+s+t+u+v+w+x+y+z+
      aa+ab+ac+ad+ae+af+ag+ah+ai+aj+ak+al+am+an+ao+ap+aq+ar+_as+at+au+av+aw+ax+ay+az+
      ba+bb+bc+bd+be+bf+bg+bh+bi+bj+bk+bl+bm+bn+bo+bp+bq+br+bs+bt+bu+bv+bw+bx+by+bz+
      ca+cb+cc+cd+ce+cf+cg+ch+ci+cj+ck+cl+cm+cn+co+cp+cq+cr+cs+ct+cu+cv+cw+cx+cy+cz+
      da+db+dc+dd+de+df+dg+dh+di+dj+dk+dl+dm+dn+_do+dp+dq+dr+ds+dt+du+dv+dw+dx+dy+dz+
      /* Snip 20 lines */
      ya+yb+yc+yd+ye+yf+yg+yh+yi+yj+yk+yl+ym+yn+yo+yp+yq+yr+ys+yt+yu+yv+yw+yx+yy+yz+
      za+zb+zc+zd+ze+zf+zg+zh+zi+zj+zk+zl+zm+zn+zo+zp+zq+zr+zs+zt+zu+zv+zw+zx+zy+zz;
}

http://thedailywtf.com/forums/thread/81930.aspx

Bizarro!

No código-fonte de um compilador razoavelmente famoso:

Agora me explica: pq v().getMethod() em vez de getMethod()????

JavaEvangelist Membro desde: 27/02/2005 17:04:22 Mensagens: 372 Offline

é daqui do forum... Quando eu vi isso quase chorei! pq ele não fez assim

GUJ Master Membro desde: 16/12/2004 17:09:19 Mensagens: 1787 Offline

Ahhh essa aí é fraca info_god

JavaEvangelist Membro desde: 27/02/2005 17:04:22 Mensagens: 372 Offline

renato3110 wrote:Ahhh essa aí é fraca info_god

aubhauihauihauiohauihauio é fraca mais quando eu vi deu um aperto no coração.. O cara parece q nunca viu logica

Eu não vejo nada demais em fazer:

É por uma questão de clareza ou de gostar de escrever mais.

Mesmo porque a alternativa correta seria:

E não como você sugeriu.

danieldestro wrote:Eu não vejo nada demais em fazer:

if( a == false )

É por uma questão de clareza ou de gostar de escrever mais.

Mesmo porque a alternativa correta seria:

if( !a )

E não como você sugeriu.

Acho que esse caso não é uma questão de "Burrice", e sim de falta de conhecimento.
Normalmente quem está começando faz isso.

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