JList zeigt die Daten aus dem Array nicht an!

[Hades]

Hallo zusammen,

komme gerade nicht mehr weiter. Das Programm dient dazu Bilddateien auszuwählen diese einzulesen und anschließend umzubenennen. Dabei will ich den ausgewählten Inhalt auf einer JList anzeigen lassen.
Nur leider komme ich gerade nicht mehr weiter.

Das Frame:

public class Frame extends JFrame {
    
    Container c;

    MenuBar menu = new MenuBar();
    Buttons button = new Buttons();
    List dataList = new List();

    
    public Frame() {
        
        final Toolkit kit = Toolkit.getDefaultToolkit();
        final Dimension screenSize = kit.getScreenSize();
        
        final int screenHeight = screenSize.height;
        final int screenWidth = screenSize.width;
        
        setSize(screenWidth /2, screenHeight /2);
        setLocation(screenWidth / 4, screenHeight / 4);
        
        setTitle("Softwareprojekt");
        
        c = getContentPane();
        c.setLayout(new BorderLayout());

        
        add(button, BorderLayout.SOUTH);


        add(dataList.dataList, BorderLayout.CENTER);
        dataList.dataList.setBorder(new EtchedBorder(Color.black, Color.black));
        
        setJMenuBar(menu.menuBar);
        
    }
}

Das File-Array:

public class FileArray {
    
    protected static File[] fileArray;
    protected String datName;
    
    List list = new List();
    
    public void setFileArray(File[] fA) {
        fileArray = fA;
    }

    public File[] getFileArray() {
        return fileArray;
    }
    
    public void showContent() {
        for(int i = 0; i < fileArray.length; i++) {
            list.data.addElement(fileArray[i]);
            list.dataList.updateUI();
        }
    }
}

Die Tabelle:

public class List {

    final DefaultListModel data;
    JList dataList;
    JScrollPane scrollList;
    
    
    public List() {
        
        data = new DefaultListModel();
        dataList = new JList(data);
        scrollList = new JScrollPane(dataList);
        
    }
}

Würde mich sehr über euere Hilfe freuen.
Vielen Dank schon ma im Vorraus.

 

[Michael...]

Die JList wird ja nirgends gefüllt. Warum eine eigene Klasse List? Verwende in der Klasse Frame doch direkt JList. Ansonsten hilft vielleicht mal ein Blick in die Grundlagen der objektorientierten Programmierung.

 

[Hades]

In der Methode showContent wird diese doch gefüllt, habe ich gedacht.
Da wird doch durch addElement die einzelnen Inhalte übergeben oder irre ich mich?

 

[SlaterB]

showContent wird nie aufgerufen, die Klasse FileArray nirgendwo benutzt?
du musst schon etwas vollständiger die Vorgänge aufzeigen, am besten in einem kleinen lauffähigen Programm

 

[Michael...]

Selbst wenn sie da befüllt werden würde: Wo wird die Methode aufgerufen, wo wird das Array befüllt?

Bei dem in der Klasse FileArray erzeugten List-Objekt handelt es sich im Übrigen nicht um das selbe Objekt wie in der Klasse Frame.

 

[Hades]

Die Methode wird über einen Button aufgerufen.

Und hier wird das Array gefüllt.

public class FileChooser implements ActionListener{
    
    JFileChooser fileChooser = new JFileChooser();
    File dir;
    FileArray fA = new FileArray();
    
    FileFilter filter = new FileExtension("JPG, JPEG", new String[] { "JPG", "JPEG"});
    FilenameFilter fileName = new FilenameFilter() {
        
        @Override
        public boolean accept(File dir, String name) {
            // TODO Auto-generated method stub
            return ((name.endsWith(".jpg")) || (name.endsWith(".jpeg")));
        }
    };    
    
    public void actionPerformed(ActionEvent e) {
        fileChooser.setCurrentDirectory(new File("."));
        //fileChooser.setFileFilter(filter);
        fileChooser.setMultiSelectionEnabled(true);
        fileChooser.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);
        if(fileChooser.showDialog(null, "Bitte auswählen") == JFileChooser.APPROVE_OPTION) {
   
                   dir = fileChooser.getSelectedFile();
                        
                        if(dir.isDirectory()) {
                            fA.setFileArray(dir.listFiles(fileName));   
                        }
                        else if(dir.isFile()) {
                            fA.setFileArray(fileChooser.getSelectedFiles());
                        }            
        }    
    }    
}

 

[JensArpe]

auch hier befüllst du die Liste wieder nicht. Du rufst da zwar setFileArray auf. Aber nirgends showContent

 

[Hades]

Hier wird die Methode aufgerufen.

public class Buttons extends JPanel implements ActionListener{
    
    FileArray file = new FileArray();
    List datenList = new List();
    
    JButton butnRename, butnShow, but2, but3;
    JPanel butPanel;
    
    public Buttons() {        
        
        butnRename = new JButton("Umbenennen");
        butnShow = new JButton("Anzeigen");

        
        
        
        
        add(butnRename);
        add(butnShow);

        
        butnRename.addActionListener(this);
        butnShow.addActionListener(this);
        
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        // TODO Auto-generated method stub
        String action = e.getActionCommand();
        if(action == "Umbenennen") {
            
        }
        if(action == "Anzeigen") {
            file.showContent();
        }
    }
}

 

[Michael...]

Mir scheint Dir ist der Umgang mit Objekten bzw. deren Verwendung in mehreren Klassen nicht ganz klar. Du erzeugst in unterschiedlichen Klassen Objekte von List und FileArray, die nichts miteinander zu tun haben, ausser dass es sich um Objekte des selben Typs handelt.

 

[JensArpe]

Bei dem in der Klasse FileArray erzeugten List-Objekt handelt es sich im Übrigen nicht um das selbe Objekt wie in der Klasse Frame.

Ich glaub auch, daß da dein Problem liegt. Aber dein Code ist schwer zu durchschauen, da du in fast jedem Teil eine List und ein FileArray anlegst.

 

[Hades]

ja, das kann sein.
Aber wie gehe ich dann vor?

 

[JensArpe]

Zuerst mal alle FileArray und Lists aus allem ausser Frame löschen.. dann
z.B. kopierst du mal alles aus FileArray in List (natürlich ohne die List)
danach verpasst du der Einfachheit halber, deiner Frame Klasse ne getter und setter Methode für die List. Und dann rufst du in deinen Buttons nur die getter/setter auf

 

[Michael...]

ja, das kann sein.
Aber wie gehe ich dann vor?

Deine Aufteilung des Codes auf die einzelnen Klassen ist so nicht unbedingt sinnvoll.

Hier eine Demo zum Arbeiten mit einem gemeinsamen Objekt (in dem Fall JList).
IDabei habe ich den Code künstlich auf zwei Klassen aufgeteilt - den Code also nicht als Programmiervorlage verwenden!!!

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

import javax.swing.DefaultListModel;
import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JScrollPane;

public class FileChooseListDemo extends JFrame {
	
	private DirectorySelector selector;
	
	public FileChooseListDemo() {
		JList list = new JList();
		JButton button = new JButton("Open Directory");
		this.getContentPane().add(new JScrollPane(list), BorderLayout.CENTER);
		this.getContentPane().add(button, BorderLayout.SOUTH);
		
		/*******************
		 * Hier wird bei der Erzeugung eines DirectorySelector Objekts
		 * die Referenz auf die JList an dieses uebergeben.
		 * Dadurch koennen das FileChooseListDemo Objekt und dasDirectorySelector Objekt
		 * gemeinsam mit der selben JList arbeiten.
		********************/
		selector = new DirectorySelector(list);
		
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				selector.selectDirectory();
			}
		});
	}
	
	class DirectorySelector {
		private JFileChooser fileChooser;
		private JList list;
		
		public DirectorySelector(JList list) {
			this.list = list;
			fileChooser = new JFileChooser();
			fileChooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
		}
		
		public void selectDirectory() {
			DefaultListModel model = new DefaultListModel();
			
			int retValue = fileChooser.showOpenDialog(list);
			
			if (retValue==JFileChooser.APPROVE_OPTION) {
				File dir = fileChooser.getSelectedFile();
				for(File file : dir.listFiles())
					model.addElement(file);
			}
			list.setModel(model);
		}
	}
	
	public static void main(String[] args) {
		JFrame frame = new FileChooseListDemo();
		frame.setBounds(0, 0, 500, 300);
		frame.setLocationRelativeTo(null);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
	}
}

 

[Hades]

Danke
habe jetzt auch ein, zwei Klassen zusammengelegt und es funktioniert alles.