EVGD: Códigos Toscos

This message was edited 1 time. Last update was at 20/04/2009 12:29:41

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.

Na verdade, o foco deste tópico é vermos algumas práticas não recomendadas, a fim de vermos como evitar erros comuns na programação.

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));

Moderador Membro desde: 29/07/2004 16:10:13 Mensagens: 17539 Offline

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ê???"

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