Char-Array - NullPointerException

[java_every_day]

Hallo Liebe Forenmitgleder,

für eine Aufgabe soll ich aus einer Datei Text lesen, diesen verschlüsseln und dann wieder auf einer Datei ausgeben. Dabei soll der Text einfach-verschlüsselt werden(Caesar-Verschlüsselung). Hierzu brauch ich jeden enzelnen Buchstaben aus dem Text.

ich bin noch im ersten Schritt:
Text aus datei lesen = klappt
Text mit split zerkleinern = klappt
Wörter zu einem Sinnvollen Char-array umwandalne = nicht !

public class cesar2 {

	public static void main(String[] args) {
		
		String [] lineArray = new String [99];
		String [] wortArray = new String [99];
		char [] letterArray = new char [99];
		
		lineArray = getLine(lineArray);
		wortArray = getWortArray(lineArray);
		letterArray = getLetterArray(wortArray);
		}	 
		
	public static String[] getLine(String array []) {
		
		array =	EasyFileAccess.getFileContent("C:/Users/lebert/Desktop/test.txt");
		return array;
	}
	public static String[] getWortArray(String [] line) {
		boolean isLine = true;
		String [] puffer = new String [line.length];
		String [] wortArray = new String [line.length];
		
		for (int i = 0; isLine ; i++) {
			puffer = line[i].split(" ");
		//	System.out.println(puffer[0]);
			for (int j = 0; j < puffer.length; j++) {
				wortArray[j] = puffer[j];
		//		System.out.println(puffer.length);
		//		System.out.println(wortArray[j]);
				isLine = false;
			}
		}
		return wortArray;
	}
	public static char [] getLetterArray(String wortArray []) {
		char [] letterArray = new char [wortArray.length];
		String wort = "puffer";
		
		for (int i = 0; i < wortArray.length; i++) {
			wort = wortArray[i];
			System.out.println(wort + " wort " + wort.length()  + " = wort.length");
			for (int j = 0; j <= wort.length() - 1; j++) {
	
				letterArray = wort.toCharArray();
				
			}
		}
		return letterArray;
	}
}

Ich bekomme eine NullPointerException, aber ich verstehe das nicht:
2.5. wort4 = wort.length
Bedeutung wort9 = wort.length
der wort3 = wort.length
OCR-Kennzeichen wort15 = wort.length
in wort2 = wort.length
der wort3 = wort.length
Datenbank wort9 = wort.length
Exception in thread "main" java.lang.NullPointerException
at cesar2.getLetterArray(cesar2.java:44)
at cesar2.main(cesar2.java:12)

 

[kaoZ]

Du solltest es nochmal überdenken, du übergibst der methode ein String array, dieses überschreibst du in der for schleife solange bis i == wortArray.length ist, erstens bringt das nix und 2tens kann es dann passieren das du auf einen index zugreifen willst der in dem array nicht existiert, du müsstest also ein char[][] zurückgeben welches weitere char arrays für jedes einzelne wort in dem String array enthält, logischerweise, außer du möchtstest alle buchstanben aller wörter in einem char array haben.

 

[ceving]

für eine Aufgabe soll ich aus einer Datei Text lesen, diesen verschlüsseln und dann wieder auf einer Datei ausgeben. Dabei soll der Text einfach-verschlüsselt werden(Caesar-Verschlüsselung). Hierzu brauch ich jeden enzelnen Buchstaben aus dem Text.

Mal ne Frage: wozu braucht man für Caesar (ich denke Rot13) ein Array? Das geht in einem Stream, da jedes Zeichen unabhängig von allen anderen Zeichen verschlüsselt wird. Du brauchst die Eingangsdatei also nur zeichenweise einlesen und die Ausgangsdatei zeichenweise schreiben.

Und noch ein Tipp:

[99]

geht gar nicht! Wenn man nicht vorhersehen kann, wie groß das Datenvolumen wird, muss man entweder eine Liste oder einen Iterator nehmen. Darauf zu vertrauen, dass 99 schon reichen wird, endet zwangsläufig in einer RuntimeException. Diese Art von Programmier-Stil ist verantwortlich für unzählige Sicherheitslöcher in Software auf der ganzen Welt. Es ist somit eine der übelsten Todsünden, die man als Programmierer begehen kann.

Und wenn es unbedingt ein Array sein muss: mit Java 7 kann man sehr einfach eine ganze Datei in einen Byte-Array oder String einlesen:

new String(Files.readAllBytes(Paths.get(filePath)));

java - What is simplest way to read a file into String? - Stack Overflow
Wenn du das Byte-Array gar nicht erst zu einem String zusammen fügst, sparst du dir, es wieder zerlegen zu müssen.

 

[ceving]

Ich habe dein Beispiel mal um eine Implementation der Klasse EasyFileAccess angereichert, damit auch andere es testen können:

import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.List;
import java.io.IOException;

public class cesar2
{
  static class EasyFileAccess
  {
    static String[] getFileContent(String filename)
    {
      try {
        List<String> lines = Files.readAllLines (Paths.get(filename),
                                                 StandardCharsets.US_ASCII);
        String[] array = new String[lines.size()];
        int i = 0;
        for (String line : lines)
          array[i++] = line;
        return array;
      }
      catch (IOException ex) {
        throw new RuntimeException(ex);
      }
    }
  }

  public static void main(String[] args)
  {
    String [] lineArray = new String [99];
    String [] wortArray = new String [99];
    char [] letterArray = new char [99];
           
    lineArray = getLine(lineArray);
    wortArray = getWortArray(lineArray);
    letterArray = getLetterArray(wortArray);
  }    
           
  public static String[] getLine(String array [])
  {
    array = EasyFileAccess.getFileContent("test.txt");
    return array;
  }

  public static String[] getWortArray(String [] line)
  {
    boolean isLine = true;
    String [] puffer = new String [line.length];
    String [] wortArray = new String [line.length];
           
    for (int i = 0; isLine ; i++) {
      puffer = line[i].split(" ");
      //  System.out.println(puffer[0]);
      for (int j = 0; j < puffer.length; j++) {
        wortArray[j] = puffer[j];
        //      System.out.println(puffer.length);
        //      System.out.println(wortArray[j]);
        isLine = false;
      }
    }
    return wortArray;
  }

  public static char [] getLetterArray(String wortArray [])
  {
    char [] letterArray = new char [wortArray.length];
    String wort = "puffer";
           
    for (int i = 0; i < wortArray.length; i++) {
      wort = wortArray[i];
      System.out.println(wort + " wort " + wort.length()  + " = wort.length");
      for (int j = 0; j <= wort.length() - 1; j++) {
       
        letterArray = wort.toCharArray();
                   
      }
    }
    return letterArray;
  }
}

Wenn ich das Programm mit der folgenden Testdatei laufen lasse

Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam
nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat,
sed diam voluptua. At vero eos et accusam et justo duo dolores et ea
rebum. Stet clita kasd gubergren, no sea takimata sanctus est Lorem
ipsum dolor sit amet. Lorem ipsum dolor sit amet, consetetur
sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et
dolore magna aliquyam erat, sed diam voluptua. At vero eos et accusam
et justo duo dolores et ea rebum. Stet clita kasd gubergren, no sea
takimata sanctus est Lorem ipsum dolor sit amet.

knallt es schon in

getWortArray

:

javac -Xlint:unchecked cesar2.java && java cesar2
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 9
	at cesar2.getWortArray(cesar2.java:55)
	at cesar2.main(cesar2.java:35)

Der Grund für diesen Fehler ist relativ einfach gefunden: du baust einen Puffer auf der Basis von

line.length

und gehst dann jede Zeile durch und füllst den Puffer mit dem Ergebnis von

split

für die betreffende Zeile. Das Problem ist aber, dass

line.length

nicht die Länge der betreffenden Zeile ist, sondern die Anzahl der Zeilen, denn

line

hätte besser

lines

heißen sollen, weil du an

getWortArray

das Ergebnis von

getLine

übergibst.

getLine

hätte aber auch besser

getLines

geheißen, weil es ein Array aller Zeilen der Datei liefert und nicht nur eine Zeile.