Threads Objekt bewegt sich nicht.

[Pansa]

Hallo
Ich habe nun schon seit gestern ein Problem,auf dessen Lösung ich nicht komme.Ich habe mich nämlich letztens an Threads herangewagt,und wollte/will eine kleine Wolke Programmieren,die über den Bildschirm fliegt.Leider will sich die Wolke nicht bewegen.Die Klasse in der die Wolke erstellt worden ist,erbt von der Klasse JLabel.Die Wolke ist ebenfals als Label in einem JPanel Dargestellt.

Hier mal der Code

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

import javax.imageio.ImageIO;
import javax.swing.JLabel;


public class jWolkenThread extends JLabel implements Runnable {
public int x=0;
private Image Wolken;
private boolean vor=true;
    public jWolkenThread() {
	
	
	       try {                
	          Wolken = ImageIO.read(new File("C:/Users/Admin/workspace/Bergsteigerspiel/src/Wolke.png"));
	       } catch (IOException e) {
	           
	       }
	    }

	    @Override
	    protected void paintComponent(Graphics g) {
	        super.paintComponent(g);
	        g.drawImage(Wolken, 0, 0, null);
	        repaint();
	
    }

    @Override
    public void run() {
	while(true){
	    synchronized (this) {
		if (x > getWidth() - Wolken.getWidth(this)) {
		vor = false;
		} else if (x <= 0)
		    vor = true;
		if (vor) {
		    x += 1;
		
		}
	    }
	    repaint();
	    try{
		Thread.sleep(20);
	    } catch (InterruptedException e){
	    break;
	    }
	    jWolkenThread t= new jWolkenThread();
		Thread wolken = new Thread(t);
		wolken.start();
	}
	
    }

}

Danke für eure Hilfe^^
LG Pansa

 

[Gucky]

Du musst auch die Wolke bewegen. Momentan veränderst du nur Variablen, die als Vorlage für die Koordinaten dienen sollen aber nicht die Koordinaten selbst.
Die Zeichneposition in paintComponent() musst du von x abhängig machen.

 

[Pansa]

Du musst auch die Wolke bewegen. Momentan veränderst du nur Variablen, die als Vorlage für die Koordinaten dienen sollen aber nicht die Koordinaten selbst.
Die Zeichneposition in paintComponent() musst du von x abhängig machen.

Danke für die Hilfe! Aber wie genau mache ich das?
LG Pansa

 

[Gucky]

Erstmal bitte Variablen immer klein und Klassennamen immer groß. Beide nach dem KamelCase.

Das ist eigentlich so einfach, dass ich dir nicht direkt die Lösung sagen werde.
Es ist das drawImage(...), wo du ein Zeichen austauschen musst. Dann müsste es hinhauen.

 

[Pansa]

Ich habe es jetz so gemacht:

g.drawImage(wolken, x, 0, null);

Allerdings bewegt sich die Wolke immer noch nicht.Ich weiß echt nichtmehr weiter :rtfm:.
LG Pansa

 

[Chloroplast]

das repaint() bewirkt das neuzeichnen des fensters. paintComponent zeichnet neu. Da gehört das repaint nicht hin, denn wenn du gerade gezeinet hast, muss ja nicht sofort neugezeichnet werden. das repaint gehört in den Thread in dem du deine Variable änderst

 

[Pansa]

Ich habe die Methode repaint() entfernt,funktioniert aber immernoch nocht .
LG Pansa

 

[Gucky]

Kannst du noch mal den neuen Code posten?

 

[Pansa]

Okay.Hier die Klasse von der Wolke:

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

    import javax.imageio.ImageIO;
import javax.swing.JLabel;
     
     
    public class JWolkenThread extends JLabel implements Runnable {
    public int x=0;
    private Image wolken;
    private boolean vor=true;
        public JWolkenThread() {
    
       
               try {                
                  wolken = ImageIO.read(new File("C:/Users/Admin/workspace/Bergsteigerspiel/src/Wolke.png"));
            
               } catch (IOException e) {
                   
               }
            }
     
            @Override
            protected void paintComponent(Graphics g) {
                super.paintComponent(g);
                g.drawImage(wolken, x, 0, null);
                g.setColor(Color.black);
                
            }
            
       
       
     
        @Override
        public void run() {
        while(true){
            synchronized (this) {
            if (x > getWidth() - wolken.getWidth(this)) {
            vor = false;
            } else if (x <= 0)
                vor = true;
            if (vor) {
                x += 1;
                repaint();
            }
            }
           
            try{
            Thread.sleep(20);
            repaint();
            } catch (InterruptedException e){
            break;
            }
            JWolkenThread t= new JWolkenThread();
            Thread wolken = new Thread(t);
            wolken.start();
            
        }
       
        }
     
    }

 

[Gucky]

Hast du schon mal debugged? Für den Code mal mit dem Debugger zeilenweise durch.
Ich kann grade leider nicht selber testen, da ich mit dem Handy online bin.

 

[Pansa]

Da kam auch nichts.Ist mit der while Schleife von der Logik her auch alles richtig,würde sich die Wolke nach rechts bewegen?Ich mache dann auch ersmal für heute Feierabend.,und suche morgen weiter nach einer Lösung ;-)
LG Pansa

 

[Chloroplast]

Achja, hast du den Thread auch gestartet irgentwo?

 

[Pansa]

Ja,hier in der Hauptklasse.Das Label in der die Wolke drinne ist heißt JWolkenVersuch.

Thread t=new Thread(JLWolkenVersuch);
	t.start();

LG Pansa