Ich lerne seit ca. 1 Woche programmieren mit dem Buch C++ von A bis Z
bin daher noch relativ am anfang , meine IDE ist codeblocks auf XP

Ich hab mich dann nun für ein Erstprojekt entschieden den "Taschenrechner"
Mit + - * / Ich versuche meine "erlerntes" in den Code einfließen zu lassen.

Ich hoffe ihr könntet mir behilflich sein den Code zu "optimieren" nur ein paar vorschläge wie ich es besser machen könnte , bitte auf anfänger basis

Ich versuche Morgen / Übermorgen nen neuen Rechner zu machen mit Case und Switch funktionen .

#include <iostream>

using namespace std;

int main(void)

{
    int var1, var2, var3, var4;
    cout << "Taschenrechner 0.1 by Farcry  '\n' ";
    cout << "Bitte geben sie die erste Zahl ein: " ;
    cin >> var1;
    cout << "Bitte geben sie die zweite Zahl ein:  '\n' " ;
    cin >> var2;
    cout << " 1 = Addition '\n'  " ;
    cout << " 2 = Subtration '\n'  " ;
    cout << " 3 = Division '\n'  " ;
    cout << " 4 = Multiplikation '\n'  " ;
    cout << "Bitte geben sie die rechenart ein: '\n' ";
    cin >> var3;

    if( var3 == 1 )
    {
        var4 = var1 + var2;
        cout << var1 << " + " << var2 << " = " << var4 << '\n' ;
    }

    else if ( var3 == 2 )
    {
        var4 = var1 - var2;
        cout << var1 << " - " << var2 << " = "<< var4 << '\n' ;
    }

    else if ( var3 == 3 )
    {
        cout << var1 << " / " << var2 << " = "<< (var1/var2) << '\n' ;
    }

    else if ( var3 == 4 )
    {
        cout << var1 << " * " << var2 << " = " << (var1*var2) << '\n' ;
    }

    if ( !(cin >> var1) )
    {
        cerr << " ERROR " ;
    }

    return 0;
}

Ich fand es bisher nicht schlecht , die beispiele waren zwar nicht die besten aber für mich ist es leicht verständlich und ich komme gut mit diesem Buch klar
Auserdem geht es hier in dem Thread doch garnicht um das Buch.

Du findest es vielleicht gut. Als Anfaenger merkt man ja auch nicht, dass man absoluten Muell lernt.
Im Thread steht genug negatives zum Buch.

und zwar folgendermassen: man arbeitet ein kapitel aus einem guten buch durch (kA dem primer oder so) und nimmt sich dann das entsprechende kapital aus dem JW buch und streicht solange alle fehler an bis die ganze seite rot ist

Ich hoffe ihr könntet mir behilflich sein den Code zu "optimieren" nur ein paar vorschläge wie ich es besser machen könnte , bitte auf anfänger basis

Was hälst du denn hiervon?

#include <iostream>
#include <algorithm>

int main()
{
    using namespace std;

    double zahl1; // int für Division macht nicht das, das du willst. 
    double zahl2;
    const char ops[] = "AaSsDdMm"; // Erlaubte Operationen
    char auswahl;

    cout << "Taschenrechner 0.1 by Farcry\n\n"; // Du musst \n innerhalb einer Zeichenkette nicht in Hochkommas setzen.

    cout << "A = Addition\n";
    cout << "S = Subtration\n";
    cout << "D = Division\n";
    cout << "M = Multiplikation\n";
    cout << "\nBitte geben sie die Rechenart ein: ";
    cin >> auswahl;

    if( find(ops,ops+sizeof(ops),auswahl) == ops+sizeof(ops) ) // Wurde eine erlaubte Operation eigegeben?
    {
        cout << "Fehlerhafte Eingabe";
        return -1;
    }

    cout << "\nBitte geben sie die erste Zahl ein: ";
    cin >> zahl1;

    cout << "\nBitte geben sie die zweite Zahl ein: ";
    cin >> zahl2;

    switch(auswahl) // Für Menüsachen eignet sich doch eine switch-Anweisung besser.
    {
        case 'A':
        case 'a':
            cout << "\nRechnung: " << zahl1 << " + " << zahl2 << " = " << zahl1+zahl2;
            break;

        case 'S':
        case 's':
            cout << "\nRechnung: " <<  zahl1 << " - " << zahl2 << " = " << zahl1-zahl2;
            break;

        case 'D':
        case 'd':
            cout << "\nRechnung: " <<  zahl1 << " / " << zahl2 << " = " << zahl1/zahl2;
            break;

        case 'M':
        case 'm':
            cout << "\nRechnung: " <<  zahl1 << " * " << zahl2 << " = " << zahl1*zahl2;
            break;
    }

    return 0;
}

Farcry schrieb:

Ich hoffe ihr könntet mir behilflich sein den Code zu "optimieren" nur ein paar vorschläge wie ich es besser machen könnte , bitte auf anfänger basis

Was hälst du denn hiervon?

...

Magische Konstanten die mehrfach auftauchen? Das wäre eine passende Stelle um Aufzählungen einzuführen.

Was hälst du denn hiervon?

Wenn du hier schon Array Hacks mit std::find auspackst, hättest du doch zumindest so etwas schreiben können:

int main()
{
  while (true)
  {
    double var1, var2;
    char operation;
    std::cout << "Please enter calculation: ";
    std::cin >> var1 >> operation >> var2;
    if (std::cin.good())
    {
      switch (operation)
      {
      case '+':
        std::cout << var1 + var2 << '\n';
        break;
      case '-':
        std::cout << var1 - var2 << '\n';
        break;
      case '*':
        std::cout << var1 * var2 << '\n';
        break;
      case '/':
        std::cout << var1 / var2 << '\n';
        break;
      default:
        std::cerr << "Unkown operation: " << operation << '\n';
        break;
      }
    }
    else
    {
      std::cerr << "Input error.\n";
      std::cin.clear();
      std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    }
  }
}

Gugelmoser schrieb:

Farcry schrieb:

Ich hoffe ihr könntet mir behilflich sein den Code zu "optimieren" nur ein paar vorschläge wie ich es besser machen könnte , bitte auf anfänger basis

Was hälst du denn hiervon?

...

Magische Konstanten die mehrfach auftauchen? Das wäre eine passende Stelle um Aufzählungen einzuführen.

Du meinst eine Aufzählungen hierfür?

const char ops[] = "AaSsDdMm";

Aber dann muss man doch noch den operator>> überladen für die Eingabe und überprüfen welcher Buchstabe nun eingegebn wurde und ein enum-Objekt entsprechend setzen. Das bläht das Ganze doch voll auf, oder hab hab ich dich nun falsch verstanden, falls ja, kannst dann kurz ein Beispiel anführen :).