Keylistener reagiert nicht

[AsterixdGallier]

hallo mein keylistener reagiert nicht hier erstmal der code:

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.io.File;
import java.util.ArrayList;
import java.util.List

import javax.swing.AbstractAction;
import javax.swing.JMenuItem;
import javax.swing.JPopupMenu;
import javax.swing.KeyStroke;
import javax.swing.SwingUtilities;

public class PadController 
{
	List<PadView> list1 = new ArrayList<PadView>();
	PadModel padModel;
	File datei;
	SampleChooser chooser;
	SoundPlayer tmpplayer;
	boolean isMouseOnButton;
	boolean isMousePressed = false;
	PadView padView;
	PadController padController;
	boolean isAlreadyPressed = false;
	char taste;
	String tasteString;
	
	public PadController(PadModel padModel)
	{
		this.padModel = padModel;
		tmpplayer = new SoundPlayer();
	}
		
	public void addToList(PadView padView)
	{
		this.padView = padView;
		addMouseListenerToPad(padView);
		list1.add(padView);
	}
	
	private void addMouseListenerToPad(PadView padView)
	{	
		padView.addMouseListener(new MouseListener()
		{
			
			@Override
			public void mouseClicked(MouseEvent arg0) 
			{
				
			}

			@Override
			public void mouseEntered(MouseEvent arg0) 
			{
				for (int i = 0; i < list1.size(); i++)
				{
					if(isMousePressed == false)
					{
						list1.get(i).changeButtonImage(PadView.ButtonStyle.OVERLAY);
					}
					else
					{
						list1.get(i).changeButtonImage(PadView.ButtonStyle.CLICKED);
						try
						{
							tmpplayer.play();
						}
						catch(Exception e)
						{
							
						}
						
					}
				}
				isMouseOnButton = true;
			}

			@Override
			public void mouseExited(MouseEvent arg0) 
			{
				for (int i = 0; i < list1.size(); i++)
				{
					list1.get(i).changeButtonImage(PadView.ButtonStyle.NORMAL);
				}
				isMouseOnButton = false;
			}

			@Override
			public void mousePressed(MouseEvent arg0) {
				try
				{
					if (SwingUtilities.isLeftMouseButton (arg0))
					{
						for (int i = 0; i < list1.size(); i++)
						{
							((PadView) list1.get(i)).changeButtonImage(PadView.ButtonStyle.CLICKED);
						}
						isMousePressed = true;
						tmpplayer.play();
						
					}	
					else if (SwingUtilities.isRightMouseButton (arg0))
					{
						showMenu(arg0);
					}
					else if(SwingUtilities.isMiddleMouseButton(arg0))
					{
						for (int i = 0; i < list1.size(); i++)
						{
							((PadView) list1.get(i)).changeButtonImage(PadView.ButtonStyle.CHOOSELETTER);
						}
					}
				}
				catch (Exception e)
				{
					
				}
			}

			@Override
			public void mouseReleased(MouseEvent arg0) 
			{
				for (int i = 0; i < list1.size(); i++)
				{
					if (isMouseOnButton == false)
					{
						list1.get(i).changeButtonImage(PadView.ButtonStyle.NORMAL);
					}
					else
					{
						list1.get(i).changeButtonImage(PadView.ButtonStyle.OVERLAY);
					}
					isMousePressed = false;
				}
			}
		});		
		padView.requestFocus(true);
	}
	
	public void showMenu(MouseEvent evt)
	{
		tasteString = null;
		isAlreadyPressed = false;
		JPopupMenu menu = new JPopupMenu();
		JMenuItem sample = new JMenuItem("Sample hinzufügen");
		menu.add((sample));
		sample.addActionListener(new ActionListener()
		{

			@Override
			public void actionPerformed(ActionEvent arg0) 
			{
				chooser = new SampleChooser(new ActionListener () 
				{
					public void actionPerformed (ActionEvent e) 
					{
						datei = chooser.getSelectedFile ();
						padModel.setDateiName(datei.getName());
						tmpplayer.initialisieren(datei);
						chooser.dispose();
					}
				}, new ActionListener ()
				{

					@Override
					public void actionPerformed(ActionEvent e) 
					{
							chooser.dispose();		
					}
					
				});
			}
		});
		JMenuItem bindkey = new JMenuItem("Taste zuweisen");
		menu.add(bindkey);
		System.out.println(padView);
				

		bindkey.addActionListener(new ActionListener()
		{

			@Override
			public void actionPerformed(ActionEvent e) 
			{
				System.out.println("Los geht´s");
				padView.addKeyListener(new KeyListener()
				{

					@Override
					public void keyPressed(KeyEvent e) 
					{
						char key = e.getKeyChar();
						System.out.println(key);
					}

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

					@Override
					public void keyTyped(KeyEvent e) {
						// TODO Auto-generated method stub
						
					}
					
				});
			}
			
		});
		
		menu.show(padView, evt.getX(), evt.getY());
	}
}

Dieser keylistener soll erst gestartet werden, nachdem die actionperformed methode aufgerufen wurde.
das ganze wird mehreren buttons zugewiesen. das lustige ist bei dem ersten button funktioniert es, klicke ich jedoch andere buttons an nicht. ich erzeuge die buttons in einer for-schleife in einer anderen klasse es muss also an dieser methode hier liegen. vllt kann mir diesmal jemand helfen

 

[Gossi]

Entweder ich habs übersehen oder [c]addToList()[/c] wird nie aufgerufen

 

[AsterixdGallier]

sorry habe die klasse nicht gepostet, addtolist() wird hier aufgerufen:

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Toolkit;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Main 
{
//	private PadView padView;
//	private PadViewBig padViewBig;
//	private PadModel padModel;
	public static void main(String[] args) 
	{
		Main main = new Main();
		main.buildAFewPads();
	}
	
//	public void build()
//	{
//		final Dimension screenSize = Toolkit.getDefaultToolkit ().getScreenSize ();
//		JFrame frame = new JFrame();
//		JPanel panel = new JPanel();
//		frame.getContentPane ().add (panel, BorderLayout.CENTER);
//		panel.setVisible(true);
//		panel.setBackground(Color.gray);
//		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
//		padModel = new PadModel();
//		PadModel modelacc = padModel;
//		padView = new PadView(modelacc);
//		padViewBig = new PadViewBig(modelacc);
//		modelacc.addObserver (padView);
//		modelacc.addObserver (padViewBig);
//		PadController controller = new PadController(padModel);
//		controller.addToList(padView);
//		controller.addToList(padViewBig);
//		panel.add(padView);	
//		frame.pack();
//		frame.setLocation (((int)screenSize.getWidth () / 2 - frame.getWidth () / 2), ((int) screenSize.getHeight () / 2) - frame.getHeight () / 2);
//		frame.setVisible(true);
//	}
//	
	public void buildAFewPads()
	{
		final int anzahlPads = 4;
		final int anzahlProReihe = 4;
		final Dimension screenSize = Toolkit.getDefaultToolkit ().getScreenSize ();
		JFrame frame = new JFrame();
		JPanel panel = new JPanel();
		frame.getContentPane ().add (panel, BorderLayout.CENTER);
		panel.setVisible(true);
		panel.setFocusable(false);
		frame.setFocusable(false);
		panel.setBackground(Color.gray);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		panel.setLayout(new GridBagLayout());
		GridBagConstraints c = new GridBagConstraints();
		for (int i = 0; i < anzahlPads; i++)
		{
			c.gridx = i % anzahlProReihe;
			c.gridy = i / anzahlProReihe;
			PadModel padModel = new PadModel();
			PadModel modelacc = padModel;
			PadView padView = new PadView(modelacc);
			padView.setFocusable(true);
			modelacc.addObserver (padView);
			PadController controller = new PadController(padModel);
			controller.addToList(padView);
			panel.add(padView, c);
		}	
		frame.pack();
		frame.setLocation (((int)screenSize.getWidth () / 2 - frame.getWidth () / 2), ((int) screenSize.getHeight () / 2) - frame.getHeight () / 2);
		frame.setVisible(true);
	}
}

 

[Swoop]

Hey,

weist du ob die ActionPerformed-Methode aufgerufen wird ? Dein Key-Listener sollte eigentlich eine Ausgabe machen.
Mach mal eine ausgabe vor den Key-Listener ...
Ich glaub aber dein Ablauf stimmt nicht ganz...

Wenn ich es richtig verstanden habe, müsstest du es so wollen:
Action-Performed wird ausgeführt -> Key-Listener wird aktiviert-> ab jetzt wird auf die Tasten reagiert?
Willst du das erreichen oder hab ich dich falsch verstanden?

Gruß Swoop

 

[AsterixdGallier]

richtig ich probiere es kurz aus
achso doch klar die actionperformed methode wird ausgeführt. das "los gehts" wird geschrieben in der console. meinst du nicht genau sowas?

 

[Swoop]

ja genau das mein ich,
dann weis ich nicht genau wie der Ablauf ist aber eigentlich sollte dein Listener hinzugefügt werden. Leider weis ich nicht wie man das bei Swing testen kann...
Versuch doch erstmal dein KeyListener direkt auf den padView zu legen. Wenn dann die ganze Zeit das gedrückte Zeichen ausgegeben wird stimmt etwas bei deiner actionperformed Methode nicht. (Also beim Ablauf) Wenn nein Funktioniert der KeyListener allgemein nicht...

 

[AsterixdGallier]

wenn ich es direkt in die padview klasse lege wird alles ausgegeben, der keylistener soltle richtig sein.
bei meinem ersten pad(button) funktiert das ganze ja auch, nur halt nur bei dem ersten. die actionperformed methode wird ja auch aufgerufen, irgendwie funktioniert nur die combination der beiden nicht.
also nochmal:
wenn ich "Taste hinzufügen" auf dem ersten button auswähle funktioniert es wie ich es mir vorstelle.
wähle ich "Taste hinzufügen" auf einem der anderen button (pads) aus passiert nichts.
obwohl ich die anderen genau so wie den ersten anlege, in der forschleife der Main klasse.

 

[Swoop]

Wann wird denn ActionPerformed aufgerufen... Sorry kenn den Listener nicht da ich nicht aus Swing sondern der SWT ecke komme !

 

[AsterixdGallier]

also actionperformed wird aufgerufen sobald ich im kontextmenu auf den eintrag "taste zuweisen" drücke. ja kein problem ist ja super dass du überhaupt dich damit beschäftigst. =)

 

[Swoop]

hmm ... ah ok ... eig sollte es funktionieren ...
Dann weis ich leider auch nicht mehr weiter -.-

 

[AsterixdGallier]

alles klar trotzdem danke vllt hat ja noch jemand eine idee