Listener funktioniert nicht

[MrBlues]

Hallo zusammen,

ich bin gerade auf ein kleines Problem gestoßen. Ich habe zwei Buttons. Auf dem einen möchte ich die Anwendung schließen, wenn ich auf den Exit-Button klicke. Allerdings tut sich leider nichts. Habe nun schon lange überlegt und komme nicht weiter. Wäre nett, wenn mir hier jemand helfen könnte. Danke!

Freundliche Grüße

Hier der Code:

package jetedit.main;

import java.awt.BorderLayout;
import java.awt.Font;


import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;

public class Gui {
	
	protected JFrame frame;
	protected JPanel controls;
	protected JPanel content;
	protected JTextArea text;
	protected JButton open;
	protected JButton exit;
	protected Font f;
	
	private JetListener jetlistener;
	

	
	
	

	public Gui(){
		super();
		SwingUtilities.invokeLater
		(
				new Runnable()
				{
					@Override
					public void run(){
						buildGui();
					}
				}
		);
				
		
	}
	
	protected void buildGui(){
		frame = new JFrame("Jetedit  V 0.0.0.1");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setSize(800, 600);
		frame.setLocationRelativeTo(null);
		
	    content = new JPanel(new BorderLayout());
		frame.setContentPane(new JScrollPane(content));
		
		controls = new JPanel();
		open = new JButton();
		//open.addMouseListener(jetlistener);
		open.setText("Open...");
		
	    controls.add(open);
	    exit = new JButton();
	    exit.setText("Exit");
	    controls.add(exit);
	    content.add(controls, BorderLayout.LINE_START);
	    
	    text = new JTextArea();
	    content.add(new JScrollPane (text), BorderLayout.CENTER);
	    
	    f = new Font("Verdana", Font.PLAIN, 14);
	    text.setFont(f);
	    
		frame.setVisible(true);
	}
	
		
	
	
	public JButton getButton1(){
		return exit;
	}
	
	
	
	
	public static void main(String[] args) {
		new Gui();

	}

}

und die Listener Klasse:

package jetedit.main;


import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;








public class JetListener implements MouseListener{
	
	
	
	public JetListener(){
	Gui gui= new Gui();
	gui.getButton1().addMouseListener(this);
	
	}

	@Override
	public void mouseClicked(MouseEvent arg0) {
		// TODO Automatisch generierter Methodenstub
		
	}

	@Override
	public void mouseEntered(MouseEvent arg0) {
		// TODO Automatisch generierter Methodenstub
		
	}

	@Override
	public void mouseExited(MouseEvent arg0) {
		// TODO Automatisch generierter Methodenstub
		
	}

	@Override
	public void mousePressed(MouseEvent arg0) {
		System.exit(0);
		
	}

	@Override
	public void mouseReleased(MouseEvent arg0) {
		// TODO Automatisch generierter Methodenstub
		
	}
	

	
		
	}

 

[NoWitzness]

Hallo dir fehlt der ActionListener ...
Ich habe ihn dir eingefügt.

Lyncht mich nicht wenn es nicht 100% richtig ist - funktionieren sollte es dennoch!

package startPackage;

import java.awt.BorderLayout;
import java.awt.Font;


import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;

public class Gui implements ActionListener{

	protected JFrame frame;
	protected JPanel controls;
	protected JPanel content;
	protected JTextArea text;
	protected JButton open;
	protected JButton exit;
	protected Font f;

	private JetListener jetlistener;






	public Gui(){
		super();
		SwingUtilities.invokeLater
		(
				new Runnable()
				{
					@Override
					public void run(){
						buildGui();
					}
				}
				);


	}

	protected void buildGui(){
		frame = new JFrame("Jetedit  V 0.0.0.1");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setSize(800, 600);
		frame.setLocationRelativeTo(null);

		content = new JPanel(new BorderLayout());
		frame.setContentPane(new JScrollPane(content));

		controls = new JPanel();
		open = new JButton();
		//open.addMouseListener(jetlistener);
		open.setText("Open...");

		controls.add(open);
		exit = new JButton();
		exit.setText("Exit");
		exit.addActionListener(this);
		controls.add(exit);
		content.add(controls, BorderLayout.LINE_START);

		text = new JTextArea();
		content.add(new JScrollPane (text), BorderLayout.CENTER);

		f = new Font("Verdana", Font.PLAIN, 14);
		text.setFont(f);

		frame.setVisible(true);
	}




	public JButton getButton1(){
		return exit;
	}




	public static void main(String[] args) {
		new Gui();

	}

	@Override
	public void actionPerformed(ActionEvent e) {

		if (exit == e.getSource()) 
		{
			System.exit(0);
		}

	}
}

Lg

 

[MrBlues]

Danke für die Antwort aber das bringt mich nicht weiter. Ich würde das gerne mit dem MouseListener machen, wie oben gezeigt.

 

[NoWitzness]

Du hast nur geschrieben, damit der Listener nicht funkt ^^
Bin grad nicht zuhause aber ich werd in den nächsten Stunden versuchen, dir zu helfen

 

[NoWitzness]

Du hast nur geschrieben, damit der Listener nicht funkt ^^
Bin grad nicht zuhause aber ich werd in den nächsten Stunden versuchen, dir zu helfen

 

[Harry Kane]

Nach dem ersten Drüberlesen hätte ich vorgeschlagen, daß du im Konstruktor deiner Gui-Klasse einfach die bereits vorhandene Variable jetlistener vom Typ JetListener initialisierst.

Wahrscheinlich hast du das schon versucht und dich gewundert, warum das Programm nicht läuft. Der Grund dafür ist einfach ein stack overflow: Im Konstruktor deiner JetListener Klasse wird eine neue Instanz von Guin angelegt, in dieser wiederum eine neue Instanz von JetListener, dort wieder von Gui usw.
Lösung: Lösche den Konstruktor von JetListener komplett, dann entferne in der Gui-Klasse das Kommentarzeichen vor //open.addMouseListener(jetlistener); und schliesslich schreibe in der JetListener-Klasse in einen Methodenrumpf deiner Wahl den Befehl System.exit();

 

[MrBlues]

Ok, danke. Wirklich gute Idee. Habe nun aber schon einen anderen Lösungsweg gefunden.

 

[Harry Kane]

Ok, danke. Wirklich gute Idee. Habe nun aber schon einen anderen Lösungsweg gefunden.

Aha. Angesichts der Tatsache, daß ohne die drei von mir beschriebenen Änderungen das Programm definitiv nicht laufen wird (es sei denn du strickst den Code komplett um), würde mich der "andere Lösungsweg" schon interessieren.