Probleme mit Arrays

[Volnick]

public class Klassenzimmer {
        Schueler[] mates;
        int number = 0;

        public Klassenzimmer() {
            mates = new Schueler[5];
        }

        public void add(Schueler x) {

            mates[number] = x;
            number += 1;

            if (number <= 4) {
                System.out.println(x.forename + " " + x.name + " kommt rein");
            } else if (number > 4) {
                System.out.println("Tut uns leid, " + x.forename + " " + x.name + ". Das Klassenzimmer ist voll");
            }

        }

        public Schueler[] allGetOut() {
            // if(taxi.length == 0){
            // Ich soll ausgeben, wie viele ausgestiegen sind, d.h. wie viele im Taxi saßen
            Schueler[] taxi0 = new Schueler[0];
            Schueler[] taxi1 = new Schueler[1];
            Schueler[] taxi2 = new Schueler[2];
            Schueler[] taxi3 = new Schueler[3];
            Schueler[] taxi4 = new Schueler[4];

            if(number == 1)
                return new Schueler[1];
            else if (number == 2)
                return new Schueler[2];
            else if (number == 3)
                return new Schueler[3];
            else if (number == 4)
                return new Schueler[4];
            else
                return taxi0;

        }

    @Override
    public String toString() {
        if (number == 1)
            return "Das ist das Klassenzimmer von Dr.Jung. Er unterrichtet Marek Kurz ";
        else if (number == 2)
            return "Das ist das Klassenzimmer von Dr.Jung. Er unterrichtet Marek Kurz und Paul Alles";
        else if (number == 3)
            return "Das ist das Klassenzimmer von Dr.Jung. Er unterrichtet Marek Kurz, Paul Alles und Lena Weil.";
        else if (number == 4)
            return "Das ist das Klassenzimmer von Dr.Jung. Er unterrichtet Marek Kurz, Paul Alles, Lena Weil und Fabien Lehm. ";
        else
            return "Das ist das Klassenzimmer von Dr.Jung. Er unterrichtet niemanden.";

    }
}

_______________________________________________________________________________________________________________________

public class Schueler {

    String forename;
    String name;

    public Schueler(String forename, String name) {
        this.forename = forename;
        this.name = name;
    }

    public String getName() {
        return name;
    }
    public String getForename(){

        return forename;
    }

    @Override
    public String toString() {
        return forename+ " " + name;
    }

}

_______________________________________________________________________________________________________________________

public class Main {
    public static void main(String[] args) {

        Schueler m1 = new Schueler("Marek", "Kurz");
        Schueler m2 = new Schueler("Paul","Alles");
        Schueler m3 = new Schueler("Lena","Weil");
        Schueler m4 = new Schueler("Susi","Born");
        Schueler m5 = new Schueler("Fabien","Lehm");

        Klassenzimmer classe = new Klassenzimmer();

        classe.add(m1);
        classe.add(m2);
        classe.add(m3);
        classe.add(m4);
        classe.add(m5);

        System.out.println(classe.allGetOut());

    }
}

_______________________________________________________________________________________________________________________
Ausgabe:

Marek Kurz kommt rein
Paul Alles kommt rein
Lena Weil kommt rein
Susi Born kommt rein
Tut uns leid, Fabien Lehm. Das Klassenzimmer ist voll
[LclassroomForExample.Schueler;@59a6e353

Problem:
Nachdem die Methode allGetOut() ausgeführt wird, soll in einem Array angezeigt werden, wer alles das Klassenzimmer verlassen hat.
Das heißt, falls nur Marek Kurz im Klassenzimmer war, dann soll [Marek Kurz] ausgegeben werden.

Ich erhalte jedoch immer einen Fehler:

allGetOut() returned an array that contains null instead of a proper Schueler object for one of the schueler!

 

[kneitzel]

Schau Dir doch die Funktion einmal im Detail an.

Du erzeugst ein Array von Schueler einer bestimmten Größe und gibt dann das Array zurück. Die Referenzen sind also mit null gefüllt.

Das Array, das Du erzeugst, musst Du auch mit den Schülern füllen, ehe Du es zurück gibst.

 

[Volnick]

Schau Dir doch die Funktion einmal im Detail an.

Du erzeugst ein Array von Schueler einer bestimmten Größe und gibt dann das Array zurück. Die Referenzen sind also mit null gefüllt.

Das Array, das Du erzeugst, musst Du auch mit den Schülern füllen, ehe Du es zurück gibst.

Und wie fülle ich es mit den Schülern?

 

[kneitzel]

Wie hast Du denn das Array "mates" gefüllt, wenn da ein Schüler in den Klassenraum gekommen ist?
Prinzipiell geht es hier genau so, nur eben musst Du hier nicht einen Schüler, der als Parameter übergeben wird, einfügen, sondern die Schüler, die bereits in mates enthalten sind.

 

[Volnick]

Wie hast Du denn das Array "mates" gefüllt, wenn da ein Schüler in den Klassenraum gekommen ist?
Prinzipiell geht es hier genau so, nur eben musst Du hier nicht einen Schüler, der als Parameter übergeben wird, einfügen, sondern die Schüler, die bereits in mates enthalten sind.

Ich komme nicht auf die Lösung. Könnten Sie mir bitte weiterhelfen?

 

[kneitzel]

Du hast ein Array mates in dem Schueler gespeichert sind. Desweiteren hast Du eine Variable number, in der gespeichert ist, wie viele Schüler gespeichert wurden.

Somit sieht der Pseudo-Code wie folgt aus:
1. Erzeuge ein neues Array von Schueler mit number elementen. (Also ein Java Konstrukt mit "new" Befehl)
2. Für zähler = 0 bis number-1: Setze neuesArray[zaehler] = mate[zähler] (Also for Schleife mit einer Zuweisung im Schleifenkörper)
3. gib neuesArray zurück. (Also ein return Befehl)
Ich habe in Klammern einmal Hinweise auf Java Befehle gegeben.

Kommst Du mit diesen Hinweisen jetzt weiter? Sorry, wenn es für Dich etwas umständlich scheint, aber mir ist es wichtiger, die Hinweise zu der Lösung zu geben, so dass Du da eher ein Verständnis entwickelst, als Dir nur fertigen Code "um die Ohren zu hauen".

 

[Volnick]

Du hast ein Array mates in dem Schueler gespeichert sind. Desweiteren hast Du eine Variable number, in der gespeichert ist, wie viele Schüler gespeichert wurden.

Somit sieht der Pseudo-Code wie folgt aus:
1. Erzeuge ein neues Array von Schueler mit number elementen. (Also ein Java Konstrukt mit "new" Befehl)
2. Für zähler = 0 bis number-1: Setze neuesArray[zaehler] = mate[zähler] (Also for Schleife mit einer Zuweisung im Schleifenkörper)
3. gib neuesArray zurück. (Also ein return Befehl)
Ich habe in Klammern einmal Hinweise auf Java Befehle gegeben.

Kommst Du mit diesen Hinweisen jetzt weiter? Sorry, wenn es für Dich etwas umständlich scheint, aber mir ist es wichtiger, die Hinweise zu der Lösung zu geben, so dass Du da eher ein Verständnis entwickelst, als Dir nur fertigen Code "um die Ohren zu hauen".

Vielen Dank, dass Sie helfen!
Ich habe jetzt folgenden Code geschrieben:

Human[] neuesArray;

public Schueler[] allGetOut() {

    Schueler[] mates = new Schueler[number];

    for(int zaehler =0; zaehler <= number-1; neuesArray[zaehler] = mates[zaehler] ){
        return neuesArray;
    }
    return mates;

}

Ist die for-Schleife so richtig?
Bekomme nun leider einen NullPointerException ausgeworfen.

 

[kneitzel]

Also Dein lokales mates array überdeckt das mates Array aus der Klasse. Das ist etwas unschön.

Die Schleife ist aber noch nicht korrekt. Ich verbessere es jetzt einmal mit Kommentaren:

public Schueler[] allGetOut() {

//    Schueler[] mates = new Schueler[number];
    Schueler[] neuesArray = new Schueler[number]; // neuesArray habe ich es mal genannt. mates ist ja das Arary in der Klasse

//    for(int zaehler =0; zaehler <= number-1; neuesArray[zaehler] = mates[zaehler] ){
    for(int zaehler =0; zaehler <= number-1; zaehler++) { // in der for Schleife wird an der dritten Stelle meistens die Variable erhöht.
        neuesArray[zaehler] = mates[zaehler]; // Hier weisen wir den schüler aus mates dem neuen Array zu.
//        return neuesArray; // Wir wollen die Schleife ja ganz durchlaufen, daher kein return.
    }
//    return mates; // Wäre ok gewesen, wenn ich mates oben nicht umbenannt hätte.
    return neuesArray;
}

Ist das so verständlich? Ich habe jetzt gerade keine Möglichkeit das zu testen. Falls es da noch Probleme geben sollte, dann zeig noch mal den ganzen Code von Dir.

 

[Volnick]

Also Dein lokales mates array überdeckt das mates Array aus der Klasse. Das ist etwas unschön.

Die Schleife ist aber noch nicht korrekt. Ich verbessere es jetzt einmal mit Kommentaren:

public Schueler[] allGetOut() {

//    Schueler[] mates = new Schueler[number];
    Schueler[] neuesArray = new Schueler[number]; // neuesArray habe ich es mal genannt. mates ist ja das Arary in der Klasse

//    for(int zaehler =0; zaehler <= number-1; neuesArray[zaehler] = mates[zaehler] ){
    for(int zaehler =0; zaehler <= number-1; zaehler++) { // in der for Schleife wird an der dritten Stelle meistens die Variable erhöht.
        neuesArray[zaehler] = mates[zaehler]; // Hier weisen wir den schüler aus mates dem neuen Array zu.
//        return neuesArray; // Wir wollen die Schleife ja ganz durchlaufen, daher kein return.
    }
//    return mates; // Wäre ok gewesen, wenn ich mates oben nicht umbenannt hätte.
    return neuesArray;
}

Ist das so verständlich? Ich habe jetzt gerade keine Möglichkeit das zu testen. Falls es da noch Probleme geben sollte, dann zeig noch mal den ganzen Code von Dir.

Danke!!!
Es hat alles geklappt.
Schönes Wochenende noch.