Erste Schritte Berechnung des Paketportos - Problem/Frage

[Serano11]

Abend,

ich habe ein Problem bei der folgenden Aufgabe

Hinweis: Angaben sind in int Milimeter und Gram, Euro in double

"
......( alles davor funktioniert und habe ich gemacht aber das Gurtmaß bereitet mir Probleme )
• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 5kg: 5,99 EUR
• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 10kg: 7,99 EUR"
• Paket bis 120x60x60 und bis 31,5kg: 14,99 EUR

und zwar habe ich folgendes:
".....
} else if ((pack.length <= 1200) && (pack.width <= 600) && (pack.height <= 600) &&
((pack.length + (2 * pack.width) + (2 * pack.height) ) <= 300) && (pack.weight <= 5000)) {
shippingCosts = 5.99;

} else if ((pack.length <= 1200) && (pack.width <= 600) && (pack.height <= 600)
&& ((pack.length + (2 * pack.width) + (2 * pack.height) ) <= 300) && (pack.weight <= 10000 ) {
shippingCosts = 7.99;

} else {
shippingCosts = 14.99;
}
return shippingCosts;
}

}
"
Entschuldigt, wenn der Code schlecht dargestellt wird.

Mein Problem ist, dass es den Schritt zu 5,99€ einfach überspringt. Es fehlen Befehle aber ich weiß nicht welche. Edit: Ich verstehe, was und wo mein Fehler ist, lediglich die Lösung für den "Max. Gurtmaß" finde ich nicht.

Könntet ihr mir bitte helfen? Ich komme echt nicht weiter.

Grüße

Anmerkung: Bei Fehlern entschuldige ich mich ( wie falsches Thema o.ä. ).

 

[JStein52]

((pack.length + (2 * pack.width) + (2 * pack.height) ) <= 300)

Wenn alles Millimeter sind, muss es dann nicht <= 3000 heissen ?

 

[Serano11]

Wenn alles Millimeter sind, muss es dann nicht <= 3000 heissen ?

Ja, das stimmt! - Danke, den habe ich übersehen!

Edit: Gibt trotzdem den Betrag 14,99€ wieder.

 

[JStein52]

Mit welchen Input-Daten ?

 

[Serano11]

Mit welchen Input-Daten ?

Sind damit die Eingabe-daten gemeint? ( sorry, wenn ich eine dumme Frage ist, wie gesagt, bin Anfänger )
Weil eine Eingabe Möglichkeit ist vorhanden, da eine GUI erstellt wurde.

Edit: Eine weitere Frage: Ich habe beim GUI nun einen Button für Exit erstellt aber wie heißt der Befehl, dass beim Klicken des Buttons, das Fenster tatsächlich schließt?

 

[JStein52]

ich meinte wie sind die Paketdaten ?

 

[Serano11]

Das hier?:


public class Packet {

// length of package in millimeters
public int length;

// width of package in millimeters
public int width;

// height of package in millimeters
public int height;

// weight of package in grams
public int weight;

// constructor
public Packet(int length, int width, int height, int weight) {
this.length = length;
this.width = width;
this.height = height;
this.weight = weight;
}

 

[JStein52]

Nein, welche Werte hat das Paket (Länge, Breite, Höhe etc) bei deinem Test wo du sagst es ergibt immer 14,99€

 

[Xyz1]

• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 5kg: 5,99 EUR
• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 10kg: 7,99 EUR"
• Paket bis 120x60x60 und bis 31,5kg: 14,99 EUR

Mir fällt da auf, dass nur nach kg vergleicht werden muss......
(120x60x60 gleich, Gurtmaß insofern verstanden auch)

 

[Serano11]

Nein, welche Werte hat das Paket (Länge, Breite, Höhe etc) bei deinem Test wo du sagst es ergibt immer 14,99€

Ah entschuldige

Bei den Werten

- 250x150x90mm und 790 gramm = 3,89€
- 450x250x125mm und 1500 gramm = 4,39€ <- bis hier hin funktioniert es.

Mit den Werten

- 1200x600x600mm und 5000 gramm = 14,99€

- Die 7,99€ wird ignoriert.

Liegt es an "((pack.length + (2 * pack.width) + (2 * pack.height) ) <= 300) "?

Edit: DerWissende:

Wie kann ich es am Besten vergleichen lassen?

 

[JStein52]

Na rechne mal nach 1200 +2*600+2*600 = 3600 ! Also, Bedingung nicht erfüllt

 

[Serano11]

Na rechne mal nach 1200 +2*600+2*600 = 3600 ! Also, Bedingung nicht erfüllt

Das sind aber nur die Paketmaße, wie ist es dann mit den Gurtmaßen?

 

[JStein52]

Wie kann ich es am Besten vergleichen lassen?

Deine Vergleiche sind richtig !

 

[JStein52]

wie ist es dann mit den Gurtmaßen?

Na eben, die Gurtmasse passen nicht. Ist doch nicht <= 3000. Also ist 14,99 € richtig

 

[Serano11]

Größten Dank für deine Zeit! ( Sofern duzen in Ordnung ist )

Wie ist es mit der letzten Bedingung " <= 31500 gram "?

 

[JStein52]

Wie ist es mit der letzten Bedingung " <= 31500 gram "

Nun ja ... die fragst du ja nicht explizit ab sondern alles was bei den vorigen Vergleichen durchfällt kostet dann bei dir 14,99 € Du hast aber auch oben nicht gesagt was mit Paketen ist die grösser sind bzw. die schwerer als 31,5 kilo sind. Das müsstest du dir noch "ausdenken" und evtl. deien Abfragen erweitern.
Und duzen ist natürlich ok und hier üblich.

 

[Serano11]

Nun ja ... die fragst du ja nicht explizit ab sondern alles was bei den vorigen Vergleichen durchfällt kostet dann bei dir 14,99 € Du hast aber auch oben nicht gesagt was mit Paketen ist die grösser sind bzw. die schwerer als 31,5 kilo sind. Das müsstest du dir noch "ausdenken" und evtl. deien Abfragen erweitern.
Und duzen ist natürlich ok und hier üblich.

Danke dir.

ich habe es probiert mit:

} else if ((pack.length <= 1200) && (pack.width <= 600) && (pack.height <= 600)
&& ((pack.length + (2 * pack.width) + (2 * pack.height) ) <= 3000) && pack.weight >= 10000 && pack.weight <= 31500 ) {
shippingCosts = 14.99;
} else {
shippingCosts = 0;
}

Aber die "0" wäre jedoch falsch?

oder eine If-Bedingung, die per Println (" Paketgewicht zu hoch, wir führen keine Pakete mit einem Gewicht höher als 31,5 Kg!" ) o.ä.?

 

[Xyz1]

@Serano11 :

class Paket {
    public static void main(String[] args) {
        Random ra = new Random(0);
        for (int i = 0; i < 20; i++) {
            Paket pa = new Paket(ra.nextInt(130), ra.nextInt(60), ra.nextInt(60), ra.nextFloat() * 32);
            System.out.println(pa + "\t"+ pa.preis() );
        }
    }
    enum Preis {
        K(5.99), M(7.99), G(14.99), UNDEFINED(-1);
        private final double p;
        private Preis(double p) {
            this.p = p;
        }
        @Override
        public String toString() {
            return "Preis{" + "p=" + p + '}'; // hier ne schöne Ausgabe
        }
    }
    int L, B, H;
    float kg;
    public Paket(int L, int B, int H, float kg) {
        this.L = L;
        this.B = B;
        this.H = H;
        this.kg = kg;
    }
    int gurtmass() {
        return L + B + B + H + H;
    }
    Preis preis() {
        if (L <= 120 && B <= 60 && H <= 60) {
            if (gurtmass() <= 300) {
                if (kg <= 5) {
                    return Preis.K;
                } else if (kg <= 10) {
                    return Preis.M;
                } else if (kg <= 31.5) {
                    return Preis.G;
                } else {
                    return Preis.UNDEFINED;
                }
            } else {
                return Preis.G;
            }
        } else {
            return Preis.UNDEFINED;
        }
    }
    @Override
    public String toString() {
        return "Paket{" + "L=" + L + ", B=" + B + ", H=" + H + ", kg=" + kg + '}';
    }
}

Die Ausgabe schon gut:

Paket{L=70, B=28, H=49, kg=19.403046}	Preis{p=14.99}
Paket{L=65, B=53, H=11, kg=3.7442112}	Preis{p=5.99}
Paket{L=29, B=14, H=17, kg=8.088837}	Preis{p=7.99}
Paket{L=73, B=22, H=35, kg=31.450224}	Preis{p=14.99}
Paket{L=44, B=15, H=41, kg=5.631256}	Preis{p=7.99}
Paket{L=53, B=48, H=44, kg=11.775223}	Preis{p=14.99}
Paket{L=12, B=40, H=3, kg=0.04437256}	Preis{p=5.99}
Paket{L=62, B=23, H=5, kg=18.014389}	Preis{p=14.99}
Paket{L=77, B=47, H=2, kg=9.1489525}	Preis{p=7.99}
Paket{L=75, B=53, H=58, kg=31.25433}	Preis{p=14.99}
Paket{L=70, B=55, H=35, kg=27.765717}	Preis{p=14.99}
Paket{L=128, B=51, H=34, kg=18.241096}	Preis{p=-1.0}
Paket{L=32, B=42, H=19, kg=13.190968}	Preis{p=14.99}
Paket{L=26, B=6, H=27, kg=31.509571}	Preis{p=-1.0}
Paket{L=28, B=17, H=3, kg=5.951668}	Preis{p=7.99}
Paket{L=27, B=28, H=25, kg=21.47444}	Preis{p=14.99}
Paket{L=48, B=57, H=31, kg=11.49329}	Preis{p=14.99}
Paket{L=118, B=18, H=38, kg=21.339178}	Preis{p=14.99}
Paket{L=17, B=11, H=26, kg=26.11}	Preis{p=14.99}
Paket{L=67, B=42, H=55, kg=29.835777}	Preis{p=14.99}

Ich habe aber das Gefühl, wenn es nicht gegurtmaßt ist, noch etwas vergessen zu haben - (wenn es besonders schwer ist).
Also der Paketträger wäre vielleicht bezüglich des Gewichts übernachteilt.......

 

[Xyz1]

Überarbeitet:

/*
• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 5kg: 5,99 EUR
• Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 10kg: 7,99 EUR"
• Paket bis 120x60x60 und bis 31,5kg: 14,99 EUR
 */
class Paket {

    public static void main(String[] args) {
        Random ra = new Random(0);
        for (int i = 0; i < 20; i++) {
            Paket pa = new Paket(ra.nextInt(130), ra.nextInt(60), ra.nextInt(60), ra.nextFloat() * 32);
            System.out.println(pa + "\t" + pa.preis());
        }
    }

    enum Preis {
        K(5.99), M(7.99), G(14.99), UNDEFINED(-1);
        private final double p;

        private Preis(double p) {
            this.p = p;
        }

        @Override
        public String toString() {
            return "Preis{" + "p=" + p + '}'; // hier ne schöne Ausgabe
        }
    }
    int L, B, H;
    float kg;

    public Paket(int L, int B, int H, float kg) {
        this.L = L;
        this.B = B;
        this.H = H;
        this.kg = kg;
    }

    int gurtmass() {
        return L + B + B + H + H;
    }

    Preis preis() {
        int g = gurtmass();
        if (L <= 120 && B <= 60 && H <= 60) {
            if (kg <= 5) {
                if (g <= 300) {
                    return Preis.K;
                } else {
                    return Preis.G;
                }
            } else if (kg <= 10) {
                if (g <= 300) {
                    return Preis.M;
                } else {
                    return Preis.G;
                }
            } else if (kg <= 31.5) {
                // das Grutmas hier nicht mehr wichtig
                // schwer
                return Preis.G;
            } else {
                // zu schwer
                return Preis.UNDEFINED;
            }
        } else {
            // Mase ueberschritten
            return Preis.UNDEFINED;
        }
    }

    @Override
    public String toString() {
        return "Paket{" + "L=" + L + ", B=" + B + ", H=" + H + ", kg=" + kg + '}';
    }
}

Paket{L=70, B=28, H=49, kg=19.403046}	Preis{p=14.99}
Paket{L=65, B=53, H=11, kg=3.7442112}	Preis{p=5.99}
Paket{L=29, B=14, H=17, kg=8.088837}	Preis{p=7.99}
Paket{L=73, B=22, H=35, kg=31.450224}	Preis{p=14.99}
Paket{L=44, B=15, H=41, kg=5.631256}	Preis{p=7.99}
Paket{L=53, B=48, H=44, kg=11.775223}	Preis{p=14.99}
Paket{L=12, B=40, H=3, kg=0.04437256}	Preis{p=5.99}
Paket{L=62, B=23, H=5, kg=18.014389}	Preis{p=14.99}
Paket{L=77, B=47, H=2, kg=9.1489525}	Preis{p=7.99}
Paket{L=75, B=53, H=58, kg=31.25433}	Preis{p=14.99}
Paket{L=70, B=55, H=35, kg=27.765717}	Preis{p=14.99}
Paket{L=128, B=51, H=34, kg=18.241096}	Preis{p=-1.0}
Paket{L=32, B=42, H=19, kg=13.190968}	Preis{p=14.99}
Paket{L=26, B=6, H=27, kg=31.509571}	Preis{p=-1.0}
Paket{L=28, B=17, H=3, kg=5.951668}	Preis{p=7.99}
Paket{L=27, B=28, H=25, kg=21.47444}	Preis{p=14.99}
Paket{L=48, B=57, H=31, kg=11.49329}	Preis{p=14.99}
Paket{L=118, B=18, H=38, kg=21.339178}	Preis{p=14.99}
Paket{L=17, B=11, H=26, kg=26.11}	Preis{p=14.99}
Paket{L=67, B=42, H=55, kg=29.835777}	Preis{p=14.99}

Action findet in der Methode preis() statt. Methode preis() könnte optimiert werden. Stelle einfach einen Entscheidungsbaum auf. - Was sagt @JStein52 dazu?

 

[JStein52]

Ich würde sagen, preis() so lassen wie du sie jetzt hast. Ist doch übersichtlich und verständlich. Wenn du jetzt mit Entscheidungsbäumen anfängst verwirrt es nur und macht das einfache Problem wieder unübersichtlich.

 

[mrBrown]

Action findet in der Methode preis() statt. Methode preis() könnte optimiert werden. Stelle einfach einen Entscheidungsbaum auf. - Was sagt @JStein52 dazu?

Alternativ so :

import java.util.Random;
import java.util.function.Predicate;

/*
* Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 5kg: 5,99 EUR
* Paket bis 120x60x60 und max. Gurtmaß (=L + 2xB + 2xH) <= 300cm und bis 10kg: 7,99 EUR
* Paket bis 120x60x60 und bis 31,5kg: 14,99 EUR
*/
class Paket {

    public static void main(String[] args) {
        Random ra = new Random(0);
        for (int i = 0; i < 20; i++) {
            Paket pa = new Paket(ra.nextInt(130), ra.nextInt(60), ra.nextInt(60), ra.nextFloat() * 32);
            System.out.println(pa + "\t" + pa.preis());
        }
    }

    enum Preis implements Predicate<Paket> {
        K(5.99) {
            /**
             * Prüft, ob die Größe des Paket für diesen Preis gültig ist.
             */
            @Override
            public boolean test(final Paket paket) {
                return (paket.L <= 120 && paket.B <= 60 && paket.H <= 60)
                        && (paket.kg <= 5) && (paket.gurtmass() <= 300);
            }
        }, M(7.99) {
            @Override
            public boolean test(final Paket paket) {
                return (paket.L <= 120 && paket.B <= 60 && paket.H <= 60)
                        && (paket.kg <= 10) && (paket.gurtmass() <= 300);
            }
        }, G(14.99) {
            @Override
            public boolean test(final Paket paket) {
                return (paket.L <= 120 && paket.B <= 60 && paket.H <= 60) && (paket.kg <= 31.5);
            }
        }, UNDEFINED(-1) {
            @Override
            public boolean test(final Paket paket) {
                return true;
            }
        };

        private final double p;

        private Preis(double p) {
            this.p = p;
        }

        @Override
        public String toString() {
            return "Preis{" + "p=" + p + '}'; // hier ne schöne Ausgabe
        }


    }

    int L, B, H;
    float kg;

    public Paket(int L, int B, int H, float kg) {
        this.L = L;
        this.B = B;
        this.H = H;
        this.kg = kg;
    }

    int gurtmass() {
        return L + B + B + H + H;
    }

    Preis preis() {
        for (final Preis preis : Preis.values()) {
            if (preis.test(this)) {
                return preis;
            }
        }
        return Preis.UNDEFINED;
    }

    @Override
    public String toString() {
        return "Paket{" + "L=" + L + ", B=" + B + ", H=" + H + ", kg=" + kg + '}';
    }
}

 

[Xyz1]

Alternativ so

Dann hat er zur Auswahl.

 

[JStein52]

Alternativ so

Bestimmt könnte man das doch noch als Einzeiler formulieren. Aber ich weiss ja nicht ob das einem relativen Anfänger der Probleme hat seine if/else-Konstrukte korrekt zu formulieren wirklich so viel weiterhilft

 

[Javinner]

Ich gebe mal mein Senf dazu und weil hier schon Code gepostet wurde, lege ich meine Idee offen.

edit: die Preise habe ich frei erfunden.

Spoiler: Main && Konsolenausgabe

public class Versandstation
{

    public static void main(String[] args)
    {

        Paket[] pakets = new Paket[5];
        pakets[0] = new Paket(Paketmasse.MAX_60_x_30_x_15, Paketgewicht.MAX_2Kg, true);
        pakets[1] = new Paket(Paketmasse.MAX_120_x_60_x60, Paketgewicht.MAX_5Kg, false);
        pakets[2] = new Paket(Paketmasse.MAX_60_x_30_x_15, Paketgewicht.MAX_10Kg, true);
        pakets[3] = new Paket(Paketmasse.MAX_120_x_60_x60, Paketgewicht.MAX_31_5Kg, true);
        pakets[4] = new Paket(Paketmasse.MAX_120_x_60_x60, Paketgewicht.MAX_31_5Kg, false);

        //Info ueber alle Pakete;
        for (Paket paket : pakets) {
            System.out.println(paket.toString());
        }
    }
}

/** Konsolenausgabe */
run:
[Paketgroesse: MAX_60_x_30_x_15][Paketgewicht: MAX_2Kg][Inland: true]Paketkosten: 4.99]
[Paketgroesse: MAX_120_x_60_x60][Paketgewicht: MAX_5Kg][Inland: false]Paketkosten: 16.99]
[Paketgroesse: MAX_60_x_30_x_15][Paketgewicht: MAX_10Kg][Inland: true]Paketkosten: 16.99]
[Paketgroesse: MAX_120_x_60_x60][Paketgewicht: MAX_31_5Kg][Inland: true]Paketkosten: 45.0]
[Paketgroesse: MAX_120_x_60_x60][Paketgewicht: MAX_31_5Kg][Inland: false]Paketkosten: 54.0]
BUILD SUCCESSFUL (total time: 0 seconds)

Spoiler: Klasse: Paket

final class Paket
{

    /**
     * prueft, ob das Paket im Inland versendet werden soll;
     */
    private boolean inland;
    /**
     * Versandkosten des jeweiligen Paketes;
     */

    private final double kosten;
    /**
     * dient rein zu Informationszwecken, siehe {@code toString}
     */

    private final Object[] info;

    /**
     * initialisiert ein Konstruktor mit: berechnet die Kosten fuer jedes Paket
     * anhand der mitgelierten Informationen ueber die Paketmasse, Paketgewicht
     * und Lieferort des Paketes;
     *
     * @param pm Paketmasse (L x B x H)
     * @param pg Paketgewicht
     * @param inland {@code inland} true Inland/ false Ausland
     */
    Paket(Paketmasse pm, Paketgewicht pg, boolean inland)
    {
        this.kosten = berechneKosten(pm, pg, inland);
        this.info = getInfo(pm, pg, inland);
    }

    /**
     * liefert {@code kosten} des jeweiligen Paketes;
     *
     * @return {@code kosten} Versandkosten fuer das Paket
     */
    protected double getKosten()
    {
        return this.kosten;
    }

    /**
     * berechnet die Kosten fuer das jeweilige Paket;
     *
     * @param pm Paketmasse (L x B x H)
     * @param pg Paketgewicht
     * @param inland {@code inland} true Inland/ false Ausland
     *
     * @return {@code kosten} Versandkosten fuer das Paket;
     */
    private double berechneKosten(Paketmasse pm, Paketgewicht pg, boolean inland)
    {
        switch (pg) {
            case MAX_2Kg:
                switch (pm) {
                    case MAX_60_x_30_x_15:
                        return inland ? 4.99 : 8.99;
                    default:
                        return inland ? 8.99 : 10.99;
                }
            case MAX_5Kg:
                switch (pm) {
                    case MAX_60_x_30_x_15:
                        return inland ? 10.99 : 12.99;
                    default:
                        return inland ? 12.99 : 16.99;
                }
            case MAX_10Kg:
                switch (pm) {
                    case MAX_60_x_30_x_15:
                        return inland ? 16.99 : 21.99;
                    default:
                        return inland ? 21.99 : 25.99;
                }
            case MAX_31_5Kg:
                switch (pm) {
                    case MAX_60_x_30_x_15:
                        return inland ? 25.99 : 45;
                    default:
                        return inland ? 45 : 54;
                }
            default:
                return 0;
        }
    }

    /**
     * speichert die Informationen fuer das jeweilige Paket;
     *
     * @param pm Paketmasse (L x B x H)
     * @param pg Paketgewicht
     * @param inland {@code inland} Inland/Ausland
     *
     * @return {@code info} Information des Paketes, siehe {@code toString};
     */
    private Object[] getInfo(Paketmasse pm, Paketgewicht pg, boolean inland)
    {
        Object[] o = new Object[3];
        o[0] = pm;
        o[1] = pg;
        o[2] = inland;
        return o;
    }

    @Override
    public String toString()
    {
        return "[Paketgroesse: " + this.info[0] + "]" + "[Paketgewicht: " + this.info[1] + "]" + "[Inland: " + this.info[2] + "]" + "Paketkosten: " + this.getKosten() + "]";
    }

}

Spoiler: Enum: Paketgewicht

enum Paketgewicht
{
    MAX_2Kg, MAX_5Kg, MAX_10Kg, MAX_31_5Kg;
}

Spoiler: Enum: Paketmasse

 enum Paketmasse
{
    MAX_60_x_30_x_15, MAX_120_x_60_x60;
}

 

[mrBrown]

Ich gebe mal mein Senf dazu und weil hier schon Code gepostet wurde, lege ich meine Idee offen.

edit: die Preise habe ich frei erfunden.

geschachtelte switch? ist ja abartig...

Wie viele Stellen müsstest du ändern, wenn es ein zusätzliches Maß oder Gewicht gibt oder sich der Auslandsaufschlag ändert?

 

[Javinner]

geschachtelte switch? ist ja abartig...
Wie viele Stellen müsstest du ändern, wenn es ein zusätzliches Maß oder Gewicht gibt oder sich der Auslandsaufschlag ändert?

Natürlich ist es nicht perfekt und keine Musterlösung. Da ich selbst, wie ich dir ständig beweise, ein Anfänger bin, ist es jedenfalls auch nur eine sehr begrenzte Anfängerlösung, welche jedoch den Zweck hat, eine andere Möglichkeit aufzuzeigen. Und eins steht fest: meine Vorschläge werden (hoffentlich) mit der Zeit deinen Anforderungen genügen. Für den Moment erfreue ich mich daran, dass die Lösung,
so unvollkommen die auch sein mag, mich vom Lesen der Aufgabe bis zum erstellen des Programms keine 30 Minuten gekostet hat.
Da du aber selbst weiß, dass der Weg ein langer ist, bitte ich um Nachsicht und natürlich eine Alternative nebst der Kritik

 

[mrBrown]

Natürlich ist es nicht perfekt und keine Musterlösung. Da ich selbst, wie ich dir ständig beweise, ein Anfänger bin, ist es jedenfalls auch nur eine sehr begrenzte Anfängerlösung, welche jedoch den Zweck hat, eine andere Möglichkeit aufzuzeigen. Und eins steht fest: meine Vorschläge werden (hoffentlich) mit der Zeit deinen Anforderungen genügen. Für den Moment erfreue ich mich daran, dass die Lösung,
so unvollkommen die auch sein mag, mich vom Lesen der Aufgabe bis zum erstellen des Programms keine 30 Minuten gekostet hat.
Da du aber selbst weiß, dass der Weg ein langer ist, bitte ich um Nachsicht und natürlich eine Alternative nebst der Kritik

So bös war das auch nicht gemeint, ich wollte nur Hinweise geben, in welche Richtung man das verbessern kann - da bringt ne passende Frage meist mehr, als einfach eine Lösung hinklatschen

 

[Javinner]

So bös war das auch nicht gemeint
ich wollte nur Hinweise geben, in welche Richtung man das verbessern kann - da bringt ne passende Frage meist mehr, als einfach eine Lösung hinklatschen

Habe es nicht negativ aufgefasst, alles ist gut
Das stimmt schon. Ich habe mir überlegt, ob ich es umfangreich mache, dann wäre aber die Übersicht verloren gegangen. Letztendlich wollte ich die Nützlichkeit von Enum hervorheben und die Vorteile hervorheben.
Aber du hast ja auch recht, wenn es darum geht, dass meine Lösung
nicht pflegeleicht ist. Bin bei Generics angekommen und musste bei der Aufgabe daran denken, dass man hier wohl besser Map nehmen sollte. Bin aber noch nicht soweit, daher habe ich es gelassen.

 

[mrBrown]

Habe es nicht negativ aufgefasst, alles ist gut
Das stimmt schon. Ich habe mir überlegt, ob ich es umfangreich mache, dann wäre aber die Übersicht verloren gegangen. Letztendlich wollte ich die Nützlichkeit von Enum hervorheben und die Vorteile hervorheben.
Aber du hast ja auch recht, wenn es darum geht, dass meine Lösung
nicht pflegeleicht ist. Bin bei Generics angekommen und musste bei der Aufgabe daran denken, dass man hier wohl besser Map nehmen sollte. Bin aber noch nicht soweit, daher habe ich es gelassen.

Enums sind schon das passendere
Aber grad Switch und enum kann man wunderbar anders schreiben - eben in dem man das Verhalten in die Enums auslagert, dann spart man sich das Switch-Statement und hat gleichzeitig Daten und Funktion näher zusammen gebracht.

 

[Javinner]

So in etwa dann?

public enum SuperEnum
{
   
    INLAND_MAX_60_x_30_x_15_BIS_2KG, 
    AUSLAND_MAX_60_x_30_x_15_BIS_2KG;

    private final double inland_preis_MAX_60_x_30_x_15_BIS_2KG = 4.99;
    private final double ausland_preis_MAX_60_x_30_x_15_BIS_2KG = 6.99;

    protected double getPreis()
    {
        switch (this) {
            case INLAND_MAX_60_x_30_x_15_BIS_2KG:
                return inland_preis_MAX_60_x_30_x_15_BIS_2KG;
            case AUSLAND_MAX_60_x_30_x_15_BIS_2KG:
                return ausland_preis_MAX_60_x_30_x_15_BIS_2KG;
                /** Und so weiter und so fort */
            default:
                return 0;
        }
    }
}

 

[mrBrown]

Enums selbst können Methoden und Attribute haben
Guck mal in mein Beispiel weiter oben, da haben die Enums zB eine Methode, ob die Pakete zu ihrer Kategorie gehören.
So ähnlich kann man es auch für den Preis machen

 

[Xyz1]

lege ich meine Idee offen

Das ist auch ganz schön, allerdings:
Paketmasse.MAX_60_x_30_x_15
hast du das frei erfunden(?) und in Kategorien eingeteilt? Man kann bei der Post mWn auch Pakete beliebiger Maße verschicken und eine Waage ist auch immer dabei. Ebenso wird ein Auge zugedrückt, wenn ein Paket zum Besenstiel 31,51kg schwer ist.

Übertrieben wäre es, wenn man für Paket noch eine Factory hätte.

mWn.... mhm.... sind Enums mehr oder weniger Konstanten. Ich nicht genau, ob die Logik unbedingt bei den Daten sein muss - zumal die enum bereits Bestandteil der Klasse Paket ist.

 

[Javinner]

@DerWissende
War einer DHL-Pakettabelle entnommen, ob diese noch aktuell ist, spielte für mich keine Rolle, da es hier nur um eine Übung ging.
Meine Lösung entspricht so auch nicht ganz der Aufgabe, da ja genaues Gewicht, bzw. Abmaße gefordert werden. Letztendlich
wollte ich einem Anfänger von einem Anfänger eine weitere Idee vermitteln. Da gibt es sicher bessere Lösungen.

 

[Xyz1]

Meine Lösung entspricht so auch nicht ganz der Aufgabe, da ja genaues Gewicht, bzw. Abmaße gefordert werden. Letztendlich

Entspricht gar nicht der Aufgabe....
Es ist auch richtig, dass der findige Mitarbeiter durch bloßes sehen und anschließendes wiegen, weiß, um welches Paket es sich handelt. Dennoch wirds so einen Calkulator geben.

 

[mrBrown]

mWn.... mhm.... sind Enums mehr oder weniger Konstanten. Ich nicht genau, ob die Logik unbedingt bei den Daten sein muss - zumal die enum bereits Bestandteil der Klasse Paket ist.

Enums sind schon Konstanten, aber eben auch mehr als nur Konstanten.

Und das Logik und Daten zusammen gehören, ist doch eins der Prinzipen von Objekt-Orientiertheit...

 

[Javinner]

@DerWissende
Ja so kenne ich es auch
Enums benutze ich sehr gerne, wenn es darum geht, etwas zu bestimmen oder die Auswahl zu begrenzen. Hab ich richtig lieb gewonnen!

 

[Serano11]

Hui, das Thema ist ja explodiert Vielen Dank für all die Antworten und Lösungsvorschläge

Edit: Welche Befehle gibt es, um einen Exit Button auf einem GUI funtionstüchtig zu machen? Den Button erstellte ich auf dem GUI, doch er funktioniert nicht, da ich keine Anweisung habe.

 

[temi]

Weil es hier gerade so schön ist, mein Ansatz:

import java.util.ArrayList;
import java.util.Random;

public class Paket
{
    private int l;
    private int b;
    private int h;
    private float kg;
    private static ArrayList<PackageUnit> packages = new ArrayList<>();

    public Paket(int l, int b, int h, float kg)
    {
        this.l = l;
        this.b = b;
        this.h = h;
        this.kg = kg;
    }

    public static void main(String[] args)
    {
        addPackage(new PackageUnit(120, 60, 60, 300, 5f, 5.99f));
        addPackage(new PackageUnit(120, 60, 60, 300, 10f, 7.99f));
        addPackage(new PackageUnit(120, 60, 60, Integer.MAX_VALUE, 31.5f, 14.99f));

        Random ra = new Random(0);
        for (int i = 0; i < 20; i++) {
            Paket pa = new Paket(ra.nextInt(130), ra.nextInt(60), ra.nextInt(60), ra.nextFloat() * 32);

            System.out.println(pa + "\t" + pa.getPrice(pa));
        }
    }

    public static void addPackage(PackageUnit p)
    {
        packages.add(p);
    }

    public int calcGurtmass()
    {
        return l + b + b + h + h;
    }

    public float getPrice(Paket p)
    {
        for(final PackageUnit pu : packages)
        {
            if (pu.test(p))
            {
                return pu.getPrice();
            }
        }

        return 0;
    }

    public int getL()
    {
        return l;
    }

    public int getB()
    {
        return b;
    }

    public int getH()
    {
        return h;
    }

    public float getKg()
    {
        return kg;
    }

    @Override
    public String toString() {
        return "Paket{" + "L=" + l + ", B=" + b + ", H=" + h + ", kg=" + kg + '}';
    }

public class PackageUnit
{
    private final int maxL;
    private final int maxB;
    private final int maxH;
    private final int maxG;
    private final float maxKg;
    private final float price;

    public PackageUnit(int maxL, int maxB, int maxH, int maxG, float maxKg, float price)
    {
        this.maxL = maxL;
        this.maxB = maxB;
        this.maxH = maxH;
        this.maxG = maxG;
        this.maxKg = maxKg;
        this.price = price;
    }

    public boolean test(Paket p)
    {
        return p.getL() <= maxL && p.getB() <= maxB && p.getH() <= maxH && p.getKg() <= maxKg && p.calcGurtmass() <= maxG;
    }

    public float getPrice()
    {
        return price;
    }
}
}

Gibt noch Optimierungsbedarf, aber ich hab grad keine Zeit mehr dazu...

 

[Xyz1]

Willkommen bei schreiben Sie etwas das andere nicht mehr verstehen.

 

[JStein52]

Ich weiss ja nicht ob das jemanden interessiert, aber der TE hat eine Frage gestellt ...

 

[JStein52]

Den Button erstellte ich auf dem GUI, doch er funktioniert nicht, da ich keine Anweisung habe.

Handelt es sich um eine Swing-GUI ? Hast du für den Exit-Button einen ActionListener registriert ?

 

[Serano11]

Handelt es sich um eine Swing-GUI ? Hast du für den Exit-Button einen ActionListener registriert ?

Sie wurde im Eclipse erstellt und ich kann sie im Eclipse aufrufen. Ich habe es versucht aber ich weiß nicht, wie die Anweisung/Befehl hierfür heißt und nach Recherche habe ich es trotzdem nicht verstanden.. Ich dachte, ich konnte das Gegenteil für " openProjectButton.setOnAction(e -> ProjectHandling.openProject()); " mit "exitProjectButton.setOnAction(e -> ProjectHandling.exitProject());" machen, war aber falsch

Danke für die HIlfe nochmals !

 

[mrBrown]

Das geht in die richtige Richtung, es muss halt nur die Funktion zum schließen geben...

Willst du das ganze Programm beenden?

 

[Xyz1]

@Serano11 :
Das geht in die richtige Richtung. Siehe dir zum Bleistift .dispose() in der Doc an.

 

[mrBrown]

@Serano11 :
Das geht in die richtige Richtung. Siehe dir zum Bleistift .dispose() in der Doc an.

Der Tipp bringt ihm genau gar nichts...

 

[Xyz1]

bringt ihm genau gar nichts

Doch schon wenn er es richtig machen möchte.
Ein System.exit(0); finde ich nicht gerad gracefully.... Aber ich kann auch nicht Hellsehen, deswegen muss man natürlich raten.

 

[JStein52]

Der Tipp bringt ihm genau gar nichts...

... eine kleine Begründung wäre nicht schlecht gewesen weil dann auch jeder Leser wüsste was tief in deinem Inneren vor sich geht.

 

[mrBrown]

Er benutzt JavaFX, nicht Swing...


Platform.exit könnte das gesuchte sein, aber das ist halt nicht wirklich das Äquivalent zu der anderen Action...

 

[Serano11]

Vielen Dank, ich habe den Befehl:

"exitButton.setOnAction(e -> System.exit(1));"

genutzt und es funktioniert. Ich drücke den Exit Button und er beendet es

 

[Xyz1]

ich habe den Befehl:
genutzt

Das ist nicht gracefully.