Taschenrecher Code, für Vorstellungsgespräch

whenyouDIE

Also wie oben der Headder schon sagt, ich hab einen Taschenrechner Code geschrieben mit C++, sehr einfach gehalten, da ich noch nicht gut in C++ bin!. Daher wollte ich euch fragen, ob ihr euch den Code vielleicht mal anschauen könntet, und Tipps geben könntet, zum Verbessern (bitte aber sehr einfach halten) da ich den Taschenrechner auch erklären muss. So der Code:

#include <iostream>

using namespace std;

int main ()
{
    float  fltNumber=0;
    float  fltResult=0;
    int intMode=0;
    bool first=true;

    cout<<"Willkommen bei meinem Taschenrechner, viel Erfolg beim Rechnen!";

    do
    {
        if (first==false)
        {

            system("cls");
            cout<<"\n\n------------------";
            cout<<"\nErgebnis: "<<fltResult<<"!";
            cout<<"\n------------------";
            cout<<"\n\n-------------------------------------------------";
            cout<<"\nSie koennen nun mit Ihrem Ergebnis weiterrechnen!";
            cout<<"\n-------------------------------------------------";
        }

        do
        {
            cout<<"\n\n\nWas moechten sie nun tun?";
            cout<<"\ndruecken Sie:";
            cout<<"\n\n(1) fuer eine Addition";
            cout<<"\n(2) fuer eine Subtraktion";
            cout<<"\n(3) fuer eine Multiplikation";
            cout<<"\n(4) fuer eine Division";
            cout<<"\n(5) wenn Sie Ihr Ergebnis auf Null setzten moechten";
            cout<<"\n(6) wenn Sie dass Programm beenden wollen";
            cout<<"\n\nNach der Zahleneingabe immer Enter druecken!!";
            cout<<"\n\nAuswahl: ";
            cin >>intMode;
            cout<<"----------";

            if (first==true&&intMode<1||intMode>6&&first==true)
            {
                cout<<"\n\aIhre Eingabe war nicht korrekt, versuchen Sie es erneut!";
                cout<<"\n---------------------------------------------------------";
            }
            else if (first==false&&intMode<1||intMode>6&&first==false)
            {
                cout<<"\n\aIhre Eingabe war nicht korrekt, versuchen Sie es erneut!";
                cout<<"\n---------------------------------------------------------";
            }

            if (intMode==5)
            {
                fltResult=0;
                cout<<"\n\n------------------";
                cout<<"\nErgebnis: "<<fltResult<<"!";
                cout<<"\n------------------";
                cout<<"\n\nNun koennen Sie ein neues Ergebnis bestimmen!";
                cout<<"\n---------------------------------------------";
                first=false;
            }

            if (intMode==6)
            {
                first=false;
            }

        } while (intMode<1||intMode>6);

        if (first==true && intMode!=5 && intMode!=6)
        {
            cout<<"\n\nBitte tippen Sie nun Ihre erste gewuenschte Zahl ein:  ";
            cin >>fltResult;
        }

        if (intMode!=5 && intMode!=6)
        {
            cout <<"\nBitte tippen Sie nun Ihre zweite gewuenschte Zahl ein: ";
            cin >>fltNumber;
        }

        switch (intMode)
        {
            case 1:
                 fltResult=fltResult+fltNumber;
                 first=false;
            break;

            case 2:
                 fltResult=fltResult-fltNumber;
                 first=false;
            break;

            case 3:
                fltResult=fltResult*fltNumber;
                first=false;
            break;

            case 4:
                if (fltNumber==0)
                {
                    cout<<"\n\n\aMan darf nicht durch Null teilen!!!";
                    cout<<"\n-----------------------------------";
                    first=true;
                    break;
                }
                else
                {
                    fltResult=fltResult/fltNumber;
                }
                first=false;
            break;

            case 5:
                fltResult=0;
                first=true;
            break;
         }
    } while (intMode<=5);

    cout<<"\nVielen Dank fuer Ihren Besuch!\n";
    system("pause");
    return 0; 
}

thx schonmal!

mfg whenyou

sothis_

das ist doch jetzt nicht dein ernst, oder?

whenyouDIE

hmm, war nur eine Frage, ich bin gestern erst zu euch gekommen, es war nur eine höfliche frage, ob mir vll jemand weiterhelfen könnte. Keine Aufforderung, also muss sich niemand verpflichtet fühlen. Aber wenn dass so anfängt sollte ich warsch. den ganzen Thread einfach wieder löschen.

Warum muss ich wenn ich 6 zum beenden eingebe noch zwei zahlen eingeben und bekomm dann ein regebnis angezeit?

sothis_

ich meinte nicht die art der frage, die ist ja gang und gebe. aber das firmen den code eines subtilen taschenrechners für das vorstellungsgespräch verlangen, zauberte ein breites grinsen auf meinem gesicht

whenyouDIE

achso habe deine Frage falsch verstanden tut mir leid, hatte mich etwas angegriffen gefühlt: Nein ich war schon dort beim Vorstellungsgespräch, und die haben mir nun die Aufgabe gestellt bis zum Montag einen Taschenrechner zu programieren und bei ihnen vorzustellen (sozusagen wie ich finde einstellungstest)

mfg whenyou

@anfängercodierer ich bin grade dabei den fehler zu beheben, obwohl ich grade net seh wo ^^

und die einrückung ist müll. als was bewirbst du dich da?

whenyouDIE

als Fachinformatiker als Anwendungsentwicklung: kurze frage^^, also der eine fehler is behoben musste =!6 heißen anstat =!5 frage: wie kann ich es machen wenn ich bei der 5 wenn ich alles auf 0 setzten will, es hinkriegen dass ich dann normal weiterrechnen kann, weil danach fragt er mich was ich rechnen will und wenn ich z.b. 1 +2+ rechne gibt er mir 0 aus und danach gehts erst richtig weiter? Hoffe ihr versteht mich

mfg
whenyou

bin grade dabei den fehler zu beheben, obwohl ich grade net seh wo ^^

Zeile 44

und warum machst du ne bool variable nur um die ergebnis ausgabe ganz oben zu machen?

also wenn ich mir den code so anschaue dann frag ich mich
welche Firme so einen softwareentwickler einstellt

mode!=5 && mode!=6

whenyouDIE

Da ich beim ersten Vorgang nicht dass Ergebniss ausgegeben haben möchte erst beim 2ten durchlauf

@ anfängercode warum mode=!5 && mode=! 6 =?was bringt dat`?
und @ rofl wie meinst du dass?

mfg whenyou

du willst dich als Softwareentwickler bewerben und brauchst hilfe
bei einem taschenrechner?

whenyouDIE

Wenn du meine frage richtig gelesen hättest hättest du gesehen dass ich noch nicht sehr gut in C++ bin, also es wäre sehr schön wenn du mich nicht kritisieren würdest!

... oder wenn die kritik konstruktiv wäre...

whenyouDIE

oblig schrieb:

... oder wenn die kritik konstruktiv wäre...

richtig...

rofl schrieb:

du willst dich als Softwareentwickler bewerben und brauchst hilfe
bei einem taschenrechner?

du vollidiot es geht hier wohl zweifelsohne um einen ausbildugsplatz. und um den bewirbt man sich bekanntermaßen, weil man erst noch etwas lernen will, soll und wird.

whenyouDIE schrieb:

frage: wie kann ich es machen wenn ich bei der 5 wenn ich alles auf 0 setzten will, es hinkriegen dass ich dann normal weiterrechnen kann, weil danach fragt er mich was ich rechnen will und wenn ich z.b. 1 +2+ rechne gibt er mir 0 aus und danach gehts erst richtig weiter? Hoffe ihr versteht mich

anfängercode schrieb:

mode!=5 && mode!=6

Google: c++ boolsche verknüpfung

whenyouDIE

@ anfängercode:
thx funktioniert ;), sonst vll noch irgend n vorschlag zum ausbessern/verbessern?

mfg whenyou

Ja.

Gib das Ergebnis wie schon erwähnt erst unten aus und spar dir das rumgebastle mit der Bool-Variablen.
Außerdem sind deine Variablennamen nicht ganz gut, nach der ungarischen Notation schreibt man eigentlich:
iName für Ints
fName für Floats
bName für Bools
...

Dann kann ich dir noch raten, die Funktion "getchar()" aus der Header-Datei "cstdio" zu verwenden, anstatt "getch()" aus "conio.h", letztere ist nämlich veraltet.