Mein erstes Programm (Taschenrechner)...

  • Mein erstes Programm (Taschenrechner)...

    Huhu
    Bin neu hier und hoffe das ich hier richtig bin...
    Ich bin seit 3 Tage dabei, C++ zu lernen.... _naja_ ... Und ich habe auch schon was "programmiert", nämlich einen Taschenrechner... das übliche Anfängerprogramm...
    Ich würde gerne eure Meinung wissen, da ich glaube das ich warscheinlich total "unsauber" gearbeitet habe und mir das viel zu kompliziert gemacht habe... bzw. Anfängerfehler gemacht habe, die einfach nicht sein müssen :)
    Das Teil hat mich 3-4 Stunden gekostet, hat aber sehr viel spaß gemacht und ich bin motiviert weiter zu lernen :)
    Ich benutze größtenteils das Tutorial hier : cplusplus.com/doc/tutorial/
    Das ist leider auf Englisch, soll aber wohl sehr gut sein. Ich verstehe auch alles mehr oder weniger... (wollte mir später aber noch ein Buch kaufen... um genauer darauf einzugehen)...

    Das Programm funktioniert bestens... nur zwei Fehler hat es noch und ich weiß nicht wie ich diese behebe...
    1.) z.B Zeile 59... Bei der Abfrage ob eine neue Rechnung erfolgen soll, muss der Benutzer zwingend klein schreiben... leider weiß ich nicht wie man das ändert...
    2.) ... wenn der Benutzer z.B. x oder y eingeben soll (sprich eine Zahl) und man anstatt einer Zahl einen Buchstaben eingibt... dann kommt ein Ergebnis, das aber die ganze Zeit erscheint... also das man nicht mehr lesen kann was angezeigt wird da sich das unendlich lang wiederholt ... mhmpf...

    Also nochmal im Klartext : Was haltet ihr von dem Programm...? Irgendwelche Ratschläge, Tipps oder so? Wäre euch sehr dankbar...

    Gruß Necron9


    Quellcode

    1. #include <string>
    2. #include <iostream>
    4. using namespace std;
    6. int main ()
    7. {
    9. double ergebnis;
    10. double x;
    11. double y;
    12. double zeichen;
    13. string antwortja = "ja";
    17. Start:
    18. cout << "\n\n";
    19. cout << "****************************\n";
    20. cout << "****** Taschenrechner ******\n";
    21. cout << "****************************\n\n\n";
    24. //Eingabe von Rechenart
    25. cout << "Druecken Sie [ ] und Enter...\n";
    26. cout << "(1) fuer Addition\n";
    27. cout << "(2) fuer Subtraktion\n";
    28. cout << "(3) fuer Multiplikation\n";
    29. cout << "(4) fuer Division\n\n";
    30. rechenart:
    31. cout << "\n\n";
    32. cout << "Geben Sie ihre Rechenart an: ";
    33. cin >> zeichen;
    34. cout << "\n";
    37. //Eingabe der Rechenart
    38. if (zeichen > 4)
    39. {
    40. cout << "Sie muessen 1,2,3 oder 4 eingeben!";
    41. goto rechenart;
    42. }
    44. //Rechenart 1
    45. if (zeichen == 1)
    46. {
    47. cout << "Geben sie x an: ";
    48. cin >> x;
    49. cout << "Geben sie y an: ";
    50. cin >> y;
    51. ergebnis=x+y;
    52. cout << "------------------\n";
    53. cout << "Das Ergebnis lautet: " << ergebnis;
    54. cout << "\n\n\n\n\n\n";
    57. cout << "Neue Rechnung? ";
    58. cin >> antwortja;
    59. if (antwortja == "ja")
    60. {
    61. goto Start;
    62. }
    65. }
    66. //Rechenart 2
    67. if (zeichen == 2)
    68. {
    69. cout << "Geben sie x an: ";
    70. cin >> x;
    71. cout << "Geben sie y an: ";
    72. cin >> y;
    73. ergebnis=x-y;
    74. cout << "------------------\n";
    75. cout << "Das Ergebnis lautet: " << ergebnis;
    76. cout << "\n\n\n\n\n\n";
    78. cout << "Neue Rechnung? ";
    79. cin >> antwortja;
    80. if (antwortja == "ja")
    81. {
    82. goto Start;
    83. }
    84. }
    86. //Rechenart 3
    87. if (zeichen == 3)
    88. {
    89. cout << "Geben sie x an: ";
    90. cin >> x;
    91. cout << "Geben sie y an: ";
    92. cin >> y;
    93. ergebnis=x*y;
    94. cout << "------------------\n";
    95. cout << "Das Ergebnis lautet: " << ergebnis;
    96. cout << "\n\n\n\n\n\n";
    98. cout << "Neue Rechnung? ";
    99. cin >> antwortja;
    100. if (antwortja == "ja")
    101. {
    102. goto Start;
    103. }
    104. }
    106. //Rechenart 4
    107. if (zeichen == 4)
    108. {
    109. cout << "Geben sie x an: ";
    110. cin >> x;
    111. cout << "Geben sie y an: ";
    112. cin >> y;
    113. ergebnis=x/y;
    114. cout << "------------------\n";
    115. cout << "Das Ergebnis lautet: " << ergebnis;
    116. cout << "\n\n\n\n\n\n";
    118. cout << "Neue Rechnung? ";
    119. cin >> antwortja;
    120. if (antwortja == "ja")
    121. {
    122. goto Start;
    123. }
    125. }
    126. }
    Alles anzeigen

  • Hallo und Willkommen im Forum,

    Dein Quellcode ist an sich ganz in Ordnung. Nur eine Fatale Sache ist da drinn, die solltest du dir schleunigst abgewöhnen.

    Quellcode

    1. goto Start;


    Sowas macht man nicht, die goto zeiten sind vorbei :)

    Du sagtest ja, das du noch Anfänger bist, also mache dir mal keine Sorgen um die Optik, das kann mit etwas übung von ganz alleine. Du wirst deinen eigenen Style finden müssen. Eigne dir erst mal mehrere Techniken an, dann wirst du automatisch dahinter kommen wie man sachen anders lösen kann.
    Erst wenn der letzte FTP Server kostenpflichtig, der letzte GNU-Sourcecode verkauft, der letzte Algorithmus patentiert, der letzte Netzknoten kommerzialisiert, die letzte Newsgroup moderiert wird, werdet Ihr merken, dass man mit Geld allein nicht programmieren kann.

  • Ein weiter Vorschlag ist Redundanzen vermeiden - sieht dann etwa so aus:

    Quellcode

    1. #include <string>
    2. #include <iostream>
    4. using namespace std;
    6. int main ()
    7. {
    9. double ergebnis;
    10. double x;
    11. double y;
    12. double zeichen;
    13. string antwortja = "ja";
    17. Start:
    18. cout << "\n\n";
    19. cout << "****************************\n";
    20. cout << "****** Taschenrechner ******\n";
    21. cout << "****************************\n\n\n";
    24. //Eingabe von Rechenart
    25. cout << "Druecken Sie [ ] und Enter...\n";
    26. cout << "(1) fuer Addition\n";
    27. cout << "(2) fuer Subtraktion\n";
    28. cout << "(3) fuer Multiplikation\n";
    29. cout << "(4) fuer Division\n\n";
    30. rechenart:
    31. cout << "\n\n";
    32. cout << "Geben Sie ihre Rechenart an: ";
    33. cin >> zeichen;
    34. cout << "\n";
    37. //Eingabe der Rechenart
    38. if (zeichen > 4)
    39. {
    40. cout << "Sie muessen 1,2,3 oder 4 eingeben!";
    41. goto rechenart;
    42. }
    44. cout << "Geben sie x an: ";
    45. cin >> x;
    46. cout << "Geben sie y an: ";
    47. cin >> y;
    48. //Rechenart 1
    49. if (zeichen == 1)
    50. {
    51. ergebnis=x+y;
    52. }
    53. //Rechenart 2
    54. if (zeichen == 2)
    55. {
    56. ergebnis=x-y;
    57. }
    58. //Rechenart 3
    59. if (zeichen == 3)
    60. {
    61. ergebnis=x*y;
    62. }
    63. //Rechenart 4
    64. if (zeichen == 4)
    65. {
    66. ergebnis=x/y;
    67. }
    68. cout << "------------------\n";
    69. cout << "Das Ergebnis lautet: " << ergebnis;
    70. cout << "\n\n\n\n\n\n";
    73. cout << "Neue Rechnung? ";
    74. cin >> antwortja;
    75. if (antwortja == "ja")
    76. {
    77. goto Start;
    78. }
    80. }
    Alles anzeigen


    Gross- und Kleinschreibung:
    support.microsoft.com/kb/815783/de

    Division durch 0 ?

    goto steht schon in deinem Tutorial drin, das es kritisch werden kann. ;)

    Mal nach "switch" suchen.

    MfG bcc-fan

  • C-Quellcode

    1. #include <string>
    2. #include <iostream>
    4. using namespace std;
    6. int main ()
    7. {
    9. double ergebnis;
    10. double x;
    11. double y;
    12. double zeichen = 1;
    13. string antwortja = "j";
    16. cout << "\n\n";
    17. cout << "****************************\n";
    18. cout << "****** Taschenrechner ******\n";
    19. cout << "****************************\n\n\n";
    22. //Eingabe von Rechenart
    23. cout << "Druecken Sie [ ] und Enter...\n";
    24. cout << "(1) fuer Addition\n";
    25. cout << "(2) fuer Subtraktion\n";
    26. cout << "(3) fuer Multiplikation\n";
    27. cout << "(4) fuer Division\n\n";
    28. cout << "\n\n";
    29. cout << "Geben Sie ihre Rechenart an: ";
    30. cin >> zeichen;
    31. cout << "\n";
    34. //Eingabe der Rechenart
    35. if (zeichen > 4)
    36. {
    37. cout << "Sie muessen 1,2,3 oder 4 eingeben!";
    38. return main();
    39. }
    41. cout << "Geben sie x an: ";
    42. cin >> x;
    43. cout << "Geben sie y an: ";
    44. cin >> y;
    45. //Rechenart 1
    46. if (zeichen == 1)
    47. {
    48. ergebnis=x+y;
    49. }
    50. //Rechenart 2
    51. if (zeichen == 2)
    52. {
    53. ergebnis=x-y;
    54. }
    55. //Rechenart 3
    56. if (zeichen == 3)
    57. {
    58. ergebnis=x*y;
    59. }
    60. //Rechenart 4
    61. if (zeichen == 4)
    62. {
    63. ergebnis=x/y;
    64. }
    65. cout << "------------------\n";
    66. cout << "Das Ergebnis lautet: " << ergebnis;
    67. cout << "\n\n\n\n\n\n";
    69. /*
    70. cout << "Neue Rechnung? ";
    71. cin >> antwortja;
    72. if (antwortja == "ja") // hier haben wir das problem, dass "JA" und "jop" nicht gehen!
    73. {
    74. goto Start; // goto ist keine gute Loesung.
    75. }
    76. */
    78. cout << "Neue Rechnung? j/n:\n";
    79. cin >> antwortja;
    80. if (antwortja == "j")
    81. {
    82. return main();
    83. }
    84. }
    Alles anzeigen


    Man kann das auch noch verbessern, aber das würde den Code zu stark verändern.
    Aber fürs erst Programm nicht schlecht.
    Auch wenn man jetzt durch return main() zum Anfang kommt ist es immer noch besser als goto. Wenn du lernst neue Funktionen zu machen , dann kann man mit return alles "schöner" machen.
    mfg

  • Also hier eine einfache Version ohne goto oder ähnliche Verrenkungen:

    Todo hießt da sollte noch mal nachgedacht werden ;)

    Quellcode

    1. // Taschenrechner simpel
    2. // ToDo:
    3. // Operanten -> int oder besser ?
    4. // Bei der Division Sicherheitsabfrage einfügen!
    5. // Wenn der Rechner mehr leisten soll, dann ergaenzt ihn
    7. #include <iostream>
    8. using namespace std;
    10. int main ()
    11. {
    12. int zeichen;
    13. int O1, O2;
    15. bool bigloop;
    16. do
    17. {
    18. cout << "Bitte geben Sie den 1. Operanten ein: ";
    19. cin >> O1;
    21. bool loop = false;
    22. do
    23. {
    24. cout << "Geben Sie die Rechenart ein ( (1) -> + (2) -> - (3) -> * (4) -> / )" << endl;
    25. cout << ", nur Zahlen von 1 bis 4 werden aktzeptiert. ";
    26. cin >> zeichen;
    27. if ((zeichen >= 1) && (zeichen <=4))
    28. {
    29. loop = true;
    30. }
    31. else
    32. {
    33. cout << endl << " Rechenart nicht implementiert" << endl;
    34. }
    35. }while(!loop);
    37. cout << "Bitte geben Sie den 2. Operanten ein: ";
    38. cin >> O2;
    40. cout << endl;
    41. switch(zeichen)
    42. {
    43. case 1: cout << O1 << " + " << O2 << " = " << O1 + O2 << endl; break;
    44. case 2: cout << O1 << " - " << O2 << " = " << O1 - O2 << endl; break;
    45. case 3: cout << O1 << " * " << O2 << " = " << O1 * O2 << endl; break;
    46. case 4: cout << O1 << " / " << O2 << " = " << O1 / O2 << endl; break;
    48. default: cout << "\n\tSomething go's wrong";
    49. break;
    50. }
    51. bigloop = true;
    52. cout << endl << " Zeichen eingeben - 9 für neue Aufgabe" << endl; // statt der 9 etwa in J/N oder Y/N aendern
    53. cin >> zeichen;
    54. if (zeichen == 9)
    55. {
    56. bigloop = false;
    57. }
    59. }while(!bigloop);
    60. }
    Alles anzeigen


    MfG bcc-fan

  • Ich finde dieses Ergebnis am leichtesten :D





    Quellcode

    1. #include <cstdlib>
    2. #include <iostream>
    3. #include <string>
    6. using namespace std;
    8. int eingabe;
    9. float zahl1;
    10. float zahl2;
    11. float ergebnis;
    12. string eingabe1;
    15. int main()
    16. {
    17. cout << " " << endl;
    18. cout << " " << endl;
    19. cout << " TASCHENRECHNER " << endl;
    20. cout << " " << endl;
    21. cout << " " << endl;
    23. cout << "Möchten sie eine Addition(1) Multiplikation(2) Division(3)\n oder Subraktion(4) durchführen?"<< endl;
    24. cin >> eingabe;
    25. cout << "Erste Zahl :" << endl;
    26. cin >> zahl1;
    27. cout << "Zweite Zahl : " << endl;
    28. cin >> zahl2;
    30. switch (eingabe)
    31. {
    32. case 1:
    33. cout << "Ergebnis : " << zahl1+zahl2 << endl;
    34. break;
    35. case 2:
    36. cout << "Ergebnis : " << zahl1*zahl2 << endl;
    37. break;
    38. case 3 :
    39. cout << "Ergebnis : " << zahl1/zahl2 << endl;
    40. break;
    41. case 4 :
    42. cout << "Ergebnis : " << zahl1 - zahl2 << endl;
    43. break;
    46. }
    48. cout << "Möchten sie nochmal" << endl;
    49. cin >> eingabe1;
    52. if (eingabe1=="ja" || eingabe1=="Ja")
    53. {
    54. return main();
    55. }
    59. system("PAUSE");
    60. return EXIT_SUCCESS;
    61. }
    Alles anzeigen