Swing Bilder anzeigen in Gridlayout

[javapklein]

Ich möchte mit der Anwendung 24 Bilder in ein Gridlayout 4x6 einfügen.
Leider wird nichts auf dem Borderlayout.CENTER angezeigt. Die Listausgabe der Dateien erfolgt korrekt.
Wo liegt das Problem?
Bei der Anzeige der 3 Buttons funktioniert die Anzeige richtig.
Ich habe bisher nicht einen einfachen Code gefunden, wo dies so dargestellt wird.
Ich hoffe mit Eurer Hilfe das Problem zu lösen.

public class Simcity1 extends JFrame {
	private JButton buttonNeu, buttonSpeichern, buttonLaden;
	private JPanel panelButton, panelAnzeige;
	private int ausgabeZaehler = 0;	
	public Simcity1(){
		panelButton = new JPanel(new GridLayout(1,3,25,1));
		panelAnzeige = new JPanel(new GridLayout(4,6));
		setTitle("           Simcity           ");	
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(200, 100, 900, 800);		
		buttonNeu = new JButton("Neu");
		add(buttonNeu);
		buttonSpeichern = new JButton("Speichern");
		add(buttonSpeichern);
		buttonLaden = new JButton("Laden");
		add(buttonLaden);
		panelButton.add(buttonNeu);
        panelButton.add(buttonSpeichern);
        panelButton.add(buttonLaden);
        getContentPane().add(BorderLayout.NORTH, panelButton);
       
		setVisible(true);	
		buttonNeu.addActionListener(new ActionListener() {              
            public void actionPerformed(ActionEvent e) {
                   gebaeudeAnzeigen();  
            }
        });			
	}
	public void gebaeudeAnzeigen() {
		File dir = new File("images128");
				
		for (File file : dir.listFiles()) {
			String name = file.getName();
			System.out.println(name);
			ImageIcon icon = new ImageIcon(name);	
			JLabel jicon = new JLabel(icon);
			panelAnzeige.add(jicon);			
		}
		getContentPane().add(BorderLayout.CENTER, panelAnzeige);		
	}	

	public static void main(String[] args) {
		Simcity1 sc = new Simcity1();	
	}

 

[SlaterB]

eine GUI zur Laufzeit zu ändern ist nicht üblich, nicht so leicht,
verteile lieber die JLabel und das Center-Panel schon am Anfang und setze dann dort die Icons beizeiten

ansonsten brauchst du validate() zur Erneuerung des Layouts:

    public void gebaeudeAnzeigen()
    {
        for (int i = 0; i < 15; i++)
        {
            panelAnzeige.add(new JLabel("" + i));
        }
        getContentPane().add(BorderLayout.CENTER, panelAnzeige);
        validate();
    }

 

[javapklein]

Das habe ich überhaupt nicht verstanden.
validate() habe ich eingebaut, aber es wird immer noch nichts angezeigt.
Was soll die Schleife < 15??
Wo muss ich die Bilder einfügen - das ist mir immer noch nicht verständlich?

 

[SlaterB]

gehen die Bilder grundsätzlich? versuche zunächst EIN Bild zu laden und anzuzeigen, und zwar gleich am Anfang, nicht erst per Button,
danach deine Schleife, auch am Anfang, nicht erst per Button

meine Schleife war nur ne Alternative, da ich das mit den Bildern verständlicherweise nicht so leicht testen konnte,

Bilder gehören in JLabel, wie du es ja auch hast, wenn es die JLabel schon gibt wird bestimmt irgendeine set-Methode helfen,
andere Dinge wie das Merken der JLabel gehört auch dazu, das war nur eine allgemeine Richtung, zu der dann auch
Stunden neues Denken + Programmieren + Nachschlagen usw. gehört, falls zu verfolgen

 

[jgh]

du musst natürlich dir auch die Bilder aus dem richtigen Pfad holen....und dann diesen auch übergeben

ImageIcon icon = new ImageIcon("images128/" + name);

...so sollte es klappen, wahrscheinlich

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class Simcity1 extends JFrame {
	private JButton buttonNeu, buttonSpeichern, buttonLaden;
	private JPanel panelButton, panelAnzeige;
	private int ausgabeZaehler = 0;

	public Simcity1() {
		panelButton = new JPanel(new GridLayout(1, 3, 25, 1));
		panelAnzeige = new JPanel(new GridLayout(4, 6));
		setTitle("           Simcity           ");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(200, 100, 900, 800);
		buttonNeu = new JButton("Neu");
		add(buttonNeu);
		buttonSpeichern = new JButton("Speichern");
		add(buttonSpeichern);
		buttonLaden = new JButton("Laden");
		add(buttonLaden);
		panelButton.add(buttonNeu);
		panelButton.add(buttonSpeichern);
		panelButton.add(buttonLaden);
		getContentPane().add(BorderLayout.NORTH, panelButton);

		setVisible(true);
		buttonNeu.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				gebaeudeAnzeigen();
			}
		});
	}

	public void gebaeudeAnzeigen() {
		File dir = new File("images128");

		for (File file : dir.listFiles()) {
			String name = file.getName();
			System.out.println(name);
			ImageIcon icon = new ImageIcon("images128/" + name);
			JLabel jicon = new JLabel(icon);
			panelAnzeige.add(jicon);
		}
		getContentPane().add(BorderLayout.CENTER, panelAnzeige);
		validate();
	}

	public static void main(String[] args) {
		Simcity1 sc = new Simcity1();
	}
}

[edit]das laden der Bilder passiert nun im EDT => das führt dazu, dass die GUI während dessen einfriert, so sollte man es eigentlich nicht machen. Lies dir die Tipps von slater dazu am besten nochmal durch. Schlechtes Bsp, aber es funzt^^[/edit]

 

[javapklein]

Es wird kein Bild angezeigt. Ich habe das direkt am Programmanfang eingebaut und Button ausgebaut.
Wo steckt das Problem??

 

[javapklein]

War erst mit testen und antworten dran, sodass ich die eine antwort nicht mitbekommen habe.
Es werden jetzt die Bilder angezeigt.
Ich weiß jedoch nicht was GUI friert ein bedeuten soll.
Ich kann erst die Bilder anzeigen, wenn der Button neu gedrückt wird.
Wie soll ich das den sonst handhaben?

 

[jgh]

bspw. indem du dir eine Methode machst [c]initGebauedeAnzeigen[/c] in der du deine jetztigen Sachen halt auslagerst...wird dann der Button gedrückt, fügst du das Panel nur noch hinzu...mal so auf die schnelle hingeklatscht...

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class Simcity1 extends JFrame {
	private JButton buttonNeu, buttonSpeichern, buttonLaden;
	private JPanel panelButton, panelAnzeige;
	private int ausgabeZaehler = 0;

	public Simcity1() {
		panelButton = new JPanel(new GridLayout(1, 3, 25, 1));
		panelAnzeige = new JPanel(new GridLayout(4, 6));
		setTitle("           Simcity           ");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(200, 100, 900, 800);
		buttonNeu = new JButton("Neu");
		add(buttonNeu);
		buttonSpeichern = new JButton("Speichern");
		add(buttonSpeichern);
		buttonLaden = new JButton("Laden");
		add(buttonLaden);
		panelButton.add(buttonNeu);
		panelButton.add(buttonSpeichern);
		panelButton.add(buttonLaden);
		getContentPane().add(BorderLayout.NORTH, panelButton);
		initGebauedeAnzeigen();
		setVisible(true);
		buttonNeu.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				gebaeudeAnzeigen();
			}
		});
	}

	private void initGebauedeAnzeigen() {
		File dir = new File("Bilder");
		for (File file : dir.listFiles()) {
			String name = file.getName();
			System.out.println(name);
			ImageIcon icon = new ImageIcon("Bilder/" + name);
			JLabel jicon = new JLabel(icon);
			panelAnzeige.add(jicon);
		}
	}

	public void gebaeudeAnzeigen() {
		getContentPane().add(BorderLayout.CENTER, panelAnzeige);
		validate();
	}

	public static void main(String[] args) {
		Simcity1 sc = new Simcity1();
	}
}

btw Einfrieren bedeutet im Kontext von GUI´s dass die GUI sich während des -in deinem Fall- Ladevorgangens der Bilder nicht bedienen lässt, sie friert ein...zu sehen daran, dass z.B. der Button gedrückt bleibt, bevor es weitergeht

 

[javapklein]

Das habe ich jetzt verstanden. Von der Logik her macht es aber keinen Sinn einen initGebaeudeAnzeigen schon zu benutzen, ohne dass der Button Neu gedrückt ist. Wenn ich z.B. Laden drücke, dann benötige ich initGebaeudeAnzeigen ja gar nicht. Wenn ich große Dateien hätte, würde das Programm dann ja eingefroren sein. Anzeigen macht für mich daher nur Sinn, wenn ich auch auf Neu gedrückt habe.
Ich hoffe, ich habe da keinen Denkfehler gemacht.
Wenn ja, bitte korrigieren.
Eine andere Frage, wie bekomme ich in das Gridlayout Leerzeichen. Im Moment habe ich 24 Bilder. Wenn ich jedoch nur 12 habe und diese verteilen möchte. In der Form, das ein Bild und dann leer und wieder Bild usw. erscheint. Oder auch 1 Bild 2 x leer dann 1 Bild 1 x lerr 1 Bild oder noch ganz anders.

 

[jgh]

da missverstehen wir uns...du sollst ja die Bilder nicht anzeigen, sondern das Panel vorbereiten und die Bilder in vorraus laden, damit wenn auf den entsprechende Button gedrückt wird, die Bilder angezeigt werden können...aber für den Anfang kann man sicherlich auch mit deiner Idee leben.

Willst du einen Leerraum haben, musst du dir eine Idee einfallen lassen, wie man das lösen könnte. Wenn jedes zweite nur angezeigt werden soll, bspw so:

File dir = new File("Bilder");
		boolean jedesZweiteBild = false;
		for (File file : dir.listFiles()) {
			String name = file.getName();
			System.out.println(name);
			ImageIcon icon = new ImageIcon("Bilder/" + name);
			JLabel jicon = new JLabel(icon);
			panelAnzeige.add(jicon);
			if (jedesZweiteBild) {
				panelAnzeige.add(new JPanel());
			}
			jedesZweiteBild = !jedesZweiteBild;
		}

 

[javapklein]

Soweit funktioniert die Erstellung mit dem Neu-Button. Es wird auch eine Datei mit Speichern-Button erstellt. Die Datei wird auch ohne Fehler geladen über den Laden-Button jedoch wird das JPanel aber nicht angezeigt. Ich habe jetzt schon alles möglich ausprobiert, aber jedoch ohne Erfolg. Vielleicht kann mir ja jemand auf die Sprünge helfen, woran das liegt. Hier mein Code:

public class Simcity1 extends JFrame {
	private JButton buttonNeu, buttonSpeichern, buttonLaden;
	private JPanel panelButton, panelAnzeige;
	private int[] bilder; 
	private JTextArea ta = new JTextArea();
	public Simcity1(){
		panelButton = new JPanel(new GridLayout(1,3,30,1));		
		setTitle("           Simcity           ");	
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(10, 5, 1000, 700);		
		buttonNeu = new JButton("Neu");
		add(buttonNeu);
		buttonSpeichern = new JButton("Speichern");
		add(buttonSpeichern);
		buttonLaden = new JButton("Laden");
		add(buttonLaden);
		panelButton.add(buttonNeu);
        panelButton.add(buttonSpeichern);
        panelButton.add(buttonLaden);
        getContentPane().add(BorderLayout.NORTH, panelButton);            
		setVisible(true);
		ta = new JTextArea();
		
		buttonNeu.addActionListener(new ActionListener() {              
            public void actionPerformed(ActionEvent e) {
                   gebaeudeAnzeigen();  
            }
        });	
		buttonSpeichern.addActionListener(new ActionListener() {              
            public void actionPerformed(ActionEvent e) {
                   gebaeudeSpeichern();  
            }
        });	
		buttonLaden.addActionListener(new ActionListener() {              
            public void actionPerformed(ActionEvent e) {
                   gebaeudeLaden();  
            }
        });	
		if (ta != null)
			add(ta);
	}
	public void gebaeudeAnzeigen() {
		File dir = new File("images128");
		int count = 0;			
		panelAnzeige = new JPanel(new GridLayout(4,6,1,1));
		bilder = new int[24];
		vorbelegen(bilder);
						
		while (true) {		
			for (File file : dir.listFiles()) {
				if (file.isFile()) {
					String name = file.getName();
					ImageIcon icon = new ImageIcon("images128/" + name);	
					JLabel jicon = new JLabel(icon);
					if (bilder[count] == 0)
						panelAnzeige.add(new JPanel());
					else
						panelAnzeige.add(jicon);
					count++;
					if (count == 24) break;
				}
			}
			if (count == 24) break;	
		}
		getContentPane().add(BorderLayout.CENTER, panelAnzeige);
		validate();
		}	
	public void vorbelegen(int[] bilder) {
		Random rm = new Random();
		int zzahl = rm.nextInt(23) + 1; 
		int zzahl1 = zzahl;
		for (;;) {
			if (zzahl1 == 0)
				break;
			else { 
				zzahl = rm.nextInt(23) + 1;
				if (bilder[zzahl] == 0) {
					bilder[zzahl] = 1;
					zzahl1--;
				}				
			}				
		}			
	}
	public void gebaeudeSpeichern() {
		if (panelAnzeige != null) {			
			try {
				ObjectOutputStream stream = new ObjectOutputStream(new FileOutputStream("simcity.stadt"));
				stream.writeObject(panelAnzeige);
				stream.close();
				ta.setText("Speichern von simcity.stadt erfolgreich.");
				
			} catch (Exception e) {
				ta.setText("Speichern nicht möglich. " + e.getMessage());			
			}
		}
		else
			ta.setText("Kein Inhalt, daher speichern nicht möglich.");	
	}	
	
	public void gebaeudeLaden() {
		try {
			ObjectInputStream stream = new ObjectInputStream(new FileInputStream("simcity.stadt"));
			panelAnzeige = (JPanel)stream.readObject();
			stream.close();
			getContentPane().add(BorderLayout.CENTER, panelAnzeige);
			validate();
		}
		catch (Exception e) {
			ta.setText("Laden nicht möglich. " + e.getMessage());
			}
	}				

	public static void main(String[] args) {
		Simcity1 sc = new Simcity1();	
	}
}

 

[javapklein]

Das mit dem JPanel funktioniert nicht irgendwie überhaupt nicht und eine grosse Datei wird erzeugt. Ich habe dies jetzt anders gelöst mit einer kleinen Tabelle (24), wo ich den Namen des Bilder oder Leer schreibe. Beim Laden suche ich mir die entsprechenden Bilder wieder und baue das JPanel neu auf und jetzt geht es schnell und korrekt.
Vielen Dank für die Unterstützung.