Problem in Code

[Simon N. (ardi)]

Hi,
ich wollte ein einfaches Getränkeautomatenprogramm machen.
Wenn nicht der Betrag eingegeben wird wie erwünscht soll kommen:
Bitte geben sie den passenden Betrag ein oder.
Das tut es auch. Aber wenn ich den richtigen Wert eingeben kommt das 
auch und wenn ich dann da nochmal den richtigen Wert eingeben dann erst 
ist das Programm erfolgreich:

.
.
.
if(wahl==3){
    printf("Bitte geben sie 1.50 Euro ein:");
    scanf("%f", &summe);
    do{
        printf("Bitte geben sie den vollen Betrag ein:");
        scanf("%f", &summe);
    }while(summe!=1.50);
        if(summe==1.50){
            printf("Bitte entnehmen Sie Ihr Getreank");
        }

}
.
.
.

Ich möchte wenn ich als erstes den Betrag richtig eingegeben habe gleich 
zum Ende.
Danke und Gruß?

[Helfer (Gast)]

Hallo Simon,
dann darfst du keine do-while Schleife benutzen. Diese wird nämlich erst 
einmal durchlaufen, bevor das Abbruchkriterium geprüft wird.
Ersetzt du sie durch eine While-Schleife, dann sollte dein Code laufen.

[Simon N. (ardi)]

Jetzt jetzt kommt nach jedem richtigen eingeben ich solle den passenden 
Preis eingeben:

if(wahl==2){
    printf("Bitte geben sie 2 Euro ein:");
    scanf("%d", &summe);
    while(summe!=2){
        printf("Bitte geben sie den vollen Betrag ein:");
        scanf("%d", &summe);
    }
        if(summe==2){
            printf("Bitte entnehmen Sie Ihr Getreank");
        }

}

[Dr. Sommer (Gast)]

Mit diesen kleinen "einfachen" Werten funktioniert das vermutlich noch, 
aber generell ist es eine schlechte Idee float's so zu benutzen, denn 
wenn man zwei floats summiert kommt nicht unbedingt die mathematisch 
Summe heraus, sondern z.B. etwas das um 0.00001 abweicht. Und wenn man 
dann auf exakt 1.5 prüft, wäre 1.4999999 schon falsch... Daher prüft 
man entweder einen Bereich ala 1.4<x<1.6, oder verwendet doch lieber 
Integer (und rechnet komplett in Cent), denn dort hat man dieses Problem 
nicht...

[Simon N. (ardi)]

Ok ich habe jetzt mal alle Werte auf Integer gestellt, also ohne Komma, 
und es funktioniert.
Aber was ist wenn ich jetzt Kommas haben möchte?

[Peter II (Gast)]

Simon Name schrieb:
> Aber was ist wenn ich jetzt Kommas haben möchte?

dann musst du etwas unscharf abfragen:

if( abs(summe-2) < 0.001 ) {

[Simon N. (ardi)]

Ok Danke mal

[Dr. Sommer (Gast)]

Simon Name schrieb:
> Aber was ist wenn ich jetzt Kommas haben möchte?
Einfach komplett in Cent rechnen (also ohne Komma) und dann einfach nur 
die Anzeige mit Komma machen:

printf ("%d,%d€\n", cents/100, cents%100);

Und die Eingabe auch:

int euros, cents;
scanf("%d,%d\n", &euros, &cents);
cents += euros * 100;

Und dann mit nur mit "cents" weiterrechnen.

So vermeidest du alle Probleme mit "float" komplett, vor allem eben die 
Ungenauigkeit in Peter II's Antwort. float sollte grundsätzlich nur 
benutzt werden wenn man ganz genau weiß wie das funktioniert und wie die 
Probleme entstehen... Integer funktionieren hier ja wunderbar da man 
alles in Cents rechnen kann, das Komma ist nur eine Frage der 
Ein/Ausgabe.