Interface-Frage

[pg1337]

Sers liebe Community.

Ich habe eine Methode endPreis() mit der der Endpreis berechnet wird:

public double endPreis(){
		
		double result= 0.0;
		
		if (lieferung.equals("schnell")) {
		result = einkaufswagen.gesamtPreis() * 1.03;
		} else if(lieferung.equals("standard")) {
			result = einkaufswagen.gesamtPreis() + 3;
		} else if(lieferung.equals("overnight")) {
			result = einkaufswagen.gesamtPreis() * 1.05;
		} else {
			System.out.println("Lieferart nicht vorhanden!\nFolgende Lieferarten verfügbar: standard, schnell, overnight");
		}
		return result;
	}

Nun möchte ich jedoch lieber 3 Klassen erstellen (SchnelleLieferung{}, StandardLieferung{} und OvernightLieferung{}) die alle ein Interface mit der Methode endPreis() implementieren aus einem neuen Interface:

public interface Lieferung {
	
	double result= 0.0;
	
	public double endPreis();

}

Ich weiß nicht wie ich das machen soll.. Was kommt in die 3 Klassen rein? implements Lieferung... ist klar, nur dann?

Regards

 

[faetzminator]

public class SchnelleLieferung implements Lieferung {
    public double endPreis() {
       return einkaufswagen.gesamtPreis() * 1.03;
    }
}

Etc...

 

[pg1337]

public class SchnelleLieferung implements Lieferung {
    public double endPreis() {
       return einkaufswagen.gesamtPreis() * 1.03;
    }
}

Etc...

Das sieht schonmal gut aus, Problem ist nur ich kann auf "einkaufswagen." nicht in dieser Klasse direkt zugreifen, da einkaufswagen ein Objekt einer Klasse Einkaufswagen ist.

Regards

Edit:

Habe das so gedacht:

public class SchnelleLieferung implements Lieferung {
	
	private Einkaufswagen einkaufswagen;
	
	public SchnelleLieferung(Einkaufswagen einkaufswagen) {
		super();
		this.einkaufswagen = einkaufswagen;
}

	@Override
	public double endPreis() {
		return einkaufswagen.gesamtPreis()*1.03;
	}
	
	

}

Aber funkt nicht.

 

[SlaterB]

was soll denn daran nicht funktionieren?
welche Kompilerfehler hast du, Exceptions zur Laufzeit, welches Fehlverhalten?

das hat doch nichts mehr mit dem Interface zu tun oder? ob im komplett anderen Programmteilen, man bedenke
http://www.java-forum.org/allgemeine-java-themen/129996-ubungsaufgabe-online-shop.html
der Einkaufswagen nicht befüllt wird, oder was auch immer, ist hier doch überhaupt nicht zu beurteilen, oder?

besteht eine Frage?

 

[pg1337]

SlaterB, das Programm funktioniert einwandfrei jetzt (Übung Online Shop). es geht mir nur drum dass ich die Methode endPreis() in ein Interface auslagern will und 3 neue Klassen erstelle für die verschiedenen Lieferarten.

Allerdings kann ich ja nicht von der neuen Klasse auf das einkaufswagen-Objekt zugreifen.

Fehler: einkaufswagen cannot be resolved.

Der Einkaufswagen wurde in der Klasse Test definiert:

Einkaufswagen einkaufswagen= new Einkaufswagen();
		einkaufswagen.addArtikel(laptop, 5);
		einkaufswagen.addArtikel(dvdPlayer, 2);
		einkaufswagen.addArtikel(fernseher, 1);

Geht mir nur drum wie ich die Methode endPreis() in ein Interface auslagern kann (Ich auf das Objekt einkaufswagen zugreifen kann???:L), auf das 3 Klassen dann zugreifen können, sprich SchnelleLieferung-Klasse, StandardLieferung-Klasse und OvernightLieferung-Klasse.

Gruß

 

[ARadauer]

[qoute]Fehler: einkaufswagen cannot be resolved.[/qoute]
oder
[qoute]Fehler: Einkaufswagen cannot be resolved.[/qoute]
Was fehlt dir die Variable oder die Klasse

Der Einkaufswagen wurde in der Klasse Test definiert:

Aber du hast schon eine Klasse Einkaufswagen oder?

und in Test machst du dann ein
SchnelleLieferung sl = new SchnelleLieferung (einkaufswagen);


das muss gehen...
zeig mal den ganzen Code

 

[SlaterB]

gemeckert wurde schon wieder, eine Fehlermeldung ohne Code dazu ist fast sinnlos,
aber immerhin noch ein komplett lauffähiges Programm dazu:

public class Test
{
    public static void main(String[] args)
    {
        Einkaufswagen einkaufswagen = new Einkaufswagen();
        Lieferung l = new SchnelleLieferung(einkaufswagen);
        System.out.println(l.endPreis());
    }
}


class Einkaufswagen
{
    double gesamtPreis()
    {
        return 2;
    }
}


interface Lieferung
{
    public double endPreis();
}


class SchnelleLieferung
    implements Lieferung
{

    private Einkaufswagen einkaufswagen;

    public SchnelleLieferung(Einkaufswagen einkaufswagen)
    {
        super();
        this.einkaufswagen = einkaufswagen;
    }

    public double endPreis()
    {
        return einkaufswagen.gesamtPreis() * 1.03;
    }

}

Ausgabe: 2.06

 

[pg1337]

SchnelleLieferung sl = new SchnelleLieferung (einkaufswagen);

Das wars was gefehlt hat, jetzt funktionierts!
Vielen Dank .

Ich lasse das mal kurz noch als "unerledigt" weil ich schnell einkaufen gehen muss..*Welch Zufall mit einkaufswagen * , dann probier ichs und schreibe hier nochmal rein falls n Fehler kommt und wenn nicht dass es funkt.

Danke an Alle, echt n klasse Forum hier!

 

[pg1337]

Die Klasse Test (Eine von 4 Klassen vom Ausgangsprogramm), in der die Main Methode seinen Sitz hat sieht so aus:

import java.util.List;


public class Test {
	
	

	public static void main(String[] args)
	{
		
		Artikel laptop= new Artikel("Neu", 500,"5");
		Artikel dvdPlayer= new Artikel(null, 100, null);
		Artikel fernseher= new Artikel(null, 300, null);
	
		Einkaufswagen einkaufswagen= new Einkaufswagen();
		einkaufswagen.addArtikel(laptop, 5);
		einkaufswagen.addArtikel(dvdPlayer, 2);
		einkaufswagen.addArtikel(fernseher, 1);	
		
		SchnelleLieferung sl = new SchnelleLieferung (einkaufswagen);
		
		Bestellung bestellung= new Bestellung(einkaufswagen, "schnell");
		
		System.out.println("Gesamtpreis der Artikel im Warenkorb:  " + Math.round(einkaufswagen.gesamtPreis()) + " €");
		System.out.println("Gesamtpreis der Artikel bei Lieferung: " + Math.round(bestellung.endPreis()) + " €");
		
		List<Artikel> artikel= einkaufswagen.getEinkaeufe();
		for(Artikel a:artikel) {
			System.out.println("Preis: " + a.getPreis());
		}
		
	}	
}

Greetz

 

[faetzminator]

Ich denke, dass in der Klasse [c]Bestellung[/c] anhand des String (hier [c]"schnell"[/c]) die jeweilige Lieferung erstellt werden soll. Dann kannst du das dort in [c]endPreis()[/c] implementieren.

 

[SlaterB]

spitzfindig:
> SchnelleLieferung sl = new SchnelleLieferung (einkaufswagen);
> Das wars was gefehlt hat, jetzt funktionierts!

ohne diese Zeile gäbe es eigentlich nicht die genannte Fehlermeldung

----

eher Grund des Postings:
> Ich lasse das mal kurz noch als "unerledigt" weil ich schnell einkaufen gehen muss..
damit nicht andere unnötig reingeschauen habe ich auf 'erledigt' gesetzt,
sobald du eine neue Frage hier stellen willst, kann man das ja wieder aufheben

 

[pg1337]

Sers,

nochmals eine Frage:

Wie mach ich das, das diese Definition hier:

public class SchnelleLieferung implements Lieferung {
	
	   private Einkaufswagen einkaufswagen;
	    
	    public SchnelleLieferung(Einkaufswagen einkaufswagen) {
	        super();
	        this.einkaufswagen = einkaufswagen;
	}
}

nicht in jeder Klasse (StandardLieferung, SchnelleLieferung, OvernightLieferung) einzeln steht????:L


Gruß

 

[faetzminator]

Geht nicht. Du kannst zwar eine abstrakte Klasse machen, welche den Konstruktor mit Parameter verwendet und das Feld setzt (und einen Getter zur Verfügung stellt), aber du musst dann trotzdem Konstruktoren erstellen, welche [c]super(einkaufswagen);[/c] aufrufen.

 

[SlaterB]

du kannst auch eine allgemeine Oberklasse einführen, dann ist das Interface vielleicht obsolet,
und dort den Code in etwa einfügen, allerdings nicht per Konstruktor, weil Konstruktoren in Subklassen zu wiederholen sind,
sondern per init-Methode

eher unnötiger Aufwand, lasse es lieber so,



übrigens fast derselbe Beitrag von mir wie in
http://www.java-forum.org/java-basi...griff-klasse-interface-objekt.html#post850753

aber dort bisschen komplizierter, wäre unfair nur dorthin zu verweisen,
dennoch nicht komplett falsch das Thema mal zu lesen

 

[pg1337]

Ok dann lasse ich das mal so

Bevor ich die Methoden ausgelagert habe in 3 Klassen habe ich diese Ja über ein If Statement aufgerufen (Einen String ausgelesen und überprüft ob er übereinstimmt, mit schnell o.ä.).

Nun habe ich das soweit in der Klasse test, also in der main-methode:

SchnelleLieferung sl = new SchnelleLieferung (einkaufswagen);
		StandardLieferung st= new StandardLieferung(einkaufswagen);
		OvernightLieferung ol= new OvernightLieferung(einkaufswagen);
		
		
		Bestellung bestellung= new Bestellung(einkaufswagen, "Schnelle Lieferung");
		
		
		System.out.println("Gesamtpreis der Artikel im Warenkorb:  " + Math.round(einkaufswagen.gesamtPreis()) + " €");
		System.out.println("Gesamtpreis der Artikel bei Lieferung: " + Math.round(sl.endPreis()) + " €");

Allerdings hat der Parameter bei bestellung "Schnelle Lieferung" nun keinen Einfluss mehr sondern ich muss das am Ende festlegen, was genommen wird, wobei ich das schon noch nach dem String haben wollte. Den Teil meine ich:

sl.endPreis()

Wie kann ich das wieder nach dem String automatisch machen lassen?
Zuvor stand da:

bestellung.getPreis()

.

Geht das wieder über das bestellung-Objekt?

Gruß

 

[Gast2]

Übergib doch statt dem String direkt das Lieferungsobjekt. Dann kann die Bestellung darauf zugreifen.

 

[faetzminator]

Wieso übergibt man dem Konstruktor von einer [c]Bestellung[/c] nicht auch gleich die [c]Lieferung[/c] ? Dann kannst du so was machen:

Bestellung bestellung = new Bestellung(einkaufswagen, new SchnelleLieferung());

 

[pg1337]

Wieso übergibt man dem Konstruktor von einer [c]Bestellung[/c] nicht auch gleich die [c]Lieferung[/c] ? Dann kannst du so was machen:

Bestellung bestellung = new Bestellung(einkaufswagen, new SchnelleLieferung());

In der Klasse Lieferung steht aber nur die Methode endPreis() drin.

Verstehe ich nicht ganz, mit Übergeben hab ich mir das auch schon gedacht aber weiß nicht genau wie das funkt.

Wie meinste das?

 

[pg1337]

Übergib doch statt dem String direkt das Lieferungsobjekt. Dann kann die Bestellung darauf zugreifen.

Das würde wie aussehen?

 

[Gast2]

So wie faetzminator schon geschrieben hat.
Du musst dann natürlich den Konstruktor anpassen. Auch das Interface würde ich dann anpassen in irgendwie sowas:

interface Lieferung {
    public double berechneVersandkosten(Einkaufswagen e);
}

berechneVersandkosten() kannst du dann in der Bestellung bei endPreis() aufrufen.

 

[faetzminator]

@EikeB: ich hab eher daran gedacht, den [c]Einkaufswagen[/c] und die [c]Lieferung[/c] an die [c]Bestellung[/c] zu übergeben, also so:

public Bestellung {
    private final Einkaufswagen einkaufswagen;
    private final Lieferung lieferung;

    public Bestellung(Einkaufswagen einkaufswagen, Lieferung lieferung) {
        this.einkaufswagen = einkaufswagen;
        this.lieferung = lieferung;
    }

    public double endPreis() {
        return einkaufswagen.gesamtPreis() * lieferung.faktor(); // irgendwie macht Faktor mehr Sinn als Endpreis für die Lieferung
    }
}

 

[pg1337]

Hab das jetzt mal so geändert:

		Bestellung bestellung= new Bestellung(einkaufswagen, new SchnelleLieferung(einkaufswagen));

So sieht das Objekt jetzt aus.

Klasse Bestellung sieht der Konstruktor jetzt so aus:

	private Lieferung lieferung;
	
	public Bestellung(Einkaufswagen einkaufswagen, Lieferung lieferung) {
		
		super();
		this.einkaufswagen = einkaufswagen;
		this.lieferung = lieferung;
}

Hier wird ja dann lieferung übergeben.

Interface sieht so aus:

public interface Lieferung {
	
	double result= 0.0;
	

	    public double endPreis(Einkaufswagen e);
	}

Sowie die 3 LieferKlassen ungefähr so:

public class SchnelleLieferung implements Lieferung {
	   private Einkaufswagen einkaufswagen;
	    
	   
	    public SchnelleLieferung(Einkaufswagen einkaufswagen) {
	        super();
	        this.einkaufswagen = einkaufswagen;
	}
	    
		 
		public double endPreis(Einkaufswagen e) {
			return einkaufswagen.gesamtPreis()*1.03;
		}
}

Stimmt das so?
Wenn ja wie rufe ich das dann im System.out.println("...") auf?

bestellung. ?

Gruß

 

[Gast2]

@EikeB: ich hab eher daran gedacht, den Einkaufswagen und die Lieferung an die Bestellung zu übergeben, also so:

Jep ich auch.

// irgendwie macht Faktor mehr Sinn als Endpreis für die Lieferung

Die Lieferart muss mindestens noch irgendwo den Gesamtwert der Bestellung herbekommen. Daher habe ich noch den Einkaufswagen übergeben. Oft findet man ja sowas wie:
Bestellwert < 100€ -> 4,90€ Lieferkosten
Bestellwert >= 100€ -> 0€ Lieferkosten

 

[pg1337]

In der Lieferung wird aber der Endpreis schon berechnet.
Nicht nur der Faktor festgelegt.

 

[faetzminator]

In der Lieferung wird aber der Endpreis schon berechnet.
Nicht nur der Faktor festgelegt.

Das sollte aber eben nicht so sein Oder aber wenn du es doch so willst, dann benötigst du gar keine Klasse [c]Bestellung[/c] mehr, da eine [c]Lieferung[/c] dein Preis selbst anhand eigener Daten und derer des [c]Einkaufswagen[/c] berechnet.