Programm funktioniert nicht richtig

shivan

Danke, daran hab ich noch gar nicht gedacht, ich werde einfach einmal die While Schleife löschen, die ist unnötig.

Mentras

Jop ziemlich

Wobei ich dir raten muss, solche GoTo-Anweisungen nicht zu oft zu verwenden.
Bei 1000 Zeilen Code kann da schon einige Verwirrung entstehen.

Eine Möglichkeit wäre, das in eine Funktion reinzustecken.

Ist halt Geschmackssache bzw. wenn du gute / scharfe Augen hast, dann kannst du es ruhig lassen^^

Schönen Tag noch!

Lg Mentras

shivan

Mhh.. jetzt funktioniert gar nix mehr
Hast du eine Ahnung, wie ich das Problem beheben kann?

shivan

Okey die Schleife ist weg und trotzdem hab ich das selbe Problem wie vorhin. Ich denke, dass die Schleife nicht das Problem war, denn alles war richtig, er hat sie nur nicht ausgeführt, aber das ist ja irrelevant.

Ich werde jetzt erstmal schlafen gehen und hoffe auf neue Comments morgen..

gn8

Mentras

Totaler Humbuk^^ Siehe 2.Post!

shivan

Danke, werd ich gleich mal ausprobieren!

Mentras

shivan schrieb:

Okey die Schleife ist weg und trotzdem hab ich das selbe Problem wie vorhin. Ich denke, dass die Schleife nicht das Problem war, denn alles war richtig, er hat sie nur nicht ausgeführt, aber das ist ja irrelevant.

do{ //Er kommt in die Schleife...
            system("color F0");  //Kenn den Befehl nicht^^ Er führt ihn aus..
            system("cls");       //Kenn den Befehl auch nicht^^ Er führt ihn aus..
            cout << " Temperatur umrechnen" <<endl;  //Führt aus...
            cout << "----------------------" <<endl; //Führt aus...
            cout << " 1) Grad Celcius in Kelvin"<<endl; //Führt aus...
            cout << " 2) Kelvin in Grad Celcius"<<endl; //Führt aus...
            cout << " Bitte w"<<ae<<"hlen Sie Ihre gew"<<ue<<"nschte Option aus (1-2): "; //Führt aus...
            cin >> tempauswahl; //Führt aus...
            if(tempauswahl > 2 || tempauswahl < 1) // Falls Eingabe > 2 oder Eingabe < 1 dann...
            {
             fehleingabe();  //Funktion fehleingabe() ausführen.
            }
            //Sonst:
            {
             goto temperaturmenue; //Wieder rauf!
            }
         }
         while(tempauswahl > 2 || tempauswahl < 1); //Wenn ich aber jetzt was richtiges eingebe, komm ich NICHT AUS DER SCHLEIFE RAUS! Weil er erst nur rauskommt,
//wenn ich eine Zahl größer als 2 oder kleiner als 1 eingeben muss!
//Die Schleife ist unnötig, weil du schon mit der If-Abfrage prüfst ob die Eingabe passend ist.

€DIT: Schau auf Seite 2...

shivan

Leider hab ich immer noch das selbe Ergebnis wie vorher!

Aber danke dir für deine Bemühungen!

Mentras

temperaturmenue:
            system("color F0");
            system("cls");
            cout << " Temperatur umrechnen" <<endl;
            cout << "----------------------" <<endl;
            cout << " 1) Grad Celcius in Kelvin"<<endl;
            cout << " 2) Kelvin in Grad Celcius"<<endl;
            cout << " Bitte w"<<ae<<"hlen Sie Ihre gew"<<ue<<"nschte Option aus (1-2): ";
            cin >> tempauswahl;

            if(tempauswahl > 2 || tempauswahl < 1)
            {
             fehleingabe();
            }
            else
            {      
              if(tempauswahl == 1)
              {
               system("cls");
               cout << " Grad Celcius in Kelvin" <<endl;
               cout << "------------------------" <<endl;
               cout << " Bitte geben Sie die Temperatur in Celcius an: ";
               cin >> temp;
               cout << " Die Temperatur in Kelvin betr"<<ae<<"gt " << temp - 273 << "."<<endl;
               cout << " M"<<oe<<"chten Sie noch noch eine Umrechnung durchf"<<ue<<"hren? (j/n)"<<endl;
               cout << " Dr"<<ue<<"cken Sie H um zum Hauptmen"<<ue<<" zur"<<ue<<"ck zu kehren.";
               wiederholen = getch();
                    if(wiederholen == 'j' || wiederholen == 'J')
                    {
                     goto temperaturmenue;         //Sprung zu Waehrungsgemenue
                    }
                    if(wiederholen == 'h' || wiederholen == 'H')
                    {
                     goto hauptmenue;
                    }
                    else if(wiederholen == 'n' || wiederholen == 'N')
                    {
                     danke();
                    }
              }
              else if(tempauswahl == 2)
              {
               system("cls");
               cout << " Kelvin in Grad Celcius" <<endl;
               cout << "------------------------" <<endl;
               cout << " Bitte geben Sie die Temperatur in Kelvin an: ";
               cin >> temp;
               cout << " Die Temperatur in Celcius betr"<<ae<<"gt " << temp + 273 << "."<<endl;
               cout << " M"<<oe<<"chten Sie noch noch eine Umrechnung durchf"<<ue<<"hren? (j/n)"<<endl;
               cout << " Dr"<<ue<<"cken Sie H um zum Hauptmen"<<ue<<" zur"<<ue<<"ck zu kehren.";
               wiederholen = getch();
                    if(wiederholen == 'j' || wiederholen == 'J')
                    {
                     goto temperaturmenue;         //Sprung zu Waehrungsgemenue
                    }
                    if(wiederholen == 'h' || wiederholen == 'H')
                    {
                     goto hauptmenue;
                    }
                    else if(wiederholen == 'n' || wiederholen == 'N')
                    {
                     danke();
                    }
              }
    } 
           }

Oh mein Gott, was ich da angestellt habe! Sorry, jetzt sollte es gehen!

Er überprüft ob die Eingabe nicht passt:

if(tempauswahl > 2 || tempauswahl < 1)
            {
             fehleingabe();
            }
            else
            {      
            //...
           }

Wenn Sie passt, dann macht er den eigentlichen Teil...

Lg Mentras

Tut mir leid nochmal!

shivan

Nene, bin doch froh, das du mir hilfst! Danke, mal gucken obs jetzt funzt

shivan

Vielen Dank, jetzt funktioniert wirklich alles!

Mentras

shivan schrieb:

Vielen Dank, jetzt funktioniert wirklich alles!

Wirklich?

Wie ich mich freue

Muss mich nochmal echt entschuldigen, bin müde und hab dir da immer ein Wisch Wasch gegeben.

Schönen Tag noch,...ich geh mal ins Bett!

Lg Mentras

shivan

Jo egal, jetzt funktioniert ja alles