Taschenrechner Problem

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

  • Taschenrechner Problem

    Habe gerade mit C++ begonnen
    Zahlen werden kontinuirlich addiert subtrahiert etc
    Reset funktioniert auch
    mein problem ist das wenn ich 6 drücke um die schleife zu beenden springe das programm noch einmal in die else verzweigung
    hat irgendjemand eine idee wie sich das vermeiden liesse ?
    ich darf keine goto anweisung verwenden
    cin >> value2; gefolgt cin >> value2; darf ich im switch auch nicht verwenden :p
    ich bin ehrlich gesagt mit meinem latein am ende und wiess nicht mehr weiter

    Quellcode

    1. void add(float &x, float y)
    2. {
    3. float erg;
    4. erg = x+y;
    5. cout << x << "+" << y << "=" << erg <<endl ;
    6. x = erg;
    8. }
    11. void subtract(float &x, float y)
    12. {
    13. float erg;
    14. erg = x-y;
    15. cout << x << "-" << y << "=" << erg <<endl;
    16. x = erg ;
    18. }
    20. void multiply (float &x,float y)
    21. {
    22. float erg;
    23. erg = x*y;
    24. cout << x << "*" << y << "=" << erg <<endl;
    25. x = erg;
    27. }
    29. void divide (float &x, float y)
    30. {
    31. if(y==0)
    32. {
    33. cout << "0 ist nicht zulässig"<<endl;
    34. }
    35. else
    36. {
    37. float erg;
    38. erg = x/y;
    39. cout << x << "/" << y << "=" << erg <<endl;
    40. x = erg ;
    42. }
    44. }
    46. int main ()
    47. {
    49. int calc;
    50. float value1;
    51. float value2;
    53. cout << "Wähle 1(+) 2(-) 3(*) 4(/) 5(R) 6(END)"<<endl;
    54. cin >> calc;
    56. cout << "1: ";
    57. cin >> value1;
    59. cout << "2: ";
    60. cin >> value2;
    62. while(calc<6)
    63. {
    65. switch(calc)
    66. {
    67. case 1: add(value1,value2);break;
    68. case 2: subtract(value1,value2);break;
    69. case 3: multiply(value1,value2);break;
    70. case 4: divide(value1,value2);break;
    72. }
    73. cin>>calc;
    75. if(calc == 5)
    76. {
    77. cout << "Wähle 1(+) 2(-) 3(*) 4(/) 5(R) 6(END)"<<endl;
    78. cin >> calc;
    80. cout << "1: ";
    81. cin >> value1;
    83. cout << "2: ";
    84. cin >> value2;
    85. }
    87. else
    88. {
    89. cout<<"2:";
    90. cin>>value2;
    91. }
    94. }
    95. cout<<"Programm beendet";
    96. return 0;
    97. }
    Alles anzeigen

    Dieser Beitrag wurde bereits 3 mal editiert, zuletzt von Sylar ()

  • hm. also erstmal hast du Redundanz im Code, also du hast haargenau die gleichen Befehle mehrmals drinstehen, also diese hier:

    Quellcode

    1. cout << "Wähle 1(+) 2(-) 3(*) 4(/) 5(R) 6(END)"<<endl;
    2. cin >> calc;
    4. cout << "1: ";
    5. cin >> value1;
    7. cout << "2: ";
    8. cin >> value2;

    das solltest du auf jedenfall vermeiden, Redundanz ist immer sehr fehleranfällig und schwierig zu warten. Du könntest z.B. innerhalb der while Schleife einfach die Switch-Block ans Ende setzen und evtl um die Cases 5,6 und am besten noch nen default-case, der ausgeführt wird wenn die eingabe falsch ist.

    und folgenden Teil versteh ich nicht:

    Quellcode

    1. if(calc == 5)
    2. {
    3. [...]
    4. }
    6. else
    7. {
    8. cout<<"2:";
    9. cin>>value2;
    10. }

    Wenn die Eingabe ungleich 5 ist wird nochmal value2 eingelesen? Warum?

  • hallo,

    klar, wenn du 6 eingiebst hast du keinen case dafür und dein if wird auch nicht ausgeführt sondern er landet im else zweig!

    1) ich würde immer (außer in super ausnahmesituationen) einen default zweig im switch machen... du kannst dir fast nie sicher sein das einer deiner cases zutrifft!
    2) mach doch einfach einen case für 6 der dann das programm beendet... z.b.:

    Quellcode

    1. case 1:...
    2. case 2:...
    3. case 3:...
    4. case 4:...
    5. case 5:...
    6. case 6: return 0;


    dann sollte das auch funktionieren.
    hoffe ich konnte helfe!

    thx, truespin

  • Quellcode

    1. cout<<"|***CALCULATOR***|"<<endl;
    2. cout<< "enter digit 1 > (+) (-) (*) (/) > digit 2 > solve with '=' ENTER "<<endl;
    4. do
    5. {
    6. cout << endl;
    7. cout << "<: ";
    8. cin >> value1;
    10. bool iloop;
    11. bool show; //var zeige letztes erg
    12. //bool wait;
    14. do
    15. {
    16. cout<<"<: ";
    18. if(show)
    19. {
    20. cout<<value1;
    21. }
    23. cin >> calc;
    25. if ((calc =='+') || (calc =='-') || (calc =='*') || (calc =='/'))
    27. {
    28. iloop = true;
    29. show = true;
    31. cin >> value2 >> solve;
    33. if(solve == '=')
    34. {
    35. switch(calc)
    36. {
    37. case '+': add(value1,value2);break;
    38. case '-': subtract(value1,value2);break;
    39. case '*': multiply(value1,value2);break;
    40. case '/': divide(value1,value2);break;
    41. }
    42. //mem = value1;
    43. //cout<<endl<<mem<<"zwischenspeicher";
    44. cout << endl << "1[Continue]"<<"--"<<"2[Reset]"<<"--"<<"3[END]"<< endl;
    45. cin >> options;
    46. }
    49. }
    50. else
    51. {
    52. cout << endl << "eingabe nicht definiert" << endl;
    54. }
    56. if(options == 1)
    57. {
    58. iloop=false;
    60. }
    62. }while(!iloop);
    64. if (options == 2)
    65. {
    66. oloop = false;
    67. }
    69. if(options == 3)
    70. {
    71. cout<<"program terminated";return 0;
    72. }
    74. }while(!oloop);
    Alles anzeigen


    habe ihn mal komplett neugeschrieben da der rechner recht fehleranfällig und auch mein lehrer recht unzufrieden war

    es ist nun möglich 2/34= einzugeben und vor allem hab ich hoffentlich nun die redudanzen grossteils entfernt :p

    nun muss allerdings der rechner noch erweitert werden ich soll nun 1+2+4+3+4+2 eingeben können und wie schon in meinem quellcode zu sehen soll bei eingabe des "=" berechnet werden
    es soll aber auch möglich sein 2+2 direkt danach eingeben zu können dann "=" und zu lösen
    ich habe aber keine ahnung wie ich lösen könnte mein plan value1 in einer zwischnvariable zu speichern hat nicht funktioniert zumindest hatte ich keine idee mehr wie man weiter vorgehen könnte/müsste

    funktions prototypen :

    Quellcode

    1. void add(float &x, float y)
    2. {
    3. float erg;
    4. erg = x+y;
    5. cout << x << "+" << y << "=" << erg <<endl ;
    6. x = erg;
    8. }
    11. void subtract(float &x, float y)
    12. {
    13. float erg;
    14. erg = x-y;
    15. cout << x << "-" << y << "=" << erg <<endl;
    16. x = erg ;
    18. }
    20. void multiply (float &x,float y)
    21. {
    22. float erg;
    23. erg = x*y;
    24. cout << x << "*" << y << "=" << erg <<endl;
    25. x = erg;
    27. }
    29. void divide (float &x, float y)
    30. {
    31. if(y==0)
    32. {
    33. cout << "0 ist nicht zulässig"<<endl;
    34. }
    35. else
    36. {
    37. float erg;
    38. erg = x/y;
    39. cout << x << "/" << y << "=" << erg <<endl;
    40. x = erg ;
    42. }
    44. }
    Alles anzeigen


    ich beginne konsolen anwendungen langsam zu hassen komme von Java und habe hauptsächlich programme mit oberflächen erstellet was wesentlich besser von der hand ging

    Dieser Beitrag wurde bereits 3 mal editiert, zuletzt von Sylar ()