mal wieder ein Taschenrechner

  • mal wieder ein Taschenrechner

    Hallo,
    ich habe vor kurzem mit dem programmieren in C++ begonnen (ist meine erste Programmiersprache).
    Das Buch ist "Magnum C++", ein ziemlich gutes Buch wie ich finde.

    Jedenfalls habe ich gerade einen Taschenrechner programmiert (ohne Vorlage) und möchte wissen ob das ein guter "Programmierstil" ist,
    oder ob ich/ oder was ich verbessern sollte.

    Ich bedanke mich jetzt schon mal für eure Hilfe.




    Quellcode

    1. #include <iostream>
    2. using namespace std;
    4. int addieren(int a, int b)
    5. {
    6. return(a + b);
    7. }
    9. int subtrahieren(int a, int b)
    10. {
    11. return(a - b);
    12. }
    14. int dividieren(int a, int b)
    15. {
    16. return(a / b);
    17. }
    19. int multiplizieren(int a, int b )
    20. {
    21. return(a * b);
    22. }
    24. int main()
    25. {
    26. char entscheidung;
    27. int zahl1, zahl2;
    29. cout << "Ein einfaches Rechenprogramm mit ganzen Zahlen" << endl << endl;
    31. for (int i = 1; i > 0;)
    32. {
    35. cout << "Menue:" << endl;
    36. cout << "(A)ddieren" << endl;
    37. cout << "(D)ividieren" << endl;
    38. cout << "(M)ultiplizieren"<< endl;
    39. cout << "(S)ubtrahieren" << endl << endl;
    40. cout << "(B)eenden" << endl << endl;
    42. cin >> entscheidung;
    45. switch (entscheidung)
    46. {
    47. case 'A':
    48. case 'a':
    49. cout << "Bitte geben Sie die zwei Zahlen ein, die Sie miteinander addieren moechten: " << endl;
    50. cin >> zahl1;
    51. cin >> zahl2;
    52. cout << "Ergebnis: ";
    53. cout << addieren(zahl1, zahl2) << endl << endl;
    54. break;
    56. case 'D':
    57. case 'd':
    58. cout << "Bitte geben Sie die zwei Zahlen ein, die Sie miteinander dividieren moechten: " << endl;
    59. cin >> zahl1;
    60. cin >> zahl2;
    61. cout << "Ergebnis: ";
    62. cout << dividieren(zahl1, zahl2) << endl << endl;
    63. break;
    65. case 'M':
    66. case 'm':
    67. cout << "Bitte geben Sie die zwei Zahlen ein, die Sie miteinander multiplizieren moechten: " << endl;
    68. cin >> zahl1;
    69. cin >> zahl2;
    70. cout << "Ergebnis: ";
    71. cout << multiplizieren(zahl1, zahl2) << endl << endl;
    72. break;
    74. case 'S':
    75. case 's':
    76. cout << "Bitte geben Sie die zwei Zahlen ein, die Sie miteinander subtrahieren moechten: " << endl;
    77. cin >> zahl1;
    78. cin >> zahl2;
    79. cout << "Ergebnis: ";
    80. cout << subtrahieren (zahl1, zahl2) << endl << endl;
    81. break;
    83. case 'B':
    84. case 'b':
    85. return 0;
    87. default:
    88. cout << endl << "Ungueltiger Wert" << endl << "Bitte versuchen Sie es erneut: " << endl << endl;
    89. }
    90. }
    91. }
    Alles anzeigen

  • hallo,

    was mir sofort auffällt, du hast überhaubt keine kommentare drin. es ist mir klar das es sich hier lediglich um ein übungsprojekt handelt, aber gewöhn dir an, immer alles zu kommentieren.
    wenn du irgendwann mal komplexere programme schreibst oder sogar im team an einem projekt arbeitest ist es sehr wichtig das man immer schön alles kommentiert hat!

    thx, truespin

  • truespin schrieb:

    es ist mir klar das es sich hier lediglich um ein übungsprojekt handelt, aber gewöhn dir an, immer alles zu kommentieren.

    Klingt so, als würdest du zu den Programmierern gehören, die für meinen Geschmack mit dem Kommentieren übertreiben. Meiner Meinung nach ist guter, selbsterklärender Code meistens die beste Dokumentation und zumindest der zweite Punkt ist hier erfüllt.

  • Hallo PakMan,
    Was ich zur besserenLesbarkeit verändern würde und auch empfehle ist in der Zeile 31 die for Schleife durch eine do while Schleife zu ersetzen, oder daraus eine while Schleife zu machen.

    statt dieser for Schleife

    for (int i = 1; i > 0;)
    {
    //...
    }

    eine do while Schleife

    do {
    //...
    } while(entscheidung != 'b' || entscheidung != 'B')

    oder eine while Schleife

    while (true) {
    //...
    }


    Zudem kannst du nach jeder neuen Auswahl den Bildschirm wieder löschen, was zur besseren Lesbarkeit führt.
    #include <windows.h>
    system("CLS"); // clear screen


    Anmerkung:
    // sind Kommentare

    eine for Schleife benutzt man meistens dann, wenn die Anzahl der Wiederholungen festgelegt sind.

  • danke

    Danke für die Tipps!

    Das mit den Kommentaren werd ich mir wohl angewöhnen müssen,
    wenn ich den Quellcode anderer lese finde ich Kommentare ja auch sinnvoll,
    nur selbst bin ich leider etwas faul;-)

    Der Befehl "CLS" wurde in meinem Buch noch garnicht erwähnt, obwohl schon ich bei Seite 400 bin.
    Werde ich mir aber merken.

    Also ein guter Programmierstil definiert sich darunter dass man keine "Gotos" benutzt,
    keine Endlosschleifen einfügt und den Quellcode nachvollziehbar für andere Programmierer gestaltet?

    Es wurde und wird zwar immer wieder behauptet (in meinem Buch), dass man unbedingt einen guten Programmierstil schreiben sollte,
    nur wurde nicht so verständlich erklärt was ein guter Programmierstil denn überhaupt ist.

  • @ Florian:
    nein gehörte ich sicher nicht. ich bin selbst immer recht kommentierfaul, aber es ist einfach wichtig wenn man in einem team arbeitet. ich sage nicht das man jede zeile kommentieren muss, aber man sollte funktionen kurz beschreiben. alleine schon weil verschiedene neue entwicklungsumgebungen später beim aufrufen der funktion dir z.b. die übergabeparameter usw. die du ja kommentiert hast per intellisens anzeigt und du dann eben nicht erst nachsehen muss was da so alles übergeben werden muss bzw. was das überhaubt würd parameter sind.
    auch wenn man schleifen oder rekursionen hat sollte man die doch einfach kurz erklären... so kann man sich viel schneller in den code einarbeiten. ebenso halte ich es für sinnvoll, wichtige variablen zu kommentieren. ich rede nicht von irgendwelchen countern die keinen menschen interessiern...

    thx,
    truespin


    edit:
    @pakman:
    das kannst du so nicht sagen... auch endlosschleifen werden in der softwareentwicklung verwendet z.b. in steuergeräten für autos oder irgendwelchen fertigungsmaschinen.

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von truespin ()