JLabels löschen (sollen nicht mehr gezeichnet werden)

[manuche]

Hallo!
In meinem Spiel werden beim Mausklick an der richtige Stelle JLabels erzeugt und gezeichnet... Wenn sie nun an die falsche Stelle geschoben werden, sollen sie wieder gelöscht werden! Ich fand es einleuchtend die null-referenz zuzuweisen da es ja im Prinzip nur Instanzen sind die gelöscht werden sollen! Allerdings werden die sichtbaren Labels nicht gelöscht (auch dann nicht wenn ich den GarbageCollector direkt danach ausführe)! Hier der entsprechende Code:

private void guiMouseReleased (MouseEvent evt){
	if( hitLabel != null ){
		System.out.println ("Released: " + evt.getPoint());
       		if (contentPanel.getComponentAt (evt.getPoint()) instanceof GamePanel){
			System.out.println ("On GamePanel");
            		hitLabel.setLocation (evt.getX() - labelX, evt.getY() - labelY);
            		hitLabel.repaint();
            	}else{
            		System.out.println ("Else-Zweig");
            		backupX = hitLabel.getX();
            		backupY = hitLabel.getY();
            		String currentLabel = hitLabel.getName();
            		if (currentLabel.equals ("s1copy")){
            			s1copy = null;
	            	}else if (currentLabel.equals ("s2copy")){
        	    		s2copy = null;
            		}else if (currentLabel.equals ("s3copy")){
            			s3copy = null;
	            	}else if (currentLabel.equals ("s4copy")){
        	    		s4copy = null;
            		}else if (currentLabel.equals ("s5copy")){
            			s5copy = null;
	            	}
        	        hitLabel = null;
                	this.repaint();
            	}
        }
}

 

[SlaterB]

wenn du deine lokale Variable oder Exemplarvariable auf null setzt,
interessiert das andere Stellen, die Referenzen auf das Objekt haben, wie z.B. das JPanel in der GUI, nicht die Bohne,

auch der GC kann dann (zum Glück) nix machen, das Objekt wird an anderer Stelle noch verwendet,
du musst herausfinden, wo das Objekt in der GUI steckt und dann z.B. jPanel.remove(deletedLabel);

 

[manuche]

Also ich hab das ganze mal ausprobiert...
Also hinzugefügt wird es mit this.add! daraus habe ich mal gefolgert, dass es auch this.remove(xyz) heissen muss... Fehlanzeige! Auch ein contentPanel.remove.... hilft da nichts... leider?
wie kann ich denn rausfinden wo es ist?

 

[SlaterB]

ganz ordinär fängst du mit dem JContentPane an,
schaust ob es dadrin ist,
wenn nicht, dann suchst du im JContentPane rekursiv nach anderen Containern, die das gesuchte JLabel enthalten könnten,
allen voran JPanels, aber im Grunde jedes Component-Objekt

---------

das aber nur allgemeinstmöglich,
viel mehr Sinn macht es, dein Programm besser kennenzulernen,

wenn du die Objekte direkt ins ContentPane einfügst, dann müssen sie auch noch darin sein,
alles andere deutet auf Fehler oder Missdeutung hin, wie Mehrfacherzeugung, doch Unterpanel, Suche mit der falschen Variablen usw.,
beliebige Fehler denkbar, ohne Code von dir nicht weiter nachzuvollziehen

----------

aber immerhin:
nur ein remove reicht nicht, du brauchst noch repaint() und/ oder validate():

public class TestGUI
    extends JFrame
{
    JLabel a = new JLabel("test");

    public TestGUI()
        throws Exception
    {
        setLayout(new FlowLayout());

        JButton b = new JButton("Test");
        b.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
                {
                    getContentPane().remove(a);
                    // validate();
                    // muss ausgeführt werden, wenn sich die Komponenten
                    // neu anordnen sollen

                    // repaint();
                    // muss ausgeführt werden damit das alte JLabel nicht
                    // mehr zu sehen ist
                }
            });

        add(a);
        add(b);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(400, 200);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {
        new TestGUI();
    }
}

 

[manuche]

Hab nun mal geschaut... mit this.getComponents() hab ich mir bestätigen lassen, dass sich hier das JLabel befindet
Ein this.remove (xyz) funktioniert allerdings tortzdem nicht...

 

[SlaterB]

weitere wichtige Schritte habe ich oben gepostet,
ansonsten weißt du ja, was du zu tun hast, außer deinen aktuellen Stand 'geht nicht' zu wiederholen

 

[manuche]

Könnte es evtl am null-Layout liegen?

 

[SlaterB]

na supi, eine kleine Mini-Information, wie du wahrscheinlich im Laufe der Zeit noch viele herausrücken wirst (statt 100% klaren Code zu posten),

und dann auch noch etwas, was du mühelos selber testen könntest:

public class TestGUI
    extends JFrame
{
    JLabel a = new JLabel("test");

    public TestGUI()
        throws Exception
    {
        // setLayout(new FlowLayout());
        setLayout(null);


        JButton b = new JButton("Test");
        b.setBounds(150, 50, 100, 50);
        b.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
                {
                    getContentPane().remove(a);
                    // validate();
                    // muss ausgeführt werden, wenn sich die Komponenten
                    // neu anordnen sollen

                    // repaint();
                    // muss ausgeführt werden damit das alte JLabel nicht
                    // mehr zu sehen ist
                }
            });

        a.setBounds(50, 50, 100, 50);
        add(a);
        add(b);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(400, 200);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {
        new TestGUI();
    }
}

war das schwer?

das regt mich jetzt mal auf, so dass ich nicht weiter antworte

 

[manuche]

Sorry... War kein Aufruf das ganze nochmal für ein null-Layout zu posten! War nur eine vermutung, dass es was ausmachen könnte je nachdem wie ich in meinem Programm rumgefentert hab! Ein ja oder nein hätte mir eigentlich gereicht...
Naja torztdem danke!

 

[SlaterB]

> Ein ja oder nein hätte mir eigentlich gereicht...

was ja niemand wirklich 100% im Kopf hat,

also entweder
1)
nicht antworten, Thread versandet, jemand wartet auf Antwort usw
2)
ich muss es testen..
3)
du machst dir dir minimale Mühe, es mit minimalen Änderungen selber zu testen

3) wolltest du nicht, 1) wollte ich nicht -> 2) + Ärger

 

[manuche]

Jop hast ja recht hätte ich selbst testen können... war nur zu sehr auf meinen quellcode fixiert und hab da rum gebastelt!
Mein Fehler, sorry!!!

*edit: Hier ist der komplette Quelltext des Programms... Zwar an einigen Stellen schon wieder überarbeitet, aber im großen und ganzen noch relativ der selbe! Quelltext
Ansonsten hab ichs jetzt erstmal mit setVisible (boolean b) gelöst... Nicht schön aber erfüllt seinen Zweck!