Arduino Read FeedBack using JSerialComm

[Scott Eric Catalano]

Hello Group!

I would like to say thanks for all the help received thus far it has been greatly appreciated. I've written another java program that communicates with an Arduino. Now I can send data to the Arduino just fine however receiving data has been a bit trickier. I would like to change a button color based on the arduino sending data to the java program. My Java code is below. Thanks!

//Listener for the blink button.
		btnBlink.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				
				btnOn.setBackground(null);
				btnBlink.setBackground(Color.yellow);
				btnOff.setBackground(null);
				//Turns on the #13 pin LED and flashes it 10 times.
				if(comPort.isOpen() == true) {
					comPort.setComPortTimeouts(SerialPort.TIMEOUT_READ_SEMI_BLOCKING, 100, 0);
					InputStream in = comPort.getInputStream();
					//Send a 2 to the Arduino and update the user.
					System.out.println("STATUS: LED BLINKING!!");
					lblStatus.setText("Status: LED Blinking");
					outPut.print("2");
					outPut.flush();

try {
   
      
  byte[] buffer = new byte[3];
    String message = "";
    
        int len = in.read(buffer);
        if (len > 0) {
            message = new String(buffer);
            System.out.println(message);
        }
		
        len = 0;
        String messages[];
        messages = message.split("\r\n");
        
            if (message.equals("OFF")) {
                btnBlink.setBackground(Color.red);
            }
        outPut.flush();
    in.close();
	
  
} catch (Exception ep) { ep.printStackTrace(); }
  				
				} else {
					//Update the status/console if the Arduino hasn't been connected.
					System.out.println("CONNECT ARDUINO");
					lblStatus.setText("Status: Connect Arduino");
				}
			}
		});

The button does NOT change color when the message from the Arduino is sent which is OFF. Now the word OFF does get print to the console only after the BLINK button has been selected again as well as the button changing color. This needs to happen once the blinking stops.

[Tony Docherty]

Not sure that this will solve your problem but it's generally bad practice to perform long running tasks on the EDT, you should use a worker thread to run the task in the background. If your task is also interacting with the GUI (as yours is) you can use a SwingWorker see this tutorial: https://docs.oracle.com/javase/tutorial/uiswing/concurrency/worker.html.

BTW why are you calling message.split() and then not using the returned array anywhere.