Przekazywanie obiektów do tablicy.

Question

Witam.

Mam problem z zadaniem zamówienie: http://www.jkozak.pl/przedmioty/java/zadania7.pdf .

Problemem jest przekazanie obiektu z klasy Pozycja do metody z klasy Zamówienie "void dodajPozycje(Pozycja p)". Nie bardzo wiem jak to ma wyglądać. Chciałem w klasie Zamówienie zrobić tablicę obiektów Pozycja i w/w metodzie dodawać kolejno obiekty. Niestety coś mi to nie działa. Może ktoś mnie naprowadzić co i jak :D?

W klasie Zamówienie nie działa też metoda obliczWartosc. Tzn. bez pętli for jest ok, a z problem przy zwróceniu 'x'. 

Class Pozycja-

public class Pozycja {
    private String nazwaTowaru;
    private static int ileSztuk;
    private static double cena;
    
    public Pozycja(String n, int s,double c){
        this.nazwaTowaru=n;this.ileSztuk=s;this.cena=c;
    }
    
    public double obliczWartosc(){
        double x=ileSztuk*cena;
        return x;
    }
    public String toString(){
        return nazwaTowaru+"         "+cena+"zł  "+ileSztuk+"szt.    "+obliczWartosc()+"zł";
    }
    
}

Class Zamówienie- 

public class Zamownienie {
    private static int ileDanych,maxRozmiar,nextID;
    private Pozycja[] lista;
    
    public Zamownienie(){this(10);}
    public Zamownienie(int maxRozmiar){
        this.maxRozmiar=maxRozmiar;
    }
   
    public void dodajPozycje(Pozycja p){
        this.lista[nextID]=p;
        nextID++;
    }
    public double obliczWartosc(){
        double x;
       for(int i=0;i<lista.length;i++)
            x+=lista[nextID].obliczWartosc();
        return x;  
    }
    public String toString(){
        return "  Razem: "+obliczWartosc();
    }
    static { nextID=0;}
}

Main- 

package zadanie.zamownienie;
public class ZadanieZamownienie {
    
    public static void main(String[] args) {
        Pozycja k=new Pozycja("Cukier",3,5);
        System.out.println(k);
        Pozycja x=new Pozycja("Chleb",2,4);
        System.out.println(x);
        
        Zamownienie z= new Zamownienie(20);
        z.dodajPozycje(k);
        z.dodajPozycje(x);
        System.out.println(z);
    }
    
}

 

Comments

co gdy next_id będzie większe niż rozmiar tablicy ?

---

Na wejściu jest 0 dopiero potem inkrementacja, więc nie powinno się tak zdarzyć. No ale w klasie Zamówienie metoda obliczWartość powinna zaczynać się od 0 a nie nextID to tu jest błąd :D. Pozostał jednak ten z przekazywaniem obiektów w tablice :/

Answer 1

Cześć!

Problemem jest przekazanie obiektu z klasy Pozycja do metody z klasy Zamówienie "void dodajPozycje(Pozycja p)". Nie bardzo wiem jak to ma wyglądać. Chciałem w klasie Zamówienie zrobić tablicę obiektów Pozycja i w/w metodzie dodawać kolejno obiekty. Niestety coś mi to nie działa. Może ktoś mnie naprowadzić co i jak :D?

Zgadza się. Problemem było przekazanie obiektu do tablicy. Popatrz na ten kod:

private Pozycja[] lista;

Utworzyłeś zmienną referencyjną do obiektu tablicowego lista, ale....! Referencja jest pusta. Co chcesz dodawać do tablicy która jeszcze nie istnieje? Ba! nawet podajesz jej konkretny index :P

public void dodajPozycje(Pozycja p){
        this.lista[nextID]=p;
        nextID++;
    }

 Podczas kompilowania programu na pewno dostałeś NullPointerException. Dlaczego? Bo nigdzie w Twoim kodzie nie przypisałeś do referencji utworzonego obiektu tablicowego o określonym rozmiarze.
Zrób to w konstruktorze w ten sposób:

 

public Zamownienie(int maxRozmiar){
        this.maxRozmiar=maxRozmiar;
        this.lista = new Pozycja[maxRozmiar];
    }

 

 

Comments

Faktycznie. Zapomniałem o tym. Dziękuję za pomoc :).

---

public void dodajPozycje(Pozycja p){
        this.lista[nextID]=p;
        nextID++;
}

Tyle, że teraz mam w tablicy nazwę obiektu ale bez jego wartości (bez pól). Tj. ostatni utworzony obiekt typu Pozycja dodaje w mojej tablicy takie same wartości cen i sztuk w każdym elemencie tablicy zmienia się tylko nazwa obiektu. 

---

Zauważyłem, że już w pewien sposób rozwiązałeś swój problem w innym wątku ;) Zmienne statyczne i wszystko jasne :P