Przekazanie metody z argumentem do toString

Question

Witam, mam pewien problem, posiadam dwie klasy Position, Order. W klasie Order mam stworzoną metodę calculateProduct(name), gdzie za pomocą Hashmapy obliczam wartość danej pozycji. Chciałbym wartość tej pozycji umieścić w metodzie toString w klasie Order. Niestety z racji tego że metoda toString nie przyjmuje argumentu, nie mam jak przekazać wartości z metody CalculateProduct. Poniżej kod:

package programming.com.pl;

import java.util.Objects;

public class Position  {

    private String name;
    private double price = 0;
    private int quantity = 0;


    public Position(String name, double price, int quantity){
        this.name = name;
        this.price = price;
        this.quantity = quantity;
    }

    public double getPrice() {
        return price;
    }

    public int getQuantity() {
        return quantity;
    }

    public String toString(){
        String str = String.format("%4s,%4s zł,%2s szt.", name,price,quantity);
        return  str;
    }

    @Override
    public boolean equals(Object other){
        if(this == other){
            return true;
        }
        if( other instanceof Position){
            Position p = (Position) other;
            return name.equals(p.name) && price == p.price && quantity == p.quantity;
        }
        return false;
    }
    @Override
    public int hashCode(){
        return Objects.hash(name,price,quantity);
    }






}


package programming.com.pl;

import java.util.ArrayList;
import java.util.HashMap;

public class Order {
    //final private ArrayList<Position> positions = new ArrayList<>();
    final private HashMap<String,Position> positions = new HashMap<String,Position>();




    double calculateProduct(String name){
        double value = positions.get(name).getPrice()*positions.get(name).getQuantity();
       return value;
    }


    void addPosition(String name, Position p) {
        positions.put(name,p);
    }

    void deletePosition(String name) {
        positions.remove(name);
    }



    public String toString() {
        return String.format("%f",calculateProduct(name));
    }

}

 

 

 

Comments

W linii 27 masz dziwnie - same %s, gdy tylko 1 argument to String.

toString() też jest @override.

Zawsze możesz zdefiniować inne metodę o takiej samej nazwie, ale z innymi parametrami. Oczywiście na nowa metoda nie zastąpi tej pierwotnej, będzie istnieć w sumie niezależnie i będzie używana gdy podasz odpowiednie parametry.

Answer 1

Napisać własną implementacje metody przyjmującej parametr. Np taką

public class Order {
    //final private ArrayList<Position> positions = new ArrayList<>();
    final private HashMap<String,Position> positions = new HashMap<String,Position>()

 //... pominiety kod
 
    public String valueToString(String name) {
        return String.format("%f",calculateProduct(name));
    }
 
}

a jeśli mass użyc dokładnie to string to odwołać się do zmiennej która przetrzyma to 

public String toString() {
    String result = this.mapa.getElementzMapy(klucz) + "," + this.mapa.getElementzMapy(klucz); 
    return result;
}

korzystajac z this -> odwolujacego sie do "pola klasy tego obiektu, tej instancji" zabezpieczyc jednoczesnie przed faktem BRAKU KLUCZA, BRAKU ELEMENTU i upewnieniu sie ze elementy sa juz dodane 
 

Comments

Dziękuje bardzo za rozjaśnienie problemu, zastanawiałem się czy własna implementacja toString(parametr) jest zgodna z dobrymi praktykami programowania. Jeśli chodzi o błędy w toString %s już poprawiam, rzeczywiście popełniłem tam błąd :)

---

sama w sobie implementacja poprzez nadpisanie niestety nie, nie możesz też tej metody przeciążyć, ale własne metody możesz dobrowolnie pisać, nawet w taki sposób, że wewnętrznie wywołasz toString

---

Tylko widzę pewien zgrzyt projektowy - metoda toString ma wypisać obiekt, cały obiekt.

Jeśli wypisujesz tam coś na kształt cena*wartość to dlaczego dotyczy to jednej pozycji zamówienia a nie np. wartości całego zamówienia (suma po pozycjach).

Wypisanie wartości pojedynczej pozycji powinno być metodą klasy pozycja.

---

tzn ja odwołałem się tylko do pytania, bez wglądu w mechanizm implementacji, dla mnie sam fakt takiego podejścia wypisania czegoś, co potem i tak jest w kolekcji nie ma sensu.

---

Rozumiem, z drugiej strony, czy wypisywanie w Klasie pozycja jest odpowiednie? Ogólnie robiąc to zadanie słyszałem, że w Klasie pozycja powinna być tylko nazwa i cena, natomiast w Zamówienie: ilość, wartość zamówienia. Teraz sam już zgupiałem.

---

@Wiciorny,  Z drugiej strony spróbowałem sposobu :

public String toString() {
    String result = this.mapa.getElementzMapy(klucz) + "," + this.mapa.getElementzMapy(klucz); 
    return result;
}

Niestety, ale problem polega na tym, że w miejscu(klucz), trzeba umieścić name, który tak jak mówiłem jest niewidoczny dla metody toString.

 

---

jest niewidoczny bo go tak przekazujesz,  to słowo KLUCZ u mnie to nie nazwa zmiennej samej w sobie, a przyklad klucz musisz przekazać jako POLE KLASY TEGO OBIKETU.
Odwołuje się do poczytania o 'this'
nie potrafisz tego zaimplementować po prostu, możesz nawet zrobić dodatkową metodę która w runtime to zwróci.
https://stackoverflow.com/questions/27143951/how-make-tostring-method-return-super-class-private-fields-also-along-with-its

---

@Wiciorny,  Posiłkując się Twoim tematem ze Stackoverflow zrobiłem coś takiego, tylko nie wiem czy to miałeś dokładnie na myśli?

 

package programming.com.pl;

import java.util.ArrayList;
import java.util.HashMap;

public class Order extends Position {
    //final private ArrayList<Position> positions = new ArrayList<>();
    final private HashMap<String,Position> positions = new HashMap<String,Position>();

    public Order(String name, double price, int quantity) {
        super(name, price, quantity);
    }


    double calculateProduct(String name){
        double value = positions.get(name).getPrice()*positions.get(name).getQuantity();
       return value;
    }


    void addPosition(String name, Position p) {
        positions.put(name,p);
    }

    void deletePosition(String name) {
        positions.remove(name);
    }


    public String toString() {
        String result = this.positions.get(getName()) +"," + this.positions.get(getName());
        return result;
    }

}