Frage zur NullPointerException

[El Kabong]

Hallo!

Irgendwie werd ich nicht schlau draus warum ich eine NullPointerException geworfen bekomme.

public class BuchTest {

    Buch buch;
    public static void druckeBuecher(Buch[] buecher) {
        for (int i=0; i<=buecher.length; i++) {
        buch = buecher[i];
            if (buecher != null) {
                System.out.println(buch.getAutor() + buch.getTitel() + buch.getPreis());
            
        }
        }
    }

    public static void main(String[] args) {

        Buch[] b1 = new Buch[4];
        b1[0].setTitel("Maier");
        b1[1].setAutor("Maier");
        b1[2].setSeitenzahl(200);
        b1[3].setPreis(30.90);

        System.out.println(b1[0].getTitel());

Wo liegt der Fehler?

 

[Haave]

Du musst in einer Schleife über das Buch-Array gehen und die Elemente darin initialisieren. Bisher hast du nur das Array initialisiert.
Wahrscheinlich Fehler ab Zeile 16, oder?

 

[Raum114]

Hier ist ein komplettes Beispiel.

public class Main {

    public static void druckeBuecher(Buch[] buecher) {
        for (Buch buch : buecher) {
            if (buch != null) {
                System.out.println(buch);
            }
        }
    }

    public static void main(String[] args) {

        Buch[] b1 = new Buch[4];
        b1[0] = new Buch("Meier", "Meier", 300, 80.99);
        b1[1] = new Buch("Meier2", "Meier2", 300, 80.99);
        b1[2] = null;
        b1[3] = new Buch("Meier3", "Meier3", 300, 80.99);

        druckeBuecher(b1);
    }

    public static class Buch {

        private String autor;
        private String titel;
        private int seitenzahl;
        private double preis;

        public Buch(String autor, String titel, int seitenzahl, double preis) {
            this.autor = autor;
            this.titel = titel;
            this.seitenzahl = seitenzahl;
            this.preis = preis;
        }

        public String getAutor() {
            return autor;
        }

        public void setAutor(String autor) {
            this.autor = autor;
        }

        public double getPreis() {
            return preis;
        }

        public void setPreis(double preis) {
            this.preis = preis;
        }

        public int getSeitenzahl() {
            return seitenzahl;
        }

        public void setSeitenzahl(int seitenzahl) {
            this.seitenzahl = seitenzahl;
        }

        public String getTitel() {
            return titel;
        }

        public void setTitel(String titel) {
            this.titel = titel;
        }

        @Override
        public String toString() {
            return "[" + titel + "]";
        }
    }
}

 

[El Kabong]

@Raum114

Das Beispiel mit dem Konstruktor bekomme ich auch so hin. Nur mich wurmt es das es doch auch ohne Konstruktor funktionieren muss.

 

[Raum114]

Dann musst du zu C++ wechseln, dort geht es ohne.
Wenn man will dass, das Array fertig initialisiert ist, muss man sich in Java schon eine Klasse bauen die das macht z.B. class FeritgInitalisiertesBuchArray { ... }
new FeritgInitalisiertesBuchArray.get(0).setPreis(...) usw..

 

[Firephoenix]

Hi,
nochmal zur genaueren Erklärung:

mit Buch[] bla = new Buch[4] reservierst du im Array ja nur Speicher für Bücher, es werden aber
keine Buchobjekte erzeugt.
Das wäre in etwa so wie dieser Code:

Buch a1;
Buch a2;
Buch a3;
a1.setIrgendwas();

a1 ist hier null, daher kann man nichts setzen, wie das Array aufgebaut wird wurde oben ja schon erklärt.

Dann zu der Schleife (Raum hat direkt eine foreach draus gemacht )

for (int i=0; i<=buecher.length; i++) {
        buch = buecher[i];
            if (buecher != null) {
                System.out.println(buch.getAutor() + buch.getTitel() + buch.getPreis());
            
        }
        }

überleg mal was beim letzten durchlauf passiert. i<= length, deine Länge ist 4, das heißt du hast Objekte von 0 -> 3. in der nächsten Zeile greifst du auf das i-te Feld zu und spätestens hier knallt die ArrayIndexOutOfBounds
beim Array durchlaufen mit for immer nach dem Schema:

for(int i = 0; i < length; i++)

rückwärts auslesen wäre so in etwa:

for(int i = length-1; i >= 0; i--)

Gruß