Dodawanie pozycji oraz wyświetlanie zamówienia

Question

Witam, zaczynam przygodę z programowaniem obiektowym i trafiłem na takie zadanie:

public class Pozycja {

    String nazwaTowaru;
    int ileSztuk;
    double cena;
    double wartosc;
    String orderSummary;

    Pozycja(String nazwaTowaru, double cena, int ileSztuk){
        this.nazwaTowaru = nazwaTowaru;
        this.cena= cena;
        this.ileSztuk = ileSztuk;
    }


    double obliczWartosc(){
        wartosc = ileSztuk*cena;
        return wartosc;
    }

   public String toString(){
        String orderSummary = String.format("%-20s   %10szł    %4d szt.  %10szł", nazwaTowaru, cena, ileSztuk, wartosc);
        return orderSummary;
    }




}

public class Zamówienie {
    int ileDodanych;
    int maksRozmiar;
    Pozycja[] pozycje = new Pozycja[maksRozmiar];
    double suma = 0;

    Zamówienie() {
        maksRozmiar = 10;
    }

    Zamówienie(int maksRozmiar) {
        this.maksRozmiar = maksRozmiar;
        pozycje = new Pozycja[this.maksRozmiar];
    }

    void dodajPozycje(Pozycja p) {
        if (ileDodanych <= maksRozmiar) {
            pozycje[ileDodanych] = p;
            ileDodanych++;
        }
    }

    double obliczWartosc(){
            for(int i = 0;i<ileDodanych;i++){
                suma = pozycje[i].obliczWartosc();
            }
            return suma;
    }

    public String toString(){
    
    }

Mój problem to klasa zamówienie, nie mam pojęcia jak zrobić metodę toString w tej klasie, w dodatku nie mam pewności do poprawności dodajPozycje i obliczWartosc(). Proszę o jakieś podpowiedzi, siedzę już trochę nad tym i nadal nie mam pomysłu.

Answer 1

Jest dobrze. Jedyne co nie jest do końca dobre to w klasie Zamowienie zmienna:

double suma = 0;

raczej nie musi być klasowa, zadeklaruj ją wewnątrz metody obliczWartosc. Podobnie z:

String orderSummary;

Co do toString. Po prostu jeśli umiesz wyświetlić tablicę, np. za pomocą System.out.println to właściwie trzeba zrobić to samo, tylko System.out.println będzie String'em, do którego będziesz dodawał kolejny element tablicy tak żeby po przejściu jej całej mieć jeden długi string i na końcu zwrócisz go z metody. Do sklejenia stringa najlepiej będzie jeśli użyjesz StringBuilder (poszukaj w necie jak go użyć).

Answer 2

Poradziłem sobie z zadaniem, w dodatku zmieniłem tablicę na listę. Tylko teraz mam inny problem, chciałbym edytować pozycję, czyli(nazwę towaru, cene oraz liczbę sztuk) za pomocą metody void edytujPozycje(int index). Myślałem o metodzie get w linkedlist, ale chyba nie tędy droga. Kod poniżej:

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

        Pozycja pozycja1 = new Pozycja("Chleb",2,2);
        System.out.println(pozycja1);
        Pozycja pozycja2 = new Pozycja("Samochód",100000,1);
        System.out.println(pozycja2);




        Zamówienie zamówienie1 = new Zamówienie();
        zamówienie1.dodajPozycje(pozycja1);
        zamówienie1.dodajPozycje(pozycja2);
        zamówienie1.usunPozycje(1);
        zamówienie1.edytujPozycje(0);
        System.out.println(zamówienie1);

    }

}

public class Pozycja {

   private String nazwaTowaru;
   private int ileSztuk;
   private double cena;
   private double wartosc;

    Pozycja(String nazwaTowaru, double cena, int ileSztuk){
        this.nazwaTowaru = nazwaTowaru;
        this.cena = cena;
        this.ileSztuk = ileSztuk;
        this.wartosc = ileSztuk*cena;
    }


    double podajWartosc(){
        return wartosc;
    }

   public String toString(){
        String spisPozycji = String.format("%-20s   %10szł    %4d szt.  %10szł \n", nazwaTowaru, cena, ileSztuk, podajWartosc());
        return spisPozycji;
    }




}

import java.util.List;
import java.util.LinkedList;

public class Zamówienie {

   final private List<Pozycja> listapozycji;

   Zamówienie(){
       listapozycji = new LinkedList<>();
   }


    void dodajPozycje(Pozycja p){
          listapozycji.add(p);
    }



    double obliczWartosc(){
        double suma = 0 ;
        for (Pozycja p : listapozycji){
            suma += p.podajWartosc();
        }
        return suma;
    }

    public String toString(){
        System.out.println("Zamówienie: "  );
        for (Pozycja p : listapozycji) {
            System.out.println(p);
        }
        return "Razem: \n" +obliczWartosc();
    }

    void usunPozycje(int index){
        listapozycji.remove(index);
    }
    void edytujPozycje(int index){
    listapozycji.get(0);
    }


}

 

Comments

Generalnie to programuje hobbystycznie. Mam trochę braki w wiedzy, ale często stoję w miejscu i nie wiem co dalej, chociaż analizuje debuggerem. W main coś nie tak? Dlaczego nie rozumiem:o. W każdym razie dziękuję za odpowiedzi :)

---

Jeśli chodzi o błędy w mainie, to chyba  rozumiem to w ten sposób usuwam indeks z listy zamówień i edytuje dlatego wyświetla mi się tylko jedna pozycja w zamówienie. Niestety dalej nie rozumiem aluzji dlaczego toString w klasie zamówienie jest źle?

---

Chodzi o to że toString w klasie zamówenie wypisuje także jakieś dane na konsoli. Dużo intuicyjnej i poprawniej byłoby gdyby metoda toString przygotowywała to co ma zostać wydrukowane.

Opis metody toString z dokumentacji:

Returns a string representation of the object. In general, the toString method returns a string that "textually represents" this object. The result should be a concise but informative representation that is easy for a person to read. It is recommended that all subclasses override this method.

 https://docs.oracle.com/en/java/javase/13/docs/api/java.base/java/lang/Object.html#toString()

---

Ok, a możesz mi jeszcze podpowiedzieć jedną rzecz? W metodzie dodajPozycje(Pozycja p), oprócz tego, że pozycja dodawana jest tylko wtedy gdy się nie powtarza, to jeszcze założenie jest takie, że jeśli się powtórzy, to należy zwiększyć liczbę sztuk pozycji powtarzanej. Tak też zrobiłem. Utworzyłem metodę w klasie Pozycja, która inkrementuje sztuki. Pętla w metodzie dodajPozycję nie wchodzi w grę(brak sensownego warunku by wykonać pętle). Metody z dwoma parametrami dodajPozycje(Pozycja,index) też nie biorę pod uwagę. Brak mi indeksu przy każdym wywołaniu metody, dzięki któremu można by pobrać pozycję i przypisać zwiększoną ilość sztuk.

void  zwiekszIleSztuk(){
        ileSztuk++;
    }
  

void dodajPozycje(Pozycja p) {
        if (!listapozycji.contains(p)) {
            listapozycji.add(p);
        } else {
            listapozycji.get(0).zwiekszIleSztuk;
        }
    }
}

 

---

Sprawdź w javadoc'u jakie metody ma interfejs List, może coś będzie pasować.