Swing JLabels entfernen und neue einfügen

[CMGbR]

Hallo zusammen,

ich arbeite gerade für mein Studium an der grafischen Oberfläche für Schiffe-Versenken.

Genauer gesagt an den Schiffen selbst. Ich habe dafür einen neue Klasse Ship geschrieben, die von JPanel erbt.
Man übergibt dem Konstuktor die Größe des Schiffs (Anzahl der Felder), Variation (es gibt 3 verschiedene grafikvariationen), Feldgröße und Richtung (true = vertikal)...

Den Code für die Ship-Klasse habe ich der Übersicht halber ganz unten angehängt.

Der Konstruktor initialisiert nur die Variablen, setzt die Hintergrundfarbe und ruft dann buildShipGraphics() auf, welche dann die Größe und das Layout festlegt und dann addTile aufruft.
AddTile holt sich die Grafiken für die Schiffe und fügt die JLabels mit den entsprechenden Icons ein. Jedes Schiff besteht aus einem Anfangsstück, beliebig vielen Mittelstücken und einem Endstück.

Ich kann in meinem Frame problemlos ein neues Schiff anlegen:

private void jButton4ActionPerformed(java.awt.event.ActionEvent evt) {                                         
        newship = new ship(10,1,30,true);
        newship.setLocation(0,0);
        add(newship);
        newship.setVisible(true);
        pack();
    }

Das funktioniert sowohl vertikal als auch horizontal und in jeder Größe.
Dann habe ich eine weitere Methode zum Schiff hinzugefügt: rotateShip();
rotateShip soll alle Tiles (also die Labels) löschen, den Wert von direction ändern und dann wieder buildShipGraphics() aufrufen.

Das funktioniert aber nicht! Sobald ich die Methode rotate() aufrufe verschwindet das ganze Schiff vom Bildschirm. Es taucht auch nicht wieder auf wenn ich zurückrotiere.

Ich hab schon an allen möglichen Stellen Kombinationen aus
invalidate() & validate()
revalidate()
repaint()
updateUI()
usw. versucht.

Das Hilft alles nichts...

Kann mir jemand helfen?

Vielen Dank im Voraus und viele Grüße
Kornelius

public class ship extends JPanel {

    int variation, length, fieldsize;
    boolean direction;
    /** Creates new form BeanForm */
    public ship(int length, int variation, int fieldsize, boolean direction) {
        super();
        this.length=length;
        this.variation=variation;
        this.fieldsize=fieldsize;
        this.direction=direction;
        

        this.setBackground(new Color(255,255,255,0));
        buildShipGraphics();

    }

    public void buildShipGraphics() //Setzt die Größe des Schiffs (vertikal oder horizontal) und fügt die Labels ein
    {
        int i;
        if(direction)
        {
            this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));

            setSize(fieldsize,fieldsize*length);
            setPreferredSize( new Dimension(fieldsize, fieldsize*length) );
            addTile(0,true,0,0,fieldsize);
            for(i=1;i<length-1;i++)
            {
               addTile(1,true,0,fieldsize*i,fieldsize);
            }
            addTile(2,true,0,fieldsize*(i+1),fieldsize);
        }
        else
        {
            this.setLayout(new BoxLayout(this, BoxLayout.X_AXIS));
            setSize(fieldsize*length,fieldsize);
            setPreferredSize( new Dimension(fieldsize*length, fieldsize) );
            addTile(0,false,0,0,fieldsize);
            for(i=1;i<length-1;i++)
            {
               addTile(1,false,fieldsize*i,0,fieldsize);
            }
            addTile(2,false,fieldsize*(i+1),0,fieldsize);

        }
    }

    //posX und posY werden seit dem BoxLayout nicht mehr verwendet:
    private void addTile(int type, boolean vertical, int posX, int posY, int size) 
    {
        String dir = (vertical ? "v" : "h");
        ImageIcon icon = new ImageIcon(getClass().getResource("images/ship_"+Integer.toString(this.variation)+"_" + dir + "_"+Integer.toString(type)+".png").getPath() );
        JLabel temp = new JLabel(scaleTileIcon(icon,fieldsize));
        //temp.setLocation(posX,posY);
        temp.setSize(size,size);
        temp.setPreferredSize(new Dimension(size,size));
        //temp.setIconTextGap(0);
        temp.setBackground(new Color(255,255,255,0));
        this.add(temp);
        temp.setVisible(true);
        temp.validate();
    }

    private ImageIcon scaleTileIcon(ImageIcon icon, int size)
    {
        BufferedImage bi = new BufferedImage(size,size, BufferedImage.TYPE_INT_ARGB);
        bi.getGraphics().drawImage(icon.getImage(), 0, 0, size, size, null);
        return new ImageIcon(bi);
    }

    public void rotateShip()
    {
        this.removeAll();
        direction=(direction?false:true);
        //this.removeAll();
        
        System.out.println(direction);
        //direction=false;
        buildShipGraphics();
        this.validate();
    }
    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">
    private void initComponents() {

        setName("ship"); // NOI18N
        setLayout(new javax.swing.BoxLayout(this, javax.swing.BoxLayout.Y_AXIS));
    }// </editor-fold>


    // Variables declaration - do not modify
    // End of variables declaration

}

 

[SlaterB]

ich empfehle
validate();
repaint();
am JFrame aufzurufen, siehe Beispiel unten,
nur innerhalb des JPanels funktioniert es bei mir diesmal nicht, doppelte Anzeige

deine diversen setSize/PreferredSize-Aufrufe habe ich rausgenommen, für meine Text-Label zu klein,
setVisible(true) + validate() auf die JLabel kannst du dir komplett sparen,
Klassen groß schreiben!

public class TestGUI extends JFrame {
    public TestGUI() {
        ship p = new ship(10, 10, 10, true);
        add(p);
        setSize(950, 300);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setVisible(true);

        while (true) {
            try{ Thread.sleep(2000);  }  catch (Exception e) {}
            p.rotateShip(); // alle 2 sec rotieren
            validate();
            repaint();
        }
    }

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


class ship
    extends JPanel
{

    int variation, length, fieldsize;
    boolean direction;

    public ship(int length, int variation, int fieldsize, boolean direction)
    {
        this.length = length;
        this.variation = variation;
        this.fieldsize = fieldsize;
        this.direction = direction;
        this.setBackground(new Color(255, 255, 255, 0));
        buildShipGraphics();
    }

    public void buildShipGraphics()
    {
        int i;
        if (direction)
        {
            this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));

//            setSize(fieldsize, fieldsize * length);
//            setPreferredSize(new Dimension(fieldsize, fieldsize * length));
            addTile(0, true, 0, 0, fieldsize);
            for (i = 1; i < length - 1; i++)
            {
                addTile(1, true, 0, fieldsize * i, fieldsize);
            }
            addTile(2, true, 0, fieldsize * (i + 1), fieldsize);
        }
        else
        {
            this.setLayout(new BoxLayout(this, BoxLayout.X_AXIS));
//            setSize(fieldsize * length, fieldsize);
//            setPreferredSize(new Dimension(fieldsize * length, fieldsize));
            addTile(0, false, 0, 0, fieldsize);
            for (i = 1; i < length - 1; i++)
            {
                addTile(1, false, fieldsize * i, 0, fieldsize);
            }
            addTile(2, false, fieldsize * (i + 1), 0, fieldsize);

        }
    }

    // posX und posY werden seit dem BoxLayout nicht mehr verwendet:
    private void addTile(int type, boolean vertical, int posX, int posY, int size)
    {
        String dir = (vertical ? "v" : "h");
        JLabel temp = new JLabel("<boot> ");
        // temp.setLocation(posX,posY);
//        temp.setSize(size, size);
//        temp.setPreferredSize(new Dimension(size, size));
        // temp.setIconTextGap(0);
        temp.setBackground(new Color(255, 255, 255, 0));
        add(temp);
    }

    public void rotateShip()
    {
        removeAll();
        direction = (direction ? false : true);
        System.out.println(direction);
        buildShipGraphics();       
    }

    private void initComponents()
    {
        setName("ship");
        setLayout(new javax.swing.BoxLayout(this, javax.swing.BoxLayout.Y_AXIS));
    }
}

 

[cmgbr]

Hi,

erstmal vielen Dank für deine schnelle Antwort!

Ich habe es mal so versucht und validate() und repaint() nur im JFrame aufgerufen.
Leider funktioniert es aber immernoch nicht.

Sobald rotateShip() aufgerufen wird verschwindet das Schiff vom Bildschirm...

 

[SlaterB]

merke: wenn ich ein vollständiges Programm poste, das ohne Fehler läuft, dann bringt es nichts über Fehler in deinem unbekannten Programm mit unbekannter Umsetzung bei dir zu philosophieren,

ich habe deine Vorlage so gut es geht kopiert, mehr ist nicht machbar, du musst schon ein vollständiges Programm mit JFrame und main-Methode posten,
am besten wie bei mir testweise ohne Grafiken und mit mit Text, es sei denn dadurch verschwindet der Fehler, was dann immerhin ein guter Hinweis ist

 

[CroniD]

Ich habe mal den Code von cmgbr und den von SlaterB ausprobiert. Beide funktionieren soweit, aber SlaterBs Variante, dass man das JFrame revalidieren sollte ist nicht ganz richtig und auch inperformant. Das Panel zu validieren reicht völlig. Daher könnte ich mir folgendes vorstellen woran es liegen könnte dass du nichts siehst:
1. Der Pfad zu den Images stimmt nicht,
2. die png Dateien sind komplett transparent oder
3. der EDT wird irgendwo blockiert, bevor er zu dem repaint kommt.

Wenn du deinen kompletten Source Code in einer Zip in einem Forum Post von dir anhängen könntest, dann schau ich gerne mal rüber.

 

[SlaterB]

meinen Quellcode kann ja jeder ausprobieren und dann die beiden Befehle von Zeile von Zeile 12 + 13 etwa ans Ende von rotateShip() verschieben

bei mir siehts dann so aus, woran es liegt will ich aber nicht beschwören

 

[CroniD]

Vorweg: Sry, SlaterB. Ich lag in der Tat falsch.

Eigentlich sollte er schon die komplette Fläche des Ships neu zeichnen. Hmm ...

Oh man, ich bin auch bissle döschig. ^^ Setze beim Ship mal setOpaque(false). Swing denkt, dass das Ship das zeichnen des Hintergrundes dem Container (in diesem Falle dem JFrame) überlassen soll. Bis man auf sowas kommt. ^^