Boolean Werte - immer "false"

[Nuss]

Hallo,

ich stehe vor einem Problem, das eigentlich keines sein sollte.
Ich habe eine main-Class, in der eine while-Schleife mit Sys.out.println
existiert, die bei einem bestimmten Tastendruck true und sonst false liefern sollte.

package gui;

import javax.swing.JFrame;

public class main {

	public static void main(String[] args) {
		Frame f = new Frame();
		Player player = new Player();
		Keyboard keyboard = new Keyboard();
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setSize(800, 600);
		f.setVisible(true);
		f.setResizable(false);
		 while(true){
			 System.out.println(keyboard.getUp());
			 
			 f.repaintScreen();
			 try {
				Thread.sleep(15);
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		 }
	}

}

Dann gibt es noch die Keyboard-Klasse, die den Tastendruck prüft und die
Boolean-Werte festlegt. Dann wird über einen getter der Wert an die Main
übergeben, zum Ausgeben.

package gui;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class Keyboard implements KeyListener{
	private boolean left=false;
	private boolean right=false;
	private boolean up=false;
	private boolean down=false;
	@Override
	public void keyTyped(KeyEvent e) {
		// TODO Auto-generated method stub
		
	}
	@Override
	public void keyPressed(KeyEvent e) {
		if(e.getKeyCode()==KeyEvent.VK_W)up=true;
		if(e.getKeyCode()==KeyEvent.VK_A)left=true;
		if(e.getKeyCode()==KeyEvent.VK_S)down=true;
		if(e.getKeyCode()==KeyEvent.VK_D)right=true;
		
		
	}
	@Override
	public void keyReleased(KeyEvent e) {
		if(e.getKeyCode()==KeyEvent.VK_W)up=false;
		if(e.getKeyCode()==KeyEvent.VK_A)left=false;
		if(e.getKeyCode()==KeyEvent.VK_S)down=false;
		if(e.getKeyCode()==KeyEvent.VK_D)right=false;
		
	}
	boolean getUp(){
		return up;
	}
	boolean getDown(){
		return down;
	}
	boolean getLeft(){
		return left;
	}
	boolean getRight(){
		return right;
	}
	
}

Irgendwie wird aber immer nur ein false-Wert ausgegeben. Ich verstehe die
Welt nicht mehr, wo liegt der Fehler? Ich wäre jedem sehr verbunden,
der mir da weiterhelfen könnte.

 

[SilverClaw]

Hast du deinen KeyListener denn irgendwo auch mal deinem Container hinzugefügt? Sonst kannst du da lange warten.

 

[Kiri]

Adde den KeyListener!

Keyboard keyboard = new Keyboard();
f.addKeyListener(keyboard);

 

[Nuss]

Achja, genau!
Mit

f.addKeyListener(keyboard);

funktioniert's natürlich. Aber warum funktioniert das nicht, wenn ich bei der Frame-Klasse den KeyListener "adde"?

package gui;

import java.awt.Graphics;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Frame extends JFrame{
	
	Screen screen;
	Player player = new Player();
	Keyboard keyboard = new Keyboard();
	
	public Frame(){
		super("Test");
		screen = new Screen();
		screen.setBounds(0,0,800,600);
		add(screen);
		addKeyListener(keyboard); //damit würd's nicht funktionieren
	}
	
	public class Screen extends JLabel{
		protected void paintComponent(Graphics g){
			super.paintComponent(g);

			g.drawImage(player.getLookNormal(), (int)player.getX(), (int)player.getY(), null);
	}
		
	
}
	
	public void repaintScreen(){
		screen.repaint();
	}
		
	
}

Warum muss ich über die Main auf Frame zugreifen und den KeyListener hinzufügen, und nicht gleich von Frame aus?

 

[Kiri]

Jedes Mal wenn du dies

Keyboard keyboard = new Keyboard();

schreibst, erstellst du eine neue Instanz von Keyboard. Wenn du in der Frame-Klasse den KeyListener addest, musst du dafür sorgen, das genau diese Instanz auch in deiner Main-Klasse oder wo auch immer du sie benötigst verwendet wird und keine neue Instanz erstellen.

Anders gesagt, wenn du in deiner Frame-Klasse so einen KeyListener addest - in der Main-Klasse aber auf einen neuen erstellst, diesen nicht addest, dann funktioniert das natürlich nicht - da die Verbindung fehlt.

 

[Nuss]

Ah, gut zu wissen. Vielen Dank! Jetzt funktioniert's.
-----
Ich habe allerdings noch eine Frage dazu. Ich soll
also in jeder Klasse in der ich auf Keyboard zugreifen
möchte, auch den KeyListener einbinden?

f.addKeyListener(keyboard);

In meiner Player-Klasse würde ich gerne die Position
des Spielers auf Tastendruck ändern. Bisher gab es keine
Reaktion, es hat sich nichts bewegt. Deshalb habe ich
nun auch jenen Code von oben in die Player-Klasse eingebunden,
in der die Positionen geändert werden sollen.

Da gibt's allerdings eine Fehlermeldung
(VariableDeclarationId expected) von Eclipse. Ich binde hier einmal
die vier Klassen ein, die mir gerade Probleme bereiten. Obwohl es mir Leid tut,
dass das so viel Code ist, würde es mich sehr freuen, wenn jmd. so gütig
wäre, um mich auf meine fehlenden Kompetenzen und Fehler hinzuweisen.

Player-Class:

package gui;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.image.BufferedImage;
import java.io.IOException;

import javax.imageio.ImageIO;

public class Player {
	private static BufferedImage [] look = new BufferedImage [5];
	 float posx=30;
	 float posy=30;
	 Frame f = new Frame();
	 Keyboard keyboard = new Keyboard();
	 f.addKeyListener(keyboard);
	 
	static{
	try {
		look[0] = ImageIO.read(Player.class.getClassLoader().getResourceAsStream("gfx/programmer.png"));
		look[1] = ImageIO.read(Player.class.getClassLoader().getResourceAsStream("gfx/programmer_left1.png"));
		look[2] = ImageIO.read(Player.class.getClassLoader().getResourceAsStream("gfx/programmer_left2.png"));
		look[3] = ImageIO.read(Player.class.getClassLoader().getResourceAsStream("gfx/programmer_right1.png"));
		look[4] = ImageIO.read(Player.class.getClassLoader().getResourceAsStream("gfx/programmer_right2.png"));
		
	
	} 
	catch (IOException e) {e.printStackTrace();
	}
	}
	
	
	BufferedImage getLookNormal(){
		return look[0];
	}

	
	public void updatePlayer(){
		if(keyboard.getLeft())posx-=5;
		if(keyboard.getRight())posx+=5;
		if(keyboard.getUp())posy-=5;
		if(keyboard.getDown())posy+=5;
	}
	
	float getX(){
		return posx;
	}
	float getY(){
		return posy;
	}
	

	}

Frame-Class:

package gui;

import java.awt.Graphics;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Frame extends JFrame{
	
	Screen screen;
	Player player = new Player();
	Keyboard keyboard = new Keyboard();
	
	public Frame(){
		super("Test");
		screen = new Screen();
		screen.setBounds(0,0,800,600);
		add(screen);
		addKeyListener(new Keyboard());
		
	}
	
	public class Screen extends JLabel{
		protected void paintComponent(Graphics g){
			super.paintComponent(g);
			g.drawImage(player.getLookNormal(), (int)player.getX(), (int)player.getY(), null);
			
	}
		
	
}
	
	public void repaintScreen(){
		screen.repaint();
	}
}

main-Class:

package gui;

import javax.swing.JFrame;

public class main {

	public static void main(String[] args) {
		Frame f = new Frame();
		Player player = new Player();
		
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setSize(800, 600);
		f.setVisible(true);
		f.setResizable(false);
		 while(true){
			player.updatePlayer();
			//System.out.println(player.posx);
			 f.repaintScreen();
			 try {
				Thread.sleep(15);
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		 }
	}

}

Keyboard-Class:

package gui;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class Keyboard implements KeyListener{
	private boolean left=false;
	private boolean right=false;
	private boolean up=false;
	private boolean down=false;
	
	@Override
	public void keyTyped(KeyEvent e) {
		// TODO Auto-generated method stub
		
	}
	@Override
	public void keyPressed(KeyEvent e) {
		if(e.getKeyCode()==KeyEvent.VK_W)up=true;
		if(e.getKeyCode()==KeyEvent.VK_A)left=true;
		if(e.getKeyCode()==KeyEvent.VK_S)down=true;
		if(e.getKeyCode()==KeyEvent.VK_D)right=true;
		
		
	}
	@Override
	public void keyReleased(KeyEvent e) {
		if(e.getKeyCode()==KeyEvent.VK_W)up=false;
		if(e.getKeyCode()==KeyEvent.VK_A)left=false;
		if(e.getKeyCode()==KeyEvent.VK_S)down=false;
		if(e.getKeyCode()==KeyEvent.VK_D)right=false;
		
	}
	boolean getUp(){
		return up;
	}
	boolean getDown(){
		return down;
	}
	boolean getLeft(){
		return left;
	}
	boolean getRight(){
		return right;
	}
	
}

Vielen Dank!