Thread pausieren

Chiller · 11. Juni 2011

Hi Leute,

ich bin gerade dabei ein kleines Spiel zu schreiben. Bisher habe ich 2 Klassen:
"Mastermind.java" und "Stoppuhr.java".

die Stoppuhr soll einfach die Zeit hochzählen im Format: m:s "00:00"

die Stoppuhrklasse sieht so aus:

Stoppuhr.java

Code

public class Stoppuhr{
private int m=0;
private int s=0;
private boolean stop;
private boolean isRunning = false;
private int zaehler;
private long save = System.currentTimeMillis();
private Zähler z = new Zähler();
private boolean pause = false;
public String getZeit(){
if(m<10&&s<10)return("0"+m+":0"+s);
else if(m<10&&s>10)return("0"+m+":"+s);
else if(m>10&&s<10)return(""+m+":0"+s);
else return(""+m+":"+s);
}
public void weiter(){
stop = false;
}
public void stoppe(){
stop = true;
}
public void pauseZähler(){
pause = true;
}
public void resumeZähler(){
pause = false;
}
public void starte(){
z = new Zähler();
z.start();
}
public boolean getPause(){
return pause;
}
public boolean getStop(){
return stop;
}
class Zähler extends Thread{
public void run(){
if(!isRunning){
isRunning=true;
stop = false;
zeitSetzen();
isRunning=false;
}
}
public void zeitSetzen(){
while(!stop){
long tmp = System.currentTimeMillis();
if(!pause){
zaehler = (int)(tmp-save);
m = zaehler/60000;
s = (zaehler-(m*60000))/1000;
}else{
long tmp2 = save;
save += tmp-tmp2;
}
}
}
}
}

Alles anzeigen

und die Zeit soll mit in einer Swing-Oberfläche angezeigt werden:

Mastermind.java

Java

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import saufg24.Stoppuhr;
public class MasterMind extends JFrame{
//.... verschieden Swing-Komponenten
private JLabel zeit = new JLabel("00:00");
private JMenuBar menuBar = new JMenuBar();
private JLabel zeit = new JLabel("00:00");
public static void main(String[] args){
MasterMind gui = new MasterMind();
gui.setVisible(true);
gui.setLocationRelativeTo(null);
gui.setDefaultCloseOperation(EXIT_ON_CLOSE);
}
public MasterMind(){
super("MasterMind");
zThread.start();
setJMenuBar(menuBar);
JMenu fileMenu = new JMenu("File");
JMenuItem newGameItem = new JMenuItem("Neues Spiel");
JMenuItem pauseGameItem = new JMenuItem("Pause");
fileMenu.add(newGameItem);
fileMenu.add(pauseGameItem);
//Menu Items ActionListener
newGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
s = new Stoppuhr();
s.starte();
}});
pauseGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
if(s.getPause()){
s.pauseZähler();
}
else if(!s.getPause()){
s.resumeZähler();
}
}});
//... anderer Code zur Zusamensetzung der Swing-Komponenten
}
class Zeit extends Thread{
public void run(){
while(true){
zeit.setText(s.getZeit());
}
}
}
}

Alles anzeigen

Ich kann die Zeit zwar mit dem "newGameItem" starten aber das mit pause und resume funktioniert leider nicht.
Kann mir vielleicht jemand weiterhelfen und/oder Verbesserungsvorschläge machen?

Danke schon mal im Vorraus,

Mfg,
Chiller

Edit by Awesome:
Habe den code-Tag mal verbessert & Spoiler für die Übersicht hinzugefügt.

BigBob · 11. Juni 2011

Also so wie ich das sehe, machst du gar kein Thread Objekt und rufst damit auch die run() Methode auf.

Das geht so:
Thread thread1 = new Thread(Hier kommt das Objekt rein, dass in einem Thread laufen soll);
thread1.start(); //Damit wird die run() Methode gestartet.

Dann noch was du besser machen solltest:
mach lieber implements Runnable anstatt extends Thread, ist das gleich aber dann kannst du noch von einer anderen Klasse erben.
Und verwende nie Umlaute wie ä, ö und ü.

Chiller · 11. Juni 2011

ich habe die beiden threads leider vergessen noch beim code mit anzugeben (habe ihn gekürzt)

private Stoppuhr s = new Stoppuhr();
private Zeit zThread = new Zeit();

der compiler meckert über nichts aber es geht einfach nicht^^

kompletter code von Mastermind

[

Java

package saufg24;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import saufg24.Stoppuhr;
public class MasterMind extends JFrame{
private String sName;
private boolean gewonnen = false;
//Allgemeines Panel
private JPanel spielPanel = new JPanel(new BorderLayout());
//Status-Labels
private JLabel name = new JLabel("Name");
private JLabel zeit = new JLabel("00:00");
//Menu
private JMenuBar menuBar = new JMenuBar();
private Stoppuhr s = new Stoppuhr();
private Zeit zThread = new Zeit();
public static void main(String[] args){
MasterMind gui = new MasterMind();
gui.setVisible(true);
gui.setLocationRelativeTo(null);
gui.setDefaultCloseOperation(EXIT_ON_CLOSE);
}
public MasterMind(){
super("MasterMind");
zThread.start();
//Namen abfragen und setzen
do{
sName = JOptionPane.showInputDialog(null,"Geben Sie Ihren Namen ein","Namensabfrage",JOptionPane.PLAIN_MESSAGE);
}while(sName==null || sName.isEmpty());
name.setText(sName);
//Menubar
setJMenuBar(menuBar);
JMenu fileMenu = new JMenu("File");
JMenu settingsMenu = new JMenu("Settings");
JMenu helpMenu = new JMenu("?");
menuBar.add(fileMenu);
menuBar.add(settingsMenu);
menuBar.add(helpMenu);
JMenuItem newGameItem = new JMenuItem("Neues Spiel");
JMenuItem pauseGameItem = new JMenuItem("Pause");
JMenuItem highscoreItem = new JMenuItem("Highscore");
JMenuItem aboutItem = new JMenuItem("About");
JMenuItem rulesItem = new JMenuItem("Spielregeln");
JMenuItem exitItem = new JMenuItem("Exit");
JCheckBox beginCB = new JCheckBox("Beginnen");
JCheckBox colorCB = new JCheckBox("Farbe: rot");
fileMenu.add(newGameItem);
fileMenu.add(pauseGameItem);
fileMenu.add(highscoreItem);
fileMenu.add(exitItem);
settingsMenu.add(beginCB);
settingsMenu.add(colorCB);
helpMenu.add(rulesItem);
helpMenu.add(aboutItem);
//Menu Items ActionListener
newGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
s = new Stoppuhr();
s.starte();
}});
pauseGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
if(s.getPause()){
s.pauseZähler();
}
else if(!s.getPause()){
s.resumeZähler();
}
}});
exitItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
System.exit(0);
}
});
//JPanel Gridlayout Spiel 7,7
JPanel spiel = new JPanel();
spiel.setLayout(new GridLayout(1,2));
//JPanel Boxlayout Status
JPanel status = new JPanel();
status.setLayout(new BorderLayout());
status.add(name,BorderLayout.WEST);
status.add(zeit,BorderLayout.EAST);
//Zusammensetzen zu spiel panel
spielPanel.add(spiel,BorderLayout.CENTER);
spielPanel.add(status,BorderLayout.SOUTH);
//zu Hauptfenster
getContentPane().add(spielPanel);
this.setSize(500,500);
}
class Zeit extends Thread{
public void run(){
while(true){
zeit.setText(s.getZeit());
}
}
}
}

Alles anzeigen

BigBob · 11. Juni 2011

Das liegt daran, dass du einfach nur Objekte erstellst.

Also so müsste es gehen:

private Stoppuhr uhr;

uhr = new Stoppuhr();

Thread thread1 = new Thread(uhr);
thread1.start();

Chiller · 11. Juni 2011

ok jetzt sieht es so aus... (geht aber immer noch nicht):

Mastermind.java

Java

package saufg24;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import saufg24.Stoppuhr;
public class MasterMind extends JFrame{
private String sName;
//Allgemeines Panel
private JPanel spielPanel = new JPanel(new BorderLayout());
//Status-Labels
private JLabel name = new JLabel("Name");
private JLabel zeit = new JLabel("00:00");
//Menu
private JMenuBar menuBar = new JMenuBar();
//Zeit
private Zeit zThread = new Zeit();
private Thread tZeit = new Thread(zThread);
private Stoppuhr s = new Stoppuhr();
private Thread tStoppuhr = new Thread(s);
public static void main(String[] args){
MasterMind gui = new MasterMind();
gui.setVisible(true);
gui.setLocationRelativeTo(null);
gui.setDefaultCloseOperation(EXIT_ON_CLOSE);
}
public MasterMind(){
super("MasterMind");
//Namen abfragen und setzen
do{
sName = JOptionPane.showInputDialog(null,"Geben Sie Ihren Namen ein","Namensabfrage",JOptionPane.PLAIN_MESSAGE);
}while(sName==null || sName.isEmpty());
name.setText(sName);
tZeit.start();
//Menubar
setJMenuBar(menuBar);
JMenu fileMenu = new JMenu("File");
JMenu settingsMenu = new JMenu("Settings");
JMenu helpMenu = new JMenu("?");
menuBar.add(fileMenu);
menuBar.add(settingsMenu);
menuBar.add(helpMenu);
JMenuItem newGameItem = new JMenuItem("Neues Spiel");
JMenuItem pauseGameItem = new JMenuItem("Pause");
JMenuItem highscoreItem = new JMenuItem("Highscore");
JMenuItem aboutItem = new JMenuItem("About");
JMenuItem rulesItem = new JMenuItem("Spielregeln");
JMenuItem exitItem = new JMenuItem("Exit");
JCheckBox beginCB = new JCheckBox("Beginnen");
JCheckBox colorCB = new JCheckBox("Farbe: rot");
fileMenu.add(newGameItem);
fileMenu.add(pauseGameItem);
fileMenu.add(highscoreItem);
fileMenu.add(exitItem);
settingsMenu.add(beginCB);
settingsMenu.add(colorCB);
helpMenu.add(rulesItem);
helpMenu.add(aboutItem);
//Menu Items ActionListener
newGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
s = new Stoppuhr();
tStoppuhr = new Thread(s);
tStoppuhr.start();
}});
pauseGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
s.triggerPause();
}});
exitItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
System.exit(0);
}
});
//JPanel Gridlayout Spiel 7,7
JPanel spiel = new JPanel();
spiel.setLayout(new GridLayout(1,2));
//JPanel Boxlayout Status
JPanel status = new JPanel();
status.setLayout(new BorderLayout());
status.add(name,BorderLayout.WEST);
status.add(zeit,BorderLayout.EAST);
//Zusammensetzen zu spiel panel
spielPanel.add(spiel,BorderLayout.CENTER);
spielPanel.add(status,BorderLayout.SOUTH);
//zu Hauptfenster
getContentPane().add(spielPanel);
this.setSize(500,500);
}
class Zeit implements Runnable{
public void run(){
while(true){
zeit.setText(s.getZeit());
}
}
}
}

Alles anzeigen

'Stoppuhr

Code

package saufg24;
public class Stoppuhr implements Runnable{
private int m=0;
private int s=0;
private boolean isRunning = false;
private int zaehler;
private long save = System.currentTimeMillis();
private boolean pause = false;
private boolean run;
public String getZeit(){
if(m<10&&s<10)return("0"+m+":0"+s);
else if(m<10&&s>10)return("0"+m+":"+s);
else if(m>10&&s<10)return(""+m+":0"+s);
else return(""+m+":"+s);
}
public void stoppe(){
run = false;
}
public void triggerPause(){
pause = !pause;
}
public void run(){
if(!isRunning){
isRunning=true;
run = true;
while(run){
long tmp = System.currentTimeMillis();
if(!pause){
zaehler = (int)(tmp-save);
m = zaehler/60000;
s = (zaehler-(m*60000))/1000;
}else{
long tmp2 = save;
save += tmp-tmp2;
}
}
isRunning=false;
}
}
}

Alles anzeigen

gerade ein bisschen was verändert... funktioniert leider immer noch nicht.

BigBob · 11. Juni 2011

Also, ich hab zwar immer noch nicht wirklich kapiert, was du willst aber denke du hast das hier falsch:

Code

pauseGameItem.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
if(s.getPause()){
s.resumeZähler();
}
else if(!s.getPause()){
s.pauseZähler();
}
}});

Du hast, wenn pause true, es wieder auf true gesetzt und wenn pause false war, wieder auf false.
Also wenn man jetzt Pause drückt, pausiert die Zeit aufjedenfall mal.

Chiller · 12. Juni 2011

Das klingt jetzt vielleicht wie eine Anfänger-Frage aber ich stelle sie trotzdem mal:

In der "Mastermind"-Klasse erzeuge ich mir ja ein Objekt "gui".
Wie kann ich in Klasse "StoppUhr" die Methode gui.update(); aufrufen?
Wenn es nicht geht, wie kann ich dann in dem Thread von Stoppuhr ein JLabel von Mastermind updaten?

Mastermind-Klasse-update

'StoppUhr-Code

Code

package saufg24;
public class Stoppuhr implements Runnable{
private int m=0;
private int s=0;
public boolean isRunning = false;
private int zaehler;
private long save = System.currentTimeMillis();
private boolean pause;
private boolean run;
public String getZeit(){
if(m<10&&s<10)return("0"+m+":0"+s);
else if(m<10&&s>10)return("0"+m+":"+s);
else if(m>10&&s<10)return(""+m+":0"+s);
else return(""+m+":"+s);
}
public void stoppe(){
run = false;
}
public void triggerPause(){
pause = !pause;
}
public void calcZeit(){
while(run){
long tmp = System.currentTimeMillis();
if(!pause){
zaehler = (int)(tmp-save);
m = zaehler/60000;
s = (zaehler-(m*60000))/1000;
//TODO hier soll er die update() Methode von MasterMind ausführen oder eben das JLabel updaten.
}else{
save = tmp - (zaehler+1);
}
}
}
public void run(){
if(!isRunning){
isRunning=true;
run = true;
calcZeit();
isRunning=false;
}
}
}

Alles anzeigen

Mfg

EDIT:// ok ich habe es anders gelöst...^^ funktioniert jetzt

Teilen