Co poprawić w tym zadaniu?

Question

http://www.jkozak.pl/przedmioty/java/zadania7.pdf

 

błąd:

Exception in thread "main" java.lang.NullPointerException
    at Zamówienie.toString(Zamówienie.java:28)
    at Test.main(Test.java:7)

 

public class Pozycja {

  

    String nazwaTowaru;

    int ileSztuk;

    double cena;

     

    public Pozycja(double cena, int ileSztuk, String nazwaTowaru){

        this.cena=cena;

        this.ileSztuk=ileSztuk;

        this.nazwaTowaru=nazwaTowaru;

    }

     

    double obliczWartosc(){

        return this.cena*this.ileSztuk;

    }

     

    public String toString(){

        String a="";

       a+=nazwaTowaru;

       return a;

    }

}



___________________________________________





public class Zamówienie {

     

  Pozycja[] pozycje;

  int ileDodanych=0;

  int maksRozmiar;

  double suma;

  public Zamówienie(){

      maksRozmiar=10;

      pozycje = new Pozycja[maksRozmiar];

  }

   

  public void dodajPozycje(Pozycja p){

      pozycje[ileDodanych]=p;

  }

   

  double ObliczWartosc(){

      suma=0;

      for(Pozycja p: pozycje){

          suma+= p.obliczWartosc();

      }

      return suma;

  }

   

  public String toString(){

      String a="";

       for(Pozycja p: pozycje){

          a+=(p.toString());

       }

       a+="Wartość zamówienia wynosi" + ObliczWartosc();

       return a;

     }

       

   

}

__________________________________________________



public class Test {

 public static void main(String[] args){

     Pozycja a = new Pozycja(3.20, 2, "kupa");

     Zamówienie z = new Zamówienie();

     z.dodajPozycje(a);

     System.out.println(z.toString());

 }

}

 

Answer 1

NullPointerException jest wyrzucany, gdy odczytujesz lub zmieniasz obiekt, który jest nullem.

Zauważ co robisz:

1. Tworzysz tablicę o rozmiarze 10 w klasie Zamówienie.
2. Gdy dodajesz pozycję, zawsze dodajesz ją w miejsce 0 tablicy.
3. W metodzie toString iterujesz po całej tablicy (10 miejsc), ale tylko te 1 (zerowe) nie jest nullem.

Comments

Dziękuję działa ;)

btw. Myślałem że for each jeśli nie ma obiektu w tablicy to przechodzi tylko przez wszystkie miejsca które są wypełnione, ale jak widać trzeba czasami porzucić teorię i zacząć ćwiczyć :D

---

PS. Chyba lepiej jest to zrobić na liście zamiast na tablicy?

---

Tak, też mi to przeszło przez myśl