EVGD: Códigos Toscos

Mais um EVGD (Então Você Gostaria De...), porém este trata de registrar as pérolas da tosquice da programação Java.

Compartilhem suas experiências conosco e vamos dar boas risadas e ver como não se programa.

O que será que um cara pensa fazendo um código como o de cima?

Era bem melhor fazer:

Não é?

Olhem para este if()... hahaahahaahaha...

Porque diabos colocar um return deste de um finally?

Pérola nível 5: O cara simplesmente cria um novo array a cada iteração do ResultSet e copia a array antiga para a nova. Matem-no, por favor.

Rapaz, dessa vez pegaram pesado mesmo com você né Daniel?

Que projetinho bizarro esse viu

Alguém me explica o que a primeira linha faz alí?

E esse código aqui...montando a grid na action....rsrs

ClasseX classeX = new ClasseX ();
 classeX .addColumn("Tipo Réu", "%dsTipo" );		
 classeX .addColumn("Sequência", "%cdSequencia" );
 classeX .addColumn("Documento", "%cdDocumento" );
 classeX .addColumn("Nome", "%dsNome" );		
 classeX .addColumn("Período", "%periodo" );		
 classeX .addColumn("Advogado", "%dsNomeAdvogado" );		
 classeX .addColumn("OAB", "%cdOABAdvogado" );		
 classeX .addColumn("UF", "%cdSiglaUFDocumentoAdvogado" );	
 
 classeX .setTitle("Detalhes");
 classeX .setSubTitle("Réu");
 classeX .setWindowHeight("400");
 classeX .setWindowWidth("500");
 
 TableGrid grid = new TableGrid();
 GridText label;
 GridText field;
 
 grid.setTitle( new GridText("Réu", "ttltab1") );
 grid.setMultiSelection( true ); 
 grid.setVisibleCheckBox( true, "ttltab2" );
 grid.setVisibleError( false, "ttltab2" );
 grid.setVisibleStatus( false, "ttltab2" );
 grid.setVisibleStatus( true, "ttlTab2" );
 grid.setMaxRecordsShown( 5 );
 grid.setMaxPagesBlock( 5 );
 grid.setPaginacao( true, "ttltab1", ACAO_PAGINACAO );
 grid.processPage( "0" );
 grid.setImageGridPath( "/gcpj/imagens/" );
 grid.setImagePath("/imagens/");
 
 if(pIndTipoOperacao.equalsIgnoreCase("A")){
 
 label = new GridText("Nº Documento", "ttlTab2");
 field = new GridText("cdDocumento", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 140));
 
 label = new GridText("Nome", "ttlTab2");
 field = new GridText("dsNome", "txtTab1");
 grid.addColumn( new GridColumn( label, field, 370, "", "", 
    "javascript:abrirPopUpDetalhes('%cdCodigoReu', '%cdCodigoGCC', '%cdCodigoFilialGCC', '%cdCodigoControleGCC', 
    '%cdIndicativoOrigemTabela','%dsTipo', '%flagFunc', '%cdEmpresaIncorporada' )"));
 
 label = new GridText("Seq.", "ttlTab2");
 field = new GridText("cdSequencia", "txtTab1");
 grid.addColumn( new GridColumn(label, field, 46));
 
 } else {
 
 label = new GridText("Nº Documento", "ttlTab2");
 field = new GridText("cdDocumento", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 130));
 
 label = new GridText("Nome", "ttlTab2");
 field = new GridText("dsNome", "txtTab1");
 grid.addColumn( new GridColumn( label, field, 340, "", "", 
    "javascript:abrirPopUpDetalhes('%cdCodigoReu', '%cdCodigoGCC', '%cdCodigoFilialGCC', '%cdCodigoControleGCC', 
    '%cdIndicativoOrigemTabela', '%dsTipo', '%flagFunc', '%cdEmpresaIncorporada' )"));
 
 label = new GridText("Período", "ttlTab2");
 field = new GridText("dataIni", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 30));
 
 label = new GridText("", "ttlTab2");
 field = new GridText("dataFim", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 30));
 
 label = new GridText("Tipo", "ttlTab2");
 field = new GridText("dsTipo", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 150));
 
 label = new GridText("Seq.", "ttlTab2");
 field = new GridText("cdSequencia", "txtTab1");
 grid.addColumn(new GridColumn(label, field, 39));

danieldestro wrote:while( true ) { publicacaoPorPeriodo = cursor.next(); if( publicacaoPorPeriodo == null ) { // é fim do cursor? break; } list.addPublicacao(publicacaoPorPeriodo); }

O que será que um cara pensa fazendo um código como o de cima?

Era bem melhor fazer:

while( (publicacaoPorPeriodo = cursor.next()) != null ) { list.addPublicacao(publicacaoPorPeriodo); }

Não é?

É, mas eu tive de participar de um projeto em que as regras do Checkstyle objetava contra atribuições dentro de expressões. Ou seja, tive de fazer algo muito parecido com o que o autor da digníssima expressão teve de fazer

Um boa documentação é sempre necessária, mas isto...

E dale tosquice

public Object[] transSAGT(String p_bit62) throws RemoteException {
        Object[] aoRetorno = new Object[3];
        String[] asPacote = (String[]) enviaMsg(p_bit62, "SAGT");
        String s = null;
        aoRetorno[0] = asPacote[0];
        ArrayList asDados = new ArrayList();
        if (aoRetorno[0].equals("011")) {
            String[][] as;
            String sTring1 = "";
            String sTring2 = "";
            int qtdeServicos = 0;

for (int i = 1; i < asPacote.length; i++) {
                qtdeServicos = (asPacote[i].length() - 70) / 51;
                if (qtdeServicos > 0) {
                    as = new String[qtdeServicos][12];
                    for (int x = 70, z = 0; x <= asPacote[i].length() - 51; x += 51, z++) {
                        int j = 0;
                        int w = 0;
                        s = asPacote[i].substring(x, x + 51);
                        if ((s.trim()).equals(""))
                            break;
                        as[z][0] = asPacote[i].substring(j, (w = j + 6));
                        as[z][1] = asPacote[i].substring((j = w), (w = j + 2));
                        as[z][2] = asPacote[i].substring((j = w), (w = j + 4));
                        as[z][3] = asPacote[i].substring((j = w), (w = j + 3));
                        as[z][4] = asPacote[i].substring((j = w), (w = j + 55));
                        j = 0;
                        w = 0;
                        as[z][5] = s.substring((j = w), (w = w + 3));
                        as[z][6] = s.substring((j = w), (w = w + 3));
                        as[z][7] = s.substring((j = w), (w = w + 10));
                        as[z][8] = s.substring((j = w), (w = w + 10));
                        as[z][9] = s.substring((j = w), (w = w + 15));
                        as[z][10] = s.substring((j = w), (w = w + 8));
                        as[z][11] = s.substring((j = w), (w = w + 2));
                    }
                    asDados.add(as);
                }
            }
        }

aoRetorno[1] = asDados;
        aoRetorno[2] = asPacote[asPacote.length - 1].substring(15, 55);

return aoRetorno;
    }

E já que começamos errado, vamos terminar errado...

Nao sei se ja coloquei isso aqui, mas...

Marcio Kuchma

Eu criei um post parecido com esse no JavaFree por causa de codigos muito "particulares" que vi em um projeto. No fim das contas, alguns problemas são mais pontuais e fáceis de resolver. Dificil é quando existem erros graves na arquitetura. Exemplo de codigo dentro de uma classe de negocio:

E, Daniel, não me surpreenderia se vc dissesse que unit tests são artigo raro ou inexistente nesse tipo de projeto.

valeuz...

Apesar de eu saber o que são Unit Tests, aqui eu diria: "Unit o quê???"