Fehlersuche actionPerformed

[DankbarerStudent ;)]

Hallo @ all.
Sollte mich wohl langsam SchlafloserStudent nennen..
Ich such seit gestern nach einem oder mehreren Fehlern in meinem Programm, find aber keinen. Das gute Proggie soll eigentlich nur nach dem druck einer der Tasten eine andere Klasse mit der "Funktion" aufrufen, funktioniert aber leider nur in stark abgewandelter Form. Es scheint so als würde es beim Tastendruck jede Funktion nacheinander abarbeiten (11 Stück). Meine Frage ist nun warum macht es das? Wie kann ich das ändern?
Mein Code der Aufrufklasse:

import java.awt.Dimension;
import javax.swing.JButton;
import java.awt.event.*;
import javax.swing.JLabel;
import javax.swing.ImageIcon;
import java.awt.Point;
import javax.swing.JFrame;


public class player extends JFrame implements ActionListener {

	private static final long serialVersionUID = 1L;

	private JLabel bg = null;
	
	private JButton play = null;
	private play sec = null;
	
	private JButton stop = null;
	private stop stp = null;  //  @jve:decl-index=0:

	private JButton ff = null;
	private ff ffo = null;

	private JButton rr = null;
	private rr rro = null;

	private JButton next = null;
	private next nex = null;

	private JButton back = null;
	private back bac = null;

	private JButton pl1 = null;
	private pl1 pla = null;
	
	private JButton pl2 = null;
	private pl2 plb = null;

	private JButton pl3 = null;
	private pl3 plc = null;

	private JButton pl4 = null;
	private pl4 pld = null;

	private JButton visual = null;
	private visual vis = null;
    
    
    
	/**
	 * This is the default constructor
	 */
	public player() {
		super();
		initialize();
	}

	/**
	 * This method initializes this
	 * 
	 * @return void
	 */

	private void initialize() {
		bg = new JLabel();
		bg.setIcon(new ImageIcon(getClass().getResource("/pics/Layout_1.jpg")));
		bg.setEnabled(true);
		bg.setLocation(new Point(4, 28));
		bg.setSize(new Dimension(635, 480));
		bg.setText("");
		bg.setVisible(false);
		this.setLayout(null);
		this.setSize(640, 480);
		this.setResizable(false);
		this.setTitle("°°ooOO^AUDI^OOoo°°");

		this.setVisible(true);
		this.add(bg, null);
		this.add(getplay(), null);
		this.add(getstop(), null);
		this.add(getff(), null);
		this.add(getrr(), null);
		this.add(getnext(), null);
		this.add(getback(), null);
		this.add(getpl1(), null);
		this.add(getpl2(), null);
		this.add(getpl3(), null);
		this.add(getpl4(), null);
		this.add(getvisual(), null);
	}

	/**
	 * This method initializes play	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getplay() {
		if (play == null) {
			play = new JButton();
	        play.setIcon(new ImageIcon(getClass().getResource("/pics/play.jpg")));
	        play.setSize(new Dimension(58, 50));
	        play.setRolloverEnabled(true);
	        play.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/play.JPG")));
	        play.setLocation(new Point(322, 408));
	        play.addActionListener(this);
	        add(play);

	        
	        
		}
		return play;
	}
    
  	    
	private JButton getstop() {
		if (stop == null) {
			stop = new JButton();
			stop.setRolloverEnabled(true);
			stop.setLocation(new Point(254, 408));
			stop.setSize(new Dimension(58, 50));
			stop.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/stop.JPG")));
	        stop.setIcon(new ImageIcon(getClass().getResource("/pics/stop.jpg")));
	        stop.addActionListener(this);
	        add(stop);
		}
		return stop;
	}

	/**
	 * This method initializes ff	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getff() {
		if (ff == null) {
			ff = new JButton();
			ff.setLocation(new Point(390, 408));
			ff.setRolloverEnabled(true);
			ff.setIcon(new ImageIcon(getClass().getResource("/pics/forward.JPG")));
			ff.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/forward.JPG")));
			ff.setSize(new Dimension(58, 50));
	        ff.addActionListener(this);
	        add(ff);
		}
		return ff;
	}

	/**
	 * This method initializes rr	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getrr() {
		if (rr == null) {
			rr = new JButton();
			rr.setPreferredSize(new Dimension(58, 50));
			rr.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/backward.JPG")));
			rr.setLocation(new Point(186, 408));
			rr.setSize(new Dimension(58, 50));
			rr.setIcon(new ImageIcon(getClass().getResource("/pics/backward.JPG")));
	        rr.addActionListener(this);
	        add(rr);
		}
		return rr;
	}

	/**
	 * This method initializes next	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getnext() {
		if (next == null) {
			next = new JButton();
			next.setLocation(new Point(458, 408));
			next.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/next.JPG")));
			next.setIcon(new ImageIcon(getClass().getResource("/pics/next.JPG")));
			next.setSize(new Dimension(58, 50));
	        next.addActionListener(this);
	        add(next);
		}
		return next;
	}

	/**
	 * This method initializes back	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getback() {
		if (back == null) {
			back = new JButton();
			back.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/back.JPG")));
			back.setLocation(new Point(119, 408));
			back.setSize(new Dimension(58, 50));
			back.setIcon(new ImageIcon(getClass().getResource("/pics/back.JPG")));
	        back.addActionListener(this);
	        add(back);
		}
		return back;
	}

	/**
	 * This method initializes pl1	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getpl1() {
		if (pl1 == null) {
			pl1 = new JButton();
			pl1.setLocation(new Point(33, 321));
			pl1.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/PL1.JPG")));
			pl1.setIcon(new ImageIcon(getClass().getResource("/pics/PL1.JPG")));
			pl1.setSize(new Dimension(58, 50));
	        pl1.addActionListener(this);
	        add(pl1);
		}
		return pl1;
	}

	/**
	 * This method initializes pl2	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getpl2() {
		if (pl2 == null) {
			pl2 = new JButton();
			pl2.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/PL2.JPG")));
			pl2.setLocation(new Point(33, 408));
			pl2.setSize(new Dimension(58, 50));
			pl2.setIcon(new ImageIcon(getClass().getResource("/pics/PL2.JPG")));
	        pl2.addActionListener(this);
	        add(pl2);
		}
		return pl2;
	}

	/**
	 * This method initializes pl3	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getpl3() {
		if (pl3 == null) {
			pl3 = new JButton();
			pl3.setLocation(new Point(549, 321));
			pl3.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/PL3.JPG")));
			pl3.setIcon(new ImageIcon(getClass().getResource("/pics/PL3.JPG")));
			pl3.setSize(new Dimension(58, 50));
	        pl3.addActionListener(this);
	        add(pl3);
		}
		return pl3;
	}

	/**
	 * This method initializes pl4	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getpl4() {
		if (pl4 == null) {
			pl4 = new JButton();
			pl4.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/PL4.JPG")));
			pl4.setLocation(new Point(549, 408));
			pl4.setSize(new Dimension(58, 50));
			pl4.setIcon(new ImageIcon(getClass().getResource("/pics/PL4.JPG")));
	        pl4.addActionListener(this);
	        add(pl4);
		}
		return pl4;
	}

	/**
	 * This method initializes visual	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getvisual() {
		if (visual == null) {
			visual = new JButton();
			visual.setLocation(new Point(212, 321));
			visual.setRolloverIcon(new ImageIcon(getClass().getResource("/pics/visual.JPG")));
			visual.setIcon(new ImageIcon(getClass().getResource("/pics/visual.JPG")));
			visual.setSize(new Dimension(206, 50));
	        visual.addActionListener(this);
	        add(visual);
		}
		return visual;
	}
	public void actionPerformed( ActionEvent evt) {

		{
    	sec = new play();
        Object source = evt.getSource();
        if (source == play) {
            sec.getClass(); }
        }
    	{
 		   stp = new stop();
 		   Object source0 = evt.getSource();
 		   if (source0 == stop) {
 		       stp.getClass(); }
 	    }   
 	    {
 	 	   ffo = new ff();
 	 	   Object source1 = evt.getSource();
 	 	   if (source1 == ff) {
 	 	       ffo.getClass(); }
 	    }   
 	   	{
  		   rro = new rr();
  		   Object source2 = evt.getSource();
  		   if (source2 == rr) {
  		       rro.getClass(); }
  	    }   
  	   	{
  	   		nex = new next();
  	        Object source3 = evt.getSource();
  	        if (source3 == next) {
  	            nex.getClass(); }
        }
      	{
   		   bac = new back();
   		   Object source4 = evt.getSource();
   		   if (source4 == back) {
   		       bac.getClass(); }
   	    }   
  	    {
  	 	   pla = new pl1();
  	 	   Object source5 = evt.getSource();
  	 	   if (source5 == pl1) {
  	 	       pla.getClass(); }
  	    }   
  	   	{
  		   plb = new pl2();
  		   Object source6 = evt.getSource();
  		   if (source6 == pl2) {
  		       plb.getClass(); }
  	    }	
   	    {
  	 	   plc = new pl3();
  	 	   Object source7 = evt.getSource();
  	 	   if (source7 == pl3) {
  	 	       plc.getClass(); }
  	    }   
  	   	{
  		   pld = new pl4();
  		   Object source8 = evt.getSource();
  		   if (source8 == pl4) {
  		       pld.getClass(); }
  	    }
  	    {
  	 	   vis = new visual();
  	 	   Object source9 = evt.getSource();
  	 	   if (source9 == visual) {
  	 	       vis.getClass(); }
  	    }   
	}	   	    	
}

Der Code einer Funktionsklasse( sind alle nach dem selben Schema gebaut nur mit unterschiedlichen Buchstaben):

import java.awt.*;
import java.awt.event.*;

public class next {
   private Robot robot;
   
   public next() {
      try {
          robot = new Robot();
      }
      catch(AWTException e) {
      }

      robot.keyPress(KeyEvent.VK_N);
      robot.keyRelease(KeyEvent.VK_N);

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

Hat einer ne Idee oder ´n Tip?

 

[clemson]

dein code in actionPerformed führt alle codefragmente aus. entweder if-else abfragen einbauen oder nach jedem gewünschten aufruf ein return; statement, dann wird der restliche code der methode nicht ausgeführt....

 

[DankbarerStudent ;)]

hi clemson,
also die anderen Klassen sehen eigentlich ganz genau so aus, nur dass halt anstatt "Tastendruck N", "Tastendruck what ever". Die Klassen pl1, pl2, pl3, pl4 sehen bischen anders aus, hier der Code:

import java.io.*;

public class pl1 {

   
   public pl1() {
	      try {
	          Runtime.getRuntime().exec("cmd /c start pl1.m3u");
	        }
	        catch(IOException e){ 
	        }
	        }
   public static void main(String[] args) {
      new pl1();
   }
}

Kannst du mir zu den if-else Abfragen oder der return sache mal´n Beispiel geben? bin ´n ziemliche Newbie
thx erstmal...

 

[TheJavaKid]

Einwurf -> Du solltest dich vielleicht hier registrieren

 

[RauwlDuke]

ganz deiner Meinug Hackl
was ich dann hiermit getan hab

so hab mal mit dem tip von clemson wat probiert, funzt aber auch noch nich so, da arbeitet das programm die schritte bis zum return ab
mir is da grad was aus C++ wieder eingefallen, CASE.
dass das in java geht weis ich aber das switch will bei mir nicht funktionieren.
wie krieg ich das jetzt am besten hin?

codeauszug:

	public void actionPerformed( ActionEvent evt) {

   
 	    {
 	 	   ffo = new ff();
 	 	   Object source1 = evt.getSource();
 	 	   if (source1 == ff) {
 	 	       ffo.getClass(); 
 	 	        return;}
 	    }   
 	   	{
  		   rro = new rr();
  		   Object source2 = evt.getSource();
  		   if (source2 == rr) {
  		       rro.getClass(); 
  		        return;}
  	    }   
      	{
   		   bac = new back();
   		   Object source4 = evt.getSource();
   		   if (source4 == back) {
   		       bac.getClass(); 
   	        return;}
   	    }   
  	   	{
  	   		nex = new next();
  	        Object source3 = evt.getSource();
  	        if (source3 == next) {
  	            nex.getClass(); 
  	          return;}
        }
  	    {
  	 	   pla = new pl1();
  	 	   Object source5 = evt.getSource();
  	 	   if (source5 == pl1) {
  	 	       pla.getClass(); 
  	         return;}
  	    }   
  	   	{
  		   plb = new pl2();
  		   Object source6 = evt.getSource();
  		   if (source6 == pl2) {
  		       plb.getClass(); 
  		        return;}
  	    }	
   	    {
  	 	   plc = new pl3();
  	 	   Object source7 = evt.getSource();
  	 	   if (source7 == pl3) {
  	 	       plc.getClass(); 
  	         return;}
  	    }   
  	   	{
  		   pld = new pl4();
  		   Object source8 = evt.getSource();
  		   if (source8 == pl4) {
  		       pld.getClass(); 
  		        return;}
  	    }
  	    {
  	 	   vis = new visual();
  	 	   Object source9 = evt.getSource();
  	 	   if (source9 == visual) {
  	 	       vis.getClass(); 
  	         return;}
  	 	{
  	 	   sec = new play();
  	       Object source = evt.getSource();
  	       if (source == play) {
  	           sec.getClass(); 
            return;}
        }
  		{
  		   stp = new stop();
  		   Object source0 = evt.getSource();
  		   if (source0 == stop) {
  		       stp.getClass(); 
  		        return;}
   	    }
  	    }   
	}
}

 

[clemson]

also das mit den verzweigungen hab ich so gemeint:

public void actionPerformed(ActionEvent evt)
	{
		Object source = evt.getSource();
		if (source == play)
		{
			play.getClass();
		}
		else if (source == stop)
		{
			stop.getClass();
		}
		else if (source == rr)
		{
			// rr wurde gedrückt
		}
	}

 

[RauwlDuke]

Hey danke clemson :toll:

habs noch leicht modifiziert und funzt hervorragend

Der Code (wen´s interessiert):

	public void actionPerformed( ActionEvent evt) {
		Object source = evt.getSource();
		 	   
 	 	   if (source == ff) {
 	 	 	   ffo = new ff();
 	 	       ffo.getClass(); 
}
  else if (source == rr) {
  			 rro = new rr();
  		     rro.getClass(); 
}
  else if (source == next) {
		 nex = new next();
	     nex.getClass(); 
}
  else if (source == back) {
  		  bac = new back();
	      bac.getClass(); 
}
  else if (source == play) {
		 sec = new play();
	     sec.getClass(); 
}
  else if (source == stop) {
  	   stp = new stop();
       stp.getClass(); 
}
  else if (source == pl1) {
	     pla = new pl1();
          pla.getClass(); 
}
  else if (source == pl2) {
	  	   plb = new pl2();
	       plb.getClass(); 
}
  else if (source == pl3) {
           plc = new pl3();
            plc.getClass(); 
}
  else if (source == pl4) {
	    pld = new pl4();
       pld.getClass(); 
}
  else if (source == visual) {
	    vis = new visual();
        vis.getClass(); 
}

}}

also nochmal tausend Dank an dich und alle anderen die mir geholfen haben [schild=6 fontcolor=000000 shadowcolor=C0C0C0 shieldshadow=1]java-forum.org RoCkS[/schild]
ohne euch hät ich´s wohl nich geschafft :wink:

bis die tage...