Ich hab in den letzten Tagen ein Programm geschrieben, das über 1000 Zeilen hat, doch leider funktionieren die letzten ca. 70 Zeilen nicht richtig und ich suche mich schon dumm und demlich ab

Habe mir gedacht, dass vielleicht jemand von Euch den Code angucken könnte, denn vielleicht guck ich einfach am falschen Ort.

else if(menuauswahl == 4)
    {
         int tempauswahl;
         int temp;
         char wiederholen;

         temperaturmenue:
         do{
            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();
            }
            {
             goto temperaturmenue;
            }
         }
         while(tempauswahl > 2 || tempauswahl < 1);

              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();
                    }
              }
    }

Das Programm soll etwa so ablaufen:

Temperatur umrechnen
----------------------

1. Grad Celcius in Kelvin
2. Kelvin in Grad Celcius
   Bitte wählen Sie Ihre gewünschte Option aus (1-2): --> 1 oder 2

Nun soll das Programm zu dem Punkt wechseln und die einzelnen Schritte auführen.
Leider aber erscheint jetzt gleich wieder das Menü:

Temperatur umrechnen
----------------------

1. Grad Celcius in Kelvin
2. Kelvin in Grad Celcius
   Bitte wählen Sie Ihre gewünschte Option aus (1-2): --> 1 oder 2

Ich bin in einer Endlosschleife, in der ich eine Option wählen soll und komme nicht raus.

Hoffe, dass mir jemand helfen kann und meine Beschreibung klar ist, ansonsten einfach nachfragen^^

Danke fürs Helfen

MfG Shivan

Deine Abfrage scheint falsch zu sein bzw. die Bedingung für die While-Schleife ganz oben.

Du fragst zuerst ob die Zahl größer ist als 2 oder kleiner ist als 1.

Und dann stellst du die SELBE Bedingung bei der While-Schleife:

while(tempauswahl > 2 || tempauswahl < 1);

Da musst du was ändern...

€dit: Fazit, er kommt gar nicht rein.

LG Mentras

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

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

gn8

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...

Aber danke dir für deine Bemühungen!

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!

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