Laufzeitfehler: NullPointerException

[webstarg]

Hallo Community,

ich verstehe nicht, wieso ich bei folgendem Codeausschnitt diese Exception erhalte:

public void loadFiles(String directory)
	{
		File f = new File(directory);
		if (! f.isDirectory())
		{
			files.add(f.getName());
		}
		else
		{
			for(File file: f.listFiles())
			{
				loadFiles(file.getName());
			}
		}
	}

files ist eine weiter oben instanzierte List<string>.
Der Parameter directory könnte z. B. "C:\" sein (wobei ich das Zeichen "\" "escaped" habe).
Die Methode ist rekursiv.

Fehlercode:

Exception in thread "main" java.lang.NullPointerException
	at java.io.File.<init>(Unknown Source)
	at FileSystem.loadFiles(FileSystem.java:28)
	at FileSystem.<init>(FileSystem.java:12)
	at MainClass.main(MainClass.java:15)

Vielen Dank im Voraus!

grüße
webstarg[/code]

 

[musiKk]

Die Rekursion funktioniert bei mir zwar nicht, aber ich erhalte auch keine NullPointerException. Es wäre mal gut zu wissen, auf welche Zeile sich die Exception im Stück oben bezieht.

 

[webstarg]

Hallo musiKk,

danke für deine Antwort.

Es wäre mal gut zu wissen, auf welche Zeile sich die Exception im Stück oben bezieht.

at FileSystem.loadFiles(FileSystem.java:28)

bezieht sich auf:

File f = new File(directory);

(in der Methode loadFiles(String directory))

at FileSystem.<init>(FileSystem.java:12)

bezieht sich auf:

this.loadFiles(path);

(Aufruf dieser Methode aus dem Konstruktor dieser Klasse)

at MainClass.main(MainClass.java:15)

bezieht sich auf:

FileSystem fs = new FileSystem("C:\\");

(Instanz der Klasse FileSystem in der Klasse MainClass in der statischen main-Methode erstellen)

mfg
webstarg

 

[diggaa1984]

glaub es ist besser wennu die Klasse insgesamt zeigen könntest, bzw wenn zu ausladend zumindest kontruktor und membervariablen

 

[musiKk]

Besser ist das. Ich kanns nicht nachvollziehen. Habs einfach rauskopiert und es gibt keinen Fehler. Wenn man den rekursiven Aufruf auch noch in

loadFiles(file.getAbsolutePath());

ändert, dann funktioniert das ganze sogar.

 

[diggaa1984]

vielleicht schon des rätsels lösung?

 

[webstarg]

Ok.

Klasse: MainClass (da ist noch nicht viel objektorientiert; kommt noch ;-))

import javax.swing.*;
import java.awt.*;

public class MainClass 
{
	JFrame jf = new JFrame("JFrame");
	JProgressBar pb = new JProgressBar(0, 100);
	
	public static void main(String[] args) 
	{
		MainClass mc = new MainClass();
		mc.defineFrame();

		FileSystem fs = new FileSystem("C:\\");
		
	}
	
	public void defineFrame()
	{
		jf.setVisible(true);
		jf.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		jf.setPreferredSize(new Dimension(600, 200));
		
		pb.setLocation(10, 10);
		pb.setVisible(true);
		pb.setStringPainted(true);
		pb.setValue(66);
		
		jf.add(pb, BorderLayout.PAGE_START);
		jf.pack();
	}
}

Klasse: FileSystem (ist aber noch unfertig, daher könnten einige Sachen noch keinen Sinn ergeben)

import java.io.*;
import java.util.*;

public class FileSystem 
{
	String path;
	List<String> files = new ArrayList<String>();
	
	public FileSystem(String file)
	{
		this.path = path;
		this.loadFiles(path);
	}
	
	public String[] getFiles()
	{
		//return files.toArray(new String[0]);
		return new String[] {};
	}
	
	public void setPath(String path)
	{
		this.path = path;
	}
	
	public void loadFiles(String directory)
	{
		File f = new File(directory);
		if (! f.isDirectory())
		{
			files.add(f.getName());
		}
		else
		{
			for(File file: f.listFiles())
			{
				loadFiles(file.getName());
			}
		}
	}
}

grüße
webstarg

 

[musiKk]

Der Konstruktor funktioniert so nicht. Du übergibst einen String, machst aber nichts mit dem. Die Variable path ist uninitialisiert (und daher null) und loadFiles(null) liefert folgerichtig die NullPointerException.

public FileSystem(String file) 
   { 
      this.path = file; 
      this.loadFiles(path); 
   }

So sollte es laufen (und mit der anderen Änderung, die ich drei Posts weiter oben erwähnt habe).

 

[diggaa1984]

konstruktor war eigentlich die einzige Möglichkeit

 

[webstarg]

this.path = path; :autsch:
Dummer Fehler. ;-)

Vielen Dank jedenfalls.

grüße
webstarg

 

[musiKk]

Oh, ein kurzer Nachtrag noch: this.path = path muss ja kein Fehler sein, wenn man das wie folgt anwendet:

public FileSystem(String path) 
   { 
      this.path = path; 
      this.loadFiles(path); 
   }

und diese Schreibweise ist eigentlich auch die gebräuchlichere. Keine Ahnung, warum ich das eben anders gemacht habe.

 

[diggaa1984]

aber sone fehler werden teilweise von IDE's angezeigt, wenn auch als Warnung. Da so etwas in dem Fall absolut 0 Sinn macht ist wie: x=x

 

[webstarg]

Oh, ein kurzer Nachtrag noch: this.path = path muss ja kein Fehler sein, wenn man das wie folgt anwendet:

public FileSystem(String path) 
   { 
      this.path = path; 
      this.loadFiles(path); 
   }

und diese Schreibweise ist eigentlich auch die gebräuchlichere. Keine Ahnung, warum ich das eben anders gemacht habe.

Eben. Ich hatte das vorher auch so, war mir dann aber nicht ganz sicher, ob das macht, was ich von einer anderen Programmiersprache gewöhnt bin (hätte es aber gemacht ;-)), dann habe ich es abgeändert. Und da habe ich leider die Hälfte vergessen.