Interpreter-Fehler NullPointerException

[lajao]

Hey Community

Meine Aufgabe war es, ein Dateiname nach Verzeichnis, Extension etc. aufzuspalten.

Dafür hab ich zwei Klassen geschreiben. Eine, die die ganze Arbeit ausführen soll und eine, die die erste testen soll.

Tester-Klasse:

import java.util.Scanner;

public class TestParser{

  public static void main(String[] args){

    Scanner scanner = new Scanner(System.in);

    System.out.println("Dateipfad eingeben: ");
    String s = scanner.nextLine();
    System.out.println(s);

    Parser parser = new Parser(s);
    parser.print();

  }
}

"Arbeiter"-Klasse

public class Parser{

 public String Pfad;

 public Parser(String s){
   Pfad = s;
 }

 int[] pos = positions();

 private String Extension(){
   String ext = Pfad.substring(pos[1]+1,Pfad.length()-1);
   return ext;
 }
 
 private String Dateiname(){
   String dat = Pfad.substring(pos[0]+1,pos[1]-1);
   return dat;
 }
 
 private String Verzeichnis(){
   String ver = Pfad.substring(0,pos[0]-1);
   return ver;
 }

 private int[] positions(){
   int[] ret = new int[2];

   int i = Pfad.length();

   while(i > 0){
    if(Pfad.charAt(i) == '.') ret[1]=i;
    if(Pfad.charAt(i) == '\\'){
      ret[0]=i;
      i = 0;
    }
   }
   
   return ret;
 }
 
 public void print(){
   System.out.println("Dateipfad:   " + Pfad);
   System.out.println("Extension:   " + Extension());
   System.out.println("Dateiname:   " + Dateiname());
   System.out.println("Verzeichnis: " + Verzeichnis());
 }
}

Beim ausführen in der Konsole krieg ich ne NullPointerException:
at Parser.positions(Parser.java:29)
at Parser.(init)(Parser.java:9)
at TestParser.main(TestParser.java:13)

Wenn ich das richtig verstehe, hat die "Arbeiter"-Klasse Schwierigkeiten, den String Pfad richtig zu lesen. Dies, obwohl der String doch über den Konstruktor definiert wurde, also frei zur Verfügung stehen solle?
Wenns wirklich daran liegt, dass die anderen Methoden den String Pfad nur durch Zeile 3 kennen, wie krieg ich die Infos aus dem Konstruktor raus? Sollte mMn eigentlich automatisch geschehen, doch wo liegt dann der Fehler?

Bin momentan komplett verwirrt

Ich habe ebenfalls versucht, die Methode positions(String s) zu definieren und dann einfach als positions(Pfad) aufzurufen, doch das funktionierte ebenfalls nicht. Irgendetwas stimmt mit dieser Methode nicht ???:L

btw: ich weiss, dass ich mir den Umweg über das int-Array mit der String Methode "lastIndexOf" sparen könnte, doch darum gehts wohl nicht, oder?

thx und lg
lajao

 

[Tomate_Salat]

dein Fehler liegt hier:

 int[] pos = positions();

das wird ausgeführt bevor dein Konstruktor [c]Pfad[/c] definieren kann.

Zudem: Konventionen einhalten! Methoden/Variablen schreibt man lowerCamelCase. Nähere infos zu Konventionen findest du in meiner Signatur

 

[Final_Striker]

Versuche es mal damit:

	public String Pfad;
	int[] pos;

	public Parser(String s) {
		Pfad = s;
		pos = positions();
	}

 

[lajao]

also muss ich das int-Array auch über den Konstruktor definieren lassen? also ca so (btw Namen angepasst )

public class Parser{

 public String pfad;
 private int[] pos;

 public Parser(String s){
   pfad = s;
   pos = positions(pfad);
 }

 private String extension(){
   String ext = pfad.substring(pos[1]+1,pfad.length()-1);
   return ext;
 }
 
 private String dateiname(){
   String dat = pfad.substring(pos[0]+1,pos[1]-1);
   return dat;
 }
 
 private String verzeichnis(){
   String ver = pfad.substring(0,pos[0]-1);
   return ver;
 }

 private int[] positions(String s){
   int[] ret = new int[2];

   int i = s.length()-1;

   while(i > 0){
    if(s.charAt(i) == '.') ret[1]=i;
    if(s.charAt(i) == '\\'){
      ret[0]=i;
      i = 0;
    }
   }
   
   return ret;
 }
 
 public void print(){
   System.out.println("Dateipfad:   " + pfad);
   System.out.println("Extension:   " + extension());
   System.out.println("Dateiname:   " + dateiname());
   System.out.println("Verzeichnis: " + verzeichnis());
 }
}

Neben den Namen habe ich noch ein paar Grenzen angepasst, die nicht den Array-Grenzen entsprachen.

Nun hab ich ein nächstes, bisher noch nie gesehenes Problem, das ich nicht verstehe.

Das Programm läuft zwar, es kommen auch keine Fehlermeldungen mehr, aber es hört nicht mehr auf zu rechnen..

Meine CPU wird seit ca. 5min zu 70% ausgelastet, aber er kommt zu keinem Ergebnis. So kompliziert ist das Programm ja auch nicht

Der System.out.println Befehl aus der Test-Klasse wird noch ausgeführt, danach bleibt er stecken.

 

[Tomate_Salat]

weil du eine endlosschleife gebaut hast:

   while(i > 0){
    if(s.charAt(i) == '.') ret[1]=i;
    if(s.charAt(i) == '\\'){
      ret[0]=i;
      i = 0;
    }
   }

angenommen s.charAt liefert einen Buchstaben: beide bedingungen sind nicht gültig. i wird nie verändert und er prüft wieder den gleichen Buchstaben. Da fehlt ein i--;

 

[lajao]

Man bin ich blöde -.-

Vielen Dank, nun funktioniert das Programm endlich einwandsfrei

Falls es jemand interessiert, hier der Quellcode des funktionierenden Programms: (Mussten noch ein paar Grenzen bei den substrings angepasst werden)

import java.util.Scanner;

public class TestParser{

  public static void main(String[] args){

    Scanner scanner = new Scanner(System.in);

    System.out.println("Dateipfad eingeben: ");
    String s = scanner.nextLine();
    System.out.println(s);

    Parser parser = new Parser(s);
    parser.print();

  }
}

public class Parser{

 public String pfad;
 private int[] pos;

 public Parser(String s){
   pfad = s;
   pos = positions(pfad);
 }

 private String extension(){
   String ext = pfad.substring(pos[1]+1,pfad.length());
   return ext;
 }
 
 private String dateiname(){
   String dat = pfad.substring(pos[0]+1,pos[1]);
   return dat;
 }
 
 private String verzeichnis(){
   String ver = pfad.substring(0,pos[0]);
   return ver;
 }

 private int[] positions(String s){
   int[] ret = new int[2];

   int i = s.length()-1;

   while(i > 0){
    if(s.charAt(i) == '.') ret[1]=i;
    if(s.charAt(i) == '\\'){
      ret[0]=i;
      i = 0;
    }
    i--;
   }
   
   return ret;
 }
 
 public void print(){
   System.out.println("Dateipfad:   " + pfad);
   System.out.println("Extension:   " + extension());
   System.out.println("Dateiname:   " + dateiname());
   System.out.println("Verzeichnis: " + verzeichnis());
 }
}