Problem mit Threads

[MasseElch]

Hallo Forum,

mir fiel kein aussagekräftiges Thema ein, verzeiht dafür.

Ich habe eine Klasse SlideShow, die Bilder in einer Laufleiste anzeigt.

Ein Thread bewegt die Bilder und repainted.

Dazu implementiere ich das Runnable-Interface in einer inneren Klasse in der Klasse SlideShow:

class moveAndPaintImages implements Runnable {
		public void run() {
			try {
				while (SlideShow.this.running) {
					Thread.sleep(8L);
					SlideShow.this.moveImages();
					repaint();
				}
			} catch (InterruptedException e) {
				//TODO - Exception Handling
			}
		}
	}

Im Konstruktor der Klasse SlideShow erzeuge ich dann den Thread:

//create Thread
Thread moveImages = new Thread(new moveAndPaintImages());
moveImages.start();

Mit einem Button wird der Wert der Variable running geändert, damit der Bildlauf läuft oder stehen bleibt:

if(pressedButton == gui.getButtonPlay()) {
			if(this.slideShow.getRunning()) {
				this.slideShow.setRunning(false);
				gui.getButtonPlay().setText("Play");
			} else {
				this.slideShow.setRunning(true);
				gui.getButtonPlay().setText("Pause");
			}

Nun passiert aber nichts, wenn ich den Button Play/Pause drücke.
Ich habe versuchsweise als Event die Methoden moveImages() und repaint() direkt mit dem Button einmalig aufgerufen. Sie funktionieren richtig.

Mag mir jemand helfen?

Lg
MasseElch

 

[torsan]

Hi,

du musst die repaint() Methode im Swing Thread ausführen.

Versuch mal folgendes:

class moveAndPaintImages implements Runnable {
	@Override
	public void run() {
		try {
			while (SlideShow.this.running) {
				Thread.sleep(8L);
				SwingUtilities.invokeLater(new Runnable() {
					@Override
					public void run() {
						SlideShow.this.moveImages();
						repaint();
					}
				});
			}
		}
		catch (final InterruptedException e) {
			// TODO - Exception Handling
		}
	}
}

Gruß,
torsan

 

[MasseElch]

Hallo,

danke für die schnelle Antwort.

Funktioniert leider nach wie vor nicht.

Ich bin auch leider echt ratlos :-(

 

[ssoul26]

Häng mal revalidate() mit rein.

 

[torsan]

kannst du bitte mal dieMethode moveImages() posten. Evtl. hilft das weiter.

 

[MasseElch]

revalidate() hat auch nichts gebracht.

Hier die moveImages():

public void moveImages() {
		for(SlideShowImage img : this.imageList.getImageList()) {
			img.move();
		}
	}

move() aus dem SlideShowImage:

public void move() {
		this.yPos = this.yPos+1;
	}

Anmerkung:
Das ganze ist zweigeteilt, weil es später 1. auswählbar sein soll, wie schnell die Bilder laufen und 2. es für jedes Bild eine verschiende Geschwindigkeit geben soll. (Die können nebeneinander laufen)

Lg

 

[KSG9|sebastian]

Wie ist die Variable "running" definiert? Diese MUSS volatile sein.
Unter Umständen reicht es wenn du im Setter ein notifyAll() einbaust.

Poste mal die komplette Klasse. Und Klassen schreibt man groß..so nebenbei.

 

[MasseElch]

running ist volatile. Und ich hab alle Klassen groß, das ist nur ein Flüchtigkeitsfehler. Verzeihung

Hier die gesamte Klasse:

package slideShow;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.io.IOException;

import javax.imageio.ImageIO;
import javax.swing.JPanel;

import slideShowObjects.ImageList;
import slideShowObjects.SlideShowImage;

public class SlideShow extends JPanel {

	private static final long serialVersionUID = 1L;

	/**
	 * openerImage; shown when program starts up
	 */
	private Image openerImage;
	
	/**
	 * the images we want to show in our slideShow
	 */
	private ImageList imageList;
	
	/**
	 * the gui
	 */
	GeneralUserInterface gui;
	
//SLIDESHOWRUNNING?
	/**
	 * true if SlideShow is 'sliding'
	 * false if it's not
	 */
	private volatile boolean running = false;

	/**
	 * contructor
	 * loads openerImage and gets gui
	 */
	public SlideShow(GeneralUserInterface gui) {
		
		this.gui = gui;
		try {
			this.openerImage = ImageIO.read(getClass().getResource("../resources/openerImageBlack.jpg"));
		} catch(IOException e) {
			//TODO - Exception Handling
		}
		//create instance of ImageList, otherwise we get problems on start up
		this.imageList = new ImageList();
		
		//create Thread
		Thread moveImages = new Thread(new moveAndPaintImages());
		moveImages.start();
	}
	
	/**
	 * we need to move the images down the screen
	 */
	public void moveImages() {
		for(SlideShowImage img : this.imageList.getImageList()) {
			img.move();
		}
	}
	
	/**
	 * now lets paint everything
	 */
	public void paint(Graphics g) {
		super.paint(g);
		
		//vars used in this method
		int screenWidth = GeneralUserInterface.getScreenWidth();
		int screenHeight = GeneralUserInterface.getScreenHeight();
		int frameWidth = GeneralUserInterface.getFrameWidth();
		int imageHeight = screenHeight/5;
		
		//set fontColor to red for the 'interpret - title' part
		g.setColor(Color.red);
		g.drawImage(	this.openerImage, 
						0,
						0,
						screenWidth,
						screenHeight,
						this);
		
		for (SlideShowImage img : this.imageList.getImageList()) {
			if(		img.getyPos() < screenHeight
				&&	img.getyPos() >= 0-imageHeight) {
						
						g.drawImage(img.getImage(),
									0,
									img.getyPos(),
									frameWidth,
									imageHeight,
									this);
						if(img.getInterpret() != null) {
							g.drawString(	img.getInterpret()+" - "+img.getTitle(),
											0,
											img.getyPos()+imageHeight+12);
						}
			} else if(img.getyPos() > screenHeight+imageHeight) {
				img.setyPos(this.imageList.getLastVerticalPos());
			}
		}
	}
	
	class moveAndPaintImages implements Runnable {
		public void run() {
			try {
				while (SlideShow.this.running) {
					Thread.sleep(8L);
					SlideShow.this.moveImages();
					repaint();
				}
			} catch (InterruptedException e) {
				
			}
		}
	}
	
	/**
	 * get the imageList
	 * @return imageList
	 */
	public ImageList getImageList() {
		return this.imageList;
	}
	
	/**
	 * set running
	 * true if we want to let the SlideShow 'slide the images'
	 * false if not
	 * @param the new value for running
	 */
	public void setRunning(boolean running) {
		this.running = running;
	}
	
	/**
	 * get running
	 */
	public boolean getRunning() {
		return this.running;
	}
}

 

[KSG9|sebastian]

Öhm ja..wie soll das auch funktionieren

while (SlideShow.this.running) {
                    Thread.sleep(8L);
                    SlideShow.this.moveImages();
                    repaint();
                }

Beim ersten Aufruf ist die Bedingung der While-Schleife false, damit war's das. Der Code-Block wird nie mehr durchlaufen.
Der Aufruf blockt ja nicht.

1. Begin der Run-Methode
2. if(running) -> false
3. Ende der Run-Methode

 

[MasseElch]

Hmmm ... okay!


Dann hab ich das mit den Threads noch nicht verstanden. ABer ich weiß wo der Fehler ist. Danke!

Ps:
Ist der Code so in Ordnung? Von der Form etc? Ich bin jung in Java und bin für jede (konstruktive) Kritik dankbar

 

[KSG9|sebastian]

Das Problem sind nicht Threads sondern der Code. Threads laufen nebenher, unterscheiden sich aber sonst nicht von allem anderen. Du kannst ne Endlosschleife machen (while ( true)) und darin dann 1. prüfen ob running ==true. Um zu verstehen was da passiert bau am besten System. out's ein.

 

[MasseElch]

Das der Code das Problem war hab ich soweit verstanden.

Nur war ich bis vorhin der Meinung ein Thread würde das, was er macht, immer wieder tun, bis man ihn beendet. Aber dem ist ja eben nicht so.

Aber dank deiner Hilfe habe ich es nun verstanden (weil Threads noch Mal angeschaut) und es läuft jetzt.

Vielen Dank!