Setter funktioniert nicht

[Nuss]

Hallo,

ich habe drei Klassen. Eine Gui-class in der es einen MouseListener gibt, durch den ich die x- und y-Position des Mausklicks in zwei Variablen speichere. Die Player-Position ist anfangs x=1, y=1.

In die CalcVector-class werden in die update-Methode über die main-class die Player und Mouseposition-Werte überliefert. Im Grunde soll ein Vektor zwischen dem Player und der Mauspositon errechnet werden.

Mein Problem liegt darin, dass ich möchte, dass die Player-Position bei Mausklick, die Position von der Maus annimmt. Irgendwie well mein Setter da nicht arbeiten.

Main:

package m_events;

import javax.swing.JFrame;

public class main {
	public static void main(String[] args) {
		Gui g = new Gui();
		CalcVector cv = new CalcVector();
		g.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		g.setSize(300, 300);
		g.setVisible(true);
		
		while(true){
		cv.update(g.getPlayerX(), g.getPlayerY(), g.getMouseX(), g.getMouseY()); //Werte aus gui nach CalcVector
		
		System.out.println(g.getPlayerX()); //die Spielerposition ändert sich durch den Setter leider nicht
		
		try {
			Thread.sleep(30);
		} catch (InterruptedException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		}
		
		
	}
}

Gui:

package m_events;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class Gui extends JFrame{
	private JPanel mousepanel;
	private float m_posx;
	private float m_posy;
	private float p_posx=1;
	private float p_posy=1;
	
	
	public Gui(){
		super("title");
		mousepanel = new JPanel();
		mousepanel.setBackground(Color.WHITE);
		add(mousepanel, BorderLayout.CENTER);
		
		HandlerClass handler = new HandlerClass();
		mousepanel.addMouseListener(handler);
		mousepanel.addMouseMotionListener(handler);
	}
	
	private class HandlerClass implements MouseListener, MouseMotionListener{
		public void mouseClicked(MouseEvent event){
			m_posx=event.getX();
			m_posy=event.getY();
		}

		@Override
		public void mouseDragged(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}

		@Override
		public void mouseMoved(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}

		@Override
		public void mousePressed(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}

		@Override
		public void mouseReleased(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}

		@Override
		public void mouseEntered(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}

		@Override
		public void mouseExited(MouseEvent e) {
			// TODO Auto-generated method stub
			
		}
	}
	
	public float getMouseX(){
		return m_posx;
	}
	
	public float getMouseY(){
		return m_posy;
	}
	public float getPlayerX(){
		return p_posx;
	}
	public float getPlayerY(){
		return p_posy;
	}
	public void setPlayerX(float x){
		p_posx=x;
	}
	public void setPlayerY(float y){
		p_posy=y;
	}
}

CalcVector:

package m_events;

public class CalcVector {
	private float playerx;
	private float playery;
	private float mousex;
	private float mousey;
	
	private float vector_x;
	private float vector_y;
	private float betrag;
	
	Gui g = new Gui();
	
	
	public void update(float px, float py, float mx, float my){
		mousex=mx;
		mousey=my;
		playerx=px;
		playery=py;
		
		
		// Calculating Vector and speed
		
		vector_x=mousex-playerx;
		vector_y=mousey-playery;
		betrag = (float) Math.sqrt(vector_x*vector_x+vector_y*vector_y);
		
		g.setPlayerX(mousex); //hier soll die Playerposition gleich der Mausposition gesetzt werden
		g.setPlayerY(mousey);
		
	}
	
	public float getVectorX(){
		return vector_x;
	}
	
	public float getVectorY(){
		return vector_y;
	}
	public float getMouseX(){
		return mousex;
	}
	public float getMouseY(){
		return mousey;
	}
	
	public float getBetrag(){
		return betrag;
	}
}

Ich hoffe ich habe alles einigermaßen verständlich erklären können. Danke schon einmal für jede Anregung und Hilfe!

 

[njans]

Es ist kein Wunder, dass das bei dir nicht geht:
Du bekommst die SpielerPosition durch

 public void mouseClicked(MouseEvent event){
            m_posx=event.getX();
            m_posy=event.getY();
        }

Und dann überschreibst du diese mit

  g.setPlayerX(mousex); //hier soll die Playerposition gleich der Mausposition gesetzt werden
        g.setPlayerY(mousey);

Wieso willst du die überhaupt überschreiben?
Ich gewinne den Eindruck, dass du deine Struktur nicht sauber durchdacht hast.

 

[Nuss]

Danke für die Antwort,

irgendwie liegt da ein Missverständnis vor. Ich bekomme die Mausposition und möchte diese bei Klick in die Spielerpositionsvariablen hineinschreiben. Dafür verwende ich bei der update-Methode den Setter.

Gui:

public void setPlayerX(float x){
		p_posx=x;// hier wird die player position behandelt
	}public void setPlayerY(float y){
		p_posy=y;
	}

CalcVector:

g.setPlayerX(mousex); //hier wird die player position durch die maus position ersetzt
g.setPlayerY(mousey);

Der Sinn davon sollte später dann jener sein, dass der Spieler zur Mausposition "geht". Deshalb möchte ich dann auch den Vektor und Betrag davon berechnen und eine Geschwindigkeit festsetzen. Bisher habe ich das einfach einmal so "gelöst", dass die aktuelle Spielerposition der Mausposition bei Klick entspricht.

 

[Daelras]

In deiner Klasse CalcVector erstellst du ein neues Objekt der Klasse Gui, auf welchem in der Methode update die Methoden setPlayerX und setPlayerY aufgerufen werden. Die Methoden zum Auslesen der Koordinaten rufst du allerdings auf dem Gui-Objekt auf, welches du in deiner main-Methode anlegst. Daher ist dein Problem weniger, dass deine Setter nicht funktioniert, sondern dass du sie auf dem falschen Objekt aufrufst.

 

[Nuss]

Dh. wie kann ich dann sonst die Positionen aus Gi and CalcVector übergeben, wenn nicht mit:

cv.update(g.getPlayerX(), g.getPlayerY(), g.getMouseX(), g.getMouseY());

Das Problem ist, ich habe ehrlich gesagt, obwohl ich mir deine Antwort öfters durchgelesen haben, meinen Fehler noch nicht wirklich erkannt. Liegt der Fehler darin, dass ich an void update() 4 Werte aus Gui übergebe, und in derselben Methode die Werte in Gui verändere? Und mit welchem Objekt soll ich dann sonst die Koordinaten ändern bzw. aufrufen, wenn nicht mit Gui?

 

[Daelras]

Das Problem liegt nicht bei der Übergabe der Werte an dein CalcVector-Objekt. Das wird so funktionieren. In der update-Methode der CalcVector-Klasse rufst du deine Setter in den Zeilen 29 und 30 auf einem Gui-Objekt auf. Dieses Objekt erstellst du in Zeile 13. Eigentlich willst du allerdings das Gui-Objekt verwenden, was du in deiner main-Methode erstellst.

 

[Nuss]

Achso, in Ordnung. Ich hab's nun verstanden. Aber wie soll ich das Problem dann lösen? Eigentlich dachte ich, dass ich beim Erstellen eines Objektes, zB.:

Gui g = new Gui();

auf Variablen aus der Klasse gui zugreifen bzw. diese auch verändern kann, selbst wenn ich das Objekt bereits in einer anderen Klasse erzeugt habe.

Wie kann ich dann auf das Objekt g aus der Klasse main zugreifen, wenn's mit dem Objekt g aus CalcVector nicht funktionieren will? Ich habe da ehrlich gesagt gerade keinen blassen Schimmer wie ich da ansetzen sollte.

Danke schonmal für jegliche Hilfe!