Hilfe beim Erzeugen eines Arrays NullPointerException wird ausgelöst

[DannyS]

Guten Morgen liebe Java-Gemeinde,
ich bin ein Java Anfänger und habe ein Verständnisproblem bei folgenden Java-Code. Worin ist mein Denkfehler beim Erzeugen eines Arrays? Ich bekomme immer eine NullPointerException. Anbei der Code:

class Caesar{
    public Caesar(String stri) {
      this.stri = stri;
    }

    public String stri;
    char[] chr= new char[(stri.length())];
    char[] testchar=new char[stri.length()];
    public char caesar(){
        for (int i=0; i < stri.length(); i++){
            chr[i]=stri.charAt(i);
            testchar[i] = (char)(chr[i]+ 3);
            System.out.println(testchar[i]);
    }
    return 1;
    }

}

Vielen Dank im Voraus.

Gruß Danny

 

[fhoffmann]

new char[(stri.length())]

Wenn du das Array anlegst, ist "stri" noch nicht gefüllt. Dies solltest du in den Konstruktor verschieben.

 

[DannyS]

Vielen Dank, für die Unterstützung. Allerding habe ich es nicht ganz verstanden, was ich in den Konstruktor schieben soll. Wenn ich char[] chr= new char[(stri.length())] in den Konstruktor schiebe, kommt trotzdem die Exception.
So hat es geklappt:

class Caesar{
    public Caesar(String stri) {
        this.stri = stri;
        //char[] chr= new char[(stri.length())];

    }

    public String stri;
    //char[] chr= new char[(stri.length())];
    //char[] testchar=new char[stri.length()];


    public char caesar(){
        char[] chr= new char[(stri.length())];
        char[] testchar=new char[stri.length()];
        for (int i=0; i < stri.length(); i++){
            chr[i]=stri.charAt(i);
            testchar[i] = (char)(chr[i]+ 3);
            System.out.println(testchar[i]);
    }
    return 1;
    }

}

 

[kneitzel]

Wo kommt genau welche Exception? Und was genau machst Du?

Im Augenblick sieht es auf den ersten Blick so aus, als dürfte keine NullPointerException kommen, so du bei der Erstellung der Instanz einen String übergibst und nicht null.

 

[DannyS]

Wo kommt genau welche Exception? Und was genau machst Du?

Im Augenblick sieht es auf den ersten Blick so aus, als dürfte keine NullPointerException kommen, so du bei der Erstellung der Instanz einen String übergibst und nicht null.

Hallo, wenn ich den Code wie eben geschrieben, ausführe, kommt es nicht zu einem Fehler. Lediglich der Code im Ausgang hat in Zeile 8 eine Exception ausgelöst. Nachdem ich char[] testchar=new char[stri.length()]; wie von fhoffmann im Konstruktor hinzufüge, erhalte ich trotzdem die Exception in Zeile 15 (vorher 8). Ich habe dann die Zuweisung vom char in der Methode ausgeführt und es funktioniert.

 

[kneitzel]

Also was @fhoffmann meinte, war lediglich die Initialisierung zu verschienben. Das hätte dann so ausgesehen:

class Caesar{
    public Caesar(String stri) {
        this.stri = stri;
        chr= new char[(stri.length())];
        testchar=new char[stri.length()];
    }

    public String stri;
    char[] chr;
    char[] testchar;

Du behältst also die Instanzvariablen, aber initialisiert werden diese im Konstruktor, wenn auch stri einen Wert hat und nicht null ist.

class Caesar{
    public Caesar(String stri) {
        this.stri = stri;
        char[] chr= new char[(stri.length())];
        testchar=new char[stri.length()];
    }

    public String stri;

macht keinen Sinn. Jetzt sind es lokale Variablen und nur im Konstruktor gültig.

Sowas:

    public char caesar(){
        char[] chr= new char[(stri.length())];
        char[] testchar=new char[stri.length()];
        for (int i=0; i < stri.length(); i++){
            chr[i]=stri.charAt(i);
            testchar[i] = (char)(chr[i]+ 3);
            System.out.println(testchar[i]);
        }
        return 1;
    }

ist natürlich gültig. Aber jetzt sind chr und testchar auch nur lokale Variablen und nur innerhalb der Methode gültig. So Du die Array nicht außerhalb der Methode benötigst, ist das also eine valide Sache.

(Ok, über den Sinn kann man noch reden, denn das return 1 macht nicht unbedingt viel Sinn ... was für ein Zeichen willst Du da zurück geben?)

 

[DannyS]

(Ok, über den Sinn kann man noch reden, denn das return 1 macht nicht unbedingt viel Sinn ... was für ein Zeichen willst Du da zurück geben?)

Vielen Dank, jetzt wurde mir einiges klarer. Das return habe ich nur erstmal reingeschrieben, um nicht void in die Methode zu schreiben. Da wird natürlich noch eine sinnvolle Verarbeitung stattfinden.
Der code ist sieht jetzt wie folgt aus:

class Caesar{
    public Caesar(String stri) {
        this.stri = stri;
        chr= new char[(stri.length())];
        testchar=new char[stri.length()];
    }

    public String stri;
    char[] chr;
    char[] testchar;
    public String s;

    public String caesar(){
        //char[] chr= new char[(stri.length())];
        //char[] testchar=new char[stri.length()];
        for (int i=0; i < stri.length(); i++){
            chr[i]=stri.charAt(i);
            testchar[i] = (char)(chr[i]+ 3);
            s+=testchar[i];
            System.out.println(s);
    }
    return s;
    }
}
[B]

[/B]

 

[Trjavnamen]

Moment char[] ist ein Array das mit einem String dimensioniert wird? char[] chr= new char[filu];

 

[DannyS]

Hallo,
Ich verstehe die Frage nicht ganz. Es funktioniert alles wie gewünscht. Der Char wird nur zu einem String hinzugefügt, aber nicht dimensioniert. Ich verwende die Länge des Strings zur Initialisierung.
Gruß Danny

 

[Trjavnamen]

[(stri.length()-1)]; das array begint mit 0 und endet mit 9. Wird der erste buchstabe in 1 gesetzt endet das wort in 10 und das ist außerhalb der Grenze =out of bond könnte da das problem sein?

 

[Trjavnamen]

Vergiss die frage?

 

[DannyS]

Danke für die Antwort, das Problem war eigentlich nur, dass beim initialisieren der String nicht initialisiert war und dadurch 0 da stand. Dadurch wurde die Exception ausgelöst.
Gruß Danny