3. Tipps fürn N00b !!!

Tipps fürn N00b !!!

  • ?

    Bin absoluter Neuling was programmieren angeht, daher hier mal mein erstes Projekt, mein Taschenrechner !!! Würde mich freuen wenn Ich tipps bekomme, die mir das Leben vll ein wenig vereinfachen, aber denkt bitte dran, bin absoluter N00bie, daher bitte alles ganz easy going 🙂 thx

    #include <stdio.h>
    #include <iostream.h>
    #include <conio.h>

    int main()
    {
    long int x;
    long int y;
    char z;
    char taste;

    do
    {
    cout << endl;
    cout << endl;
    cout << endl;
    cout << endl;
    cout << " Geben Sie die erste Zahl ein !" << endl << endl;
    cout << " ";
    cin>> x;
    cout << endl;
    cout << " Rechenart (+,-,*,/) ?" << endl;
    cout << endl;
    cout << " ";

    cin>> z;
    getch();

    if (z == '+')
    {
    cout << endl;
    cout << endl;
    cout << " "<< x <<" soll mit welcher Zahl addiert werden ? " << endl << endl;
    cout << " ";
    cin>> y;
    cout << endl;
    cout << " OK, somit lautet Ihre Rechnung also : " << x << z << y << endl;
    cout<< " (Um das Ergebnis zu erhalten bitte ENTER druecken)" <<endl << endl;
    cout << " ";
    getch();
    cout << " Das Ergebnis lautet : "<< x + y << endl;
    cout << " ";
    cout << endl;
    cout << endl;
    getch();
    cout << endl;
    cout << endl;
    cout << " Erneute Eingabe (j/n) " << endl;
    cout << endl;
    cout << " ";
    cin >> taste;
    }

    if (z == '-')
    {
    cout << endl;
    cout << endl;
    cout << " "<< x <<" soll mit welcher Zahl subtrahiert werden ? " << endl << endl;
    cout << " ";
    cin>> y;
    cout << endl;
    cout << " OK, somit lautet Ihre Rechnung also : " << x << z << y << endl;
    cout<< " (Um das Ergebnis zu erhalten bitte ENTER druecken)" <<endl << endl;
    cout << " ";
    getch();
    cout << " Das Ergebnis lautet : "<< x - y << endl;
    cout << " ";
    cout << endl;
    cout << endl;
    getch();
    cout << endl;
    cout << endl;
    cout << " Erneute Eingabe (j/n) " << endl;
    cout << endl;
    cout << " ";
    cin >> taste;
    }

    if (z == '*')
    {
    cout << endl;
    cout << endl;
    cout << " "<< x <<" soll mit welcher Zahl multiplitiert werden ? " << endl << endl;
    cout << " ";
    cin>> y;
    cout << endl;
    cout << " OK, somit lautet Ihre Rechnung also : " << x << z << y << endl;
    cout<< " (Um das Ergebnis zu erhalten bitte ENTER druecken)" <<endl << endl;
    cout << " ";
    getch();
    cout << " Das Ergebnis lautet : "<< x * y << endl;
    cout << " ";
    cout << endl;
    cout << endl;
    getch();
    cout << endl;
    cout << endl;
    cout << " Erneute Eingabe (j/n) " << endl;
    cout << endl;
    cout << " ";
    cin >> taste;
    }
    if (z == '/')
    {
    cout << endl;
    cout << endl;
    cout << " "<< x <<" soll durch welche Zahl Dividiert werden ? " << endl << endl;
    cout << " ";
    cin>> y;
    cout << endl;
    cout << " OK, somit lautet Ihre Rechnung also : " << x << z << y << endl;
    cout<< " (Um das Ergebnis zu erhalten bitte ENTER druecken)" <<endl << endl;
    cout << " ";
    getch();
    cout << " Das Ergebnis lautet : "<< x / y << endl;
    cout << " ";
    cout << endl;
    cout << endl;
    getch();
    cout << endl;
    cout << endl;
    cout << " Erneute Eingabe (j/n) " << endl;
    cout << endl;
    cout << " ";
    cin >> taste;
    }

    }
    while (taste =='j' || taste == 'J');

    return 0;
    }

    0
  • N

    Lies doch mal Shade Of Mines Tutorial durch:
    http://www.schornboeck.net/ckurs/inhalt.htm

    Was mir so auf den ersten Blick an Deinem Programm negativ auffällt ist dass Du zb. das nicht standardisierte getch() verwendest, die nicht standardisierte <conio.h>, die veraltete <iostream**.h**> und die nicht benötigte <stdio.h> einbindest. Außerdem solltest Du darauf achten Deinen Variablen halbwegs aussagekräftige Namen zu geben, Code-Tags zu verwenden, sowas:

    cout << endl;
cout << endl;
cout << "Hallo";

    duch sowas:

    cout << "\n\nHallo";

    zu ersetzen, uä.

    Davon mal ganz abgesehen hättest Du statt if-Abfragen hier besser switch verwendet!

    0
Anmelden zum Antworten