Ticketautomat in Java umprogrammieren

[infinityfan]

Hi Leute, habe im Studium gerade begonnen Java zu lernen. Evt. kennt jemand die Entwicklungsumgebung "BlueJ". Wir arbeiten im momentanen Semester damit und steigen nachher auf eclips um.
Jetzt haben wir einen Ticketautomaten vorgegeben, der mit Cent`s arbeitet, sprich:
- Ticketpreis eingeben
- bisher bezahlt ausgibt
- Ticket drucken kann
- preis des Tickets ausgibt
- und natürlich kann man Geld einwerfen
Bisher haben wir nur mit int Werten gearbeitet, jetzt haben wir aber übers WE die Aufgabe bekommen den Quelltext so umzuschreiben, dass er mit Euro und Cent arbeitet bzw. Franken und Rappen.
Einen grossen Teil habe ich schon umgeschrieben, z.B. Eingabe von zwei Werten für Franken und Rappen. Jedoch kriege ichs nicht hin, dass mir der Automat vernünftig was ausgibt.

Hier mal der Quelltext:

/**
* Die Klasse Ticketautomat modelliert einfache Ticketautomaten,
* die Tickets zu einem Einheitspreis herausgeben.
* Der Preis f¸r die Tickets eines Automaten kann ¸ber den Konstruktor
* festgelegt werden.
* Ticketautomaten dieser Klasse pr¸fen, ob sinnvolle Geldbetr‰ge
* eingeworfen werden, und drucken ein Ticket nur dann, wenn
* ausreichend Geld eingeworfen wurde.
*
* @author David J. Barnes und Michael Kˆlling
* @version 2008.03.30
*/


public class Ticketautomat
{
// Der Preis eines Tickets dieses Automaten.
private int preis;//Modifizierer; Datentyp; Bezeichung
// Der Betrag, der bisher vom Automatenbenutzer eingeworfen wurde.
private int bisherGezahlt;
// Die Geldsumme, die bisher von diesem Automaten eingenommen wurde.
private int gesamtsumme;
private int Rappen;
private int Franken;
/**
* Erzeuge einen Automaten, der Tickets zum angegebenen Preis
* (in Franken und Rappen) ausgibt.
*/
public Ticketautomat(int Franken, int Rappen)
{//Konstruktor
preis = Franken + Rappen;
bisherGezahlt = 0;
gesamtsumme = 0;
}

/**Anfang der 1. Methode
* Liefere den Preis eines Tickets dieses Automaten (in Cent).Return ist nicht gleich System.out.println()!
*/
public void gibPreis()
{
System.out.println("Der Preis beträgt:" + Franken +" Franken und " + Rappen +" Rappen");
}

/**Beginn der 2. Methode
* Liefere die Hoehe des Betrags, der fuer das naechste Ticket bereits
* eingeworfen wurde.
*/
public int gibBisherGezahltenBetrag()
{
return bisherGezahlt;//Rückgabeanweisung "return" von bisherGezahlt;, liefert einen ganzzahligen Wert
}//Ende der 2.Methode

/**Begin der 3. Methode (modifizierende Methode)
* Nimm den angegebenen Betrag (in Franken und Rappen) als Anzahlung fuer das 
* n‰chste Ticket. Pr¸fe, ob der Betrag sinnvoll ist.
*/
public void geldEinwerfen(int Franken, int Rappen)//modifizierende Methode, am Parameter erkennbar
{
if (Franken >= 0) {// bedingte Anweisung; true oder false
bisherGezahlt+=Franken + Rappen; //bisherGezahlt = bisherGezahlt + betrag;//

}
else {
System.out.println("Bitte nur positive Betraege verwenden: "//Betrag hat int Wert; Fehlermeldung wird ausgegeben....
+ Franken + Rappen);
}
}

/**
* Drucke ein Ticket, wenn gen¸gend Geld eingeworfen wurde, und 
* ziehe den Ticketpreis vom bisher gezahlten Betrag ab. Gib eine 
* Fehlermeldung aus, falls noch Geld f¸r ein Ticket fehlt.
*/
public void ticketDrucken() //modifizierende Methode
{

if (bisherGezahlt >= preis) {
// Den Ausdruck eines Tickets simulieren.
System.out.println("##################");
System.out.println("# Die BlueJ-Linie");
System.out.println("# Ticket");
System.out.println("#" + Franken +" Franken");
System.out.println("#" + Rappen +" Rappen");
System.out.println("##################");
System.out.println();

// Die Gesamtsumme um den Ticketpreis erhˆhen.
gesamtsumme = gesamtsumme + preis;
// Den Preis von der bisherigen Bezahlung abziehen.
bisherGezahlt = bisherGezahlt - preis;
}
else {
System.out.println("Sie m¸ssen noch mindestens " +
(preis - bisherGezahlt) +
" Franken einwerfen.");


}

}



/**
* Gib das Wechselgeld bzw. den bisher gezahlten Betrag zur¸ck.
*/
public int wechselgeldAuszahlen()
{
int wechselgeld = bisherGezahlt; //lokale Variable (Hilfsvariable); nur in der Methode wirksam; wird nachher wieder verworfen
bisherGezahlt = 0; //setzt bisherGezahlt auf 0 zurück
return wechselgeld; //gibt Wert von wechselgeld zurück, nicht im Textfeld
}
}

Hoffe ihr könnt mir bei meinem Problem weiterhelfen, bin leider noch n blutiger Anfänger in sachen Java . Danke schonmal für euren Rat.

 

[eRaaaa]

Ist das:

Java™ Quelltext ist bitte in

-Tags zu schreiben: [code=Java]hier Java Code einfügen[/ JAVA][/COLOR][/SIZE]

wirklich so schwer zu übersehen? :noe:

Und was ist jetzt genau dein Problem? Wie testest du überhaupt den Code (mit welchen Werten fütterst du deinen Automaten, was erhältst du als Ergebnis, was ist dein erwartetes Ergebnis etc.), eine main-Methode ist dort ja nicht vorhanden.

 

[infinityfan]

Oh Mist, sorry hab ich völlig vergessen, das in n Java-Tag zu schreiben.
Nehmen wir mal an, ich gebe beim Erstellen des Automaten in BlueJ einen Ticketpreis von 5 Franken und 40 Rappen ein. Die Zuweisung der Werte sollte doch eig. soweit ok sein. Also Zahle ich naher mit "geldEinwerfen" die 5 Franken und 40 Rappen ein. Wenn ich dann die Methode "ticketDrucken" aufrufe bekomme ich:

##################
# Die BlueJ-Linie
# Ticket
#0 Franken
#0 Rappen
##################

Es sollten aber nicht 0 Franken und 0 Rappen dastehen, sondern die 5 Franken 40 Rappen^^.

 

[T7V]

So nebenbei: Du hast zwei int rappen und franken.
Im konstruktor addierst du beide zum preis.
Bsp:
Franken =5
Rappen =40
Dann ist dein Preis 45


Edit: Du kannst dann auck 40 Frankrn und 5Rappen einzahlen. Es wäre sinnvoll die Franken immer gleich in Rappen umzurechnen.

 

[Fu3L]

Du rechnest das eingeworfene Geld immer nur zusammen, aber speicherst nie die Franken und Rappen in den entsprechenden Membervariablen, sodass diese immer 0 bleiben.

In der Methode, wo du das geld einwirfst, könntest du hinzufügen:

this.Franken = Franken;

Variablennamen schreibt man normalerweise übrigends immer klein

 

[W9ND3R]

Kann es nicht, weil deine Methode

ticketDrucken()

die Werte nicht kennt, die du dem Konstruktor übergibst. Du deklarierst ganz oben die Variablen

private int [COLOR="SeaGreen"]Rappen[/COLOR];
private int [COLOR="SeaGreen"]Franken[/COLOR];

Doch du schreibst die Werte nie da rein! Die Werte im Konstruktor sind die, die als Argumente übergeben werden:

public Ticketautomat(int [COLOR="Red"]Franken[/COLOR], int [COLOR="Red"]Rappen[/COLOR])
{//Konstruktor
preis = [COLOR="Red"]Franken [/COLOR]+ [COLOR="Red"]Rappen[/COLOR];
bisherGezahlt = 0;
gesamtsumme = 0;
}

Irgendwo im Konstruktor musst du deine Eingabewerte der ganzen Klass "bekannt" machen ...

public Ticketautomat(int Franken, int Rappen)
{//Konstruktor
this.[COLOR="SeaGreen"]Franke [/COLOR]= Franken;
this.[COLOR="SeaGreen"]Rappen [/COLOR]= Rappen;
preis = Franken + Rappen;
bisherGezahlt = 0;
gesamtsumme = 0;
}

 

[infinityfan]

Hey danke für die schnellen Antworten. Naja, hatte bis jetzt erst zwei mal Java-Unterricht. Von da her ist da noch nicht viel Wissen vorhanden, jedoch arbeite ich daran . Mal schauen ob ich das so gepackt bekomme. Werde mich später nochmal melden. Bis dann.

 

[javaDev2011]

Hi infinityfan,

Ich weis gar nicht ob du meinen verbesserten Code nach der ganzen anderen schnellen Hilfe überhaupt noch brauchst, er funktioniert aber auf jeden Fall.

public class Ticketautomat {
    // Der Preis eines Tickets dieses Automaten.
    private double preis;//Modifizierer; Datentyp; Bezeichung
    // Der Betrag, der bisher vom Automatenbenutzer eingeworfen wurde.
    private double bisherGezahlt;
    // Die Geldsumme, die bisher von diesem Automaten eingenommen wurde.
    //private double gesamtsumme;
    private int rappen;
    private int franken;
    /**
    * Erzeuge einen Automaten, der Tickets zum angegebenen Preis
    * (in Franken und Rappen) ausgibt.
    */
    public Ticketautomat(int f, int r) {
        if (f >= 0 && r >= 0 && r < 100){
            franken = f;
            rappen = r;
            preis = rappen; 
            preis /= 100;
            preis += franken;
            bisherGezahlt = 0;
            //gesamtsumme = 0;
        } 
        else {
            System.err.println("Bitte nur gültige Beträge verwenden: " + f + "," + r);
        }
    }
 
    /**Anfang der 1. Methode
    * Liefere den Preis eines Tickets dieses Automaten (in Cent).Return ist nicht gleich System.out.println()!
    */
    public void gibPreis() {
        System.out.println("Der Preis beträgt: " + franken +" Franken und " + rappen +" Rappen");
    }
 
    /**Beginn der 2. Methode
    * Liefere die Hoehe des Betrags, der fuer das naechste Ticket bereits
    * eingeworfen wurde.
    */
    public double gibBisherGezahltenBetrag() {
         return bisherGezahlt;//Rückgabeanweisung "return" von bisherGezahlt;, liefert einen ganzzahligen Wert
    }
 
    /**Begin der 3. Methode (modifizierende Methode)
    * Nimm den angegebenen Betrag (in Franken und Rappen) als Anzahlung fuer das 
    * nächste Ticket. Prüfe, ob der Betrag sinnvoll ist.
    */
    // modifizierende Methode, am Parameter erkennbar
    public void geldEinwerfen(int f, int r) {
       if (f >= 0 && r >= 0 && r < 100) {// bedingte Anweisung; true oder false
           double jetztGezahlt = r; 
           jetztGezahlt /= 100;
           jetztGezahlt += f;
           bisherGezahlt += jetztGezahlt; //bisherGezahlt = bisherGezahlt + betrag;//
       }
       else {
           //Betrag hat int Wert; Fehlermeldung wird ausgegeben....
           System.err.println("Bitte nur gültige Beträge verwenden: " + f + ","+ r);
       }
    }
 
    /**
    * Drucke ein Ticket, wenn gen¸gend Geld eingeworfen wurde, und 
    * ziehe den Ticketpreis vom bisher gezahlten Betrag ab. Gib eine 
    * Fehlermeldung aus, falls noch Geld für ein Ticket fehlt.
    */
    //modifizierende Methode
    public void ticketDrucken() { 
        if (bisherGezahlt >= preis) {
            // Den Ausdruck eines Tickets simulieren.
            System.out.println("##################");
            System.out.println("# Die BlueJ-Linie");
            System.out.println("# Ticket");
            System.out.println("# " + franken +" Franken");
            System.out.println("# " + rappen +" Rappen");
            System.out.println("##################");
            System.out.println();
            // Die Gesamtsumme um den Ticketpreis erhöhen.
            //gesamtsumme = gesamtsumme + preis;
            // Den Preis von der bisherigen Bezahlung abziehen.
            bisherGezahlt -= preis;
            if(bisherGezahlt != 0){
                System.out.println("Sie erhalten noch "+bisherGezahlt+" Franken");
            }
        }
        else {
            System.out.println("Sie müssen noch mindestens " + (preis - bisherGezahlt) + " Franken einwerfen.");
        }
    }
 
    /**
    * Gib das Wechselgeld bzw. den bisher gezahlten Betrag zur¸ck.
    */
    public double wechselgeldAuszahlen() {
         double wechselgeld = bisherGezahlt; //lokale Variable (Hilfsvariable); nur in der Methode wirksam; wird nachher wieder verworfen
         bisherGezahlt = 0; //setzt bisherGezahlt auf 0 zurück
         return wechselgeld; //gibt Wert von wechselgeld zurück, nicht im Textfeld
    }
    
}

javaDev2011

 

[infinityfan]

Morgen zusammen. Funktioniert jetzt alles tip top. @JavaDev2011, danke für deine Hilfe mit dem Gesammtcode, das hilft mir doch sehr um die Zusammenhänge besser sehen. Bin mal gespannt wie die Projektarbeit "Minesweeper" so wird .