Java Frage zur Methode

[JMe]

Hallo
ich muss für die Schule folgende Aufgabe lösen:

public void tanken(double liter)

{
benzinstand=benzinstand+liter;
}

ich soll hierzu sagen was an dieser Methode fehlerhaft ist und wie man sie verbessern könnte.

benzinstand hab ich bereits vorher als double deklariert. mir würde hierzu eventuell nur einfallen die liter auch schon vorher als atrribut festzulegen.
ich hoffe mir kann jemand helfen danke

 

[Marco13]

Och, was heißt schon "fehlerhaft"... Aber ... mit dieser Methode könnte man etwas machen, wofür man in der Realität einen Schlauch bräuchte

 

[Kim Stebel]

keine ahnung, worauf der hinaus will....vielleicht das: if (liter < 0) throw new IllegalArgumentException("negatives Tankvolumen!");

 

[Wodan]

ich als java-neuling probier mich auch mal an der lösung

public class Tanken {

public static double benzinstand = 20;

public static double tanken(double liter) {
return benzinstand = benzinstand + liter;
}

public static void main(String[] args) {
System.out.println(tanken(20));
}

}

-->Konsole: 40
^^ ich denke er wollte auf das void hinaus das ich nun durch double ersetzt habe & auf das static *g

mir würde hierzu eventuell nur einfallen die liter auch schon vorher als atrribut festzulegen.

An dem liegts nicht, da du es ja beim aufruf der methode als parameter übergibst.

System.out.println(tanken(20));
-->Konsole:40


mfg wodan

 

[Drake]

Hallo

was ist an diese Methode Fehlerhaft, hmm, sie kann be - und enttanken, ist an sich garnicht so verkehrt ist falls Diesel z.B. in einem Benziner landet.

Falls das wirklich die komplette Aufgabenstellung ist, wäre die Antwort meiner Meinung nach "nichts", von der Syntax her ist sie korrekt.
Ansonsten wenn du möchtest, dass man dir bei deinen Hausaufgaben hilft (nicht zu verwechseln mit diese für dich löst), poste die ganze Aufgabe (ich glaube nicht, dass diese so gestellt wurde ohne Auto Klasse drum herum und mit dieser merkwürdigen Fragestellung).

Zur Aufgabe:
Überleg dir, wo in einem Auto dieser Sprit hingeht, ob es da vielleicht physikalisch bedingte Grenzen gibt welche man beschten müsste bei be - und enttankung und wie du diese am besten einbaust. Desweiteren brauchst du wirklich die genauigkeit eines Double oder gäbe es da eine speicherschonendere Alternative.

 

[SlaterB]

auf keinen Fall auf static gehen, böse von Wodan,
dann könnte man im ganzen Programm nur ein Auto haben..

 

[mikachu]

auf keinen Fall auf static gehen, böse von Wodan,
dann könnte man im ganzen Programm nur ein Auto haben..

...oder alle fahrzeuge ernähren sich aus einem tank

 

[@x.l]

vllt. noch
benzinstand=benzinstand+liter;
durch
benzinstand+=liter;
ersetzen

 

[Ariol]

Man könnte sich auch noch den Preis zurückgeben lassen.

 

[SlaterB]

yippee, wenn ich das nächste Mal tanke, dann schaue ich auch in den Tank hinein um den Preis zu erfahren,
statt auf die Zapfsäule

 

[Guest]

Jetzt verunsichert den Thread-Steller doch nicht noch zusätzlich.

Er wollte wissen was da dran falsch ist. Antwort: Grundsätzlich NICHTS.

Es wäre vielleicht hier und da sinnvoll noch Abfragen oder sonstiges einzubauen. Aber an sich ist der Code nicht fehlerhaft, sondern höchstens nicht sauber bzw. nicht ganz sicher.

Von daher (wie schon mal gesagt), poste mal die gesamte Aufgabe (oder war das wirklich alles an Sourcecode und Aufgabenstellung?).

 

[Wodan]

Das mit dem "static" hätte für ein Auto doch gereicht *gg*

okok schon gut war dumm aber dafür hab ich wieder was dazugelernt

 

[JMe]

okay also was ich bisher programmiert hab so stehts in unserem script brauchte man im prinzip ja nur abschreiben:

public class Auto
{
//Attribute
double tankvolumen, benzinstand, verbrauch, kmstand;

// Konstruktor
public Auto()
{
tankvolumen = 70.0;
benzinstand = 50.0;
verbrauch = 8.6;
kmstand = 77000;
}


//Methode zum tanken
public void tanken (double liter)
{
benzinstand=benzinstand+liter;
}

}



So die Aufgaben zu der Methode sind:
1) Was ist an dieser Method enoch fehlerhaft, was könnte man verbessern?
2) Verbessere diese Methode entsprechend und baue sie in die Klasse Auto ein

 

[SlaterB]

guck, schon ist das ganze viel klarer,

meinst du nicht, dass man die Variable tankvolumen beim Tanken berücksichtigen sollte?

 

[JMe]

achso ja hätte logik*grins* also müsst eich jetzt noch kontrollieren lassen dass der benzinstand nicht über 70 in unserem falle hinaus geht bzw nicht übers tankvolumen okay dankeeee