Verbesserungen für meinen Code

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

    • Verbesserungen für meinen Code

      Hallo lieber M-Core User, ich habe am Samstag angefangen c++ zu lernen mit dem sicher wohl bekannten Buch C++ von A bis Z. Da ich nun heute die
      do...while Schleife im Buch als neues Thema hatte, dachte ich mir, das ich mal einen kleinen Taschenrechner schreibe um die Schleife sozusagen zu üben. Es klappt alles soweit wunderbar, allerdings sagte ein Freund von mir, der schon länger C++ lernt, das der Code noch zu verbessern sei, ich wollte natürlich nicht wissen wie, sondern wo. Allerdings hat er mir natürlich auch dies nicht verraten, da ich allerdings gerne wissen würde wo, damit ich daran arbeiten kann bzw. herausfinden kann was dort zu verbessern ist, dachte ich ich Frage mal hier in meinem neuen Forum nach :)

      Nun nach langer Rede hier noch endlich der Code hehe :

      C-Quellcode

      1. // Rechner by Asura Cryin v1.0.2
      2. #include <iostream>
      3. #include <cstdlib>
      4. using namespace std;
      6. int var, var1, var2;
      8. int main(void) {
      9. do {
      10. // Aufforderung zur Wahl der Rechenart
      11. cout << "\nBitte eine Rechenart waehlen : \n";
      12. cout << "-1- Addition \n";
      13. cout << "-2- Subtraktion \n";
      14. cout << "-3- Multiplikation \n";
      15. cout << "-4- Division \n";
      16. cout << "-5- Ende \n";
      18. if( !(cin >> var)) {
      19. cerr << "Fehler bei der Eingabe ! \n";
      20. exit(1);
      21. }
      22. // Beginn der Rechnungen und so
      23. switch (var) {
      24. case 1 :
      25. cout << "Sie haben Addition gewaehlt \n";
      26. cout << "Bitte Zahl eins eingeben : ";
      27. if( !(cin >> var1)) {
      28. cerr << "Fehler bei der Eingabe! ";
      29. exit(1);
      30. }
      31. cout << "Bitte Zahl zwei eingeben : ";
      32. if( !(cin >> var2)) {
      33. cerr << "Fehler bei der Eingabe! ";
      34. exit(1);
      35. }
      36. cout << var1 << " + " << var2 << " = " << (var1+var2);
      37. break;
      38. case 2 :
      39. cout << "Sie haben Subtraktion gewaehlt \n";
      40. cout << "Bitte Zahl eins eingeben : ";
      41. if( !(cin >> var1)) {
      42. cerr << "Fehler bei der Eingabe! ";
      43. exit(1);
      44. }
      45. cout << "Bitte Zahl zwei eingeben : ";
      46. if( !(cin >> var2)) {
      47. cerr << "Fehler bei der Eingabe! ";
      48. exit(1);
      49. }
      50. cout << var1 << " - " << var2 << " = " << (var1-var2);
      51. break;
      52. case 3 :
      53. cout << "Sie haben Multiplikation gewaehlt \n";
      54. cout << "Bitte Zahl eins eingeben : ";
      55. if( !(cin >> var1)) {
      56. cerr << "Fehler bei der Eingabe! ";
      57. exit(1);
      58. }
      59. cout << "Bitte Zahl zwei eingeben : ";
      60. if( !(cin >> var2)) {
      61. cerr << "Fehler bei der Eingabe! ";
      62. exit(1);
      63. }
      64. cout << var1 << " * " << var2 << " = " << (var1*var2);
      65. break;
      66. case 4 :
      67. cout << "Sie haben Division gewaehlt \n";
      68. cout << "Bitte Zahl eins eingeben : ";
      69. if( !(cin >> var1)) {
      70. cerr << "Fehler bei der Eingabe! ";
      71. exit(1);
      72. }
      73. cout << "Bitte Zahl zwei eingeben : ";
      74. if( !(cin >> var2)) {
      75. cerr << "Fehler bei der Eingabe! ";
      76. exit(1);
      77. }
      78. cout << var1 << " / " << var2 << " = " << (var1/var2);
      79. break;
      80. case 5 :
      81. cout << "Sie haben Ende gewaehlt \n";
      82. cout << "Das Programm wird beendet ";
      83. break;
      84. default :
      85. cout << "Fehler bei der Eingabe";
      86. break;
      87. }
      88. } while(var !=5);
      89. return 0;
      90. }
      Alles anzeigen

    • Werbung zur Unterstützung des Forums ( Bitte AddBlocker deaktivieren )

    • Hallo, ich meld mich mal wieder haha xP. Nun, ich hab mich dann heute Morgen dann noch einmal dran gesetzt und einen neuen Rechner gecodet(Übung macht ja bekanntlich den Meister ^^). Hab mir natürlich ein, zwei Erweiterungen einfallen lassen ^^ Allerdings macht er nun Zicken bei der Division o.o
      Alles was ich eingebe kommt mit einem total falschen Ergebnis raus, auch die Wiedergabe der Variablen ist falsch. Aber ich weiß nicht wieso.

      Hier einmal der abschnitt :

      Quellcode

      1. if (wahl1==1) {
      2. cout << "Sie möchten zwei Zahlen dividieren\n";
      3. cout << "Bitte Zahl eins eingeben\n";
      5. if( !(cin >> var)) {
      6. cerr << "Error! Fehler bei der Eingabe!\n";
      7. break;
      8. }
      9. cout << "Bitte Zahl zwei eingeben\n";
      11. if( !(cin >> var)) {
      12. cerr << "Error! Fehler bei der Eingabe!\n";
      13. break;
      14. }
      15. cout << var << " / " << var1 << " = " << (var/var1);
      16. break;
      Alles anzeigen
      könnte es daran liegen das long double nicht dafür geeignet ist?^^ (ich weiß auch das ich keinen Schutz vor 0 Divisionen habe und es dann ''crashd")
      wäre über Hilfe erfreut

      Edit: also ich hab das mal getestet, wenn ich int als Datentyp nutze geht es, mit long double allerdings nicht.
      Woran liegt das? Würde mich mal interessieren :O

    • Float ist ungenau double ist denke ich schon der richtige datentyp
      Du konntest anstatt bei jeder Aufgabe die zahlen einlesen konntest du dies schon vor dem Switch Case machen somit sparst du Code

      so würde das dann aussehen

      Quellcode

      1. cout << "[1]Addition \n" << "[2]Subtraktion\n" << "[3]Multiplikation\n" << "[4]Division\n";
      2. int select;
      3. if (!(cin >> select)) {cerr << "Fehler";cin.get();}
      4. double in1, in2;
      5. cout << "Zahl1 eingeben\n";
      6. if (!(cin >> in1)) {cerr << "Fehler";cin.get(); }
      7. cout << "Zahl2 eingeben\n";
      8. if (!(cin >> in2)) {cerr << "Fehler";cin.get(); }
      9. switch (select) {
      10. case 1 : cout << (in1+in2) << "\n"; break;
      11. case 2 : cout << (in1-in2) << "\n"; break;
      12. case 3 : cout << (in1*in2) << "\n"; break;
      13. case 4 : cout << (in1/in2) << "\n"; break;
      14. }
      Alles anzeigen