actionPerformed reagiert nicht auf das Canvas-objekt

[bristtote]

Hallo liebe Leute,

folgendes Problem:
Ich habe eine Methode, namens "drueck", die wie Ihr seht in actionperformed aufgerufen wird. Wird der Button "b_b" gedrückt, sollen zum einen der Hauptpanel unsichtbar werden und dafür das Canvas-Objekt sichtbar werden. Leider wird aber nur die erste Zeile ausgeführt also:

 Hauptpanel.setVisible(false);

Die zweite Zeile

 canvas.setVisible(true);

wird einfach nicht ausgeführt. Füge ich aber die zweite Zeile an einer anderen Stelle ein, so funktioniert sie. Sie funktioniert also nur nicht in der Methode ActionPerformed().

Weiss jemand, woran es liegt und was man da machen kann?
Hier nochmal der Code:

    b_b.addActionListener(new ActionListener()               
    {                                                         
    	 public void actionPerformed(ActionEvent e)               
         {                                                        
                drueck(e);               
         }                                                        
    });

     public void drueck(ActionEvent action)
     {
	  String command=action.getActionCommand();
	  if(command.equals("Los"))
	  {
	       Hauptpanel.setVisible(false);
              canvas.setVisible(true);	
	   }
     }

 

[Wildcard]

canvas soll vermutlich dein Hauptpanel ersetzen?
Wie hast du die Beiden in die Parent-Komponente eingefügt?
Wahrscheinlich ist setVisible nicht das richtige für dich. Versuch stattdessen mal

parent.remove(Hauptpanel);
parent.add(canvas);
parent.revalidate();

 

[bristtote]

canvas soll vermutlich dein Hauptpanel ersetzen?
Wie hast du die Beiden in die Parent-Komponente eingefügt?
Wahrscheinlich ist setVisible nicht das richtige für dich. Versuch stattdessen mal

parent.remove(Hauptpanel);
parent.add(canvas);
parent.revalidate();

Also, die Sache ist die: Für den Hauptpanel funktionieren beider Methoden, also sowohl setVisible() alsauch Remove(). Es ist das Canvas-Objekt, das in ActionPerformed Probleme macht, aber wirklich nur in ActionPerformed. Ich habe es eben, wie Du es gesagt hattes, geändert, aber auch funktioniert nicht. Tja ich weiss nicht, was ich machen soll.
Ich hatte sie vorher mit add in die parentkomponente eingefügt gehabt.

Was ich aber nicht gemacht habe, ist einen ActionListener füt das Canvas-Objekt erzeugen. Kann es daran liegen?

 

[Wildcard]

Ich hatte sie vorher mit add in die parentkomponente eingefügt gehabt.

Wenn das eine das andere ersetzen soll liegen diese Komponenten wahrscheinlich an gleicher Position (also zB BorderLayout.CENTER). Und das funktioniert eben nicht weil da nur eine Komponente liegen darf. Mit den derzeitigen Informationen kann ich aber eben nur nur Vermutungen abgeben.
Am ActionListener liegt es übrigens nicht. Der Fehler muss an anderer Stelle liegen.

 

[bristtote]

Ich hatte sie vorher mit add in die parentkomponente eingefügt gehabt.

Wenn das eine das andere ersetzen soll liegen diese Komponenten wahrscheinlich an gleicher Position (also zB BorderLayout.CENTER). Und das funktioniert eben nicht weil da nur eine Komponente liegen darf. Mit den derzeitigen Informationen kann ich aber eben nur nur Vermutungen abgeben.
Am ActionListener liegt es übrigens nicht. Der Fehler muss an anderer Stelle liegen.

Ne, ne sie liegen nicht an gleicher Position, denn wenn ich den gleichen Befehl direkt in der init einsetze, dann klappt auch alles, nur in der actionperofrmed gibt es probleme. Ich kann dir gerne den ganzen Code mitschicken.

public class Itemalsfunk extends Applet //implements ActionListener 
{    
    Panel Hauptpanel  = new Panel();
    Color bg_co       = new Color(158,158,158);   
    Button b_b        = new Button("Los");
    Panel panel    [] = new Panel[12];
    String Items   [] = {"NOP", "ADD", "LOAD"}; 
    Choice choices [] = new Choice [11];
    Choice c1      [] = new Choice [11];
    Choice c2      [] = new Choice [11];
    Choice cerg    [] = new Choice [11];
    Label labels   [] = new Label[11];
    Label g_z      [] = new Label[11];
    Label plus     [] = new Label[11];   
    TextField  t      = new TextField(8); 
    MyCanvas  canvas;
    
	public void init ()
	{
		for (int i=0; i<panel.length; i++)
		{
    		panel[i]=new Panel();
    		Hauptpanel.add(panel[i]); 
    	}
		String s="0";    
		for (int i=0; i<labels.length; i++) 
		{   
    		labels[i] =new Label(s);
    		int s1=Integer.parseInt(s);
    		s1++;
    		s=Integer.toString(s1); 
    		panel[i].add(labels[i]);
    	}
    
		String g="=";   
		for (int i=0; i<g_z.length; i++)
		{  
    		g_z[i] =new Label(g);    
    	}    
    
		String p="+";   
		for (int i=0; i<plus.length; i++)
		{ 
    		plus[i] =new Label(p);  
   		}       
        
		for ( int i = 0; i < choices.length; i++ )
		{
    		choices[i] = new Choice();  
    		panel[i].add(choices[i]);
    	}

		for (int i=0; i<choices.length; i++) 
		{  
   		 	choices[i].addItem(Items [0]);
   		 	choices[i].addItem(Items [1]);  
    	 	choices[i].addItem(Items [2]);
    	}
    
		for ( int i=0; i<choices.length; i++) 
		{
		 	final int index = i;
		 	choices[i].addItemListener(new ItemListener()
			{
    			public void itemStateChanged(ItemEvent e) 

        			if ( ItemEvent.SELECTED == 1)
        			{
      					aendere( index, e.getItem() );
    				}
    			}
  			});
		}

 	 b_b.addActionListener(new ActionListener()               
     {                                                         
    	public void actionPerformed(ActionEvent e)               
        {                                                        
            drueck(e);               
        }                                                        
   	 });   	 

		Hauptpanel.setLayout(new GridLayout(0,1));    
 		setBackground(bg_co);
 		add(Hauptpanel);
 		panel[11].add(b_b);

 		panel[11].add(t); 
 		canvas = new MyCanvas();
 		add(canvas);
 		canvas.setVisible(false);
 		     
	public void aendere( int index, Object item )
	{
  // wird immer aufgerufen, wenn "choice" ein neues "item" selektiert hat. Das item kann auch null sein!
    	// Hier nichts Relevantes
	}	
	public void drueck(ActionEvent action)
	{
		String command=action.getActionCommand();
		if(command.equals("Los"))
		{
			Hauptpanel.setVisible(false);
            canvas.setVisible(true);	
		}
	}
	
}
  
   
class MyCanvas extends Canvas 
{

	public Dimension getPreferredSize() 
 	{
    	return new Dimension(250, 120);
   	}
	
		
	public void paint(Graphics g)
	
	{	
	    super.paint(g);
		g.setColor(Color.lightGray);
		g.fillRect(150,100,100,20); 
		g.setColor(Color.black);	 
					

	}
	
	
			
}

 

[Wildcard]

Ich habe es eben, wie Du es gesagt hattes, geändert, aber auch funktioniert nicht.

Das glaube ich dir nicht weil es dann funktioniert hätte. :roll:
BTW: nicht böse gemeint, aber dein Code ist furchtbar. Das solltest du überarbeiten. Stichwort OOP...

public class Itemalsfunk extends Applet // implements ActionListener
{   
    Panel Hauptpanel  = new Panel();
    Color bg_co       = new Color(158,158,158);   
    Button b_b        = new Button("Los");
    Panel panel    [] = new Panel[12];
    String Items   [] = {"NOP", "ADD", "LOAD"};
    Choice choices [] = new Choice [11];
    Choice c1      [] = new Choice [11];
    Choice c2      [] = new Choice [11];
    Choice cerg    [] = new Choice [11];
    Label labels   [] = new Label[11];
    Label g_z      [] = new Label[11];
    Label plus     [] = new Label[11];   
    TextField  t      = new TextField(8);
    MyCanvas  canvas;
   
   public void init ()
   {
      for (int i=0; i<panel.length; i++)
      {
          panel[i]=new Panel();
          Hauptpanel.add(panel[i]);
       }
      String s="0";   
      for (int i=0; i<labels.length; i++)
      {   
          labels[i] =new Label(s);
          int s1=Integer.parseInt(s);
          s1++;
          s=Integer.toString(s1);
          panel[i].add(labels[i]);
       }
   
      String g="=";   
      for (int i=0; i<g_z.length; i++)
      { 
          g_z[i] =new Label(g);   
       }   
   
      String p="+";   
      for (int i=0; i<plus.length; i++)
      {
          plus[i] =new Label(p); 
         }       
       
      for ( int i = 0; i < choices.length; i++ )
      {
          choices[i] = new Choice(); 
          panel[i].add(choices[i]);
       }

      for (int i=0; i<choices.length; i++)
      { 
             choices[i].addItem(Items [0]);
             choices[i].addItem(Items [1]); 
           choices[i].addItem(Items [2]);
       }
   
      for ( int i=0; i<choices.length; i++)
      {
          final int index = i;
          choices[i].addItemListener(new ItemListener()
         {
             public void itemStateChanged(ItemEvent e)
             {
                 if ( ItemEvent.SELECTED == 1)
                 {
                     aendere( index, e.getItem() );
                }
             }
           });
      }

    b_b.addActionListener(new ActionListener()               
     {                                                         
       public void actionPerformed(ActionEvent e)               
        {                                                       
            drueck(e);               
        }                                                       
       });      

      Hauptpanel.setLayout(new GridLayout(0,1));   
      setBackground(bg_co);
      add(Hauptpanel);
      panel[11].add(b_b);

      panel[11].add(t);
      canvas = new MyCanvas();
//		DAS HIER MUSS WEG!!
//	    add(canvas);
//      canvas.setVisible(false);
   }
           
   public void aendere( int index, Object item )
   {
  // wird immer aufgerufen, wenn "choice" ein neues "item" selektiert hat. Das
	// item kann auch null sein!
       // Hier nichts Relevantes
   }   
   public void drueck(ActionEvent action)
   {
      String command=action.getActionCommand();
      if(command.equals("Los"))
      {
    	  remove(Hauptpanel);
    	  add(canvas);
    	  validate();
      }
   }
   
}
 
   
class MyCanvas extends Canvas
{

   public Dimension getPreferredSize()
   {
       return new Dimension(250, 120);
      }
   
      
   public void paint(Graphics g)
   
   {   
       super.paint(g);
      g.setColor(Color.lightGray);
      g.fillRect(150,100,100,20);
      g.setColor(Color.black);   
               

   }
         
}

 

[bristtote]

Wie peinlich, ich habe es echt vorher so geändert

Danke, es klappt jetzt.

 

[Wildcard]

übrigens... sowas:

      for (int i=0; i<labels.length; i++)
      {   
          labels[i] =new Label(s);
          int s1=Integer.parseInt(s);
          s1++;
          s=Integer.toString(s1);
          panel[i].add(labels[i]);
       }

macht man so:

      for (int i=0; i<labels.length; i++)
      {   
          label[i]=new Label(String.valueOf(i));
          panel[i].add(labels[i]);
      }

:wink:

 

[bristtote]

Vielen Dank für die Tips,

sowas kann ich gut gebrauchen, da ich ein Neuling bin.

 

[bristtote]

BTW: nicht böse gemeint, aber dein Code ist furchtbar. Das solltest du überarbeiten. Stichwort OOP...

Hi Wildcard, über die Sache oben komme ich nicht hinweg. :lol:
Kannst Du mir erzählen, was ich da unbedingt in meinem Code ändern müsste, wenn Du mal Zeit hast. Das wäre echt nett, weil ich nämlich selber meinen Code schon ganz gut fand und nicht weiss, was da so schlimm war.

Danke

 

[Wildcard]

ok, mal ein paar Punkte:
1. Deklarier die Variablen private
2. variablen werden immer klein geschrieben
3. keine _ verwenden (ausser bei Konstanten)
4. eigene Klasse für das MainPanel
5. eigene Klasse für das gezeichnete canvas
6. Anstatt n-Zeilen vom gleichen Aufbau zu erzeugen solltest du eine Klasse schreiben die von JPanel/Panel erbt und alle enthaltenen Komponenten kapselt (Labels/Choises)
7. Ich würde zu einem Umstieg auf Swing raten.

 

[bristtote]

Hallo Wildcard,

ich habe versucht meinen Code dem OOP anzupassen. Jetzt habe ich ein Problem:

Vorher hat die Methode "aendere" dafür gesorgt, dass beim Wählen eines Items drei neue Choices entstehen, leider geht es jetzt nicht mehr. Die Choices tauchen zwar kurz auf, verschwinden aber dann wieder. Ich denke, es hängt damit zusammen, dass jetzt die Methode "aendere" nicht in der Hauptklasse sondern in der inneren Klasse realisiert worden ist. Weisst Du woran es liegen kann?

public class Test extends Applet 
{	
	HauptPanel p= new HauptPanel();	
	
	public void	init()
	{		
	add(p);
	setBackground(p.bgCo);		
	}
	
}

class HauptPanel extends Panel 
{	

    Color bgCo        = new Color(158,158,158);   
    Button buttonL    = new Button("Los");
    TextField t       = new TextField(8);
    Panel panel    [] = new Panel[12];
    String items   [] = {"NOP", "ADD", "LOAD"}; 
    Choice choices [] = new Choice [11];
    Choice c1      [] = new Choice [11];
    Choice c2      [] = new Choice [11];
    Choice cerg    [] = new Choice [11];
    Label labels   [] = new Label[11];
    Label gleich   [] = new Label[11];
    Label plus     [] = new Label[11]; 
    String g          ="=";  
    String p          ="+";    
	
    HauptPanel()
    {		
	setLayout(new GridLayout(0,1)); 
	setBackground(bgCo);	
	panelsEinfuegen();
        labelsEinfuegen();
    	choicesEinfuegen();
    	panel[11].add(buttonL);
    	panel[11].add(t);	
    	rufItem();    	
    }    
    
    public void panelsEinfuegen()
    {
    	for (int i=0; i<panel.length; i++)
	{
    		panel[i]=new Panel();
    		add(panel[i]);     
    	}
    
    }    
        
    public void choicesEinfuegen()
    {
    	for (int i=0; i<choices.length; i++)
	{    		
    	    choices[i] = new Choice();  		
    	    choices[i].addItem(items [0]);
   	    choices[i].addItem(items [1]);  
    	    choices[i].addItem(items [2]); 
            panel[i].add(choices[i]);
    	 }
    	
    }
    
    public void labelsEinfuegen()
    {
    	for (int i=0; i<labels.length; i++)
		{
		labels[i]=new Label(String.valueOf(i));
		gleich[i] =new Label(g);
    		plus[i] =new Label(p); 
    		panel[i].add(labels[i]); 
		}	    	
    } 
	
	public void rufItem()
	{		
		for ( int i=0; i<choices.length; i++) 
		{
		 	final int index = i;
		 	choices[i].addItemListener(new ItemListener()
			{
    			public void itemStateChanged(ItemEvent e) 
    			{
     
        			if ( ItemEvent.SELECTED == 1)
        			{
      					aendere( index, e.getItem() );
    				}
    			}
  			});
		}	
		
	}
		public void aendere( int index, Object item )
	{
  // wird immer aufgerufen, wenn "choice" ein neues "item" selektiert hat. Das item kann auch null sein!
    	if(item.equals("ADD"))
    	{       
       		 if (cerg[index]!=null)
        	 {
             	  panel[index].remove(cerg[index]);
            	  panel[index].remove(gleich[index]); 
                  panel[index].remove(plus[index]);          
                  panel[index].remove(c1[index]);
                  panel[index].remove(c2[index]);      
                    
                 } 
 
      	   c1[index]  = new Choice();
       	   c2[index]  = new Choice();
       	   cerg[index]= new Choice();
       	   gleich[index] = new Label("=");
       	   plus[index]= new Label("+");
       	 
	       c1[index].add("R 0");
	       c1[index].add("R 1");
	       c1[index].add("R 2");
	       c1[index].add("R 3");
	       c1[index].add("R 4");
	       c1[index].add("R 5");
	       c1[index].add("R 6");
	       c1[index].add("R 7");
	       
	       c2[index].add("R 0");
	       c2[index].add("R 1");
	       c2[index].add("R 2");
	       c2[index].add("R 3");
	       c2[index].add("R 4");
	       c2[index].add("R 5");
	       c2[index].add("R 6");
	       c2[index].add("R 7");
	       
	       cerg[index].add("R 1");
	       cerg[index].add("R 2");
	       cerg[index].add("R 3");
	       cerg[index].add("R 4");
	       cerg[index].add("R 5");
	       cerg[index].add("R 6");
	       cerg[index].add("R 7");
	       
	       
	       panel[index].add(cerg[index]);
	       panel[index].add(gleich[index]);     
	       panel[index].add(c1[index]);
	       panel[index].add(plus[index]);
	       panel[index].add(c2[index]);    
   	   }
      
       else if(item.equals("LOAD"))..........................// Hier geht es weiter mit den anderen beiden Items LOAD und NOP
      
          
	   repaint();
	   validate(); 
	}
}