erster c++ code tipps und kretik

  • erster c++ code tipps und kretik

    Hallo zusammen,
    ich habe mich seit ungefair einer woche mit c++ beschäftigt, und wollte mal meinen code vorstellen.

    Ich
    hoffe ihr könnt mir Tipps und gute Kritik mit auf dem weg geben wie ich
    es besser machen kann was falsch oder auch nur sinnlos ist.

    Habe
    diesen code gestern abend geschriben ist derzeit mein erstes mit c++,
    bitte um verständnis bei schlechter ausführung da ich hier noch am
    anfang stehe. :thumbsup:

    Quellcode

    1. #include <iostream>
    3. void plus();
    4. void minus();
    5. void mal();
    6. void geteilt();
    8. using namespace std;
    10. int main()
    11. {
    12. char Taschenrechner;
    14. cout << "Taschenrechner Waehle eine Zahl und druecke Enter\n"
    15. << "\t1.Plus +\n"
    16. << "\t2.Minus -\n"
    17. << "\t3.Mal *\n"
    18. << "\t4.Geteilt :\n"
    19. << "\t5.Bildschirm Leeren\n"
    20. << "\t6.Beenden\n";
    21. cin >> Taschenrechner;
    22. switch(Taschenrechner)
    23. {
    24. case '1':
    25. cout << "Plus +\n";
    26. plus();
    27. break;
    28. case '2':
    29. cout << "Minus -\n";
    30. minus();
    31. break;
    32. case '3':
    33. cout << "Mal *\n";
    34. mal();
    35. break;
    36. case '4':
    37. cout << "Geteilt :\n";
    38. geteilt();
    39. break;
    40. case '5':
    41. system("cls");
    42. main();
    43. break;
    44. case '6':
    45. return 0;
    46. default:
    47. cout << "Du hast keine Option Gewaehlt.";
    48. }
    49. cin.sync();
    50. cin.get();
    51. return 0;
    52. }
    54. void plus()
    55. {
    56. int zahl1, zahl2;
    57. cout << "Erste Zahl\n";
    58. cin >> zahl1;
    59. cout << "Zweite Zahl\n";
    60. cin >> zahl2;
    62. cout << "Ergebnis\n" << zahl1 + zahl2 << endl;
    63. main();
    64. }
    66. void minus()
    67. {
    68. int zahl1, zahl2;
    69. cout << "Erste Zahl\n";
    70. cin >> zahl1;
    71. cout << "Zweite Zahl\n";
    72. cin >> zahl2;
    74. cout << "Ergebnis\n" << zahl1 - zahl2 << endl;
    75. main();
    76. }
    78. void mal()
    79. {
    80. int zahl1, zahl2;
    81. cout << "Erste Zahl\n";
    82. cin >> zahl1;
    83. cout << "Zweite Zahl\n";
    84. cin >> zahl2;
    86. cout << "Ergebnis\n" << zahl1 * zahl2 << endl;
    87. main();
    88. }
    90. void geteilt()
    91. {
    92. int zahl1, zahl2;
    93. cout << "Erste Zahl\n";
    94. cin >> zahl1;
    95. cout << "Zweite Zahl\n";
    96. cin >> zahl2;
    98. cout << "Ergebnis\n" << zahl1 / zahl2 << endl;
    99. main();
    100. }
    Alles anzeigen

  • Guten Abend amogma :)

    Schön, dass du mit C++ anfängst. Schöne Sprache ;)


    Das erste was mir auffällt ist, dass du innerhalb deiner Rechenfunktionen (plus(), minus(), ...) die Main-Funktion aufrufst. Laut C++-Standard ist das ganze verboten.
    The function main shall not be used (3.2) within a program.
    Warum genau, ist mir selbst mangels fundierter C++-Kenntnisse nicht ganz klar. Standards haben aber (meistens) ihren Sinn und sollten befolgt werden. :D


    Bau dein Programm lieber so um, dass du den ganzen Code in der Main-Methode solange ausführst, wie deine Variable "Taschenrechner" nicht 6 ist - sprich du nicht deinen 6. Menüpunkt ausgewählt hast. Dafür verwendet man die Kontrollstruktur "while" bzw. "do while". Hier ein kleines Beispiel, bei dem der Code innerhalb des While-Blocks solange ausgeführt wird, wie "i" ungleich 10 ist. Sobald i den Wer 10 annimmt, wird mit den nachfolgendem Code fortgefahren.

    Quellcode

    1. while(i != 10)
    2. {
    3. i++; //erhöht die Variable um 1
    4. cout << i << endl;
    5. }
    7. // .... nachfolgender Code



    In deinem Taschenrechner kannst du das ganze so verpacken:

    Quellcode

    1. char Taschenrechner;
    3. while(Taschenrechner != '6')
    4. {
    5. // ... Menüauswahl anzeigen lassen
    7. cin >> Taschenrechner;
    9. // switch-Statement
    10. }
    12. // Beenden des Programms
    13. return 0;
    Alles anzeigen




    Das war das grobe, was mir aufgefallen ist. Ich hoffe meine Erklärung war verständlich. ;)

    Quellen zum Nachlesen:
    de.wikibooks.org/wiki/C%2B%2B-…ung_in_C%2B%2B/_Schleifen

  • Hab jetzt nicht alles ausprobiert damit meine Compiler deinen Quelltext schlucken.

    Wie schon geschrieben: Das aufrufen von main() im Quelltext entspricht etwa einem goto. Und das sollte man möglichst meiden.
    Du kannst das eleganter mit do-while lösen.

    Wenn du deinen Compiler auf neuere Standard, wie z.B. C++11 einstellst, werden zusätzlich deine Funktionen plus() und minus() als Fehler gemeldet. Da dürfte eine Nameskollusion vorliegen.

    Da meine Compiler sich weigern kann ich die Division nicht testen. Was passiert bei Division durch 0? Was gibt das Programm bei 2/3 aus?

    Am besten du stellst deinen Compiler zum Lernen so ein das er möglichst alle Warnungen oder Fehler kommentiert. Nur, wenn du dir absolut sicher bist das dein Compiler unrecht hat, kannst du diesen speziellen Fehler oder die Warnung ignorieren.

    MfG bcc-fan