Datentypen String -> File.getAbsolutePath

[kaoZ]

Hallöchen, ich zerbreche mir grad mein Kopf und zwar habe ich folgende situation:

-JComboBox
-File[]
-ArrayList<File>
-JLabel

ich lese alle Dateien eines Verzeichnisses aus und füge diese einem File[] Array hinzu, über eine for/each schleife füge ich der ComboBox jeweils die Namen der Dateien hinzu:

Codeausschnitt

try{
	fileArray = new File("H:/Apps/Test/TestOrdner/").listFiles();
	for(File a : fileArray){
				
		comboModel.addElement(a.getName()); // liefert mir jeweils nur den Dateinamen
	}

Funktioniert bis hierhin soweit super !

nun möchte ich allerdings je nach gewähltem "Item" in der ComboBox ,über einen (Buffered)FileReader die jeweilige Datei , bzw. die erste Zeile in der Datei ausgelesen wird.

mein Ansatz :

	comboModel = new DefaultComboBoxModel<String>();
	box = new JComboBox<String>(comboModel);
	box.addItemListener(new ItemListener(){

		@Override
		public void itemStateChanged(ItemEvent item) {
				
			if(item.getSource()==comboModel.getSelectedItem()); // ebenfalls nur ein Ansatz ?!
					
                           /* Hier bin ich mir noch unsicher....
                            * Die ComboBox(das Model) enthält ja nur Strings, welche "eigentlich"
                            * ja jeweils für einen Dateinamen in meinem Verzeichnis
                            * stehen. 
                            * Theoretisch müsste ich nun Prüfen ob in meinem Verzeichnis,
                            * eine Datei mit Namen des gewählten Items vorhanden ist, oder ? */

		}
			
		});

Ausserdem bin ich mir nicht sicher ob der ItemListener auch wirklich die richtige Wahl für diesen Fall ist, hat jemand noch einen Tipp oder liege ich vollkommen daneben ? :rtfm:

 

[Ruzmanz]

public class MyFile extends File {
	private String name;
	
	public MyFile(String pathname) {
		super(pathname);
		// TODO: Namen sinvoll setzen
		this.name = pathname; 
	}
	
	@Override
	public String toString() {
		return name;
	}
}

=> JComboBox<MyFile> cbMyFiles
=> ArrayList<MyFile> listMyFiles

 

[kaoZ]

hm wenn ich mir eine zusätliche klasse MyFiles schreibe so wie in deinem Bsp.

habe ich trotzdem das gleich ein grün, wenn ich z.B

..imports...

public class Foo{

ArrayList<MyFiles> fileList;
JComboBox<MyFiles> box;
DefaultComboBoxModel<MyFiles> model;
Files[] allFiles;

public Foo(){
......
......
fileList = new FileList<MyFiles>();

model = new DefaultComboBoxModel<MyFiles>();
box = new JComboBox<MyFiles>(model);
}

public void getClient(){

  allFiles[] = new Files("H/:Test/Unterordner").listFiles();
  
    for(Files a : allFiles){
        model.addElement(a)   // oder a.toString();

/* nun könnte ich auf (a) die überschriebene toString() methode                
    aufrufen  , die mir dann aber trotzdem nur 
    denn DateiNAMEN (als String) liefert, aber nicht den PFAD,
    ausserdem lässt sich das dann überhaupt kompilieren ? <MyFile> Collection 
    vs. String ? / ist doch das gleiche wie vorher nur mit eigener klasse ?! */
}

public static void main(String[]args){

  new Foo().getClient();
}

}
}

über eine etwas ausführlichere erläuterung wäre ich nicht böse

 

[X5-599]

Keine Ahnung, ob sich in der Zwischenzeit was geändert hat, aber so war es bisher:

Die Elemente die per addItem() der ComboBox hinzugefügt wurden, wurden grafisch angezeigt indem deren toString() Methode aufgerufen wird (vom Swing Framework).

Ich gehe mal davon aus, dass das immer noch so ist. d.h. du müsstest dir einen Wrapper schreiben, der als Attribut ein File Objekt hat und eine toString() Methode in der du definierst, wie Objekte deiner neuen Klasse in der ComboBox angezeigt werden sollen.
Auch brauchst du einen Getter für das File Attribut, wenn du später damit arbeiten willst.

In deiner Schleife, wo du die gefundenen File Objekte behandelst musst du dann anstatt

for(File file : files)
{
    box.addItem(file);
}

//das hier machen

for(File file : files)
{
    box.addItem(new MyFile(file));
}

Eine Sache noch: Beim itemStateChanged war bei meinen Tests die Source immer die ComboBox selbst. Also reicht ein MyFile mine = (MyFile)combo.getSelectedItem(); um das selektierte Element zu bekommen.
Auch ist mir aufgefallen, dass zweimal ein itemStateChanged Event gefeuert wird. Ich schätze einmal für das selektieren des neuen Elementes und einmal für das de-selektieren des alten Elementes.
Da du wahrscheinlich nur einmal auf die Auswahl eines Elementes reagieren möchtest, kannst du sowas machen:

if(e.getStateChange() == ItemEvent.SELECTED)
{
    //deine Aktion
    MyFile mine = (MyFile)combo.getSelectedItem();
    File datei = mine.getFile();
    //...
}

 

[kaoZ]

Das hilft mir irgendwie grad nicht weiter, da ich nicht nur eine JComboBox habe , sondern ebend ein DefaultComboBoxModel, ich füge meine Items ja nicht der ComboBox hinzu sondern dem Modell, diesem kann ich nur einen ListDateListener hinzufügen , wenn ich allerdings debugge, passiert an der stelle rein garnichts, bzw. scheint auch kein Event abgefeuert zu werden ......


da ich meine "Elemente" dem model hinzufüge, und nicht der Box , gehe ich auch mal davon aus das das model diese verwaltet, und die ComboBox das Model, demnach müsste ich ja die Events auf dem Model abfangen und nicht auf der box, oder muss ich die Events der Box abfangen und an das model weitergeben ?!

Vielleicht mache ich mir es grad auch wieder nur zu kompliziert ....:rtfm:

 

[kaoZ]

Das hier funktioniert schonmal insoweit das er mir den namen des selektierten items in ein label ausgibt

sprich , event scheint abgefeuert und abgefangen zu werden , demnach muss ich es jetzt nurnoch irgendwie hinbekommen die Zeile in dieser datei auszulesen und darzustellen , dafür brauch ich jetzt den kompletten pfad meines "SelectedItems"

//COMBOBOX
		comboModel = new DefaultComboBoxModel<String>();
		box = new JComboBox<String>(comboModel);
		box.addItemListener(new ItemListener(){

			@Override
			public void itemStateChanged(ItemEvent e) {

				if(e.getStateChange()==e.SELECTED){
					
					labelSix.setText(comboModel.getSelectedItem().toString());
				}
				
			}
			
		});

Mein Problem ist nun quasi das ich mir in der ComboBox nur den Dateinamen, und nicht den kompletten pfad anzeigen lasse , ergo erhalte , wenn ich versuche ein neues File objekt zu erstellen auch nur

ABC.txt

anstelle von

H:/Test/Unterordner/ABC.txt

, könnte man nicht einfach was die ausgabe in der box angeht mit substing / split arbeiten und so zwar alles ausgeben aber nicht alles anzeigen lassen ? dem File könnte ich dann den angezeigten String + den gespeicherten Substring übergeben und mir somit wieder den kompletten pfad als String basteln ?!

 

[kaoZ]

Das hier funktioniert schonmal für die ausgabe :

try{
			getPrivate =  new File("H:/Apps/Kundendaten/Privatkunden/").listFiles();
			for(File a : getPrivate){
				
				String[] split = a.toString().split("\\\\");
				
				String token = split[4]; // Da ich den Speicherpfad festlege, steht der Dateiname immer an dieser Stelle 
				
				comboModel.addElement(token);
				
			}

Ausgabe : ABC.txt

Ausserdem sollte es doch jetzt möglich sein einfach über das String array zu iterieren und den Kompletten String für den FileReader zusammenzusetzen ich teste das gleich mal ^^

 

[kaoZ]

Das funktioniert schonmal soweit, das er mir bei auswahl des ersten Items die erste zeile in der Datei die durch das item referenziert wird anzeigt.

//COMBOBOX
		comboModel = new DefaultComboBoxModel<String>();
		box = new JComboBox<String>(comboModel);
		box.addItemListener(new ItemListener(){

			@SuppressWarnings("static-access")
			@Override
			public void itemStateChanged(ItemEvent e) {

				if(e.getStateChange()==e.SELECTED){
					
					String out ="";
					
					for(String a : splitPrivate){
						
						out += "/" + a ;
					}
					try{
					BufferedReader reader = new BufferedReader(new FileReader(out));
					String path = reader.readLine();
					reader.close();
					labelSix.setText(path);
					}
					catch(Exception ex){
						//TODO
					}
				}	
			}
		});

Allerdings funktioniert das nur bei der ersten auswahl, wenn ich daraufhin ein anderes Item der Liste auswähle zeigt er mir immernoch das gleiche an , ich glaube ich muss dafür ne neue Methode schreiben die ich bei auswahl der Combobox einfach aufrufen kann ....

so schauts zzt aus :



ich muss dazusagen das die angezeigte nummer allerdings aus einer anderen Datei Stammt die ich zuvor mal aufgerufen hatte , mir fehlt also noch was das beim auswählen eines neuen Elementes aktualliesiert und sich auf den neuen Pfad bezieht

 

[X5-599]

Woah, ich komm' gar nicht so schnell mit. Also, wenn sich mit neueren JDK Verisonen nichts geändert hat, dann braucht man um mit der JCombBox und dem DefaultComboBoxModel zu arbeiten das Model nicht extra erzeugen. Das wird quasi als Standard intern verwendet.
In meinem JDK sieht der Standardconstructor z.b. so aus:

public JComboBox() {
    super();
    setModel(new DefaultComboBoxModel());
    init();
}

Und das ist die addItem Methode. Man sieht, dass hier auch nur die addElement Methode ausgeführt wird auf dem Model. Vorher wird nur noch ein Check gemacht, ob das Model das Interface MutableComboBoxModel implementiert.

public void addItem(Object anObject) {
    checkMutableComboBoxModel();
    ((MutableComboBoxModel)dataModel).addElement(anObject);
}

Man muss also das Model der ComboBox nicht explizit erzeugen. Wie gesagt, zumindest in meinem JDK ist das noch so... Ich hoffe mal nicht, dass die das grundlegend ändern. Aber Möglich ist alles. Bei mir ist z.b. die JComboBox noch nicht generisch. Also kein JComboBox<String> oder JComboBox<File> möglich...


Alle Objekte, die in der ComboBox dargestellt werden sollen definieren ihre Anzeige über ihre toString() Methode. Darum der Gendanke eines Wrappers, da du die toString() Methode der File Klasse nicht überschreiben kannst.

Es gibt noch eine weitere Möglichkeit. Ein eigener Renderer. Ich würde alle gefundenen File Objekte in die JComboBox packen und dieser mit dem Renderer sagen, wie sie dargestellt werden sollen.

Da ich so schlecht erklären kann und du schon sehr viel Eigeninitiative gezeigt hast hier mal ein KSKB:

import java.awt.Component;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.io.File;

import javax.swing.Icon;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.ListCellRenderer;
import javax.swing.SwingUtilities;



public class ComboBoxRendererTest
{
	private JComboBox combo;
	private JFrame frame;
	
	public ComboBoxRendererTest()
	{
		frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		combo = new JComboBox();
		combo.setRenderer(new ListCellRenderer() {
			
			private JLabel label = new JLabel();
			
			public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus)
			{
				label.setOpaque(true);
				label.setFont(list.getFont());
				

				if (isSelected)
				{
					label.setBackground(list.getSelectionBackground());
					label.setForeground(list.getSelectionForeground());
				}
				else
				{
					label.setBackground(list.getBackground());
					label.setForeground(list.getForeground());
				}
				
				
				
				if (value instanceof Icon)
				{
					label.setIcon((Icon)value);
				}
				else
				{
					if(value == null)
						label.setText("");
					else if(value instanceof File)
						label.setText(((File)value).getName());
					else
						label.setText(value.toString());
				}

				return label;
			}
		});
		
		
		combo.addItem(new File("/full/path/to/test"));
		combo.addItem(new File("/path/file"));
		
		combo.addItemListener(new ItemListener() {

			@Override
			public void itemStateChanged(ItemEvent e)
			{
				if(e.getStateChange() == ItemEvent.SELECTED)
				{
					System.out.println(combo.getSelectedItem());
				}
			}
			
		});
		
		frame.add(combo);
		frame.pack();
	}
	
	
	/**
	 * @param args
	 */
	public static void main(String[] args)
	{
		SwingUtilities.invokeLater(new Runnable() {
			
			@Override
			public void run()
			{
				new ComboBoxRendererTest().frame.setVisible(true);
			}
		});
	}
}

In Zeile 59 findest du den Grund warum mit diesem Renderer nur der Dateiname von Files angezeigt wird. Solltest du mal andere Arten von Objekten mit diesem Renderer anzeigen wollen wird nur deren toString() Methode ausgeführt (Zeile 61). Ausser natürlich Icons. Bei denen bringt es nicht viel toString() aufzurufen. Darum wird ein Icon so wie es ist per setIcon() in Zeile 52 auf das Label gesetzt. (Das habe ich mir vom Standard Renderer abgeschaut).

Da du nur Files darstellen willst könntest du diese Zeilen natürlich auch weglassen. Ich dachte mir ich erwähne es einfach mal...

 

[kaoZ]

Schaffe ich heute zeitlich nichtmehr, probiere ich morgen früh aber gleich mal aus , das defautlcomboboxmodel habe ich nur verwendet weil ich auf Stackoverflow gelesen habe so ist es möglich eine Arraylist in verbindung mit einer combobox zu nutzen, da man die arraylist an das model übergibt und das model dann an die box deswegen habe ich die items auch dem model hinzugefügt und nicht direkt der box . ich meld mich morgen früh nochmal wenn ich mir das mal angeschaut habe

 

[kaoZ]

Soooo, ich habe mir das ganze mal angeschaut , auch nicht schlecht , ich habe allerdings ein wenig rumprobiert und bin letztendlich bei folgender lösung gelandet :

//-----------------------BOX-------------------------//
		
		box = new JComboBox<String>();
		box.addItem("");
		
		getClients();  // Neue Methode die beim Initialiesieren oder per button aufgerufen wird um die Box zu füllen / zu aktualisieren

		box.addItemListener(new ItemListener(){

			String clientFile = "";
			
			@SuppressWarnings("static-access")
			@Override
			public void itemStateChanged(ItemEvent e) {

				if(e.getStateChange() == e.SELECTED){
					
					clientFile = String.valueOf(clients + "/" + e.getItem() + ".txt"); 
					
					try{
						
						BufferedReader reader = new BufferedReader(new
FileReader(clientFile));
						String out = reader.readLine();
						reader.close();

						labelSix.setText(out);
					}
					catch(Exception ex){
						
						labelSix.setText("Datei konnte nicht gelesen werden !");
						
					}
				}
				
			}
			
		});

das model hab ich ganz weggelassen da ich jetzt nurnoch mit Arrays und nicht mit generics arbeite, ausserdem habe ich 2 neue methoden geschrieben, eine zum aktualliesieren der box(bzw.der items in der box)

getClients:

public void getClients(){
		
		File[] clientList = clients.listFiles();
		
		try{
		
			for(File x :  clientList){
				
				String[] s = x.getName().split("\\."); // Da ich den Namen ohne Dateierweiterung haben will 
			
				box.addItem(s[0]);  // deshalb hier auch index[0], alles vor dem split 
			
			}
		
		}
			catch(Exception ex){
				
				box.addItem("");
		}
	}

hier fehlt jetzt nurnoch eine Abfrage ob die Liste mehr oder die gleichen elemente erhält sonst hab ich die einträge bei betätigung alle doppelt

Zu beginn wollte ich Privat und Geschäftskunden trennen, das macht es mir aber unnötig kompliziert da ich sonst dateien aus 2 Ordnern auflisten und der Box hinzufügen müsste, nun speicher ich alle in einem Ordner , aber mit unterschiedlich generierten Kundennummern , je nachdem ob Privat oder Geschäftlich , ausserdem hab ich so die möglichkeit die liste einfach schon nach alphabet sortiert einzulesen.

danke nochmal für die ganzen lösungsansätze und Ideen zu behebung des Problems

so schaut es jetzt aus :