EVGD: Códigos Toscos

Na falta de um bom nome, não use nenhum… :smiley:

Na falta de um bom nome, não use nenhum… :smiley: [/quote]

:stuck_out_tongue: , vendo por esse lado…

Olha esse.

    public void adicionaRegistro(String s, String s1, String s2, String s3, String s4, String s5, String s6, 
            String s7, String s8, String s9, String s10, String s11, String s12, String s13, 
            String s14, String s15, String s16, String s17, String s18, String s19, String s20, 
            String s21, String s22, String s23, String s24, String s25, String s26, String s27, 
            String s28, String s29, String s30, String s31, String s32, String s33, String s34, 
            String s35, String s36, String s37, String s38, String s39, String s40, String s41, 
            String s42, String s43, String s44, String s45)
        throws Exception
    { ...

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

    public void alteraRegistro(String s, String s1, String s2, String s3, String s4, String s5, String s6, 
            String s7, String s8, String s9, String s10, String s11, String s12, String s13, 
            String s14, String s15, String s16, String s17, String s18, String s19, String s20, 
            String s21, String s22, String s23, String s24, String s25, String s26, String s27, 
            String s28, String s29, String s30, String s31, String s32, String s33, String s34, 
            String s35, String s36, String s37, String s38, String s39, String s40, String s41, 
            String s42, String s43, String s44, String s45)
        throws Exception
    {

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. :smiley:

Ele já ouviu falar em OOP?

Do TheDailyWTF, reparem no change log :lol: :lol: :lol:

/* 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;
}

Bizarro!

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

public class X {
    private static X instance = new X();

    // sim, o nome é v em vez de getInstance
    public static X v() {
        return instance;
    }

    // ...

    public void doSomething(...) {
        // ...
        Method m = v().getMethod(...);
    }
}

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

if (bStatus == false){ comandos = "copy etiqueta.txt lpt1"; child = Runtime.getRuntime().exec(comandos); Thread.sleep(500); bStatus = true; }

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

if !(bStatus) { //e bla blabla }

Ahhh essa aí é fraca info_god :stuck_out_tongue:

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

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.

[quote=danieldestro]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.[/quote]
Acho que esse caso não é uma questão de “Burrice”, e sim de falta de conhecimento.
Normalmente quem está começando faz isso.

Pois eu ja acho totalmente desnecessario comparar boolean == boolean… Mais vai da cabeça de cada um…

E obrigado pela correção não tinha reparado que coloquei fora do parentese…

if (!(bStatus)) {

}

Que me dizem deste modelo de getter?

    public ArrayList getCargaCombo() {
        if (cargaCombo != null)
          return cargaCombo;
        else
          return null;
    }

Temos um sistema onde todos os getters são assim.

Será que é para ficar mais “claro”? :smiley: :stuck_out_tongue:

[quote=danieldestro]Eu não vejo nada demais em fazer:

if( a == false )

É por uma questão de clareza ou de gostar de escrever mais.
[/quote]

Eu fazia isso no delphi antigamente :smiley:

[quote=danieldestro]Mesmo porque a alternativa correta seria:

if( !a )

E não como você sugeriu.[/quote]

Meu Deus! :lol: :lol: :lol:

[quote=rmarin] public ArrayList getCargaCombo() { if (cargaCombo != null) return cargaCombo; else return null; } [/quote]

Caraleo!!! Demorei pra entender uhauahuahauhauhauhauhauh :lol: :lol: :lol:

Pois é!!! huauhauh…o sistema inteiro tá assim, :smiley: :smiley: :smiley:

Hehehehe… infame!

Acabei de pegar um sistema para analisar e vi que o cara gerou o EAR com dois JARs dentro:

  • classes12.jar (driver do oracle); e
  • j2ee.jar

Incrível!
Sendo que o ambiente já tem o ojdbc14.jar (driver Oracle novo). E o pior ainda. O cara colocou o “j2ee” dentro do sistema.

É como fazer um programa em VB e empacotar o Windows junto.

Afff… eu mereço!

[quote=danieldestro]Hehehehe… infame!

Acabei de pegar um sistema para analisar e vi que o cara gerou o EAR com dois JARs dentro:

  • classes12.jar (driver do oracle); e
  • j2ee.jar

Incrível!
Sendo que o ambiente já tem o ojdbc14.jar (driver Oracle novo). E o pior ainda. O cara colocou o “j2ee” dentro do sistema.

É como fazer um programa em VB e empacotar o Windows junto.

Afff… eu mereço![/quote]

aff, já vi tanto isso que acho normal.

public class A {

    private static Hashtable table;

    public A() {
        table = new Hashtable();
        // ...
    }

    public static boolean validar(String str) {
        return table.contains(str);
    }

}

Marcio Kuchma

...
List list;
while (isQualquerCoisa) {
    list = new ArrayList();
    ...
    list.add(obj);
}
...

:roll: