JList + Pfad angabe

[raffnix]

Hallo!
Ich habe folgendes Problem.
meine JList wirft eine NullPointer..:cry:
Nach rumprobieren habe ich festgestellt, das es wohl an dem Separator liegt.
Der Pfad für die Methode die das array files[] bereitstellt liegt als String vor.
so sieht er z.B aus . C:\ordner\Verzeichniss\...
wenn ich nun per hand die Separatoren ändere von Backslash "\" zu Slash "/" funktioniert alles super.
wie kann ich nun erreichen, daß es mit dem Backslash funzt???

Hier die Methode die das array liefert.

private File[] getFiles(String path)
	{   
	    File file    = new File( path );
	    File files[] = file.listFiles();

	    return files;
	  }

und hier die JList.

dateiListe = new JList(getFiles(path));

:?: :?:

 

[Illuvatar]

Wo wird denn genau die NullPointerException geworfen?

 

[raffnix]

Nun ich denke mal, dass es in der JList ist.

hier der Fehlercode.


at javax.swing.JList$1.getSize(Unknown Source)
at javax.swing.plaf.basic.BasicListUI.updateLayoutState(Unknown Source)
at javax.swing.plaf.basic.BasicListUI.maybeUpdateLayoutState(Unknown Source)
at javax.swing.plaf.basic.BasicListUI.getPreferredSize(Unknown Source)


wegen (Unknown Source) dachte ich , dass es wohl der String path ist.
da ich ihn mit einem JFileChooser gewält habe weiss ich das er existiert.
und wie gesagt wenn ich path per hand eintrage und die Separatoren ändere funzt alles super.

wenn ich nun per hand die Separatoren ändere von Backslash "\" zu Slash "/" funktioniert alles super.

Es kann doch nicht sein das ich den String noch bearbeiten muss und eine methode schreiben muss die die Separatoren ändert.
habe schon in der Api nach File geschaut . aber Canonicalpath oder die anderen seltsamen dinge scheinen nicht das richtige für dieses problem zu sein.

 

[Wildcard]

Wo bekommst du diesen String denn her?

 

[raffnix]

puh also.
zunächst mal aus einer anderen Klasse.
dort ist ein JFileChooser chooser;

dann:

datei = chooser.getSelectedFile();
			 			pfadEingabe.setText(datei.getPath());

später:

 public String lieferpfad(){
			 	
			 	path = pfadEingabe.getText();
				System.out.print(pfadEingabe.getText());
			 
				return path;
	 		}

System.out liefert den Pfad mit Backslash als Separator.( OS ist WINDOWS)

Ich habe den String dann an die andere Clase übergeben . soweit so gut.
zu testen habe ich den String auch per hand eingetragen .
also z.B.
String path = "C:\Dokumente und Einstellungen\All Users.WINDOWS\Dokumente\Eigene Bilder\Beispielbilder"
es folgt die Fehlermeldung.
aber so:
String path = "C:/Dokumente und Einstellungen/All Users.WINDOWS/Dokumente/Eigene Bilder/Beispielbilder"

geht es..

 

[Wildcard]

String path = "C:\Dokumente und Einstellungen\All Users.WINDOWS\Dokumente\Eigene Bilder\Beispielbilder"

So kann das auch nicht gehen weil du den '\' maskieren musst: '\\'

 

[raffnix]

Ähh

also als Separator immer \\ wählen???
hm
ich bekomme path aber aus

 public String lieferpfad(){
			 	
			 	path = pfadEingabe.getText();
				System.out.print(pfadEingabe.getText());
			 
				return path;
	 		}

nur mit / geliefert.
dann ist wohl File.getText() die falsche wahl oder wie??
nun verstehe ich den zusammenhang nicht.
um einen ordner inhalt in JList zu zeigen muss ich zuerst einen String haben .
der mir aber Falsch geliefert wird.
wie bekomme ich denn sonst einen String von einem Pfad den ich auch verwenden kann wenn nicht mit File.getText() ??


Edit :
Mit get.Name()???

 

[Wildcard]

Warum machst du die Files denn nicht direkt in die JList???

 

[raffnix]

Wie Was? :bahnhof:

Das geht??
verd..
:autsch:
Sag nicht sowas.. das hat lange gebraucht soweit zu kommen.
Ich kann JList Files übergeben??? Wie Wie wie??
(in API schau und googl :### :### und nix find)

 

[Wildcard]

Übergib der Jlist einfach das File[] im Konstruktor, oder benutz ein ListModel und füg die Files da ein...

 

[raffnix]

Also ICH BLÖD!!

Übergib der Jlist einfach das File[] im Konstruktor

Tue ich das nicht hiermit??

private File[] getFiles(String path)
	{   
	    File file    = new File( path );
	    File files[] = file.listFiles();

	    return files;
	  }
........
.....
dateiListe = new JList(getFiles(path );

 

[Wildcard]

Ich versteh überhauptnicht was du da eigentlich machst...
was soll das lieferpfad() usw.?
Zeig mir mal dein Prog. Das führt ja so zu nichts :wink:

 

[raffnix]

Also gut.

package smile;
import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;


public class Ordnerauswahl extends JPanel implements ActionListener  {
	public String path;
	
	
	JButton search;
	JButton ok;
	JButton cancel;
	JTextField pfadEingabe;//soll editable sein und die möglichkeit geben den Pfad direkt einzugeben
	JFileChooser chooser;
	File datei;
	
	
	public Ordnerauswahl(){
		 super(new BorderLayout());
		 
		 
		 chooser = new JFileChooser();
		 chooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY );
		 
		 search = new JButton("Browse");
		 search.addActionListener(this);
		 
		 
		 ok = new JButton("OK");
		 ok.addActionListener(this);
		
		 cancel = new JButton("Cancel");
		 cancel.addActionListener(this);
		
		pfadEingabe = new JTextField(40);
		pfadEingabe.setEditable(false);
		
		
			
		JPanel browseButtonPanel = new JPanel(new GridLayout(0,1));
		browseButtonPanel.add(search);
		
		JPanel scheibeButton = new JPanel(new GridLayout(0,1));		
		scheibeButton.add(ok);
		scheibeButton.add(cancel);
		
		JPanel scheibePfad = new JPanel(new GridLayout(0,1));		
		scheibePfad.add(pfadEingabe);
		
		setBorder(BorderFactory.createEmptyBorder(20, 20, 20, 20));
		add(scheibeButton,BorderLayout.SOUTH);
		add(scheibePfad,BorderLayout.CENTER);
		add(browseButtonPanel,BorderLayout.LINE_END);
		
		
	}
	
	
			 public void actionPerformed(ActionEvent ereignis)
			 {
			 	if (ereignis.getSource()==search){
			 		int returnVal = chooser.showOpenDialog(Ordnerauswahl.this);
			 		if (returnVal == JFileChooser.APPROVE_OPTION) {
			 			datei = chooser.getSelectedFile();
			 			pfadEingabe.setText(datei.getPath());
			 			
			 			
						}
			 		
			 	}else if(ereignis.getSource()==cancel)
			 	       {
			 		    System.exit(0);
			 	       }
			 	else if ( ereignis.getSource()== ok)
			 	      {
			 		   lieferpfad();
			 		   javax.swing.SwingUtilities.invokeLater(new Runnable() 
			 	       		{
			 	             public void run() 
			 	                   {
			 	              	    BasicGui neuFenster = new BasicGui(lieferpfad());
				 		                     neuFenster.machmeinegui();
			 	                   }
			 	       		});			 		  
			 		    setVisible(false);
   			 	       }
			}
				
			 public String lieferpfad(){
			 	
			 	path = pfadEingabe.getText();
				System.out.print(pfadEingabe.getText());
			 
				return path;
	 		}
		  
		 
		

			public void Guimaker()
	 {
	 	JFrame.setDefaultLookAndFeelDecorated(true);
	    JFrame fenster = new JFrame("Ordner auswählen..");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JComponent neuekomponente = new Ordnerauswahl();
		 neuekomponente.setOpaque(true); 
		 //fenster.setSize(800,600);
	        fenster.setContentPane(neuekomponente);
	        fenster.pack();
	        fenster.setVisible(true);
	    }

}

die erste Class.


und hier die 2te.

package smile;
import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.Image;
import java.io.File;

import javax.swing.BorderFactory;
import javax.swing.JComponent;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JTree;



public class BasicGui extends JPanel 
{   
	
	
	String path;
	String bildString = "Bild: ";
	String dAuswahl = "Datei: ";
	String xmlAnsicht = "XML Daten";
	Image bilddatei;
	
	JLabel bildAnsicht = new JLabel(bildString);
	JLabel xmlTree = new JLabel(xmlAnsicht);
	
	JFileChooser dateiDialog = new JFileChooser();
	JTree dateiBaum = new JTree();
	JList dateiListe; 
	
	

	
	public BasicGui(String path)
	{
		super(new BorderLayout());
		Ordnerauswahl zeigmal = new Ordnerauswahl();
		path = zeigmal.lieferpfad();
		//path = "C:\Dokumente und Einstellungen\All Users.WINDOWS\Dokumente";
		//path = "C:/Dokumente und Einstellungen/All Users.WINDOWS/Dokumente";
		dateiListe = new JList(getFiles(zeigmal.lieferpfad()) );
		
		JPanel bildLabel = new JPanel(new GridLayout(0,1));
	           bildLabel.add(bildAnsicht);	
	    JPanel xmlLabel = new JPanel (new GridLayout(0,1));   
	           xmlLabel.add(xmlTree);
	    JPanel ordnerLabel = new JPanel (new BorderLayout(0,1));
	           ordnerLabel.add(dateiListe);
	    
	    
	    setBorder(BorderFactory.createEmptyBorder(20, 20, 20, 20));
	              add(bildLabel,BorderLayout.CENTER);
	              add(ordnerLabel,BorderLayout.SOUTH);
	              add(xmlLabel,BorderLayout.EAST);
	              System.out.print(path);
	               
	  }
	
	private File[] getFiles(String path)
	{   
	    File file    = new File( path );
	    File files[] = file.listFiles();

	    return files;
	  }

	
	

	public  void machmeinegui()
	{   
		JFrame.setDefaultLookAndFeelDecorated(true);
		JFrame fenster = new JFrame("HandBlablka ");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JComponent neuekomponente = new BasicGui(path);
		neuekomponente.setOpaque(true);
		
		fenster.setSize(800,600);
        fenster.setContentPane(neuekomponente);
        //fenster.pack();
        fenster.setVisible(true);
        
	}
		
}

Die Main auch noch falss du es starten willst.

package smile;

public class Zusammen {

	public static void main(String[] args) {
		
		javax.swing.SwingUtilities.invokeLater(new Runnable() {
            public void run()  {
            	Ordnerauswahl chooseOrdner = new Ordnerauswahl();
            	chooseOrdner.lieferpfad();
            	
            	chooseOrdner.Guimaker();
            	
            }
        });
		
	}
}

 

[Wildcard]

Was hast du da gemacht? Sowas wirres hab ich echt schon lang nicht mehr gesehen. :autsch:
Deine Klassen instanzieren sich ständig gegenseitig, du erzeugst irgendwelche neuen Frames,...
sorry das ist jetzt nicht das was du hören willst, aber drück den großen Löschknopf! Das ist wirklich
nicht mehr zu retten! :wink:
Ich schau mal das ich dir vieleicht schnell ein kleines bsp. schreibe...

 

[Wildcard]

Schön ist anders, aber ich will dir ja nur zeigen wie's grundsätzlich funktioniert.
Bei deinem Prog ist echt der Wurm drin! Vieleicht solltest du dir noch ein paar Grundlagen aneignen bevor
du GUIs machst...

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

import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JTextField;

/**
 * @author Wildcard
 * created at 30.04.2005
 */
public class FileListFrame extends JFrame
{
	
	private JTextField pathField = new JTextField();
	
	private JPanel jListPanel;
	
	FileListFrame()
	{
		
		
		JPanel inputPanel = new JPanel(new BorderLayout());
		jListPanel = new JPanel(new BorderLayout());
		
		pathField.setPreferredSize(new Dimension(200,50));
		
		JButton ok = new JButton("ok");
		ok.addActionListener(new ActionListener() {
		
			public void actionPerformed(ActionEvent e)
			{
				File f = new File(pathField.getText());
				JList list = new JList(f.listFiles());
				jListPanel.add(list, BorderLayout.CENTER);
				jListPanel.revalidate();		
			}
		
		});
		
		inputPanel.add(pathField,BorderLayout.CENTER);
		
		JButton browse = new JButton("Browse");
		browse.addActionListener(new ActionListener() {
		
			public void actionPerformed(ActionEvent e)
			{
				JFileChooser chooser = new JFileChooser();
			    chooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
				chooser.showOpenDialog(null);
				pathField.setText(chooser.getSelectedFile().getAbsolutePath());
		
			}
		
		});
		inputPanel.add(ok,BorderLayout.SOUTH);
		inputPanel.add(browse,BorderLayout.WEST);
		inputPanel.add(pathField,BorderLayout.CENTER);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		getContentPane().add(inputPanel,BorderLayout.NORTH);
		getContentPane().add(jListPanel,BorderLayout.SOUTH);
		setSize(600,500);
	}
	


	/**
	 * @param args
	 */
	public static void main(String[] args)
	{
		FileListFrame f = new FileListFrame();
		f.setVisible(true);
		
	}

}

 

[raffnix]

Hmm .
Ok dann
ich glaube das hilft ein wenig .Ich werde das prog wohl neu schreiben müssen.
Danke, daß du dir soviel zeit genommen hast.

mal sehen ob ich es in einem neuen Versuch besser mache


:### :### :###

 

[Scotty]

@Wildcard :applaus: Das scheint dir ja irgendwie Spass zu machen, dich in verbuggte Programme reinzufitzen, oder? Wenn ich ein großeres Projekt habe, das nicht nur aus einer Klasse besteht, kann ich dir das dann zur Korrektur (oder auch nur mal so zum anschauen und Meinung abgeben) per Mail schicken? :wink:

 

[Wildcard]

Hängt vom Umfang und meiner Laune ab... :wink:

 

[raffnix]

@ Wild
Ich hab da mal ein paar Fragen zu deinem Code.

Wieso extends JFrame wenn du gar keinen JFrame benutzt??

Du benutzt gar nicht

javax.swing.SwingUtilities.invokeLater(new Runnable() {
public void run() {


habe gehört das gehört in jede swingGui wegen Thread safety...

 

[Wildcard]

Dadurch das ich von JFrame extende ist FileListFrame auch ein JFrame.
Wenn ich ein JFrame instanzieren würde hätt ich mir das erben ja auch sparen können :wink:
Stimmt schon das Swing nicht threadsicher ist. invokeLater braucht man wenn Veränderungen im DatenModel vorgenommen werden.
Im Normalfall benutzt man es aber nicht wenn man nicht mit mehreren Threads arbeitet...

 

[mic_checker]

Je nach Anwendungszweck wäre es vielleicht sinnvoll - wie Wildcard es oben bereits gepostet hast - mit einem ListModel zu arbeiten.

@raffnix: Kommt drauf an was du machen willst, aber wenn du nachträglich Daten hinzufügen,löschen etc. willst wäre es noch sinnvoll z.B. ein DefaultListModel zu verwenden. Dazu gab es schon einige Threads. Solltest du nichts gutes finden (kann nicht sein ) , bzw. nicht damit klar kommen dann poste es.

 

[raffnix]

Ich danke euch beiden ..
Ihr seid echt nett. Wenn ich Probleme habe komme ich gerne darauf zurück.
:toll: