Problem mit setLocation

[michaelb]

Hi Leute,
ich hab da ein Problem mit setLocation. In meinem Code:

/*
 * Steuert die ganzen Fenster
 * Autor: Michael Bergmann
 * Version: 1.0
 * Stand: läuft nicht
 */
import java.awt.*;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.net.URL;
import javax.swing.*;

public class RobotControl extends JFrame implements ActionListener {
	private JDesktopPane desk;
	private JMenuBar menu;
	private Image bild;
	private Console console;
	private Steuerung steuer;
	RobotControl() {
		URL url = getClass().getResource("logo.png");
		bild = getToolkit().getImage(url);
		desk = new JDesktopPane();
		setContentPane(desk);
		setTitle("Roboter Steuersoftware");
		setSize(800,600);
		setIconImage(bild);
		menu = new JMenuBar();
		setJMenuBar(menu);
		JMenu programm = new JMenu("Programm");
		menu.add(programm);
		JMenuItem programmexit = new JMenuItem("Beenden");
		programmexit.addActionListener(this);
		programm.add(programmexit);
		JMenu fenster = new JMenu("Fenster");
		menu.add(fenster);
		JMenuItem anordnen = new JMenuItem("Anordnen");
		anordnen.addActionListener(this);
		fenster.add(anordnen);
		Console console = new Console();
		console.setVisible(true);
		desk.add(console);
		Steuerung steuer = new Steuerung(console);
		steuer.setVisible(true);
		desk.add(steuer);
	}
	
	public static void main(String[] args) {
		JFrame.setDefaultLookAndFeelDecorated(true);
		RobotControl RC = new RobotControl();
		RC.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		RC.setVisible(true);
	}
	
	public void actionPerformed(ActionEvent ae) {
		JMenuItem item = (JMenuItem)ae.getSource();
		String text = item.getText();
		if(text == "Beenden") {
			setVisible(false);
			System.exit(0);
		}
		
		if(text == "Anordnen") {
			steuer.setLocation(10,10);
			console.setLocation(320,10);
		}
	}	
}

wenn ich das compiliere kommt keine Fehlermeldung! Aber sobald ich das ausführe und auf den Menüeintrag Anordnen klick, dann bekomm ich das:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at RobotControl.actionPerformed(RobotControl.java:65)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1849)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2169)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)
at javax.swing.AbstractButton.doClick(AbstractButton.java:302)
at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1000)
at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1041)
at java.awt.Component.processMouseEvent(Component.java:5488)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3126)
at java.awt.Component.processEvent(Component.java:5253)
at java.awt.Container.processEvent(Container.java:1966)
at java.awt.Component.dispatchEventImpl(Component.java:3955)
at java.awt.Container.dispatchEventImpl(Container.java:2024)
at java.awt.Component.dispatchEvent(Component.java:3803)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4212)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3892)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3822)
at java.awt.Container.dispatchEventImpl(Container.java:2010)
at java.awt.Window.dispatchEventImpl(Window.java:1774)
at java.awt.Component.dispatchEvent(Component.java:3803)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Hat jemand eine Idee was falsch daran ist? Was muss ich das verändern?

Gruß Michi

 

[bummerland]

In Zeile 40 und 43 liegen die Fehler. du musst this.xxx benutzen, da sonst lokale Objekte angelegt werden und nicht die Instanzen der Klasse.

 

[michaelb]

Hi,
danke für die Antwort.
Aber leider weiß ich nicht wie ich das jetzt umsetzen soll!
Was soll ich da:

Console console = new Console();

und da

Steuerung steuer = new Steuerung(console);

umändern?
Gruß Michi

 

[michaelb]

Noch zur Info:
das ist Console.java:

/*
 * Nimmt die Befehle von der Steuerung auf 
 * und zeigt sie an
 * Autor: Michael Bergmann
 * Version: 1.0
 * Stand: läuft
 */
import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

public class Console extends JInternalFrame implements ActionListener {
	private JTextArea console;
	private JScrollPane scroll;
	private JButton clear;
	public Console() {
		super();
		setSize(200,200);
		setLocation(320,10);
		setTitle("Console");
		setResizable(false);
		setIconifiable(true);
		setFrameIcon(new ImageIcon("logo.png"));
		JPanel content = new JPanel(new BorderLayout());
		add(content);
		console = new JTextArea(8,10);
		console.setEditable(false);
		scroll = new JScrollPane(console);
		scroll.setPreferredSize(new Dimension(200,140));
		content.add(scroll, BorderLayout.NORTH);
		clear = new JButton("Verlauf löschen");
		clear.addActionListener(this);
		content.add(clear, BorderLayout.SOUTH);
	}
	
	public void setCommand(String str) {
		console.append(str);
		console.append("\n");
	}
	
	public void actionPerformed(ActionEvent ae) {
		Object source = ae.getSource();
		
		if(source == clear) {
			console.setText("");
		}
	}
}

und das ist Steuerung.java:

/*
 * Gibt die Befehle an die Console weiter
 * Autor: Michael Bergmann
 * Version: 1.0
 * Stand: läuft
 */
import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class Steuerung extends JInternalFrame implements ActionListener {
	private Console zugriff;
	private ButtonPanel BP;
/*
 * Ganz wichtig: Hier muss dem Konstruktor ein Objekt des
 * Types Console übergeben werden damit man nachher richtig
 * daraufzugreifen kann
 */
	public Steuerung(Console übergabe) {
		super();
		setSize(300,200);
		setLocation(10,10);
		setTitle("Steuerung");
		setResizable(false);
		setIconifiable(true);
		setFrameIcon(new ImageIcon("logo.png")); 
		this.zugriff = übergabe;
		BP = new ButtonPanel();
		this.getContentPane().add(BP);
		BP.vor.addActionListener(this);
		BP.links.addActionListener(this);
		BP.rechts.addActionListener(this);
		BP.zurück.addActionListener(this);
		BP.stop.addActionListener(this);
		
		BP.vor.setToolTipText("Der Roboter fährt vorwärts");
		BP.links.setToolTipText("Der Roboter fährt nach links");
		BP.rechts.setToolTipText("Der Roboter fährt nach rechts");
		BP.zurück.setToolTipText("Der Roboter fährt zurück");
		BP.stop.setToolTipText("Der Roboter stoppt");
		
	}
/* Hier springt das Programm hin wenn ein Button gedrückt wird
 *  (non-Javadoc)
 * @see java.awt.event.ActionListener#actionPerformed(java.awt.event.ActionEvent)
 */	
	public void actionPerformed(ActionEvent ae) {
		Object source = ae.getSource();
		
		if(source == BP.vor) {
			this.zugriff.setCommand("BMV (Vor)");
		} 
		
		if(source == BP.links) {
			this.zugriff.setCommand("RMV (Links)");
		}
		
		if(source == BP.rechts) {
			this.zugriff.setCommand("LMV (Rechts)");
		}
		
		if(source == BP.zurück) {
			this.zugriff.setCommand("BMR (Zurück)");
		}
		
		if(source == BP.stop) {
			this.zugriff.setCommand("BMS (Stop)");
		}
	}
}

class ButtonPanel extends JPanel{
	JButton vor;
	JButton links;
	JButton rechts;
	JButton zurück;
	JButton stop;
	
	ButtonPanel() {
		JPanel content = new JPanel(new GridBagLayout());
	      this.add(content);
	      GridBagConstraints c = new GridBagConstraints();
	      c.gridx = 1;
	      c.gridy = 0;
	      c.fill = GridBagConstraints.BOTH;
	      c.weightx = 1.0;
	      c.weighty = 1.0;
	      c.ipadx = 5;
	      c.ipady = 5;
	      vor = new JButton("Vor");
	      content.add(vor, c);
	      c = new GridBagConstraints();
	      c.gridx = 1;
	      c.gridy = 2;
	      c.weightx = 1.0;
	      c.weighty = 1.0;
	      c.ipadx = 5;
	      c.ipady = 5;
	      c.fill = GridBagConstraints.BOTH;
	      zurück = new JButton("Zurück");
	      content.add(zurück, c);
	      c = new GridBagConstraints();
	      c.gridx = 0;
	      c.gridy = 1;
	      c.weightx = 1.0;
	      c.weighty = 1.0;
	      c.ipadx = 5;
	      c.ipady = 5;
	      c.fill = GridBagConstraints.BOTH;
	      links = new JButton("Links");
	      content.add(links, c);
	      c = new GridBagConstraints();
	      c.gridx = 2;
	      c.gridy = 1;
	      c.weightx = 1.0;
	      c.weighty = 1.0;
	      c.ipadx = 5;
	      c.ipady = 5;
	      c.fill = GridBagConstraints.BOTH;
	      rechts = new JButton("Rechts");
	      content.add(rechts, c);
	      c = new GridBagConstraints();
	      c.gridx = 1;
	      c.gridy = 1;
	      c.weightx = 1.0;
	      c.weighty = 1.0;
	      c.ipadx = 5;
	      c.ipady = 5;
	      c.fill = GridBagConstraints.BOTH;
	      stop = new JButton("Stop");
	      content.add(stop, c);
	}
}

 

[Gast]

Hallo,

wie schon bummerland gesagt hat liegt der Fehler in Zeile 40 und 43. Hier werden nämlich zwei lokale Variablen 'console' und 'steuer' initialisiert. Diese Variablen gelten nur im Konstruktor.
Abhilfe: nur
console = new Console();
steuer = new Steuerung(console);

und sich mal über globale und lokale Variablen informieren.