Schleife berechnet kein Ergebnis (Vererbung)

[Kalkulator]

Hallo zusammen,

ich habe ein Problem mit einer For-Schleife, die mir statt dem Rechenergebnis immer den Wert ausgibt, mit der die Variable initialisiert wurde.
Es soll der Preis eines Kleinwagens berechnet werden, wobei sich dieser jedes Jahr um 5% gegenüber dem Vorjahr reduziert. Da das Ganze im Rahmen einer Übung zu Vererbung steht, gibt es 3 Klassen:

Übergeordnete Klasse Auto:

public abstract class Auto {
private int kaufpreis;
private int baujahr;

public abstract int preis();

public Auto(int kaufpreis, int baujahr) {
this.kaufpreis = kaufpreis;
this.baujahr = baujahr;
}

public int getKaufpreis() {
return kaufpreis;
}

public void setKaufpreis(int kaufpreis) {
this.kaufpreis = kaufpreis;
}

public int getBaujahr() {
return baujahr;
}

public void setBaujahr(int baujahr) {
this.baujahr = baujahr;
}

public static void main (String[] args) {
Kleinwagen kw = new Kleinwagen(1000, 2020, 10);
((Kleinwagen) kw).preis();
kw.jahre = 2;
System.out.println ("Der Endpreis des Kleinwagens nach " + kw.jahre + " Jahren beträgt " + kw.preis());
}

}

Hier stellt sich mir schon die Frage, warum ich bei der Instanziierung von Kleinwagen in der main-Methode für die fahrtkilometer nicht "null" angeben kann? Diese spielen für die vorliegende Berechnung ja überhaupt keine Rolle.

Untergeordnete Klasse Neuwagen:

public class Neuwagen extends Auto {
private int fahrtkilometer;
public int gefahren (int kilometer) {
return kilometer; // Da ich das erst später brauche, passt das evtl. noch nicht...
}

@Override
public int preis() {
return preis();
}
public Neuwagen(int kaufpreis, int baujahr, int fahrtkilometer) {
super(kaufpreis, baujahr);
this.setFahrtkilometer(fahrtkilometer);
}
public int getFahrtkilometer() {
return fahrtkilometer;
}
public void setFahrtkilometer(int fahrtkilometer) {
this.fahrtkilometer = fahrtkilometer;
}
}

Untergeordnete Klasse Kleinwagen:

public class Kleinwagen extends Neuwagen {

int jahre;

public Kleinwagen(int kaufpreis, int baujahr, int fahrtkilometer) { // Definition des Konstruktors (braucht man den, wenn nur super)??
super(kaufpreis, baujahr, fahrtkilometer);
// TODO Auto-generated constructor stub
}

@Override
public int preis() {
int preis = 0; // Initialisierung des Endpreises
for (int i = 0; i >= getBaujahr(); i++) {
preis = (int) (getKaufpreis() * Math.pow((1-5/100),jahre));
}
return preis;
}

}

Wenn ich die Methode preis() aus der Klasse Kleinwagen nun in der main-Methode der Klasse Auto ausführen möchte, kommt als Ergebnis immer der Wert raus, mit dem der Endpreis initialisiert wurde. Das heißt, meine eigentliche Rechnung wird nicht ausgeführt.

Kann mir jemand sagen, woran das liegt? Was muss ich ändern, sodass in der main-Methode in der Klasse Auto mein Endpreis ordnungsgemäß berechnet wird?

Vielen Dank schon mal für eure Hilfe!

 

[temi]

public int preis() {
return preis();
}

Die Methode ruft sich selbst auf.

Gib halt einfach einen Preis zurück oder implementiere in der Kindklasse eine Instanzvariable "preis", die du zurück geben kannst.

EDIT: Ups, sehe gerade, dass die Klasse Kleinwagen eine Berechnung des Preises vornimmt. Die Struktur kommt mir allerdings etwas seltsam vor. Um welche Funktionalität erweitert denn die Klasse Kleinwagen den Neuwagen? Es gibt zwar eine weitere Instanzvariable "jahre", aber die wird nie zugewiesen.

EDIT: Noch mal ups. Ich glaub ich hör heute hier auf. Preis wird in der main() zugewiesen. Schlechtes Design.

Wofür ist denn die Schleife (die auch nie ausgeführt wird) gedacht? Sie würde ja immer nur die gleiche Berechnung wiederholen.

 

[DrPils]

Bitte in Code Tags den Code posten

Abgesehen von dem was temi sagt geht dein Code nur in die Schleife wenn das Auto noch von Josef von Nazaret gebaut wurde.

Das zweite Argument einer For schleife ist die Bedingung wie lange sie laufen soll.

 

[kneitzel]

Hier stellt sich mir schon die Frage, warum ich bei der Instanziierung von Kleinwagen in der main-Methode für die fahrtkilometer nicht "null" angeben kann?

Der Paremter der Konstruktors für die Kilometer hat welchen Typ? Was für Werte kann dieser Typ annehmen? Welche Typen können den Wert null annehmen?

Neben dem Problem, das temi aufgezeigt hat, stimmt auch die Berechnung in Kleinwagen nicht:

@Override
public int preis() {
int preis = 0; // Initialisierung des Endpreises
for (int i = 0; i >= getBaujahr(); i++) {
preis = (int) (getKaufpreis() * Math.pow((1-10/100),jahre));
}
return preis;
}

Das bringt aus mehrerer Sicht nichts.
a) Baujahr ist ja ein Jahr. Also sowas wie 2019. i startet bei 0 und 0 ist nie > 2019.
b) Eine Prüfung auf >= zusammen mit ++ ist auch generell falsch. (Oder willst Du bis zum Überlauf erhöhen?)
c) Die Schleife macht so noch keinen Sinn. Vermutlich willst Du den Preis in der Schleife immer verändern. Aber Ausgangspunkt wäre dann immer der schon berechnete Preis.

 

[Kalkulator]

Wow, vielen Dank für eure schnellen Antworten!

Allerdings bin ich als totaler Java-Anfänger noch etwas überfragt von euren Rückfragen :-(

Die Schleife soll den aktuellen Preis des Kleinwagens berechnen. Also nach einem vorgegebenen Jahr. Wie ganz oben geschrieben, reduziert sich dieser jedes Jahr um 5% gegenüber dem Vorjahr. Ausgangspunkt für die Preisreduzierung im ersten Jahr ist der Kaufpreis.
Die Variable "jahre" habe ich zusätzlich eingefügt, um festlegen zu können, für welches Jahr der Preis ausgegeben werden soll. Oder kann ich das direkt über die Laufvariable i machen? Wie kann ich dann aber in der main-Methode dem i einen konkreten Wert zuweisen, wenn ich den Preis z.B. nach 2 Jahren ausgeben will?

Kann mir jemand sagen, wie die Schleife für diese Anwendung konkret ausschauen muss?

 

[kneitzel]

Generell ist es hier immer wichtig, dass Du immer erst sehr genau formulierst, was denn da zu tun ist!

Wie ganz oben geschrieben, reduziert sich dieser jedes Jahr um 5% gegenüber dem Vorjahr. Ausgangspunkt für die Preisreduzierung im ersten Jahr ist der Kaufpreis.

Das nehme ich als Beispiel: Was bedeutet das denn genau?
- Am Anfang ist der Preis der Kaufpreis.
- Für jedes Jahr vom Herstellungsjahr zum aktuellen Jahr wird der Preis um 5% reduziert.

Das kann man dann noch weiter ausformulieren. Ziel ist, dass jemand die Rechnung machen kann, der gar nicht weiss um was es geht.

Also immer erst genau den Algorithmus verstehen. Was ist verlangt? Was ist gegeben? Wie rechnet man selbst mit Stift und Papier um das Ergebnis zu bekommen? Das dann auch ausformulieren! Und ganz wichtig: All das, ganz ohne Computer oder Java!

Das mag einem am Anfang komisch vorkommen, aber das ist existenziell! Viele Dinge laufen später direkt im Kopf. Da schreibst man das nicht mehr auf. Aber das geht auch nur bis zu einem bestimmten Punkt und dann greift man als Entwickler auch zu Stift und Papier (oder zum Whiteboard).

 

[Robertop]

Wenn ich das richtig sehe, dann brauchst du in deiner preis() Funktion eigentlich weder die Schleife noch das Baujahr:

Durch "jahre" ist das alter von dem Auto ja schon angegeben, ohne dass man das Baujahr oder das aktuelle Jahr wissen muss.
Und deine einzige Zeile in der Schleife berechnet schon alleine sofort den richtigen Endpreis. Du musst also noch entscheiden, ob du das überhaupt mit einer Schleife machen willst, oder ob du es mathematisch lösen willst.