Zufallsbilder laden

[I.E.]

Moin Leute,

ich habe hier ein kleines Programm mit einem Panel für Bilder, und einem Button unten, der beim raufdrücken eins von drei Bildern auf das Panel laden soll. Er soll, tut er aber nicht.

Wenn ich direkt im GUI-Code eins der Bilder an das Bilder-Panel anknüpfe, wird es mir auch angezeigt. Wenn ich auch den Zufallsgenerator in der GUI habe, wird mir bei jedem neuen Aufmachen des Programms die App mit einem zufälligen Bild angezeigt.

Wenn ich den Zufallsgenerator in den Action-Listener-Code reinstecke (so wie im Code unten), kommt da gar nichts. Wo ist der Fehler? Ich glaube, hier liegt kein Logikfehler vor (mit anderen Objekten funktioniert dieser Code prima), sondern um einen in diesem Fall falschen Code zum Herausholen von externen Bildern. Wie mache ich das richtig?

import javax.swing.*; 
import java.util.*;
import java.awt.*;
import java.awt.event.*;
public class randomimage {


public static void main(String[] args) {
	randomimage ri = new randomimage(); 
	ri.gui(); 					
	}//main()

JFrame Rahmen; 
JPanel imageP; 
JButton button;
ArrayList<JLabel> picList; 

void gui(){
Rahmen = new JFrame(" random image app");	 	
 	
imageP = new JPanel(); 
imageP.setBackground(Color.red); 	
button = new JButton("get a new random image!"); 	
button.setPreferredSize(new Dimension(100,100)); 
button.addActionListener(new buttonListener()); 

picList = new ArrayList<JLabel>();
JLabel pic0 = new JLabel(new ImageIcon("C:\\cl0.png"));
picList.add(pic0); 
JLabel pic1 = new JLabel(new ImageIcon("C:\\cl1.png")); 
picList.add(pic1); 
JLabel pic2 = new JLabel(new ImageIcon("C:\\cl2.png"));
picList.add(pic2); 

Rahmen.getContentPane().add(BorderLayout.CENTER, imageP); 
Rahmen.getContentPane().add(BorderLayout.SOUTH, button); 

Rahmen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
Rahmen.setSize(300,450); 
Rahmen.setVisible(true);
}//gui()


public class buttonListener implements ActionListener{
	public void actionPerformed(ActionEvent e){
	
	int rand = (int)(Math.random()*picList.size()); 
	JLabel randPic = picList.get(rand);  

	imageP.add(randPic); 
	Rahmen.repaint(); 
	
}}//buttonListener






}//class randomimage

 

[Foxei]

Hi I.E,
Vorneweg muss ich dir sagen das du einige Programmier Standards ignorierst.
Erstens Initialisiert man nur die Objekte als Atribute die man auch später nochmal aufrufen möchte. (Bei dir nur das Array und das Panel) Und diese sollten auch bitte mit Private, Public, Static etc... bezeichnet werden das Dient erstens der Übersicht und zweitens kann das später zu Problemen führen das nicht zu tun..
Und zweitens sollte man Objekte zu ende erzeugen bevor man ein neues erzeugt. Du hast eine Frame erzeugt und dann alles andere bevor du die Größe etc der Frame gesetzt hast, dies hat zwar keine Auswirkungen auf das Programm aber wenn du später eine mal große Programme schreibst könnte das sehr unübersichtlich werden.

So nun zu deinem Problem ich habe dazu einen etwas anderen Lösungsansatz gewählt: Warum denn die JLabel speichern wenn man auch einfach die 3 Bilderspeichern kann und diese dann abwechselnd auf ein JLabel bringt. Bedeutet dein Array Speichert jetzt ImageIcons und dein Panel wird ein Label.

Nun zu deiner Random Funktion: Hier muss jetzt nur basierend auf deiner Nummer das bild aus dem Array gelesen werden und als Icon auf das Label gesetzt werden. Ergenzent habe ich die Funktion noch ergänzt das nach jedem Bild auf wirklich ein anderes kommt und nicht 10 mal Bild 1

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;

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

public class RandomImage {

	public static void main(String[] args) {
		try {
			UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
			RandomImage ri = new RandomImage();
			ri.gui();
		} catch (Exception e) {
			e.printStackTrace();
		}
	}

	private JLabel panelImage;
	private ArrayList<ImageIcon> picList;
	private int lastRandom=-1;
	void gui() {
		picList = new ArrayList<ImageIcon>();
		picList.add(new ImageIcon(getClass().getResource("./images/file_odt.png")));
		picList.add(new ImageIcon(getClass().getResource("./images/file_word.png")));
		picList.add(new ImageIcon(getClass().getResource("./images/file_pdf.png")));

		JFrame frame = new JFrame(" random image app");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setSize(300, 400);
		frame.setLocationRelativeTo(null);
		
		panelImage = new JLabel(){ //paintComponent überschrieben um dafür zu sogern das das Bild auf dem ganzen JLabel erscheint nicht nur in der größe der Bilddatei
			@Override
			protected void paintComponent(Graphics g) {
				if(getIcon()!=null){
					Graphics2D g2d=(Graphics2D) g;
					g2d.setRenderingHint(RenderingHints.KEY_RENDERING, RenderingHints.VALUE_RENDER_QUALITY);
					g2d.drawImage(((ImageIcon)getIcon()).getImage(), 0, 0, getWidth(),getHeight(),this);
				}
				
			}
		};
		
		JButton btnRandom = new JButton("get a new random image!");
		btnRandom.addActionListener(new buttonListener());
		btnRandom.doClick();
		
		frame.add(BorderLayout.CENTER, panelImage);
		frame.add(BorderLayout.SOUTH, btnRandom);
		frame.setVisible(true);
	}

	public class buttonListener implements ActionListener {
		public void actionPerformed(ActionEvent e) {		
			int rand;
			do{ //Schleife verhindert das ein Bilder mehrmals angezeigt werden so wird nach jedem klick ein neues angezeigt
				rand = (int) (Math.random() * picList.size());
			}while(rand==lastRandom);
			lastRandom=rand;			
			panelImage.setIcon(picList.get(rand));
		}
	}
}

 

[I.E.]

Danke!

Erstens - den ganzen private/public/static-Müll habe ich bislang erfolgreich ignoriert.

Das mit dem Frame habe ich mir irgendwie so angewöhnt. Ich glaube, in dem Buch "Java von Kopf bis Fuß" machen die das auch so. Oder auch nicht.

Jetzt habe ich die "setIcon"-Methode verwendet, und habe auch deinen Vorschlag beherzigt, nur einen Label, und dafür 3 Icons zu verwenden.

Das Programm funktioniert jetzt! Danke dir vielmals!

Allerdings passiert es in meinem Code, dass der Zufallsgenerator mehrmals hintereinander dasselbe Bild ausspuckt (eben das, wovor du eindringlich gewarnt hast). Ich habe schon mal in einem ähnlichen Programm selbst einen Code geschrieben, um solche Wiederholungen zu vermeiden, allerdings ohne die while-Schleife. Die habe ich bislang ebenfalls mit Erfolg ignoriert, kann sein, dass ich sie neulich verwendet habe, und dann aber auf if/for umgestiegen bin. Wird nun langsam Zeit, ihre Vorzüge in Angriff zu nehmen.

Hier ist mein neuer Code:

import javax.imageio.ImageIO;
import javax.swing.*; 
import java.util.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
public class randomimage {


public static void main(String[] args) {
	randomimage ri = new randomimage(); 
	ri.gui(); 					
	}//main()

JFrame Rahmen; 
JPanel imageP; 
JButton button;
JLabel label; 
Icon cl0, cl1, cl2; 
ArrayList<Icon> picList; 

void gui(){
Rahmen = new JFrame(" random image app");	 	
picList = new ArrayList<Icon>(); 
 	
imageP = new JPanel(); 
imageP.setBackground(Color.red); 	
button = new JButton("get a new random image!"); 	
button.setPreferredSize(new Dimension(100,100)); 
button.addActionListener(new buttonListener()); 

label = new JLabel(); imageP.add(label); 

cl0 = new ImageIcon("cl0.png"); 	
picList.add(cl0); 
cl1 = new ImageIcon("cl1.png"); 	
picList.add(cl1); 
cl2 = new ImageIcon("cl2.png"); 	
picList.add(cl2); 


Rahmen.getContentPane().add(BorderLayout.CENTER, imageP); 
Rahmen.getContentPane().add(BorderLayout.SOUTH, button); 

Rahmen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
Rahmen.setSize(300,450); 
Rahmen.setVisible(true);
}//gui()


public class buttonListener implements ActionListener{
	public void actionPerformed(ActionEvent e){
	
		int rand = (int)(Math.random()*picList.size()); 
		Icon randIcon = picList.get(rand); 
		label.setIcon(randIcon); 
			
}}//buttonListener






}//class randomimage

Jetzt fehlt nur noch meine while-Feuertaufe a la foxei, und dann ist der upgrade im Kasten!