Problema com Parametros - Javafree: O fórum de Java que mais cresce

Fmgasparino
Posts:21

Publicado em: 19/11/2009 02:11:43

Bom dia pessoal,
Estou com uma dificuldade em Java, pois chamo um método com um parametro, o trato e retorno. Ao retornar quero usa-lo mas o valor continua sendo o antigo do método chamado.
Para esclarecer irei escrever um codigo imaginario aqui.

Classe A{ public int linha; linha=1; if(linha == 1){ incluirNumero(); Calcular(linha); } public String incluirNumero(){ linha=linha +1; return linha; } }

Temos esta classe, agora na classe B, tenho o método calcular() que não importa o que ele faz, mas nele linha é igual a 1.

Ou seja , não esta mudando nada executar incluirNumero();
Ele altera o objeto somente na hora da execução dentro do método.

Esta claro?

Quem ja passou por isso, ou sabe uma forma de reparar, por favor me ajude, estou em um projeto e isso é essencial para um funcionamento efetivo dele.

Muito Obrigado .

staroski
Posts:3412

Publicado em: 19/11/2009 02:20:09

Olá! O seu "código imáginário" infelizmente não foi o suficiente para responder e/ou corrigir seu problema.
Posta o fonte das classes originais!
_________________
Ricardo Artur Staroski, OCP-JP
Blumenau - SC

Fmgasparino
Posts:21

Publicado em: 19/11/2009 02:11:43

staroski
Olá! O seu "código imáginário" infelizmente não foi o suficiente para responder e/ou corrigir seu problema.
Posta o fonte das classes originais!

Classe GerArquivoTR{ public static int numeroLinha; public static BufferedReader reader; public static String linha; while (linha != null) { //Log.info("Linha " + numeroLinha + ": " + linha); if (!linha.trim().equals("")) { parseLine(linha, tr, numeroLinha, isArquivoHtm); } else { //Log.info("Ignorando linha em branco"); } /*numeroLinha++; linha = reader.readLine();*/ nextLine(); } reader.close(); return gravarTR(tr); } catch (Exception e) { e.printStackTrace(); return false; } } public static String nextLine() throws Exception{ numeroLinha++; linha = reader.readLine(); return linha; } if (linha.contains(cs13)) { GerArquivoTR.nextLine(); parse.setValorOrcado(linha, tr); } }

Classe Parse

public void setValorOrcado (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_PADRAO1); indexInicio += 19; int indexFim = linha.indexOf(Constantes.BUSCA_TR_PADRAO2); tr.setValorOrcado(linha.substring(indexInicio, indexFim)); }

Linha fica com resultado anterior de executar o nextLine();
Mas quando executa peo While, funciona perfeitamente....

Obrigado

staroski
Posts:3412

Publicado em: 19/11/2009 02:20:09

Você só está postando "pedaços" de código.
A classe GerArquivoTR que você postou, está sintaticamente errada...
Sem contar que tem um monte de variáveis estáticas...
_________________
Ricardo Artur Staroski, OCP-JP
Blumenau - SC

Fmgasparino
Posts:21

Publicado em: 19/11/2009 02:11:43

staroski
Você só está postando "pedaços" de código.
A classe GerArquivoTR que você postou, está sintaticamente errada...
Sem contar que tem um monte de variáveis estáticas...

Segue classes completas...
Eu passei pedaços pois pensei que facilitaria no processo de analise.

Sobre as variaveis serem estaticas, é porque as ulo em mais de um unico método.

package sigos.processamento.parse; import sigos.bean.TR; import sigos.util.Constantes; public class ParseArquivoTRHtm implements ParseArquivoTRInterface{ public void setNumDemanda (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_NUM_DEMANDA_INICIO); indexInicio += 17; int indexFim = indexInicio + 11; tr.setNumDemanda(linha.substring(indexInicio, indexFim)); } public void setNumOS (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_NUM_OS_INICIO); indexInicio += 31; int indexFim = indexInicio + 16; tr.setNumOS(linha.substring(indexInicio, indexFim)); } public void setDataEnvioOS (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_DATA_ENVIO_OS_INICIO); indexInicio += 20; int indexFim = indexInicio + 10; tr.setDataEnvioOS(linha.substring(indexInicio, indexFim)); } public void setNumEntrega (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_NUM_ENTREGA_INICIO); indexInicio += 15; int indexFim = indexInicio + 4; tr.setNumEntrega(linha.substring(indexInicio, indexFim)); } public void setDataPrevisaoEntrega (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_DATA_PREVISAO_ENTREGA_INICIO); indexInicio += 16; int indexFim = indexInicio + 10; tr.setDataPrevisaoEntrega(linha.substring(indexInicio, indexFim)); } public void setDataEfetivaEntrega (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_DATA_EFETIVA_ENTREGA_INICIO); indexInicio += 15; int indexFim = indexInicio + 10; tr.setDataEfetivaEntrega(linha.substring(indexInicio, indexFim)); } public void setTipoEntrega (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_TIPO_ENTREGA_INICIO); indexInicio += 20; int indexFim = linha.indexOf(Constantes.BUSCA_TR_TIPO_ENTREGA_FIM); tr.setTipoEntrega(linha.substring(indexInicio, indexFim)); } public void setModalidadeServico (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_MODALIDADE_SERVICO_INICIO); indexInicio += 12; int indexFim = linha.indexOf(Constantes.BUSCA_TR_MODALIDADE_SERVICO_FIM); tr.setModalidadeServico(linha.substring(indexInicio, indexFim)); } public void setProdutividade (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_PRODUTIVIDADE_INICIO); indexInicio += 18; int indexFim = linha.indexOf(Constantes.BUSCA_TR_PRODUTIVIDADE_FIM); tr.setProdutividade(linha.substring(indexInicio, indexFim)); } public void setHorasAdicionaisMes (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_HORAS_ADICIONAIS_MES_INICIO); indexInicio += 25; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HORAS_ADICIONAIS_MES_FIM); tr.setHorasAdicionaisMes(linha.substring(indexInicio, indexFim)); } public void setQtdePF (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_QTDE_PF_INICIO); indexInicio += 18; int indexFim = linha.indexOf(Constantes.BUSCA_TR_QTDE_PF_FIM); tr.setQtdePF(linha.substring(indexInicio, indexFim)); } public void setIndicePontosDefeitos (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_INDICE_PONTOS_DEFEITOS_INICIO); indexInicio += 33; int indexFim = linha.indexOf(Constantes.BUSCA_TR_INDICE_PONTOS_DEFEITOS_FIM); tr.setIndicePontosDefeitos(linha.substring(indexInicio, indexFim)); } public void setValorOrcado (String linha, TR tr) throws Exception { //encontra parametro coluna 1 do valor //int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_ORCADO_INICIO); /*encontra parametro coluna 2 do valor, on de ele realmente se encontra*/ int indexInicio = linha.indexOf(Constantes.BUSCA_TR_PADRAO1); indexInicio += 19; int indexFim = linha.indexOf(Constantes.BUSCA_TR_PADRAO2); tr.setValorOrcado(linha.substring(indexInicio, indexFim)); } public void setValorDescontoAtraso (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_DESCONTO_ATRASO_INICIO); indexInicio += 23; int indexFim = linha.indexOf(Constantes.BUSCA_TR_VALOR_DESCONTO_ATRASO_FIM); tr.setValorDescontoAtraso(linha.substring(indexInicio, indexFim)); } public void setValorDescontoErro (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_DESCONTO_ERRO_INICIO); indexInicio += 21; int indexFim = linha.indexOf(Constantes.BUSCA_TR_VALOR_DESCONTO_ERRO_FIM); tr.setValorDescontoErro(linha.substring(indexInicio, indexFim)); } public void setValorTotal (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_TOTAL_INICIO); indexInicio += 15; int indexFim = linha.indexOf(Constantes.BUSCA_TR_VALOR_TOTAL_FIM); tr.setValorTotal(linha.substring(indexInicio, indexFim)); } public void setValorSQOrcado (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_SQ_ORCADO_INICIO); indexInicio += 15; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HTM_VALOR_SQ_ORCADO_FIM); tr.setValorSQOrcado(linha.substring(indexInicio, indexFim)); } public void setValorSQRetencao (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_SQ_RETENCAO_INICIO); indexInicio += 21; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HTM_VALOR_SQ_RETENCAO_FIM); tr.setValorSQRetencao(linha.substring(indexInicio, indexFim)); } public void setValorSQDescontoErro (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_SQ_DESCONTO_ERRO_INICIO); indexInicio += 15; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HTM_VALOR_SQ_DESCONTO_ERRO_FIM); tr.setValorSQDescontoErro(linha.substring(indexInicio, indexFim)); } public void setValorSQDescontoAtraso (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_SQ_DESCONTO_ATRASO_INICIO); indexInicio += 15; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HTM_VALOR_SQ_DESCONTO_ATRASO_FIM); tr.setValorSQDescontoAtraso(linha.substring(indexInicio, indexFim)); } public void setValorSQTotalTermoRecebimento (String linha, TR tr) throws Exception { int indexInicio = linha.indexOf(Constantes.BUSCA_TR_VALOR_SQ_TOTAL_TERMO_RECEBIMENTO_INICIO); indexInicio += 12; int indexFim = linha.indexOf(Constantes.BUSCA_TR_HTM_VALOR_SQ_TOTAL_TERMO_RECEBIMENTO_FIM); tr.setValorSQTotalTermoRecebimento(linha.substring(indexInicio, indexFim)); } }

package sigos.processamento.controller; import java.io.BufferedReader; import java.io.File; import java.io.IOException; import sigos.bean.TR; import sigos.delegate.SIGOSDelegate; import sigos.log.Log; import sigos.processamento.parse.ParseArquivoTRHtm; import sigos.processamento.parse.ParseArquivoTRInterface; import sigos.processamento.parse.ParseArquivoTRTxt; import sigos.util.Constantes; public class GerArquivoTR { private int linhaValorSQOrcado; private int linhaValorSQRetencao; private int linhaValorSQDescontoErro; private int linhaValorSQDescontoAtraso; private int linhaValorSQTotalTermoRecebimento; private ParseArquivoTRInterface parse; private SIGOSDelegate delegate = new SIGOSDelegate(); public static int numeroLinha; public static BufferedReader reader; public static String linha; public void processarArquivosTR() throws IOException { Log.info("***** PROCESSANDO ARQUIVOS TR *****"); File dirArquivos = new File(Constantes.CAMINHO_DIR_TRS_NOVAS); File[] listaArquivos = dirArquivos.listFiles(); File arquivo = null; boolean retorno = false; if (listaArquivos.length == 0) { Log.info("Diretório de TR'S vazio"); } else { for (int i = 0; i < listaArquivos.length; i++) { arquivo = listaArquivos<i>; retorno = processarArquivo(arquivo); // se conseguir fazer parse do arquivo transfere para a pasta processadas if(retorno) { transferirArquivo(arquivo); } } } Log.info("***** FIM PROCESSAMENTO ARQUIVOS TR *****"); } public boolean processarArquivo(File arquivo) { /** * PADRÃO TR * TR_SP200800001292.htm ou TR_SP200800001292.txt */ Log.info("ARQUIVO: " + arquivo.getName()); try { TR tr = new TR(); tr.setNumTR(GerArquivoUtil.getNumTRTA(arquivo.getName(), "TR_")); boolean isArquivoHtm; if (GerArquivoUtil.getExtensaoArquivoTRTA(arquivo.getName()).equalsIgnoreCase(".txt")) { parse = new ParseArquivoTRTxt(); isArquivoHtm = false; } else { parse = new ParseArquivoTRHtm(); isArquivoHtm = true; } reader= GerArquivoUtil.getReader(arquivo); numeroLinha = 1; linha = reader.readLine(); while (linha != null) { //Log.info("Linha " + numeroLinha + ": " + linha); if (!linha.trim().equals("")) { parseLine(linha, tr, numeroLinha, isArquivoHtm); } else { //Log.info("Ignorando linha em branco"); } /*numeroLinha++; linha = reader.readLine();*/ nextLine(); } reader.close(); return gravarTR(tr); } catch (Exception e) { e.printStackTrace(); return false; } } public static String nextLine() throws Exception{ numeroLinha++; linha = reader.readLine(); return linha; } private void parseLine(String linha, TR tr, int numeroLinha, boolean isArquivoHtm) throws Exception { CharSequence cs1 = Constantes.BUSCA_TR_NUM_DEMANDA_INICIO; CharSequence cs2 = Constantes.BUSCA_TR_NUM_OS_INICIO; CharSequence cs3 = Constantes.BUSCA_TR_DATA_ENVIO_OS_INICIO; CharSequence cs4 = Constantes.BUSCA_TR_NUM_ENTREGA_INICIO; CharSequence cs5 = Constantes.BUSCA_TR_DATA_PREVISAO_ENTREGA_INICIO; CharSequence cs6 = Constantes.BUSCA_TR_DATA_EFETIVA_ENTREGA_INICIO; CharSequence cs7 = Constantes.BUSCA_TR_TIPO_ENTREGA_INICIO; CharSequence cs8 = Constantes.BUSCA_TR_MODALIDADE_SERVICO_INICIO; CharSequence cs9 = Constantes.BUSCA_TR_PRODUTIVIDADE_INICIO; CharSequence cs10 = Constantes.BUSCA_TR_HORAS_ADICIONAIS_MES_INICIO; CharSequence cs11 = Constantes.BUSCA_TR_QTDE_PF_INICIO; CharSequence cs12 = Constantes.BUSCA_TR_INDICE_PONTOS_DEFEITOS_INICIO; CharSequence cs13 = Constantes.BUSCA_TR_VALOR_ORCADO_INICIO; CharSequence cs14 = Constantes.BUSCA_TR_VALOR_DESCONTO_ATRASO_INICIO; CharSequence cs15 = Constantes.BUSCA_TR_VALOR_DESCONTO_ERRO_INICIO; CharSequence cs16 = Constantes.BUSCA_TR_VALOR_TOTAL_INICIO; CharSequence cs17 = Constantes.BUSCA_TR_VALOR_SQ_ORCADO_INICIO; CharSequence cs18 = Constantes.BUSCA_TR_VALOR_SQ_RETENCAO_INICIO; CharSequence cs19 = Constantes.BUSCA_TR_VALOR_SQ_DESCONTO_ERRO_INICIO; CharSequence cs20 = Constantes.BUSCA_TR_VALOR_SQ_DESCONTO_ATRASO_INICIO; CharSequence cs21 = Constantes.BUSCA_TR_VALOR_SQ_TOTAL_TERMO_RECEBIMENTO_INICIO; if (linha.contains(cs1)) { parse.setNumDemanda(linha, tr); } if (linha.contains(cs2)) { parse.setNumOS(linha, tr); } if (linha.contains(cs3)) { parse.setDataEnvioOS(linha, tr); } if (linha.contains(cs4)) { parse.setNumEntrega(linha, tr); } if (linha.contains(cs5)) { parse.setDataPrevisaoEntrega(linha, tr); } if (linha.contains(cs6)) { parse.setDataEfetivaEntrega(linha, tr); } if (linha.contains(cs7)) { parse.setTipoEntrega(linha, tr); } if (linha.contains(cs<img src="http://javafree.uol.com.br/forum/images/smiles/icon_cool.gif">) { parse.setModalidadeServico(linha, tr); } if (linha.contains(cs9)) { parse.setProdutividade(linha, tr); } if (linha.contains(cs10)) { parse.setHorasAdicionaisMes(linha, tr); } if (linha.contains(cs11)) { parse.setQtdePF(linha, tr); } if (linha.contains(cs12)) { parse.setIndicePontosDefeitos(linha, tr); } if (linha.contains(cs13)) { GerArquivoTR.nextLine(); parse.setValorOrcado(linha, tr); } if (linha.contains(cs14)) { GerArquivoTR.nextLine(); parse.setValorDescontoAtraso(linha, tr); } if (linha.contains(cs15)) { GerArquivoTR.nextLine(); parse.setValorDescontoErro(linha, tr); } if (linha.contains(cs16)) { GerArquivoTR.nextLine(); parse.setValorTotal(linha, tr); } if (linha.contains(cs17)) { if (isArquivoHtm) { linhaValorSQOrcado = numeroLinha; linhaValorSQOrcado += 3; } else { parse.setValorSQOrcado(linha, tr); } } if (linha.contains(cs1<img src="http://javafree.uol.com.br/forum/images/smiles/icon_cool.gif">) { if (isArquivoHtm) { linhaValorSQRetencao = numeroLinha; linhaValorSQRetencao += 3; } else{ GerArquivoTR.nextLine(); parse.setValorSQRetencao(linha, tr); } } if (linha.contains(cs19)) { if (isArquivoHtm) { linhaValorSQDescontoErro = numeroLinha; linhaValorSQDescontoErro += 3; } else { GerArquivoTR.nextLine(); parse.setValorSQDescontoErro(linha, tr); } } if (linha.contains(cs20)) { if (isArquivoHtm) { linhaValorSQDescontoAtraso = numeroLinha; linhaValorSQDescontoAtraso += 3; } else { GerArquivoTR.nextLine(); parse.setValorSQDescontoAtraso(linha, tr); } } if (linha.contains(cs21)) { if (isArquivoHtm) { linhaValorSQTotalTermoRecebimento = numeroLinha; linhaValorSQTotalTermoRecebimento += 3; } else { GerArquivoTR.nextLine(); parse.setValorSQTotalTermoRecebimento(linha, tr); } } if (numeroLinha == linhaValorSQOrcado && isArquivoHtm) { GerArquivoTR.nextLine(); parse.setValorSQOrcado(linha, tr); } if (numeroLinha == linhaValorSQRetencao && isArquivoHtm) { GerArquivoTR.nextLine(); parse.setValorSQRetencao(linha, tr); } if (numeroLinha == linhaValorSQDescontoErro && isArquivoHtm) { GerArquivoTR.nextLine(); parse.setValorSQDescontoErro(linha, tr); } if (numeroLinha == linhaValorSQDescontoAtraso && isArquivoHtm) { GerArquivoTR.nextLine(); parse.setValorSQDescontoAtraso(linha, tr); } if (numeroLinha == linhaValorSQTotalTermoRecebimento && isArquivoHtm) { GerArquivoTR.nextLine(); parse.setValorSQTotalTermoRecebimento(linha, tr); } } private boolean gravarTR(TR tr) { Log.info("GRAVANDO TR"); boolean sucesso = true; if (existeTR(tr)) { sucesso = delegate.updateTR(tr); } else { sucesso = delegate.insertTR(tr); } return sucesso; } private boolean existeTR(TR tr) { return delegate.existeTR(tr); } private void transferirArquivo(File arquivo) { Log.info("TRANSFERINDO ARQUIVO " + arquivo.getName()); File arquivoProcessado = new File(Constantes.CAMINHO_DIR_TRS_PROCESSADAS + arquivo.getName()); arquivo.renameTo(arquivoProcessado); } }

staroski
Posts:3412

Publicado em: 19/11/2009 02:20:09

Fmgasparino
Sobre as variaveis serem estaticas, é porque as uso em mais de um unico método.

Acessar uma mesma variável em métodos distintos não justifica declarar ela como estática...

Não tive tempo para analisar com calma sua lógica
Mas de antemão posso afirmar que as seguintes variáveis não devem ser estáticas.
public static int numeroLinha; public static BufferedReader reader; public static String linha;
Não sei como estão suas outras classes mas é bem provável que alguém esteja alterando o conteúdo dessas variáveis e como elas são estáticas, a alteração reflete para todas as instâncias...

tira o static da declaração, elas devem ser variáveis de instância
_________________
Ricardo Artur Staroski, OCP-JP
Blumenau - SC

Fmgasparino
Posts:21

Publicado em: 19/11/2009 02:11:43

staroski
Fmgasparino
Sobre as variaveis serem estaticas, é porque as uso em mais de um unico método.

Acessar uma mesma variável em métodos distintos não justifica declarar ela como estática...

Não tive tempo para analisar com calma sua lógica
Mas de antemão posso afirmar que as seguintes variáveis não devem ser estáticas.
public static int numeroLinha; public static BufferedReader reader; public static String linha;
Não sei como estão suas outras classes mas é bem provável que alguém esteja alterando o conteúdo dessas variáveis e como elas são estáticas, a alteração reflete para todas as instâncias...

tira o static da declaração, elas devem ser variáveis de instância

Staroski, Defini estas variaveis comm variaveis de classe, pois eu quero que o valor dela seja igual em diferentes métodos. Quero que ao mudar ela no método nexLine, ja mude em todo o sistema. Não esta certa minha linha de raciocinio??

Obrigado.

staroski
Posts:3412

Publicado em: 19/11/2009 02:20:09

Não!
Não considero isso apropriado porque sua classe GerArquivoTR possui atributos não-estáticos que também são manipulados em conjunto com os estáticos.

Faz o seguinte: transforma essa classe num Singleton, aí você sempre estará acessando a mesma instância em todo seu sistema.
_________________
Ricardo Artur Staroski, OCP-JP
Blumenau - SC

Fmgasparino
Posts:21

Publicado em: 19/11/2009 02:11:43

Obrigado pela ajuda ...

Relacionados

[Swing] Navegar pelos campos ( indo ou voltando ) usando as setas
http://javafree.uol.com.br/topic-891048-Swing-Navegar-pelos-campos-indo-ou-voltando-usando-as-setas.html

programação
http://javafree.uol.com.br/topic-891109-programacao.html

Ajuda com o programa de Jokenpo
http://javafree.uol.com.br/topic-891111-Ajuda-com-o-programa-de-Jokenpo.html

Problema de Compilação (Java + SQL)
http://javafree.uol.com.br/topic-891107-Problema-de-Compilacao-Java-SQL.html

o q quer dizer ';'
http://javafree.uol.com.br/topic-891078-o-q-quer-dizer-.html