Typebound Objekte einer Arraylist hinzufügen

Diskutiere Typebound Objekte einer Arraylist hinzufügen im Java Basics - Anfänger-Themen Bereich.
N

nikva

Hallo Java Forum,
Ich soll für eine Aufgabe eine Methode implementieren um Waren einer Palette hinzuzufügen. Die Waren sollen von "konkreter Art" also typgebunden sein.
Nun habe ich aber das problem, dass ich die Methode add der ArrayList nur nutzen kann wenn ich Objekte des Typs <T> hinzufüge, aber wenn ich das dahingend ändere kann ich nicht mehr die Methode getWeight() aus der Klasse Good verwenden. Ich hoffe mir kann jemand zeigen wo mein Fehler liegt.

Java:
package edu.hm.cs.swe2.generics;

import java.util.ArrayList;

import edu.hm.cs.swe2.exceptions.NothingToAddException;
import edu.hm.cs.swe2.exceptions.StillageAlreadyClosedForShippingException;
import edu.hm.cs.swe2.exceptions.StillageWouldBeOverweightException;

public class Stillage<T> {

    private int maxWeight;
    private ArrayList<T> collection;
    private boolean readyToShip = false;

    public Stillage(int maxWeight) {
        this.maxWeight = maxWeight;
    }

    public String toString() {
        return "Lieferschein /n" + this.collection;
    }

    public void printListOfContents() {
        System.out.print(this.toString());
    }

    private int getLoadedWeight() {
        collection.forEach(null);

    }

    public boolean fitsOntoStillage(T good) {
        if (maxWeight >= (good.getWeight() + this.getLoadedWeight())) { //hier werde ich aufgefordert einen cast zu adden
            return true;
        } else {
            return false;
        }
    }

    public void addGood(T good) throws StillageAlreadyClosedForShippingException, NothingToAddException,
            StillageWouldBeOverweightException {
        if (this.readyToShip) {
            throw new StillageAlreadyClosedForShippingException();
        }
        if (good == null) {
            throw new NothingToAddException();
        }
        if (this.fitsOntoStillage(good)) {
            int overweight = good.getWeight() - (this.maxWeight - this.getLoadedWeight()); //hier werde ich aufgefordert einen cast zu adden
            throw new StillageWouldBeOverweightException();
            // hier noch Übergewicht ausgeben
        }
        collection.add(good);
        if (this.maxWeight == this.getLoadedWeight()) {
            this.readyToShip = true;
        }
    }

    public ArrayList<T> loadStillage(T... goods) {
        ArrayList<T> notLoadedGoods = new ArrayList<T>();
        int counter = 0;
        try {
            for (int i = 0; i < goods.length; i++) {
                counter += 1;
                this.addGood(goods[i]);
            }
        } catch (NothingToAddException e) {
            e.printStackTrace();
            System.out.println("");
        } catch (StillageAlreadyClosedForShippingException e) {
            e.printStackTrace();
            System.out.println("");
            notLoadedGoods.add(goods[counter - 1]);
        } catch (StillageWouldBeOverweightException e) {
            e.printStackTrace();
            System.out.println("");
            notLoadedGoods.add(goods[counter - 1]);
        }
        if (notLoadedGoods.isEmpty()) {
            return null;
        }
        return notLoadedGoods;
    }
}
Java:
package edu.hm.cs.swe2.generics;

public class Good<T> {

    int weight;
    String description;

    public Good(int weight, String description) {
        this.weight = weight;
        this.description = description;
    }

    public int getWeight() {
        return weight;
    }

    public String toString() {
        return "- " + this.description + ", " + "Gesamtgewicht " + this.weight + " kg";

    }

}
 
H

httpdigest

Wahrscheinlich möchtest du in der Klasse "Stillage" eher sagen, dass "T mindestens ein Good ist", also:
Java:
public class Stillage<T extends Good> { ... }
Und ich wüsste nicht, wieso die Klasse Good selbst typparametrisiert sein soll. Da bringt dir ein Typparameter ja nichts.
 
N

nikva

Danke für die schnelle Hilfe!
Und ich wüsste nicht, wieso die Klasse Good selbst typparametrisiert sein soll. Da bringt dir ein Typparameter ja nichts.
in der Aufgabenstellung stand
"In der Realität gibt es keine allgemeinen Waren, sondern nur Waren konkreter Art. "
was mich wohl dazu gebracht hat die Klasse erstmal zu typparametisieren.

Ich habe ein weiteres Problem: In der methode getLoadedWeight will ich den Wert weight von allen Goods in der Arraylist auslesen und dann addieren. Wie mache ich das am besten ohne dass ich jedes mal neu das gesamtgewicht in der for schleife erzeuge sondern alle Werte nacheinander aufaddiere?
 
H

httpdigest

in der Aufgabenstellung stand
"In der Realität gibt es keine allgemeinen Waren, sondern nur Waren konkreter Art. "
was mich wohl dazu gebracht hat die Klasse erstmal zu typparametisieren.
Verstehe. Diese Aussage in der Aufgabenstellung würde dann wohl eher heissen, dass die Klasse "Good" eine abstrakte Klasse ist und die konkreten Waren nicht-abstrakte Unterklassen von Good sind.
 
H

httpdigest

Ich habe ein weiteres Problem: In der methode getLoadedWeight will ich den Wert weight von allen Goods in der Arraylist auslesen und dann addieren. Wie mache ich das am besten ohne dass ich jedes mal neu das gesamtgewicht in der for schleife erzeuge sondern alle Werte nacheinander aufaddiere?
Java:
return collection.stream().mapToInt(Good::getWeight).sum();
 
N

nikva

Wenn ich das Programm jetzt ausführe wird bei mir immer nur die erste winecrate ins Array gefüllt, egal wie ich das max Gewicht einstelle.
Die Schleifen fangen bei 0 an und gehen komplett durch die Arraylist also müsste das eigentlich passen. Oder liegt der Fehler vielleicht in der printListOfContents Methode?
Java:
package edu.hm.cs.swe2;

import edu.hm.cs.swe2.generics.Stillage;
import edu.hm.cs.swe2.generics.WineCrate;

public class Main {

    public static void main(String[] args) {
        WineCrate wineCrate1 = new WineCrate("Chardonnay", 10, true);
        WineCrate wineCrate2 = new WineCrate("Rioja", 10, false);

        Stillage<WineCrate> wineStillage = new Stillage<WineCrate>(100);
        wineStillage.loadStillage(wineCrate1, wineCrate2);
        wineStillage.printListOfContents();
    }

}
Ausgabe:
Lieferschein:
- Chardonnay, Gesamtgewicht 20 kg, 10 Flaschen

müsste aber auch den Rioja ausgeben

Java:
package edu.hm.cs.swe2.generics;

import java.util.ArrayList;

import edu.hm.cs.swe2.exceptions.NothingToAddException;
import edu.hm.cs.swe2.exceptions.StillageAlreadyClosedForShippingException;
import edu.hm.cs.swe2.exceptions.StillageWouldBeOverweightException;

public class Stillage<T extends Good> {

    private int maxWeight;
    private ArrayList<T> collection = new ArrayList<T>();
    private boolean readyToShip = false;

    public Stillage(int maxWeight) {
        this.maxWeight = maxWeight;
    }

    public String toString() {
        String output = "Lieferschein:\n";
        for (int i = 0; i < collection.size(); i++) {
            output += collection.get(i).toString();
        }
        return output;
    }

    public void printListOfContents() {
        System.out.print(this.toString());
    }

    private int getLoadedWeight() {
        return collection.stream().mapToInt(Good::getWeight).sum();
    }

    public boolean fitsOntoStillage(T good) {
        if (maxWeight >= (good.getWeight() + this.getLoadedWeight())) {
            return true;
        } else {
            return false;
        }
    }

    public void addGood(T good) throws StillageAlreadyClosedForShippingException, NothingToAddException,
            StillageWouldBeOverweightException {
        if (this.readyToShip) {
            throw new StillageAlreadyClosedForShippingException();
        }
        if (good == null) {
            throw new NothingToAddException();
        }
        if (!this.fitsOntoStillage(good)) {
            int overweight = good.getWeight() - (this.maxWeight - this.getLoadedWeight());
            throw new StillageWouldBeOverweightException(overweight);
        }
        collection.add(good);
        if (this.maxWeight == this.getLoadedWeight()) {
            this.readyToShip = true;
        }
    }

    public ArrayList<T> loadStillage(T... goods) {
        ArrayList<T> notLoadedGoods = new ArrayList<T>();
        for (int i = 0; i < goods.length; i++) {
            try {
                this.addGood(goods[i]);

            } catch (NothingToAddException e) {
                e.printStackTrace();
                System.out.println("");
            } catch (StillageAlreadyClosedForShippingException e) {
                e.printStackTrace();
                System.out.println("");
                notLoadedGoods.add(goods[i]);
            } catch (StillageWouldBeOverweightException e) {
                e.printStackTrace();
                System.out.println("");
                notLoadedGoods.add(goods[i]);
            }
            if (notLoadedGoods.isEmpty()) {
                return null;
            }
        }
        return notLoadedGoods;
    }
}
Java:
package edu.hm.cs.swe2.generics;

public class WineCrate extends Good {

    private int numberOfBottles;

    public WineCrate(String description, int numberOfBottles, boolean isEuBottle) {
        super(calculateWeight(numberOfBottles, isEuBottle), description);
        this.numberOfBottles = numberOfBottles;
    }

    private static int calculateWeight(int numberOfBottles, boolean isEUBottle) {
        if (isEUBottle) {
            return numberOfBottles * 2;
        } else {
            return numberOfBottles * 3;
        }
    }

    public String toString() {
        return "- " + this.description + ", " + "Gesamtgewicht " + this.weight + " kg, " + numberOfBottles
                + " Flaschen";

    }

}
Java:
package edu.hm.cs.swe2.generics;

public class Good {

    int weight;
    String description;

    public Good(int weight, String description) {
        this.weight = weight;
        this.description = description;
    }

    public int getWeight() {
        return weight;
    }

    public String toString() {
        return "- " + this.description + ", " + "Gesamtgewicht " + this.weight + " kg";

    }

}
 
Zuletzt bearbeitet:
H

httpdigest

Liegt wahrscheinlich hieran (in der Schleife in loadStillage):
Java:
            if (notLoadedGoods.isEmpty()) {
                return null;
            }
Du brichst die gesamte Schleife und Methode ja ab, wenn die Liste der notLoadedGoods LEER ist.
Generell: Nutze einen Debugger in einer IDE. Da kannst du jede Zeile Schritt für Schritt durchgehen lassen und alle Variablen prüfen.
 
Thema: 

Typebound Objekte einer Arraylist hinzufügen

Passende Stellenanzeigen aus deiner Region:
Anzeige

Neue Themen

Anzeige

Anzeige
Oben