Menuaufruf -> panel.removeAll(); klappt nicht?

[Ashiaka]

Hi Leute!

Ich komme bei meinem GUI-bau mal wieder nicht weiter

Folgendes:

Ich hab ein JFrame und in diesem JFrame ein JPanel:

final JPanel panel2 = new JPanel();

Dann hab ich ein mir ein kleines Menu gebaut:

JMenuBar menuBar = new JMenuBar(); 
JMenu fileMenu = new JMenu( "Datei" ); 
menuBar.add( fileMenu ); 
this.setJMenuBar( menuBar );

und füge diesem Menü eine Eintrag mit einer Funktion hinzu:

fileMenu.add(new AbstractAction("New") {
			public void actionPerformed(ActionEvent e) {
		           panel2.setVisible(false);
		        }
		     });

funktioniert soweit auch alles.
Wenn ich jetzt jedoch

panel2.setVisible(false);

durch

panel2.removeAll();

ersetze, klappt dieses leider nicht

Um das zu überprüfen hab ich dem Panel einen Button vom Typ JButton hinzugefügt.

Kann mir jemand erklären, was ich da falsch mache?

MfG, Ashiaka

 

[L-ectron-X]

Was klappt denn nicht?

Willst du das Panel entfernen? Dann rufst du die Methode auf einer verkehrten Instanz auf.
Wenn du die removeAll()-Methode auf dem Panel aufrufst, werden dessen Komponenten entfernt, nicht das Panel selbst.

 

[Ashiaka]

Ja, richtig. So hätte ich das auch gerne.

Das Panel soll bleiben, nur der Inhalt von diesem Panel soll gelöscht werden, damit darauf neu gezeichnet werden kann.
Aber da tut sich leider gar nix

Ich poste nochmal den zusammenhängende Code:

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

import javax.swing.AbstractAction;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;

import com.sun.corba.se.spi.orbutil.fsm.Action;


public class TestGUI extends JFrame{

	/**
	 * @param args
	 */
	
	public JPanel panel2;
	
	public TestGUI()
	{
		//Frame:
		this.setTitle("TEST");
		this.setLayout(null);
		this.setSize(1024, 768);
		
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setResizable(false);
		
		
		//Main-panel:
	        final JPanel panel2 = new JPanel();
		panel2.setBorder(BorderFactory.createLineBorder(Color.black));
		panel2.setBounds(202, 40, 809, 680);
		this.add(panel2);
	
		panel2.add(new JButton("test"));     //Dieser button soll entfernt werden.
		
                //Menue:
		JMenuBar menuBar = new JMenuBar(); 
		JMenu fileMenu = new JMenu( "Datei" ); 
		menuBar.add( fileMenu ); 
		this.setJMenuBar( menuBar );
		fileMenu.addSeparator();
		
		
		
		
		//Aufruf von New soll Panel2 leeren
		fileMenu.add(new AbstractAction("New") {
			public void actionPerformed(ActionEvent e) {
		        panel2.removeAll();
		        }
		     });
		
		
		this.setVisible(true);  
	}
	
	
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		TestGUI gui = new TestGUI();
		
	}

	
}

 

[Ashiaka]

Ich hab irgendwo gelesen, dass die "ActionEvent-Methoden" nicht direkt auf ein Panel zeichnen können, sondern den Umweg über einen Thread gehen müssen. Stimmt das, oder hab ich was missverstanden?

Und wie genau muss das dann aussehen?

 

[MathiasBauer]

Ich habe mal versucht das Problem zu lösen.

Es funktioniert entweder, wenn man den Button mit einer anderen Komponenete überschreibt, oder man generiert ein neues Panel. Vielleicht hilft es dir.

Hier mein Code:

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.AbstractAction;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;

public class TestGUI extends JFrame{

   /**
    * @param args
    */
   
   private JPanel panel2;
   private JButton button;
   
   public TestGUI()
   {
      //Frame:
      this.setTitle("TEST");
      this.setLayout(null);
      this.setSize(1024, 768);
      
      this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      this.setResizable(false);
      
      
      //Main-panel:
      panel2 = new JPanel();
      button = new JButton("test");
      panel2.setBorder(BorderFactory.createLineBorder(Color.black));
      panel2.setBounds(202, 40, 809, 680);
      panel2.add(button);     //Dieser button soll entfernt werden.
      add(panel2);
     
      //Menue:
      JMenuBar menuBar = new JMenuBar();
      JMenu fileMenu = new JMenu( "Datei" );
      menuBar.add( fileMenu );
      this.setJMenuBar( menuBar );
      JMenuItem menuItem = new JMenuItem("New");
      fileMenu.add(menuItem);
      menuItem.addActionListener(new ActionListener() {	  //Aufruf von New soll Panel2 leeren
         public void actionPerformed(ActionEvent e) {
            /* panel2.remove(button);        // funktioniert nur, wenn der Button mit einer anderen Komponente überschrieben wird 
             JLabel label = new JLabel();
             label.setPreferredSize(new Dimension(60,50));
             label.setText("hallo");
             panel2.add(label);
             panel2.validate();*/
        	  remove(panel2);
        	  panel2 = new JPanel();
        	  panel2.setBorder(BorderFactory.createLineBorder(Color.black));
             panel2.setBounds(202, 40, 809, 680);
             add(panel2);
             repaint();     	 
           }
       });    
      
      this.setVisible(true); 
   }
   
   
   
   public static void main(String[] args) {
      // TODO Auto-generated method stub
      TestGUI gui = new TestGUI();
      
   }

   
}

 

[Ashiaka]

Vielen Dank. Das löschen klappt nun auch mit

label2.removeAll()

Hatte nur das repaint() vergessen.

Allerdings tritt dadurch auch schon wieder ein neues Problem ans Tageslicht.
Ich kann dem neuen Panel über die actionPerformed()-Funktion nichts neues hinzufuegen:

         public void actionPerformed(ActionEvent e) {
            /* panel2.remove(button);        // funktioniert nur, wenn der Button mit einer anderen Komponente überschrieben wird
             JLabel label = new JLabel();
             label.setPreferredSize(new Dimension(60,50));
             label.setText("hallo");
             panel2.add(label);
             panel2.validate();*/
             remove(panel2);
             panel2 = new JPanel();
             panel2.setBorder(BorderFactory.createLineBorder(Color.black));
             panel2.setBounds(202, 40, 809, 680);
             
             panel2.add(new JLabel("Test"));               <--- NEU
             
             add(panel2);
             repaint();        
           }
       });   
     
      this.setVisible(true);
   }

Überseh ich vielleicht insgesamt irgendwas?

MfG, Ashiaka

 

[MathiasBauer]

Diesen Teil auskommentieren:

remove(panel2);
panel2 = new JPanel();
panel2.setBorder(BorderFactory.createLineBorder(Color.black));
panel2.setBounds(202, 40, 809, 680);
add(panel2);
repaint();

und diesen Kommentar entfernen:

/* panel2.remove(button);        // funktioniert nur, wenn der Button mit einer anderen Komponente überschrieben wird
JLabel label = new JLabel();
label.setPreferredSize(new Dimension(60,50));
label.setText("hallo");
panel2.add(label);
panel2.validate();*/

 

[Ashiaka]

OMG

dieses validate() hat gefehlt!

Ich danke dir für deine Mühe!!! Hast mir den Tag gerettet