AWT Auto aus einer anderen Klasse her neu zeichnen

[alexj.]

Wir machen in der Schule Java und ich wollte an unserem Auto-Projekt weiterarbeiten und eine Steuerung hinzufügen doch ich habe jetzt schon Probleme und zwar ich fang eine beliebige Taste ab um das Auto ein bisschen nach Rechts zu bewegen doch es funktioniert nicht es passiert einfach nichts. Das Auto soll sich bewegen bei einer beliebigen Tasten erstmal, danach würde ich alles andere machen. Hier ist mein Quellcode:

Tut mir leid für die Schreibweise wie ich programmiere aber ich bin nicht gerade der größte Profi in Java.

Haupt.java (Main Methode)

import java.awt.*;
import java.awt.event.*;
import java.awt.image.ImageObserver;

public class Haupt extends Frame implements WindowListener
{
	//Frame erstellen mit dem Titel Auto
	private static Frame autoFrame = new Frame("Auto");
	
	//Größe des Fensters
	static final int H_SIZE = 800;
	static final int W_SIZE = 600;
	
	//Das/Die Auto(s)
	private static Auto car = new Auto();
	
	static ImageObserver myImageObserver;
	
	public Haupt()
	{
		autoFrame = this;
		autoFrame.setSize(H_SIZE, W_SIZE);
	    autoFrame.addWindowListener(this);
	    autoFrame.setLocation(50,50);
		//Fenster anzeigen:
		autoFrame.setVisible(true);
	}
	
	public void paint(Graphics g)
	{
		car.setg(g);
	    car.setRichtung(Auto.RECHTS);
	    car.setx(150);
	    car.sety(150);
	    car.setMyImageObserver(this);
	    car.zeichnen();
	}
	
	public static void main(String[] args) 
	{
		new Haupt();
		//AutoSteuerung starten um die Benutzer eingaben zu kontrollieren um das Auto zu steuern.
		new AutoSteuerung(autoFrame, car);
	}
	
	public void windowActivated(WindowEvent e){/*System.out.println("windowActivated");*/}
	public void windowIconified(WindowEvent e){/*System.out.println("windowIconified");*/}
	public void windowDeiconified(WindowEvent e){/*System.out.println("windowDeiconified");*/}
	public void windowDeactivated(WindowEvent e){/*System.out.println("windowDeactivated");*/}
	public void windowOpened(WindowEvent e){/*System.out.println("windowOpened");*/}
	public void windowClosing(WindowEvent e){/*System.out.println("windowClosing");*/ System.exit(0);}
	public void windowClosed(WindowEvent e){/*System.out.println("windowClosed");*/}
}

Auto.java

import java.awt.*;
import java.awt.image.*;


public class Auto 
{
	public int x;
	public int y;
	private int richtung;
	private Graphics g;
	  
	public static final int STOP   = 0;
	public static final int RECHTS = 1;
	public static final int LINKS  = 2;
	public static final int OBEN   = 3;
	public static final int UNTEN  = 4;

	Image wolke_links = Toolkit.getDefaultToolkit().getImage("C:\\rauchwolke_links.png");
	Image wolke_rechts = Toolkit.getDefaultToolkit().getImage("C:\\rauchwolke_rechts.png");
	ImageObserver myImageObserver;
	
	
	// Konstrukter Methoden
	public Auto() 
	{
	}
	public Auto(int r) 
	{
		this.setRichtung(r);
	}
	public Auto(int x, int y) 
	{
		this.setx(x);
		this.sety(y);
	}
	public Auto(int x, int y, int r) 
	{
		this.setx(x);
		this.sety(y);
		this.setRichtung(r);
	}
	
	  
	// Setter-Getter x,y
	public boolean setx(int wert) 
	{
		if (wert < 0) {return false;} 
		else {x = wert; return true;}
	}
	public boolean sety(int wert) 
	{
		if (wert < 0) {return false;} 
		else {y = wert; return true;}
	}
	public boolean setxy(int x, int y) 
	{
		if (!this.setx(x)) return false;
		if (!this.sety(y)) return false;
		zeichnen(); return true;
	}
	public int getx()
	{
	  return x;
	}
	public int gety()
	{
	  return y;
	}

	//Set Graphics - Zugriff auf Frame von Auto aus
	public void setg(Graphics g)
	{
	  this.g = g;
	}

	//Setter-Getter Auto Richtung
	public boolean setRichtung(int richtung) 
	{
		switch(richtung)
		{
			case 0: 										//STOP
			case 1: 										//RECHTS
			case 2: 										//LINKS
			case 3: 										//OBEN
			case 4: this.richtung = richtung; return true; 	//UNTEN
			default: return false; 							//Alles andere gibt es nicht
		}
	}
	public int getRichtung()
	{
		return richtung;
	}

	//Setter-Getter ImageObserver
	public void setMyImageObserver(ImageObserver io) 
	{
		this.myImageObserver = io;
	}
	public ImageObserver getMyImageObserver() 
	{
		return myImageObserver;
	}
	
	//Zeichnen vom Auto
	
	//Richtung
	public void zeichnen() {
		switch (richtung) 
		{
		case LINKS:
			this.zeichneLinks();
			break;
		case RECHTS:
			this.zeichneRechts();
			break;
		case OBEN:
			//this.zeichneoben();
			break;
		case UNTEN:
			//this.zeichneoben();
			break;
		}
	}
	//Zeichnen
	public void zeichneLinks() 
	{
		g.setColor(Color.BLACK);
		g.fillRect(x, y + 35, 120, 40);// Auto
		g.fillRect(x + 10, y + 20, 100, 20);// Auto

		g.setColor(Color.BLACK);
		g.fillOval(x + 10, y + 50, 30, 30);// Räder
		g.fillOval(x + 80, y + 50, 30, 30);// Räder

		g.setColor(Color.WHITE);
		g.fillOval(x + 13, y + 53, 24, 24);// Felge
		g.fillOval(x + 83, y + 53, 24, 24);// Felge

		g.setColor(Color.WHITE);
		g.fillRect(x + 12, y + 24, 54, 10);// Fenster
		g.fillRect(x + 68, y + 24, 36, 10);// Fenster

		g.setColor(Color.RED);
		g.fillRect(x + 110, y + 45, 10, 15);// Rückleuchte
		g.setColor(Color.YELLOW);
		g.fillRect(x + 1, y + 50, 10, 10);// Scheinwerfer

		this.g.drawImage(wolke_links, x + 117, y + 33, this.getMyImageObserver());
	}
	public void zeichneRechts() 
	{
		g.setColor(Color.BLACK);
		g.fillRect(x, y + 35, 120, 40);// Auto
		g.fillRect(x + 10, y + 20, 100, 20);// Auto

		g.setColor(Color.BLACK);
		g.fillOval(x + 10, y + 50, 30, 30);// Räder
		g.fillOval(x + 80, y + 50, 30, 30);// Räder

		g.setColor(Color.WHITE);
		g.fillOval(x + 13, y + 53, 24, 24);// Felge
		g.fillOval(x + 83, y + 53, 24, 24);// Felge

		g.setColor(Color.WHITE);
		g.fillRect(x + 50, y + 24, 54, 10);// Fenster
		g.fillRect(x + 12, y + 24, 36, 10);// Fenster

		g.setColor(Color.RED);
		g.fillRect(x + 1, y + 45, 10, 15);// Rückleuchte
		g.setColor(Color.YELLOW);
		g.fillRect(x + 110, y + 50, 10, 10);// Scheinwerfer

		this.g.drawImage(wolke_rechts, x - 50, y + 33, this.getMyImageObserver());
	}
	
}

und die Steuerung Klasse die Probleme macht:
AutoSteuerung.java

import java.awt.*;
import java.awt.event.*;
import java.awt.image.ImageObserver;

import javax.swing.JOptionPane;

public class AutoSteuerung implements KeyListener
{
	Frame f;
	Auto car;
	
	public AutoSteuerung(Frame f, Auto car)
	{
		this.f = f;
		f.addKeyListener(this);
		this.car = car;
	}

	public void keyPressed(KeyEvent e) {
		String s = "Taste gedrückt: Key = " + e.getKeyChar()
	            + " Code = " + e.getKeyCode();
		
	    car.setRichtung(Auto.LINKS);
	    car.setx(150);
	    car.sety(150);
	    car.setMyImageObserver(car.getMyImageObserver());
	    car.zeichnen();  
	    f.repaint();
	    //JOptionPane.showMessageDialog(null, s, "Test", JOptionPane.OK_CANCEL_OPTION);
	}
	public void keyReleased(KeyEvent e) {}
	public void keyTyped(KeyEvent e) {}
}

 

[GUI-Programmer]

Funktioniert deine keyPressed auch? Schreib doch mal ein System.out.println("keyPressed"); rein. Denke nämlich, dass da noch ein Aufruf wie frame.setFocusable(true); und frame.requestFocus() vorher fehlt, damit der KeyListener übehaupt wirkt.

Und noch was: Anstatt von

public static void main(String[] args) 
    {
        new Haupt();
        //AutoSteuerung starten um die Benutzer eingaben zu kontrollieren um das Auto zu steuern.
        new AutoSteuerung(autoFrame, car);
    }

Schreib doch eher sowas:

public static void main(String[] args) 
    {
        new AutoSteuerung(new Haupt());
    }

und in AutoSteuerung (Konstruktor) steht dann sowas:

public AutoSteuerung(Haupt f)
    {
        this.f = f;
        f.addKeyListener(this);
        this.car = f.getCar();
    }

Wobei die Klasse Haupt natürlich dann noch die Methode getCar() enthalten muss.


P.S. Über deine Bezeichnungen und der alzuhäufigen Verwendung sag ich jetzt lieber mal nichts!!!

 

[alexj.]

Danke aber das ändert auch nichts habe die getCar Methode reingemacht aber hat nichts genützt. Ja es fängt die Eingaben ab habe ja diese MessageBox eingebaut um zu testen ob es die Eingaben abfängt und es funktioniert. Wenn ich eine beliebige Taste drücke dann zeichnet es auch irgendetwas aber halt nicht das was ich will das Auto bleibt auf der selben Stelle stehen.

 

[GUI-Programmer]

Also bei Swing macht man das mit den Zeichnen überlicherweise so, dass nicht die paint() des JFrames überschrieben wird, sondern ein JPanel (oder Klasse die von JComponent erbt) dem JFrame hinzugefügt wird, und dessen paintComponent() überschrieben wird. Wahrscheinlich ist es bei AWT auch nicht viel anders.
Das heißt ich glaube du solltest mal ein Canvas deinem JFrame hinzufügen, und dessen paint() Methode überschreiben. Vielleicht funktionierts ja dann!

Falls es vielleicht dann doch an deiner generellen Vorgehensweise des Zeichnens liegt, also ein Graphics Objekt per set Methode zu übergeben, dann hilft vielleicht dass (meine .zip Datei), wobei du dann nur eine Figur hast, nämlich das Auto!!!

 

[bERt0r]

Also ich würde das Graphics Objekt nicht per set Methode übergeben und in der Autoklasse halten. Mach doch einfach eine public void zeichne(Graphics g);

Zur Bewegung deines Autos:

 public void paint(Graphics g)
    {
        car.setg(g);
        car.setRichtung(Auto.RECHTS);
        car.setx(150);
        car.sety(150);
        car.setMyImageObserver(this);
        car.zeichnen();
    }

Solange hier fixe Werte drinstehen kann sich dein Auto gar nicht bewegen, du setzt die Werte ja immer auf 150 bei jedem repaint.
Deshalb: Lass deinen Keylistener nur setFunktionen auf dein Auto ausführen (x,y und Richtung), dann führe ein repaint auf deinem Frame aus. In deinem Frame kümmerst du dich NUR darum dein Auto so wie es ist zu Zeichnen am einfachsten über auto.zeichne(g).

 

[alexj.]

Also soll ich in Auto.java eine paint Methode erstellen und die von Haupt entfernen. Dann soll ich die x, y, Richtung werte in Auto definieren und damit zeichnen und wenn ich dann will das er sich bisschen nach rechts bewegt mache ich:

Auto car;
car.setx(car.getx + 10);

Wenn aber die paint Methode in Auto ist muss ich irgendwas oben noch definieren das sie funktioniert ?
Die repaint() Methode mache ich dann in der AutoSteuerung, nachdem ich die x und y werte verändert habe ?

 

[GUI-Programmer]

Schau dir doch mal meinen Code aus den Link an

 

[bERt0r]

Du brauchst eine Sinnvolle Aufgabenverteilung:
Frame: Sorge dafür dass das Auto gezeichnet wird
Auto: Biete Funktionen an, mit denen man die Position des Autos ändern kann.
Biete eine Funktion an, mit der das Auto gezeichnet wird.

Wenn du jetzt in deiner Steuerung dein Auto bewegen willst sieht das so aus:

auto.setX(auto.getX()+10);
frame.repaint();

Was geschieht da? Du setzt zuerst die Position um, dann sagst du dem Frame, dass er neu gezeichnet werden soll. Also ruft der Frame die zeichneMich Funktion deines Autos auf, die zeichnet jetzt das auto auf der neuen Position.