3

Meu projeto em Java é um controle de farol com Arduíno. Até ai tudo certo. Porém fiz uma função na qual que os leds fica automático em um loop após um RadionButton ser selecionado. Meu problema é que quando eu seleciono o RadionButton meu JFrame congela e não deixa eu tirar a seleção do RadionButton.

Ou seja selecionar o botão causa um loop ativo. A ideia seria ao desmarcar o botão, fazer o loop parar.

Segue meu código.

Aqui é onde o botão será selecionado e enviará o comando para a função:

        if ( teste.isSelected() ) {
        try {
            utiArduino.enviarDados( "automatico" );
        } catch (InterruptedException | IOException ex) {
            Logger.getLogger(Jframe_farol.class.getName()).log(Level.SEVERE, null, ex);
        }
    } else {
        try {
            utiArduino.enviarDados( "DesligaAuto" );
        } catch (InterruptedException | IOException ex) {
            Logger.getLogger(Jframe_farol.class.getName()).log(Level.SEVERE, null, ex);
        }
    }

Aqui é a função do loop:

  public void enviarDados(String status) throws InterruptedException, IOException{
   int i = 8;

    if ( "automatico".equals(status) ) { // caso do automatico   
        while ( true ) {
            output.write(i);
            Thread.sleep(1000);
            i++;
            if ( i > 10 ){
                i = 8;
            }
        }
    }
} 
2
  • Leia isso: pt.stackoverflow.com/a/2095/132 - Esse Thread.sleep(1000) deve estar sendo feito na thread do AWT. 18/11/2016 às 10:08
  • Além do que o @VictorStafusa disse, eu também acho que a abordagem de checar se o componente está setado para interromper o loop não está correta, há listeners para fazer esse tipo de monitoramento. Só fica dificil até sugerir uma alternativa com o trecho de código passado.
    – user28595
    18/11/2016 às 10:21

1 Resposta 1

2

O AWT e o Swing tem uma thread que gerencia o seu funcionamento, ela é chamada de Event Dispatch Thread - EDT. Como ela é uma thread única, se você fizer ela cair em um while (true) e/ou em um Thread.sleep(1000), a interface gráfica ficará congelada.

Assim sendo, a solução é mover essas coisas para uma thread distinta. Tente fazer assim na sua classe utiArduino:

private volatile String ultimoStatus;
private volatile Thread threadArduino;

public void enviarDados(String status) {
   ultimoStatus = status;
   if (threadArduino == null) {
       threadArduino = new Thread(this::iniciarEnvioArduino);
       threadArduino.start();
   }
}

private void envioArduino() {
    int i = 8;
    try {
        while (Thread.currentThread() == threadArduino) {
            if ("automatico".equals(ultimoStatus)) { // caso do automatico   
                output.write(i);
                Thread.sleep(1000);
                i++;
                if (i > 10) i = 8;
            }
        }
    } catch (IOException e) {
        e.printStackTrace();
    } catch (InterruptedException e) {
        // Não faz nada e deixa a thread morrer.
    }
}

public void parar() {
    if (threadArduino != null) {
        Thread t = threadArduino;
        threadArduino = null;
        t.interrupt();
    }
}

A seleção do seu botão fica assim:

utiArduino.enviarDados(teste.isSelected() ? "automatico" : "DesligaAuto");

Na sua subclasse de JFrame (que suponha que seja quem tem a referência utiArduino), coloque isso também:

@Override
public void dispose() {
    super.dispose();
    utiArduino.parar();
}

Para mais informações sobre a EDT, dê uma lida nessa minha outra resposta também.

9
  • Primeiramente Obrigado por ter respondido minha pergunta , Porém a solução que você responde , ta com alguns erros, primeiro nesse trecho de código ' super.dispose();' ta com seguinte error 'cannot find symbol symbol: method dispose()' , o próximo erro é na linha 'threadArduino = new Thread(this::iniciarEnvioArduino);' , esta com seguinte error , 'method references are not supported in -source 1.7 (use -source 8 or higher to enable method references) method reference not expected here' e o método envioArduino() não esta sendo usada você Pode me ajuda?? POR FAVOR 19/11/2016 às 13:34
  • @JeffersonSantos Qual é a classe que contém a variável utiArduino? Qual é o tipo dessa variável? O seu primeiro erro indica que não eram as classes que eu estava imaginando que fossem, mas isso não está claro em sua pergunta. O segundo problema é algo de configuração, pois ele significa que você está usando o compilador do Java 8 para compilar Java 7, e não compilar Java 8 mesmo, e o meu código é Java 8. Edite a pergunta para complementar com essas informações. 21/11/2016 às 12:37
  • Obg eu irei editar a pergunta, mas @VitcorStafusa a 'utiArduino' não é uma variável é o nome da classe que contem o método enviarDados, mas irei editar a pergunta 21/11/2016 às 20:29
  • @JeffersonSantos A classe utiArduino tem alguma superclasse? Se sim, qual? Qual é a classe que contém a chamada utiArduino.enviarDados( "automatico" );? 22/11/2016 às 13:38
  • Boa noite @VictorStafusa a 'utiArduino' é uma classe simples nela contem a conexão , e o metodo utiArduino.enviarDados( "automatico" ); , ou seja no meu Jframe principal eu instancio essa classa e ultilizo o método utiArduino.enviarDados( "automatico" ); desculpe o modo de tenta explica o problema , sera que pode me ajuda? 22/11/2016 às 22:30

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .