Anropa metoder ifrån ActionPerformed (JAVA)

Det borde fungera.

Har du kopplat en lyssnare till alla knappar?
Det verkar som du har lagt alla knappar i en vektor av knappar (knapp[]).

Du kopplar en lyssnare till knappen genom:

JButton knapp = new JButton("En knapp");

knapp.addActionListener(this); // skriv this, ifall denna klass implementerar gränssnittet ActionListener

Detta måste du ha, annars händer inget, när du trycker på knappen.

sedan ska du ha metoden public void actionPerformed(ActionEvent e) { }

Du har

if (e.getSource() == knapp[3])

Du ska inte använda == när du jämför två object. Använd equals() istället.

e.getSource() returnerar en objekt av typen Object.

Skriv såhär istället:

JButton src = (JButton)e.getSource();

if(src.equals(knapp[3]) )
{
// resten av koden
}

I detta fall fungerar det utmärkt att använda ==-operatorn för att kontrollera vilken knapp som är källan till eventet. Den evaluerar om de båda referenserna pekar till samma objekt vilket är precis vad vi vill i detta fall.

Alternativ förklaring i Java Language Specification:
http://java.sun.com/docs/books/jls/second_edition/html/expressions.doc.html#236163

/Kalle

Här har du en fungerande exempel:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class Test extends JFrame implements ActionListener 
{
	JButton btn;
	
	public Test()
	{
		btn = new JButton("hej");
		
		btn.addActionListener(this);
		getContentPane().add(btn);
		
		setBounds(100, 100, 300,300);
		setVisible(true);
	}
	
	public void tryck()
	{
		System.out.println("anropade metoden tryck()");
	}

	public void actionPerformed(ActionEvent e)
	{
		if(e.getSource().equals(btn) ) // det fungerar såhär också, så som du själv hade det
		{
			tryck();
		}
	}
	public static void main(String[] args) 
	{
		new Test();
	}
}

Så det du försöker säga är at tdu kan anropa de aktuella metoderna manuellt men inte via actionPerformed?

I så fall, har du studerat exemplet som jme givit dig? Du måste dels loopa genom din knapp-array och skapa en ny knapp för varje element, och på samma sätt loopa genom hela arrayen och sätta rätt listener.

/Kalle

EDIT: Stavknas

Det beror antagligen på att actionPerformed inte anropas från din huvud-thread utan från en tråd som hanterar events.

Edit: Mer kött på benen ==> http://www.jguru.com/faq/view.jsp?EID=8963

/Kalle

Utan att ha sett hela koden så är det ju lite svårt att säga exakt vad som händer. Men actionPerformed anropas ju från Swing's eventthread. Är du helt säker på att den verkligen anropas när du trycker ned en knapp? Får du någon InterruptedException som gör att tråden inte sover den tid du angivit?

Sen tror jag att det rent allmänt inte är någon bra ide att låta Swing's Thread anropa sleep(). Det gör väl att ditt GUI inte kan ta emot några events alls under den tiden...

Detta borde göra det du vill utan problem (har testat )

Lånade lite kod här ovanför, men det är bara att byta ut setText() till setImageIcon() eller vad det kan vara.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class ButtonSwitchSleep extends JFrame implements ActionListener
{
	JButton btn;
	Timer timer;
	
	public ButtonSwitchSleep()
	{
		btn   = new JButton("Gammal text");
		timer = new Timer(500, this);
		
		btn.addActionListener(this);
		getContentPane().add(btn);
		setBounds(100, 100, 300,300);
		setVisible(true);
	}
	
	public void switchTo()
	{
		btn.setText("Ny Text");
		timer.start();
	}
	
	public void switchBack()
	{
		btn.setText("Gammal text");
		timer.stop();
	}
	
	public void actionPerformed(ActionEvent e)
	{
		if(e.getSource().equals(btn) )
			switchTo();
			
		if(e.getSource().equals(timer) )
			switchBack();
	}
	
	public static void main(String[] args)
	{
		new ButtonSwitchSleep();
	}
}

Jag vet inte exakt vad du gör. Försöker du starta samma tråd 4 gånger, eller ändras mythread i loopen? Om mythread ändras och det är olika trådar kanske detta fungerar (har inte testat dock)

for (int i = 0 i < 3; i++)
{
	// Fult att polla, men det funkar.
	while (oldthread != null && !oldThread.isAlive())
		;
		
	mythread.start();
	oldthread = mythread;
	
	// byt ut mythread mot en ny tråd
}

Ok, vad ska denna kodsnutt göra?

for(int index=0; index<4; index++)
        mythread.start();

För mig ser det ut som om du startar en tråd 4 gånger på raken. Vad skulle det göra?