Swing Swing JPanel und Deserialisierung

[Jazz]

Hallo zusammen,
im Moment stehe ich seit mehreren Tagen an einem Problem.
In meinem Programm gibt es eine ArrayList<Activities> mit mehreren JPanel Objekten darin und Activities is t ein Panel mit mehreren Komponenten drauf.
Bei Einlesen wie die ArrayList wiederhergestellt und in einem JPanel eines Frames mit add hinzugefügt und werden auch angezeigt.

Wenn ich anschließend das JPanel löschen will oder findet er nichts. Er gibt mit mit ComponetnCount 0 zurück und so gann ich auch kein remove(int i) machen.


Hat jeman eine Idee???

 

[CroniD]

Ich weiß nicht, ob dir dieses fixe Codebeispiel als Hilfe reicht, aber schau mal drauf.

public class ArrayListToJPanel extends JFrame {

	ArrayList<JPanel> alPanel;
	
	public ArrayListToJPanel() {
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setLayout(new GridLayout(0, 1));
		setLocationRelativeTo(null);
		init();
		pack();
	}
	
	private void init() {
		JPanel pane;
		JButton button;
		alPanel = new ArrayList<JPanel>(5);
		
		// erzeugen und einlesen in die array list
		for (int i = 0; i < 5; i++) {
			pane = getPanelWithButtons(i);
			alPanel.add(pane);
		}
		// Elemente der array list auf das frame packen
		for (JPanel p : alPanel) {
			getContentPane().add(p);
		}
		
		// extra panel mit einem 'remove' und 'add' button
		JButton remove = new JButton("delete last");
		remove.addActionListener(new ActionListener() {
			@Override public void actionPerformed(ActionEvent e) {
				System.out.println(getContentPane().getComponentCount());
				// das letzte panel nach dem control panel loeschen
				int position = getContentPane().getComponentCount()-2;
				getContentPane().remove(position);
				// da das frame schon sichtbar -> validieren
				getContentPane().validate();
				// weniger komponenten nun, also soll sich das frame auch anpassen
				// in seiner groesse
				pack();
				System.out.println(getContentPane().getComponentCount());
				// evtl. muesste man noch die array list updaten, aber das spare
				// ich mir mal
				// allerding muss eine schranke eingebaut werden, falls nichts zu
				// loeschendes mehr da ist, sonst regelt es
				// 'ArrayIndexOutOfBoundsException's ;)
			}
		});
		JButton add = new JButton("add last");
		add.addActionListener(new ActionListener() {
			@Override public void actionPerformed(ActionEvent e) {
				System.out.println(getContentPane().getComponentCount());
				// die position an der das panel nun eingesetzt werden soll
				// aber ueber die controls
				int prefix = getContentPane().getComponentCount()-1;
				getContentPane().add(getPanelWithButtons(prefix), prefix);
				// da das frame schon sichtbar -> validieren
				getContentPane().validate();
				// mehr komponenten nun, also soll sich das frame auch anpassen
				// in seiner groesse
				pack();
				System.out.println(getContentPane().getComponentCount());
				// evtl. muesste man noch die array list updaten, aber das spare
				// ich mir mal
			}
		});
		JPanel control = new JPanel();
		control.add(add);
		control.add(remove);
		getContentPane().add(control);
	}
	
	private JPanel getPanelWithButtons(int prefix) {
		JPanel pane = new JPanel(new FlowLayout(FlowLayout.LEFT, 10, 10));
		pane.setBorder(BorderFactory.createLineBorder(Color.pink));
		JButton button;
		for (int j = 0; j < 4; j++) {
			button = new JButton(String.valueOf(prefix)+String.valueOf(j));
			pane.add(button);
		}
		return pane;
	}
	
	public static void main(String[] args) {
		ArrayListToJPanel altjp = new ArrayListToJPanel();
		altjp.setVisible(true);
	}
}

Vorweg kann ich dir aber sagen, dass

getComponentCount()

auf dem richtigen Panel angewendet werden muss. Ich glaube da liegt auch dein Problem.
Und nicht vergessen den OberContainer nach jedem Lösch-/Hinzufügenvorgang neu zu validieren und die Größe anzupassen.

 

[Jazz]

Hallo und danke für deine fixe Antwort.

Prinzipielle ist alles klar und bei mir geht alles vor der serialisierung ich kann dan über einen plus button diese panels hinzufügen und entfernen.

beim serialisieren schreibe ich eben nur die arraylist und lese diese wieder anschließend adde ich alle arraylist componenten zu dem richtigen Panel. Trotzdem kann er diese nicht entfernen oder zählen.
füge ich dann mit plus 1 hinzu dann kann ich dieses entfernen und zählen (also 1) aber die anderen eben nicht

ich versuchsmal noch mit getContentPane() statt this.pnlMid

 

[CroniD]

Gib mal bitte Code her.

Deine Beschreibung klingt für mich etwas spanisch.

 

[Jazz]

Gern aber er ist lang. Versuche das unwichtige herauszukürzen:

Klasse Swing:

import javax.swing.*;

import java.awt.*;
import java.awt.event.*;
import java.io.File;
import java.io.FileInputStream;
import java.io.ObjectInputStream;
import java.util.*;


public class Swing extends JFrame 
					implements  WindowListener, ComponentListener {
	/**
	 * Serial UID
	 */
	//private static final long serialVersionUID = 666666;
	/**
	 * Variablen
	 */
	
	public ArrayList<Activities> reports = new ArrayList<Activities>();
	private int breite = 510;
	private int hoehe = 500;
	public String user;
	public String password;
	public final String url = "http://bliblablubb";
	public int endwahl = 1;
	
		
	
	/**
	 * Grundkomponenten fuer Oberfläche
	 */
     ......
	public JPanel pnlTop = new JPanel();
		private JButton btnPlus = new JButton("+");
	private JButton btnSend = new JButton("Senden");
	
	public JPanel pnlMid = new JPanel();
	public JScrollPane scrPnlMid = new JScrollPane(pnlMid, JScrollPane.VERTICAL_SCROLLBAR_ALWAYS, JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
	
	private JPanel pnlBottomL = new JPanel (new FlowLayout(FlowLayout.LEFT));
	private JPanel pnlBottomR = new JPanel(new FlowLayout(FlowLayout.RIGHT));
	private	JPanel pnlBottom = new JPanel(new BorderLayout());
	
	


	public Swing(String u, String p, Boolean online){
		/**
		 * Grundeinstellungen fuer Oberflaeche
		 */
		
	
		super("Blabla");
		
		this.setSize(breite, hoehe);
		this.setLayout(new BorderLayout());
		this.setMinimumSize(new Dimension(breite, hoehe));
		
	
		
		
		
		
	

	...
		this.add(pnlTop, BorderLayout.NORTH);
		pnlMid.setLayout( new BoxLayout(pnlMid, BoxLayout.Y_AXIS) );
		
		
		getSerialization();
		
		
		
		
		
		this.add(scrPnlMid, BorderLayout.CENTER);
		pnlBottomL.add(btnPlus);
	
		pnlBottomR.add(btnSend);
		pnlBottomR.add(Box.createRigidArea(new Dimension(5,5)));
		
		pnlBottom.add(pnlBottomL,BorderLayout.WEST);
		pnlBottom.add(pnlBottomR, BorderLayout.EAST);
		
		this.add(pnlBottom, BorderLayout.SOUTH);
		
		...
		
		/**
		 * ActionListener hinzufügen für Buttons
		 */
		
		btnPlus.addActionListener(new PlusButtonListener(this));
		btnSend.addActionListener(new SendButtonListener(this));
		
		
		
		
	
	
	

	
	....

	
	
	public void getSerialization(){
		File dateiTest = new File("TimeTracker.ser");
		
		if(dateiTest.exists())
		{
			try { 
				ObjectInputStream is = new ObjectInputStream(new FileInputStream("TimeTracker.ser"));
				
	
					
					reports = (ArrayList<Activities>) is.readObject();
					
					for (int i = 0; i < reports.size(); i++) {
						pnlMid.add(reports.get(i));
					}
					
			
				is.close();  
			} catch (Exception ex){
				ex.printStackTrace();
			}
		dateiTest.delete();	
		} else {
				
			reports.add(new Activities(this.getWidth()/12, this));
			pnlMid.add(reports.get(reports.size()-1));
		}
		
	}
	
	
}

Beim schliessen des frames wird dann die serialisierung folgend durchgeführt:

try {
						System.out.println("Save");
	    			 	FileOutputStream fs = new FileOutputStream("TimeTracker.ser");
						ObjectOutputStream os = new ObjectOutputStream(fs);
						os.writeObject(swing.reports);
						os.close();
						
						
				} catch (Exception ex){
						ex.printStackTrace();
				}

Wie gesagt die Anzeige funktioniert, die ArrayList reports hat auch alle Objekte drin die sich auslesen lassen. Nur beim hinzufügen zum Panel kann auf diesem Panel eben kein Objekt gelöscht werden. Selbst ein removeAll() macht gar nix. Natürlich wird repaint() und validate() danach aufgerufen.
Danke.

 

[CroniD]

Ja, okay. Schön das du unwichtiges rausgelassen hast, aber ich glaube du hast auch das wichtigste weggelassen. Nämlich die Stelle, wo du was vom Panel löschen willst. :-/

 

[Jazz]

Hi das ist ja in einer extra Klasse. hier im Anhang

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.Serializable;


public class MinusButtonListener 
					implements ActionListener, Serializable {
	
	/**
	 * 
	 */
	private static final long serialVersionUID = 555555;
	private Swing swing;
	private Activities activity;
	public MinusButtonListener(Swing gui, Activities act){
		swing = gui;
		activity = act;
	}

	public void actionPerformed(ActionEvent e) {
		System.out.println("Minus");
		
			
			
		if (! swing.reports.isEmpty()){
			int i =0;
			
			
			while(i < swing.reports.size()){
				Activities temp = swing.reports.get(i);
				if (temp.equals(activity)){
					System.out.println("Activity Remove erkannt.");
					swing.reports.remove(i);
	----->> das gibt dann nen Fehler	//swing.pnlMid.remove(i); 
					break;
				} 
					i++;
					
				
			}
			
			System.out.println("Reports: " + swing.reports.size());
--> gibt immer 0 aus nach wiedereinlesen, davor ohne Probleme  	System.out.println("Panel Activities: " + swing.pnlMid.getComponentCount());

			swing.repaint();
			swing.validate();
		}
	}
		
	

}

 

[Jazz]

Habs jetzt gelöst ich hab jetzt einfach noch die Swing Klasse mit serialisiert.
Problem war glaub ich dass ich mit swing.pnlMid im MinusButtonListener ein anderes swing.pnlMid aufgerufen als das dann zur Laufzeit war. jetzt gehts wie ich wollte.
danke für die versuchte Hilfe

 

[CroniD]

Ja, ne, is klar.

Ich würde dir aber empfehlen getter/setter zu schreiben und evtl. auf innere Klassen setzen. Ein Beispiel siehst du hier (angepasste Version mit Abspeicherung der ArrayList):

public class ArrayListToJPanel extends JFrame {
	 
    ArrayList<JPanel> alPanel;
    
    public ArrayListToJPanel() {
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        getContentPane().setLayout(new GridLayout(0, 1));
        setLocationRelativeTo(null);
        init();
        pack();
    }
    
    private void init() {
        alPanel = new ArrayList<JPanel>(5);
        
        getSerialization();
        
        // extra panel mit einem 'remove' und 'add' button
        final JButton remove = new JButton("delete last");
        remove.addActionListener(new ActionListener() {
            @Override public void actionPerformed(ActionEvent e) {
                System.out.println(getContentPane().getComponentCount());
                // das letzte panel nach dem control panel loeschen
                int position = getContentPane().getComponentCount()-2;
                getContentPane().remove(position);
                // da das frame schon sichtbar -> validieren
                getContentPane().validate();
                // weniger komponenten nun, also soll sich das frame auch anpassen
                // in seiner groesse
                pack();
                System.out.println(getContentPane().getComponentCount());
                // evtl. muesste man noch die array list updaten, aber das spare
                // ich mir mal
                // allerding muss eine schranke eingebaut werden, falls nichts zu
                // loeschendes mehr da ist, sonst regelt es
                // 'ArrayIndexOutOfBoundsException's ;)
            }
        });
        final JButton add = new JButton("add last");
        add.addActionListener(new ActionListener() {
            @Override public void actionPerformed(ActionEvent e) {
                System.out.println(getContentPane().getComponentCount());
                // die position an der das panel nun eingesetzt werden soll
                // aber ueber die controls
                int prefix = getContentPane().getComponentCount()-1;
                getContentPane().add(getPanelWithButtons(prefix), prefix);
                // da das frame schon sichtbar -> validieren
                getContentPane().validate();
                // mehr komponenten nun, also soll sich das frame auch anpassen
                // in seiner groesse
                pack();
                System.out.println(getContentPane().getComponentCount());
                // evtl. muesste man noch die array list updaten, aber das spare
                // ich mir mal
            }
        });
        final JButton save = new JButton("save");
        save.addActionListener(new ActionListener() {
			@Override public void actionPerformed(ActionEvent e) {
				System.out.println("save");
				// array liste und das panel noch synchronisieren
				alPanel.clear();
				System.out.println(getContentPane().getComponentCount());
				Component[] comps = getContentPane().getComponents();
				for (int i = 0; i < comps.length; i++) {
					// casten sicherstellen und nicht unser control panel adden
					if (comps[i] instanceof JPanel && !"control".equals(comps[i].getName())) {
						alPanel.add((JPanel) comps[i]);
					}
				}
				ObjectOutputStream oos;
				try {
					oos = new ObjectOutputStream(new FileOutputStream("TimeTracker.ser"));
					oos.writeObject(alPanel); 
					oos.close();
				} catch (FileNotFoundException e1) {
					e1.printStackTrace();
				} catch (IOException e1) {
					e1.printStackTrace();
				} 
			}
		});
        JPanel control = new JPanel();
        control.setName("control");
        control.add(add);
        control.add(remove);
        control.add(save);
        getContentPane().add(control);
        // listener um das schliessen zu ueberwachen
        addWindowListener(new WindowListener() {
			@Override public void windowOpened(WindowEvent e) {}
			@Override public void windowIconified(WindowEvent e) {}
			@Override public void windowDeiconified(WindowEvent e) {}
			@Override public void windowDeactivated(WindowEvent e) {}
			@Override public void windowClosed(WindowEvent e) {}
			@Override public void windowActivated(WindowEvent e) {}
			
			@Override public void windowClosing(WindowEvent e) {
				// wir benutzen einfach die save action vom button
				save.doClick();
				System.exit(0);
			}
		});
    }
    
    private JPanel getPanelWithButtons(int prefix) {
    	// schoenes panel mit 4 buttons drauf
        JPanel pane = new JPanel(new FlowLayout(FlowLayout.CENTER, 10, 10));
        pane.setBorder(BorderFactory.createLineBorder(Color.pink));
        JButton button;
        for (int j = 0; j < 4; j++) {
            button = new JButton(String.valueOf(prefix)+String.valueOf(j));
            pane.add(button);
        }
        return pane;
    }
    
    private void getSerialization() {
    	File dateiTest = new File("TimeTracker.ser");
    	
    	if(dateiTest.exists()) {
    		try {
    			ObjectInputStream is = new ObjectInputStream(new FileInputStream("TimeTracker.ser"));
    			Object o = is.readObject();
    			if (o instanceof ArrayList<?>) {
    				alPanel = (ArrayList<JPanel>) o;
    			} else {
    				return;
    			}
    			for (int i = 0; i < alPanel.size(); i++) {
    				getContentPane().add(alPanel.get(i));
    			}
    			is.close();
    		} catch (Exception ex) {
    			ex.printStackTrace();
    		}
    		dateiTest.delete();
    	} else {
    		JPanel pane;
    		// erzeugen und einlesen in die array list
            for (int i = 0; i < 5; i++) {
                pane = getPanelWithButtons(i);
                alPanel.add(pane);
            }
            // Elemente der array list auf das frame packen
            for (JPanel p : alPanel) {
                getContentPane().add(p);
            }
    	}
    }
    
    public static void main(String[] args) {
        ArrayListToJPanel altjp = new ArrayListToJPanel();
        altjp.setVisible(true);
    }
}

BTW: Ich musste nichts serialisieren, damit es funktioniert. ^^