Panel rapaint

[Stephan2009]

Hallo zusammen, ich habe ein Problem mit der Aktualisierung meines Panels hier mal der Teilcode :

 ImageIcon icon=new ImageIcon(imgList.get(0)); // Verkleinertes Foto
		        final ImageIcon icon_original=new ImageIcon(imgList.get(0)); // Originalgröße
		        icon.setImage(icon.getImage().getScaledInstance(70, 70, Image.SCALE_FAST));     
		        final JLabel label1=new JLabel(icon);
		        
		        label_original=new JLabel();
		        
		        //Icon "verkleinert" auf unteren Panel 
		        
		        panel1.add(label1);

		        // Panel Mitte "original Bild"
		        panel2.add(label_original);
		        
		        // Rahmenfarbe für Icons 
		        Border border = BorderFactory.createLineBorder(Color.BLACK);
		        label1.setBorder(border);
		        
		        //Hinzufügen aller Komponenten auf den Frame
		        this.add(panel1,BorderLayout.SOUTH);
		        this.add(panel2,BorderLayout.CENTER);   
		        this.add(panelnorden,BorderLayout.NORTH);
		       
		        // -------------------------------------------
		        
		        label1.addMouseListener(new MouseListener(){   
		            
		        
		        public void mouseClicked(MouseEvent arg0) {
		            
		            label_original.setIcon(icon_original);  
		            panel2.validate();
		            //app1.validate();
		            
		        }

Nun führe ich mit einem Button eine Suche durch nach Bildern diese werden auch geladen aber ich sehe die Änderungen im Panel nicht , hier mal der Button Aufruf

button1.addActionListener(new ActionListener(){
			    	public void actionPerformed(ActionEvent e) {
		            String text=field.getText();
			    	try {
						photo.suche(text);
						SelectedPhotos photos=new SelectedPhotos();
						
					} catch (Exception e1) {
						
						e1.printStackTrace();
					}
			    	}}
			    		);    
		}

Ich habe schon alles probiert validate, repaint,revalidate des Panels es tut sich nichts wäre glücklich wenn mit jm helfen könnte. Wenn ich auf den unteren Panel auf ein Foto klicke ändert sich das Foto im Center Panel ! Aber das untere Panel wo alle "kleineren Fotos" aufgelistet werden aktualisiert sich nicht.

 

[hdi]

Ist ein bisschen zuwenig Code um verstehen zu können was die Anwendung genau macht.

photo.suche(text);

Was tut das? Im Moment sehe ich nämlich keinerlei Verbindung zwischen dem Klick auf den Button und deinem Panel.

SelectedPhotos photos=new SelectedPhotos();

Sofern der Konstruktor jetzt nicht irgendwelche statischen Werte ändert (was ich doch mal ganz stark nicht hoffe!) ist die Zeile totaler Quatsch.

 

[Stephan2009]

Die Funktion suche(Text) bekommt die Eingabe des Nutzers übergeben und durchsucht damit Bilder die dann in einem Array abgelegt werden, mit dem erzeugen von SelectedPhotos wird die gespeicherte Liste aufgerufen (also die Klasse selbst wie oben im Code und die Bilder in den Labels -> panel gelegt

 

[Stephan2009]

Hier mal deutlicher gezeigt :

public class SelectedPhotos extends JPanel{
			
	       private JLabel label_original;  
		   private SelectedPhotos app1;

		    // Panel1 South, Panel2 Center
		    private JPanel panel1=new JPanel();  
		    private JPanel panel2=new JPanel();
		    
		    
		    private JTextField field;
		    ArrayList<URL> listen=null;
		   
		    private List<URL> imgList = aktuelleListe();
		    
		    public SelectedPhotos() throws Exception {
		    	final FlickrPhoto photo=new FlickrPhoto();
		    	setLayout(new BorderLayout());
		    	app1=this;
		    	
		        // Panel für Knopf zum Suchen
		        JPanel panelnorden=new JPanel(); 
		        JLabel labelleer=new JLabel("                                                                                                                     ");
		        JButton button1=new JButton("Suchen",new ImageIcon("lupe.gif"));
		        field=new JTextField("Sucheingabe...",16);
		        panelnorden.add(labelleer);
		        panelnorden.add(field);
		        panelnorden.add(button1);
		        		        
                            
		        //Flickr Bild 1
		        ImageIcon icon=new ImageIcon(imgList.get(0)); // Verkleinertes Foto
		        final ImageIcon icon_original=new ImageIcon(imgList.get(0)); // Originalgröße
		        icon.setImage(icon.getImage().getScaledInstance(70, 70, Image.SCALE_FAST));     
		        final JLabel label1=new JLabel(icon);
		        
		        label_original=new JLabel();
		        
		        //Icon "verkleinert" auf unteren Panel 
		        
		        panel1.add(label1);

		        // Panel Mitte "original Bild"
		        panel2.add(label_original);
		        
		        // Rahmenfarbe für Icons 
		        Border border = BorderFactory.createLineBorder(Color.BLACK);
		        label1.setBorder(border);
		        
		        //Hinzufügen aller Komponenten auf den Frame
		        this.add(panel1,BorderLayout.SOUTH);
		        this.add(panel2,BorderLayout.CENTER);   
		        this.add(panelnorden,BorderLayout.NORTH);
		       
		        // -------------------------------------------
		        
		        label1.addMouseListener(new MouseListener(){   
		            
		        
		        public void mouseClicked(MouseEvent arg0) {
		            
		            label_original.setIcon(icon_original);  
		            panel2.validate();
		            //app1.validate();
		            
		        }

Und hier die Funktion Suche

public ArrayList<URL> suche(String text) throws Exception{
	    	
			
	    	URLConnection connection = new URL("http://api.flickr.com/services/rest/?method=flickr.photos.search&api_key=a750a873fbd6a892f2965a8601a0a2be&sort=date-posted-asc&per_page=10&tags="+text+"&text="+text+"&extras=description,owner_name,geo").openConnection();
	        DataInputStream dis = new DataInputStream(connection.getInputStream());
	        
	        FileWriter filewriter = new FileWriter(new File("Photos1.xml"));
	        
	        BufferedReader read=new BufferedReader(new InputStreamReader(dis));
	        
	        String nextline="";  
	        
	        String[] servers = new String[100];
	        String[] ids = new String[100];
	        String[] secrets = new String[100];
	        
	        //List<URL> urlList = new ArrayList<URL>();
	        while ((nextline = read.readLine()) != null) { 
	       
	        
	        filewriter.append(nextline);
	     
	        }
	        dis.close();
	        filewriter.close();
	        
	        String filename = "Photos1.xml";
	        XMLInputFactory factory = XMLInputFactory.newInstance();

	        XMLEventReader reader = factory.createXMLEventReader(filename, new FileInputStream(filename));
	        int i = 0;
	        
	        ArrayList<URL> list=new ArrayList<URL>(); 
	        
	        while (reader.hasNext()) {

	            XMLEvent event = reader.nextEvent();
	            if (event.isStartElement()) {
	                StartElement element = (StartElement) event;
	                String elementName = element.getName().toString();
	                if (elementName.equals("photo")) {
	                    i++;
	                   
						
						Iterator<Attribute> iterator = element.getAttributes();
	                    
						
	                    while (iterator.hasNext()) {

	                        Attribute attribute = (Attribute) iterator.next();
	                        QName name = attribute.getName();
	                        String value = attribute.getValue();
	                       
	                        if ((name.toString()).equals("server")) {
	                            servers[i] = value; 
	                      
	                        }
	                        if ((name.toString()).equals("id")) {
	                            ids[i] = value;
	                        }
	                        if ((name.toString()).equals("secret")) {
	                            secrets[i] = value;
	                        }
	                        
	                    }   
	                    String flickrurl="http://static.flickr.com/"+servers[i]+"/"+ids[i]+"_"+secrets[i]+".jpg";
	                    URL u=new URL(flickrurl);
	                    list.add(u);
	                    System.out.println("Diese Liste wurde erzeugt durch Suche!"+u);

	                        
	                    }
	                  }
	               }
	                return list;              		
			}

 

[hdi]

Okay, bevor wir uns weiter deinem ursprünglichen Problem widmen müssen wir da aufräumen. Ich hab das Gefühl du machst da ganz schlimme Dinge Was ich wissen muss:

- von welchem Datentyp ist die Variable "photo"
- wie sieht die Methode photo.suche(String) aus? (Quellcode)
- die Klasse SelectedPhotos
- der Ort, an dem dieses Bilder-Array deklariert ist

edit: Da haben wir gleichzeitig gepostet.. Ich kuck mal

 

[hdi]

Oh je, das ist ziemlich übel.. Aber ich will jetzt nicht über das Design reden, da ist nix mehr zu retten.

Was die Programmlogik angeht, so gibt's folgende Probleme:

1) photo.suche() liefert dir eine Liste an URLs, die du dir aber gar nicht abspeicherst. Ich erkenne sonst auch keinen nachhaltigen Seiteneffekt in der Methode. D.h. der Aufruf tut... NIX.
2) Die in der actionPerformed erzeugte SelectedPhoto-Instanz hat keinerlei Verbindung zu der Variablen "photo", geschweige denn von der Liste an URLs die du da geladen hast (siehe 1))
3) Zudem sehe ich auch keine Verbindung zwischen dieser Instanz und der GUI, d.h. das Panel (ich rede nicht von panel1 oder panel2, sondern von der eigentlichen SelectedPhoto-Instanz, welche ja ein JPanel ist) wird in deinem Fenster gar nicht angezeigt.

 

[Stephan2009]

Wieso die photo.suche schreibt doch die Eingaben in eine Datei Photos1.xml und rufe diese mit :
private List<URL> imgList = aktuelleListe(); hier auf

 

[Stephan2009]

Außerdem ist nach jedem programmstart das Panel aktualisiert nur eben nicht zur Laufzeit

 

[hdi]

Achso ok, das habe ich übersehen. Ist etwas irreführend weil die Methode ja eine URL-List returned. Solche komischen Seiteneffekte sollte man lassen.. Wenn nötig, dann zumindest in eine eigene Methode auslagern.

Egal - ich frag mich dannr wo wird diese Datei nun in deinem SelectedPhotos-Panel verwendet? Also wie kommen die entsprechenden Bilder da rein, das sehe ich nicht.

Außerdem ist nach jedem programmstart das Panel aktualisiert nur eben nicht zur Laufzeit

Lädst du diese Datei denn nur beim Programmstart ein? Dann ist ja klar dass sich da zur Laufzeit nix tun kann. Du solltest zur Laufzeit die Bilder erzeugen, und dem SelectedPhotos-Panel übergeben.

 

[Stephan2009]

Also ich schreibe diese zur Kontrolle immer in eine Datei und gleichzeitig in ein Array in der photo.suche ist dies vorhanden daher gebe ich diese auch als liste zurück. ArrayList<URL> list=new ArrayList<URL>(); ( Zeile 33 )

Nun ist es so das ich mit diesem Aufruf in der SelectedPhotos mir die aktuelle Liste die zuletzt gespeichert wurde von den Fotos hole hier : private List<URL> imgList = aktuelleListe(); ( ZEILE 14 ) und speicher diese in einer neuen Liste greife dann mit imglist.get(0) usw auf die einzelnen Fotos zu und speicher diese in den Labels

 

[hdi]

Achsoooooo, ja ne ist klar...

Sorry, sowas musst du mir erklären. Das kommt halt davon wenn man solchen Spaghetti-Code schreibt :bae:

Gut, dann bleibt aus meiner Sicht noch immer das Problem, dass dein SelectedPhotos-Panel gar nicht Bestandteil deiner GUI ist. Oder passiert das auch irgendwo versteckt im Konstruktor oder bei der Variablen-Initialisierung? Weil ich seh davon nix.

 

[Stephan2009]

Das SelectedPhotos Panel ist Bestandteil eine JTabbedPanes die in der Gui gestartet wird

 

[hdi]

Ich glaube dir ist nicht ganz klar dass:

new SelectedPhotos();
new SelectedPhotos();

zwei unterschiedliche Instanzen erzeugt. Wenn du Instanz 1 in dein TabbedPane steckst, hat das mit Instanz 2 nix zu tun. Und beim Klick auf deinen Button erzeugst du nun mal ein neues.. Das muss dann also in deiner GUI (TabbedPane) das alte ersetzen (austauschen), sonst zeigst du noch immer das alte an.

Deswegen meinte ich anfangs schon, dass diese Zeile Quatsch ist. Weil du das neu erzeugte Panel gar nicht abspeicherst, d.h. gar nicht verwendest.

 

[Stephan2009]

achsooo ok ich dachte man könnte das alte Panel einfach überschreiben oder neuzeichnen weil das mit dem validate von dem MouseListener ja auch zu Laufzeit funktioniert hatte dachte ich das würde so auch gehen

 

[hdi]

Klar kannst du das machen, und so solltest du es auch machen, aber so machst du es nicht.

Hier mal ein kleiner Design-Tipp:
Du solltest nur ein einziges SelectedPhotos-Panel haben. D.h. in deinem gesamten Programmcode nur genau 1x new SelectedPhotos() aufrufen.

Dieses Panel steckst du in deine TabbedPane, und dieses Panel machst innerhalb der Klasse mit der actionPerformed-Methode (Button-Klick) bekannt. D.h. du legst dir dafür eine Variable an und speicherst das dort ab.

Und dann erzeugst du beim klick kein neues, sondern du änderst nur die Bilder (zB über eine Methode) und rufst dann ein repaint() auf dem Panel auf.

PS: und diese ganze validate-Sache kannst du aus dem Code rausnehmen!

 

[Stephan2009]

So ist es ja bis jetzt auch ich rufe das SelectedPhotos nur 1x auf in der Gui übergebe ich das dem TabbedPane. Was meinst du jetzt genau mit abspeichern in einer Variable beim actionPerformed? Vielen Dank für deine Hilfe schonmal

 

[hdi]

Nein du erzeugst bei jedem Klick auf den Button ein neues Panel:

button1.addActionListener(new ActionListener(){
                    public void actionPerformed(ActionEvent e) {
                    String text=field.getText();
                    try {
                        photo.suche(text);
                        SelectedPhotos photos=new SelectedPhotos(); // <------------- !!!!
                        
                    } catch (Exception e1) {
                        
                        e1.printStackTrace();
                    }
                    }}
                        );    
        }

Dieses neue Panel ist nicht das, welches du irgendwann beim Programmstart in dein TabbedPane gesteckt hast.

Aber genau das solltest du auch hier beim Klick auf den Button verwenden. Und dafür brauchst du eine Variable. Wo auch immer dieser Code-Ausschnitt steht, genau in diese Klasse musst du das Panel, welches in der TabbedPane steckt, hineinreichen und es dir in einer Variablen abspeichern, um dann in der actionPerformed darauf zugreifen zu können:

class X{

     private SelectedPhotos selectedPhotosPanel;

     public class X(SelectedPhotos selectedPhotosPanel){
          this. selectedPhotosPanel = selectedPhotosPanel;
     }

     // ...

     button1.addActionListener(new ActionListener(){
                    public void actionPerformed(ActionEvent e) {
                    String text=field.getText();
                    try {
                        ArrayList<URL> photos = photo.suche(text);
                        selectedPhotosPanel.setImgList(photos); // imgList-Referenz updaten!
                        selectedPhotosPanel.repaint();
                    } catch (Exception e1) {
                        
                        e1.printStackTrace();
                    }
                    }}
                        );    
        }
}

Und bitte formatier mal deinen Code, sodass alle Klammern usw richtig eingrückt sind. Ist echt schwer lesbar so.

 

[Stephan2009]

achsoooooooooo Prima super hast mir sehr geholfen werds versuchen

 

[hdi]

Wenn du dann keine weiteren Fragen mehr hast, bitte markier das Thema als erledigt über den entsprechenden Button hier. Und über ein Klick auf den "Danke"-Button würd ich mich auch freuen