Best Practice Button isPressed o.ä. auswerten

[julianpe]

Hallo,

ich möchte gerne die Aktionen mit einem JButton auswerten.

Ich hätte gerne folgendes verhalten:
Solange der Button mit der Maus betätigt wird, soll eine Methode immer wieder aufgerufen werden.

Sobald die Maustaste losgelassen wird, wird eine andere Methode einmalig aufgerufen.

So sieht das ganze bei mir momentan aus:

btnVoraus.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(MouseEvent arg0) {
               
                // Setze vorausfahrt
                    while(true)           
                    try {
                        forward();
                    } catch (Exception e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
            }
@Override
            public void mouseReleased(MouseEvent e) {
                // Rücksetze vorausfahrt
               
                try {
                   
                    stop();                   
                } catch (Exception e1) {
                    // TODO Auto-generated catch block
                    e1.printStackTrace();
                }
               
            }
        });

Ich hänge momentan noch in der while Bedingung für "Setze vorausfahrt".
Welche Eigenschaft des Buttons oder des MouseEvents kann ich nutzen um meine while Bedingung solange durchlaufen zu lassen, wie der Button gedrückt ist?

Vielen Dank

 

[Schesam]

Definiere dir Außerhalb ne boolische Variable "pressed" und frage in der while Schleife ab, ob Pressed True ist, vor der setzt du sie auf true. In der REaleased setzt du sie auf false.

 

[julianpe]

Definiere dir Außerhalb ne boolische Variable "pressed" und frage in der while Schleife ab, ob Pressed True ist, vor der setzt du sie auf true. In der REaleased setzt du sie auf false.

Die gleiche Idee hatte ich anfangs auch.
Scheitere jedoch an deren Umsetzung ;-)

Habe das nun wie folgt implementiert:

boolean buttonStatus = false;
(...)
btnVoraus.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(MouseEvent arg0) {
               
                // Setze vorausfahrt
                buttonStatus=true;
                    while(buttonStatus)           
                    try {
                        System.out.println("Voraus");
                    } catch (Exception e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
               
               
               
            }
            @Override
            public void mouseReleased(MouseEvent e) {
                // Rücksetze vorausfahrt
               
                try {
                    buttonStatus=false;
                    System.out.println("Stop");
                } catch (Exception e1) {
                    // TODO Auto-generated catch block
                    e1.printStackTrace();
                }
               
            }
        });

 

[Schesam]

Und was funktioniert daran nicht?

 

[julianpe]

Nehme an, dass ich in der while-Schleife stecken bleibe. Jedenfalls läuft meine Ausgabe fast mit der Meldung "voraus" über. Aber keine "Stop" Nachricht.

 

[T_T]

Das kann doch gar nicht funktionieren. Du bist in der while schleife gefangen und das Programm müsste "einfrieren".

Nimm einen Thread dafür.

 

[julianpe]

Das kann doch gar nicht funktionieren. Du bist in der while schleife gefangen und das Programm müsste "einfrieren".

Nimm einen Thread dafür.

Ohje.. habe bisher nur wenig mit Threads zu tun gehabt.
Wie würde ein grober Thread-Entwurf dafür aussehen?

Habe mehrere Buttons die ich auf gleicher Weise überprüfen muss.

 

[Schesam]

Hab ich nciht dran gedacht^^ Aber stimmt, wenn du alle sim EDT ausführst, friert das Frame dann ein. deine While-Schleife in nen neuen Thread gepackt schaut so aus:

new Thread(() -> {
while(buttonStatus)        
                   try{
                       System.out.println("Voraus");
                   }catch(Exception e){
                       // TODO Auto-generated catch block
                        e.printStackTrace();
                   }
}).start();

 

[Meniskusschaden]

Du könntest es mit einem Timer machen:

import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.Timer;
import java.util.TimerTask;
import javax.swing.JButton;
import javax.swing.JFrame;

public class MousePressedTest implements MouseListener {

    private JFrame frame;
    private JButton button;
    Timer timer;

    private void start() {
        frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(800, 600);
        button = new JButton("Testbutton");
        frame.add(button);
        button.addMouseListener(this);
        frame.setVisible(true);
    }

    public static void main(String[] args) {
        (new MousePressedTest()).start();
    }

    @Override
    public void mouseClicked(MouseEvent e) {
    }

    @Override
    public void mousePressed(MouseEvent e) {
        timer = new Timer();
        timer.scheduleAtFixedRate(new TimerTask() {
            @Override
            public void run() {
                String text = button.getText();
                text = text.charAt(text.length()-1) + text.substring(0, text.length()-1);
                button.setText(text);
            }
        }, 0, 100);
    }

    @Override
    public void mouseReleased(MouseEvent e) {
        if (timer!=null) {
            timer.cancel();
        }
    }

    @Override
    public void mouseEntered(MouseEvent e) {
    }

    @Override
    public void mouseExited(MouseEvent e) {
    }

}

 

[julianpe]

Hab ich nciht dran gedacht^^ Aber stimmt, wenn du alle sim EDT ausführst, friert das Frame dann ein. deine While-Schleife in nen neuen Thread gepackt schaut so aus:

new Thread(() -> {
while(buttonStatus)       
                   try{
                       System.out.println("Voraus");
                   }catch(Exception e){
                       // TODO Auto-generated catch block
                        e.printStackTrace();
                   }
}).start();

Genau das funktionier wie erhofft - besten Dank dafür.
Unter Eclipse für ein .jar Programm läuft das sehr gut.
Jedoch bekomme ich in Android Studio hierbei einen Fehler.
"Lambda Expressions are not supported at this language level"
Scheint also ein Feature von Java 8 zu sein?
Wie könnte man obigen Code Java 7 kompatibel umschreiben?

 

[Schesam]

Ja, Lambdas Sind aus Java 8. In Java 7, also ohne Lambda machst du ne einfache anonyme innere Klasse.

new Thread(new Runnable() {
                    @Override
                    public void run() {
                        while (buttonStatus) {
                            try {
                                System.out.println("Voraus");
                            } catch (Exception e) {
                                // TODO Auto-generated catch block
                                e.printStackTrace();
                            }
                        }
                    }
                }).start();