Swing Probleme mit repaint() bzw. JScrollPane

[TheAnfänger]

Hallo,
Ich bins mal wieder.

So diesmal habe ich das Problem, dass wenn ich ein JScrollPane

scroll.setVisible(true);

setze, dann wird mir trotz aufruf von

repaint();

nichts angezeigt.

Wenn ich das Fenster minimiere und wieder öffne wird es hingegen angezeigt.

Ich schätze also, dass das Problem bei repaint() liegt.

Hätte jemand eine Idee? Ich komme gerade echt nicht weiter.

Der Code kommt später. Ich sitze gerade nicht an meinem Laptop.

Vielen Dank für eure Hilfe

 

[Robat]

Ich würde mal ins blaue raten und sagen, dass du setVisible(true) auf deinem JFrame aufrufst und danach erst die JScrollPane hinzufügst.
Aber ohne Code kann man nur raten.
BTW: Du musst du Komponenten die du addest nicht setVisible(true) aufrufen. Die werden beim adden sowieso sichtbar gemacht.

 

[TheAnfänger]

Ich würde mal ins blaue raten und sagen, dass du setVisible(true) auf deinem JFrame aufrufst und danach erst die JScrollPane hinzufügst.

Leider falsch geraten

Ziel ist momentan, dass auf Knopfdruck auf einen JButton ein, dem Button zugeordnetes Menü bzw. in meinem Fall ein JPanel sichtbar wird. Dieses JPanel liegt aufgrund des Layoutmanagers von JScrollPane in einem weiteren JPanel mit einem FlowLayout, welches schlussendlich im JScrollPane liegt. Das JScrollPane liegt schlussendlich in einem weiteren (Haupt-)JPanel namens canvas mit BorderLayout zusammen mit einem anderen JPanel, das die Buttons enthält.

Für eine bessere Übersicht:

Spoiler

-jframe
--canvas(borderlayout)
---jpanel untereReihe
----die ganzen Buttons (momentan nur einer)
---JScrollPane canvasScrollPaneLeft
----JPanel canvasPanelLeft(FlowLayout)
-----beliebige JComponent (momentan nur ein JPanel namens buildsMenu)
------eventuell noch Unterkomponenten (in buildsMenu momentan 10 Buttons).

Hoffe es ist einigermassen verständlich geworden.

Nun noch der Code:

Spoiler: Code

public class GUI extends JFrame implements ActionListener {
    private static final long serialVersionUID = -7619621152921101049L;

    public static final int STANDARD_HEIGHT = 900, STANDARD_WIDTH = 1200;
    public static final String STANDARD_TITLE = "Rimwar";
   
    public JButton builds;
    public ArrayList<JButton> allButtons;

    public Zeichenflaeche canvas = null;
    public JPanel untereReihe = null;
   
    public JPanel buildsMenu;
   
    public JScrollPane canvasScrollPaneLeft = new JScrollPane(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED, JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
    public JPanel canvasPanelLeft = new JPanel();

    // übrigens: JA, die Methode wird (bei der Erstellung von GUI) aufgerufen
    // ich erspare euch nur die ganzen Konstruktoren
    public void init(String title, int width, int height) {
        setTitle(title);
        setSize(width, height);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
       
        canvasScrollPaneLeft.setViewportView(canvasPanelLeft);
        canvasScrollPaneLeft.setFocusable(false);
        canvasPanelLeft.setFocusable(false);

        canvas = new Zeichenflaeche();
        canvas.setPreferredSize(new Dimension(width, height));
        canvas.setFocusable(true);
        canvas.setLayout(new BorderLayout());
       
        canvas.add(canvasScrollPaneLeft, BorderLayout.WEST);
        canvasScrollPaneLeft.setVisible(false);
       
        add(canvas, BorderLayout.CENTER);
       
        allButtons = new ArrayList<JButton>();
       
        builds = new JButton("Build things");
        builds.setActionCommand("build");
        allButtons.add(builds);
       
        buildsMenu = new JPanel(new GridLayout(0, 3));
        //buildsMenu.setBackground(new Color(0, 0, 0, 0));
        buildsMenu.setBackground(Color.BLUE);
        buildsMenu.add(new JButton("1")).setFocusable(false);
        buildsMenu.add(new JButton("2")).setFocusable(false);
        buildsMenu.add(new JButton("3")).setFocusable(false);
        buildsMenu.add(new JButton("4")).setFocusable(false);
        buildsMenu.add(new JButton("5")).setFocusable(false);
        buildsMenu.add(new JButton("6")).setFocusable(false);
        buildsMenu.add(new JButton("7")).setFocusable(false);
        buildsMenu.add(new JButton("8")).setFocusable(false);
        buildsMenu.add(new JButton("9")).setFocusable(false);
        buildsMenu.add(new JButton("10")).setFocusable(false);
        initMenu(buildsMenu);
       
        initButtons();
        pack();
    }
   
    private void initMenu(JComponent menu) {
        menu.setVisible(false);
        Dimension d = menu.getLayout().minimumLayoutSize(menu);
        menu.setPreferredSize(d);
        menu.setSize(d);
        menu.setFocusable(false);
    }

    private void initButtons() {
        final int buttonHeight = 50;
        untereReihe = new JPanel();
        untereReihe.setBackground(new Color(0, 0, 0, 0));
        untereReihe.setPreferredSize(new Dimension(canvas.getPreferredSize().width, buttonHeight));
        untereReihe.setLayout(new GridLayout(1, 0));
        final Color c = new Color(0, 0, 150, 125);
        for (int i=0; i<allButtons.size(); i++) {
            allButtons.get(i).setFocusable(false);
            allButtons.get(i).setBackground(c);
            allButtons.get(i).setForeground(Color.white);
            allButtons.get(i).addActionListener(this);
            untereReihe.add(allButtons.get(i));
        }
        canvas.add(untereReihe, BorderLayout.SOUTH);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if("build".equals(e.getActionCommand())) {
            toggleLeftMenu(buildsMenu);
            /*
            if(buildsMenu.isVisible()) {
                buildsMenu.setVisible(false);
            } else {
                buildsMenu.setVisible(true);
                Dimension d = new Dimension(buildsMenu.getPreferredSize().width+3, canvas.getPreferredSize().height - this.untereReihe.getPreferredSize().height);
                canvasScrollPaneLeft.setPreferredSize(d);
                canvasScrollPaneLeft.setSize(d);
                canvasScrollPaneLeft.setViewportView(buildsMenu);
                canvasScrollPaneLeft.repaint();
                buildsMenu.repaint();
                this.repaint();
            }
            //*/
        }
    }
   
   
    private void toggleLeftMenu(JComponent menu) {
        if(menu.isVisible()) {
            menu.setVisible(false);
            canvasPanelLeft.removeAll();
            canvasScrollPaneLeft.setVisible(false);
        } else {
            menu.setVisible(true);
            canvasPanelLeft.removeAll();
            canvasPanelLeft.add(menu);
            canvasPanelLeft.setPreferredSize(menu.getPreferredSize());
            canvasPanelLeft.validate();
            canvasPanelLeft.setVisible(true);
            canvasScrollPaneLeft.setPreferredSize(new Dimension(canvasPanelLeft.getPreferredSize().width+3, 800));
            canvasScrollPaneLeft.setViewportView(canvasPanelLeft);
            canvasScrollPaneLeft.validate();
            canvasScrollPaneLeft.setVisible(true);
            canvasScrollPaneLeft.repaint();
           
        }
    }
}

 

[TheAnfänger]

Inzwischen habe ich noch festgestellt, sobald man das Fenster einmal minimiert und danach wieder anzeigen lässt, tritt der Fehler danach nicht mehr auf.

 

[Harry Kane]

Mögliche Fehlerquellen:
1. "Zeichenflaeche canvas" hat den falschen Layoutmanager. Der Klassename "Zeichenflaeche" spricht eher dafür, dass du eigene Zeichenroutinen implementieren möchtest, und canvas nicht als Container für Components nutzen möchtest. Du nutzt canvas aber als Container für canvasScrollPaneLeft.
2. canvasPanelLeft hat keine Kindkomponenten, und sollte dementsprechend eine preferred size von ungefähr 0x0 Pixeln haben. Die JScrollPane dürfte versuchen, sich in ihrer Größe dem canvasPanelLeft anzupassen, und dementsprechend unsichtbar sein.
3. Es ist unklar, wo setVisible() auf der Gui-Instanz aufgerufen wird. Irgendwo wird eine solche Instanz sicher erzeugt.

 

[TheAnfänger]

1. "Zeichenflaeche canvas" hat den falschen Layoutmanager.

canvas.setLayout(new BorderLayout());

Nope

Der Klassename "Zeichenflaeche" spricht eher dafür, dass du eigene Zeichenroutinen implementieren möchtest, und canvas nicht als Container für Components nutzen möchtest. Du nutzt canvas aber als Container für canvasScrollPaneLeft.

Wenn du es genau wissen willst: Ich programmiere ein kleines Spiel. Die "Zeichenfläche" zeichnet einfach das Spiel. Und darüber sollten dann die Komponenten gezeichnet werden, was meines Erachtens und Wissens nach auch kein Problem sein sollte. (Bei den anderen Elementen hat es ja auch geklappt.) Zudem hat das den Vorteil, dass man die Map auch im Hintergrund sehen kann.

Spoiler: Zeichenfläche Code

package main;

import java.awt.Graphics;

import javax.swing.JPanel;

public class Zeichenflaeche extends JPanel {
    private static final long serialVersionUID = 1060267560901578176L;

    public Zeichenflaeche() {
        super();
    }
   
     //*
    @Override
    protected void paintComponent(Graphics gmain) {
        super.paintComponent(gmain);
        Graphics g = gmain.create();
        Start.actualState.render(g);
        /*
        if(Start.actualState == Vars.game) {
           
        } else if(Start.actualState == Vars.menu) {
           
        } else {
            System.err.println("\nZeichenfläche.java:23\nStart.actualState is something different");
        }
        //*/
    }
    //*/
}

2. canvasPanelLeft hat keine Kindkomponenten, und sollte dementsprechend eine preferred size von ungefähr 0x0 Pixeln haben. Die JScrollPane dürfte versuchen, sich in ihrer Größe dem canvasPanelLeft anzupassen, und dementsprechend unsichtbar sein.

Das stimmt zumindest zum Teil: Anfangs weise ich canvasPanelLeft keine Komponenten und auch keine preferredSize zu. Ich habe das aber gerade ausprobiert und es ändert nichts. Anfangs wird keine zugewiesen, später schon. Es ist übrigens auch Sinn der Sache, dass am Anfang nichts angezeigt wird. Ich möchte, dass das canvasPanelLeft (bzw. seine Kindkomponenten) erst sichtbar werden, wenn ein bestimmter Knopf gedrückt wird.

Ich denke das hier ist der wichtigste Part in der Klasse GUI. Ich vermute, dass hier irgendwo ein Fehler liegt:

    private void toggleLeftMenu(JComponent menu) {
        if(menu.isVisible()) {
            menu.setVisible(false);
            canvasPanelLeft.removeAll();
            canvasScrollPaneLeft.setVisible(false);
        } else {
            canvasPanelLeft.removeAll();
            canvasPanelLeft.add(menu);
            menu.setVisible(true);
            menu.setBounds(0, (int)(canvasScrollPaneLeft.getLayout().preferredLayoutSize(canvasScrollPaneLeft).height-menu.getPreferredSize().getHeight()), (int)menu.getPreferredSize().getWidth(), (int)menu.getPreferredSize().getHeight());
            canvasPanelLeft.setPreferredSize(menu.getPreferredSize());
            canvasPanelLeft.validate();
            canvasPanelLeft.setVisible(true);
            canvasScrollPaneLeft.setPreferredSize(new Dimension(canvasPanelLeft.getPreferredSize().width+3, 800));
            canvasScrollPaneLeft.setViewportView(canvasPanelLeft);
            canvasScrollPaneLeft.validate();
            canvasScrollPaneLeft.setVisible(true);
            canvasScrollPaneLeft.repaint();
           
        }
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if("build".equals(e.getActionCommand())) {
            toggleLeftMenu(buildsMenu);
            /*
            if(buildsMenu.isVisible()) {
                buildsMenu.setVisible(false);
            } else {
                buildsMenu.setVisible(true);
                Dimension d = new Dimension(buildsMenu.getPreferredSize().width+3, canvas.getPreferredSize().height-this.untereReihe.getPreferredSize().height);
                canvasScrollPaneLeft.setPreferredSize(d);
                canvasScrollPaneLeft.setSize(d);
                canvasScrollPaneLeft.setViewportView(buildsMenu);
                canvasScrollPaneLeft.repaint();
                buildsMenu.repaint();
            }
            //*/
        }
    }

3. Es ist unklar, wo setVisible() auf der Gui-Instanz aufgerufen wird. Irgendwo wird eine solche Instanz sicher erzeugt.

Ist das so wichtig? Die GUI wird erstellt und direkt auf

setVisible(true)

gesetzt.

Trotzdem Danke.

Ich bin gerade ziemlich ratlos. Hat jemand noch weitere Ideen?
Bin für jede Hilfe dankbar.

Freundliche Grüsse
Ein (nicht mehr ganz so blutiger Java-) Anfänger

 

[Harry Kane]

Rufe mal pack() auf dem Hauptfenster auf, nachdem du die Sichtbarkeiten geändert hast.

 

[TheAnfänger]

Rufe mal pack() auf dem Hauptfenster auf, nachdem du die Sichtbarkeiten geändert hast.

Danke! Jetzt funktioniert endlich alles