Input/Output einlesen aus einer evtl. leeren Datei

[schrill]

Hey Zusammen

Ich bin gerade auf ein kleineres Problem gestossen, und zwar sollte ich aus einer .txt Datei Daten rauslesen und weiterverwenden. Nun ist es aufgrund der Aufgabenstellung möglich, dass die Datei keinen Inhalt hat. In dem Fall krieg ich eine NullPointerException , wenn ich die Datei mit

private static ...(String dataletter)
	{
		String [] data =In.readStrings(dataletter);
...

einzulesen versuche. Wie kann man das Problem umgehen?

Danke im Voraus

 

[Timothy Truckle]

Hey Zusammen

Ich bin gerade auf ein kleineres Problem gestossen, und zwar sollte ich aus einer .txt Datei Daten rauslesen und weiterverwenden. Nun ist es aufgrund der Aufgabenstellung möglich, dass die Datei keinen Inhalt hat. In dem Fall krieg ich eine NullPointerException , wenn ich die Datei mit

private static ...(String dataletter)
	{
		String [] data =In.readStrings(dataletter);
...

einzulesen versuche. Wie kann man das Problem umgehen?

Danke im Voraus

Am besten geht man das Problem in der Methode

In.readStrings(dataletter)

an, also zeig doch mal deren Code.

bye
TT

 

[schrill]

  public static String[] readStrings(String filename) {
        In in = new In(filename);
        String[] fields = in.readAll().trim().split("\\s+");
        return fields;
    }

 

[Timothy Truckle]

  public static String[] readStrings(String filename) {
        In in = new In(filename);
        String[] fields = in.readAll().trim().split("\\s+");
        return fields;
    }

Dann lehne ich mich mal aus dem Fenster und behaupte, die NPE wird in Zeile 3 geworfen, weil

in.readAll()

ein

null

zurück bringt.
Ich würde die NPE hier mit einem try/catch abfangen und in eine [JAPI]RuntimeException[/JAPI] mit einer vernünftigen Fehlermeldung umwandeln:

try{
  return in.readAll().trim().split("\\s+");
} catch (NullPointerrException npe) {
  throw new RuntimeException("Input file was empty!");
}

Damit muss sich nur noch

main

damit beschäftigen, um dem Anwender den Grund für den Programmabbruch auszugeben (wenn mann denn den Stacktrace in der Konsole unterdrücken will).
An allen anderen Stellen des Programms kann man jetzt davon ausgehen, dass die Datei Inhalt hatte.
bye
TT

 

[Gast2]

Wobei das relativ schlechter Stil ist. Besser:
Die Rückgabe von in.readAll() in eine Variable lesen, prüfen ob der Inhalt null oder leer ist. Wenn ja, dann eine Exception werfen.

 

[schrill]

nun, letzten Endes adde ich einfach alles was rauskommt, wenns nix ist adde ich halt nix, dann muss ich nicht mit exceptions um mich werfen oder?

konkret heiss das:ich habe eine Klasse Postmen, Package und Letter extenden Mailing:

public class Postmen {
	...
	private  List <Mailing> cargo;
	...

	private static List <Mailing> getpackage(String datapackage)
	{
		String [] data =In.readStrings(datapackage);
		.../*Konstruiere aus data neue Einträge des Typs Package
		cargo.add(new Package(...))*/
		return cargo;}

	private static List <Mailing> getletter(String dataletter)
	{
		String [] data =In.readStrings(dataletter);
		.../*Konstruiere aus data neue Einträge des Typs Letter
		cargo.add(new Letter(...))*/
		return cargo;}

	public Postmen (..., String dataletter,String datapackage, ...)
	{
		...
		this.cargo=getletter(dataletter);
		this.cargo.addAll(getpackage(datapackage));
		...
	}

cargo darf auch leer bleiben.