wo is der fehler? :)

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • wo is der fehler? :)

    hey ho, ich weiß, dass das programm eher trivial ist, aber es macht nich das was ich will und ich weiß nich warum...
    also aufgabe ist erstmal zwei zahlen einzugeben, die dann durch auswahl in einem menü addiert, dividiert,.. werden sollen und es dann so zu erweitern, dass immer wieder neue zahlen eingeben werden können bis zum abbruch.
    bei mir ist es jetz so, dass ich die zahlen eingeben kann und dann auch den entsprechnenden buchstaben, er aber nichts rechnet/ausgibt/... sondern einfach wieder nach 2 zahlen fragt. wenn ich in der if-bedingung ein "=" wegnehme addiert er immer, egal welcher buchstabe eingegeben wird, ich kann daraus aber nicht auf das problem schließen... sry is vll echt trivial für euch, aber ich bin mit char noch nicht vertraut.
    ich kopier euch mal meinen quelltext rein:

    Quellcode

    1. #include <stdio.h>
    2. #include <math.h>
    4. int main()
    5. {
    6. char c;
    7. float ergebnis,x,y;
    8. while(1)
    9. {
    10. printf("Geben Sie die erste Zahl ein: ");scanf("%f",&x);
    11. printf("Geben Sie die zweite Zahl ein: ");scanf("%f",&y);
    12. printf("Waehlen Sie a fuer Addition, m fuer Multiplikation, d fuer Division, s fuer Subraktion oder q um das Programm zu beenden: ");
    13. scanf("%c",&c);
    14. getchar ();
    16. if (c=='a')
    17. {
    18. printf("Addition beider Zahlen\n");
    19. ergebnis=x+y;
    20. printf("%f + %f = %f\n",x,y,ergebnis);
    21. continue;
    22. }
    23. if (c=='m')
    24. {
    25. printf("Multiplikation beider Zahlen\n");
    26. ergebnis=x*y;
    27. printf("%f * %f = %f\n",x,y,ergebnis);
    28. continue;
    29. }
    30. if (c=='d')
    31. {
    32. printf("Division beider Zahlen\n");
    33. ergebnis=x/y;
    34. printf("%f / %f = %f\n",x,y,ergebnis);
    35. continue;
    36. }
    37. if (c=='s')
    38. {
    39. printf("Subtraktion beider Zahlen\n");
    40. ergebnis=x-y;
    41. printf("%f - %f = %f\n",x,y,ergebnis);
    42. continue;
    43. }
    44. if (c=='q')
    45. break;
    46. }
    47. printf("Programm beendet!");
    48. return 0;
    49. }
    Alles anzeigen



    vll könnte man das generell eleganter machen, aber bei der aufgabe gehts um die break und continue anweisung, da die jetz eingeführt worden.
    wär lieb wenn mal jemand drüber schauen könnte. danke.

    //EDIT: highlighted by d0nut

  • der Fehler liegt bei getchar()

    printf("Geben Sie die erste Zahl ein: ");scanf("%f",&x);
    printf("Geben Sie die zweite Zahl ein: ");scanf("%f",&y);
    printf("Waehlen Sie a fuer Addition, m fuer Multiplikation, d fuer Division, s fuer Subraktion oder q um das Programm zu beenden: ");
    scanf("%c",&c);
    getchar (); <- Das hat hier nix verloren

  • Willkommen im Forum könntest du des nächste mal bitte mit highlight ner machen dann sieht mans besser

    so was hast du vergessen das

    fflush(stdin); sollte nach jedem scanf stehen(außer in manchen fällen)
    da sich sonst noch was von der vorherigen eingabe drauf befindet
    also das was in y bzw x steht also kann es nie a,s,d,m oder q sein außerdem ne struktur sollte man auch lernen

    MFG Donasa

    Quellcode

    1. #include <stdio.h>
    2. #include <math.h>
    4. int main()
    5. {
    6. char c;
    7. float ergebnis,x,y;
    8. while(1)
    9. {
    10. printf("Geben Sie die erste Zahl ein: ");
    11. scanf("%f",&x);
    12. fflush(stdin);
    13. printf("Geben Sie die zweite Zahl ein: ");
    14. scanf("%f",&y);
    15. fflush(stdin);
    16. printf("Waehlen Sie a fuer Addition, m fuer Multiplikation, d fuer Division, s fuer Subtraktion oder q um das Programm zu beenden:");
    17. scanf("%c",&c);
    18. fflush(stdin);
    19. if(c=='a')
    20. {
    21. printf("Addition beider Zahlen\n");
    22. ergebnis=x+y;
    23. printf("%f + %f = %f\n",x,y,ergebnis);
    24. continue;
    25. }
    26. if(c=='m')
    27. {
    28. printf("Multiplikation beider Zahlen\n");
    29. ergebnis=x*y;
    30. printf("%f * %f = %f\n",x,y,ergebnis);
    31. continue;
    32. }
    33. if(c=='d')
    34. {
    35. printf("Division beider Zahlen\n");
    36. ergebnis=x/y;
    37. printf("%f / %f = %f\n",x,y,ergebnis);
    38. continue;
    39. }
    40. if(c=='s')
    41. {
    42. printf("Subtraktion beider Zahlen\n");
    43. ergebnis=x-y;
    44. printf("%f - %f = %f\n",x,y,ergebnis);
    45. continue;
    46. }
    47. else
    48. break;
    49. }
    50. printf("Programm beendet!");
    51. return 0;
    52. }
    Alles anzeigen
    Meine Sprachen: C, C++,C# PHP
    E-Mail | MSN: gre.burgstaller@aon.at

  • aja ich hab das if(c=='q') mit else ersetzt das heißt das man z.b.: auch f zum beenden drücken kann und warum wenn du bei
    if(c='a') immer addiert hast ist auch klar da

    =....Zuweisung da die funktoniert in der klammer auch eine wahre aussage
    ==..Vergleich das musst du benutzen wenn du schon magst ob was gleich ist


    MFG Donasa
    Meine Sprachen: C, C++,C# PHP
    E-Mail | MSN: gre.burgstaller@aon.at

  • hey ho, habt ganz lieben dank für die superschnellen antworten!
    ok also ohne das getchar() funzt es, was mir jetz ein kleines rätsel ist, weil meine dozentin das explizit mit reingenommen hat mit der begründung, dass sonst die eingabetaste auch als zeichen ausgewertet werden würde und das von getchar unterbunden wird... aber da es ohne funktioniert, glaub ich einfach mal, dass ihr recht habt :)
    und zur switch anweisung... die hatten wir noch nich, also ich hab mir das schon durchgelesen, aber ich denke die aufgabe hat sie jetz wirklich gestellt, damit wir den unterschied zw continue und break sehen - ich bemerke hier mal am rande, dass ich nur den kurs "einführung in die programmierung" belege und ich somit erst ein paar wochen lulli C mache *g.
    so das mit dem highlight tut mir leid, ich hab mich vorhin erst angemeldet und bin jetz noch nich vertraut mit dem forum... und jaaaaa, die struktur is da bei mir, is nur durchs kopieren nich so schön geworden, ich fands jetz auch nich so komplex, das ich das hätte nochmal ändern wollen.
    für den fall dass y=0 is, mach ich noch, wollt nur erstmal zwischentesten und da da ja schon nix ging, hab ich das erstmal hinten angestellt... aber von try catch hab ich wirklich noch nichts gehört...

    also habt ganz vielen dank für eure hilfe!!!

  • "Donasa" schrieb:

    aja du solltest unbedingt noch bei der div. checklen ob y == 0 ist weil wenn das der Fall ist gibts nen error oder du machst es meiner meinung nach eleganter mit try....catch


    Division durch 0 gibt doch keine C++ - Exception ... da kannst du dann nichts mit catch() machen ...

    Wenn ich diese IEEE75? richtig verstanden habe dürfte das ergebniss einer Division durch 0 NaN sein ... möglicherweise gibts dann beim printf() einen Laufzeitfehler
    There are only 10 types of people in the world: Those who understand binary, and those who don't.

    Download meines ersten Spiels:HIER
    Über Feedback würde ich mich freuen ;)