Kann sich das bitte kurz jemand angucken? Wär echt nett!

if(waehrung == 1)
         {
          system("cls");
          double betrag;
          cout << " CHF in W"<<ae<<"hrungsumrechnung in Euro" << endl;
          cout << " -----------------------------------" << endl;
          cout << " Bitte geben Sie den Betrag ein: ";
          cin >> betrag;
          cout << " " << betrag << " CHF entsprechen ";
          cout << betrag * 0.66 << " Euro" <<endl;
          nochmals:
          do{
          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 waehrungsmenue;         //Sprung zu Waehrungsgemenue
          }
          if(wiederholen == 'h' || wiederholen == 'H')
          {
           goto hauptmenue;
          }
          else if(wiederholen == 'n' || wiederholen == 'N')
          {
           danke();
          }
          }while(wiederholen =! 'h' && wiederholen =! 'H' && wiederholen =! 'j' && wiederholen =! 'J' && wiederholen =! 'n' && wiederholen =! 'N')
         }

Das Problem liegt in der untersten while Schleife denk ich mal.

Fehlermeldung vom Compiler:

Zeile der while Schleife:
E:\Dokumente und Einstellungen\Besitzer\Desktop\C++\Universalrechner.cpp non-lvalue in assignment

Unter der while Schleife:
E:\Dokumente und Einstellungen\Besitzer\Desktop\C++\Universalrechner.cpp expected `;' before '}' token

MfG Shivan

1. vermeide goto! ließ mal nach was break ist
2. die if-else abfragen machen keinen sinn, da du ja im while kopf die ganze sache nochmal abfragst
3. der ungleichoperator lautet != und nicht =! (sonst weist du den negierten bool wert des char literals zu)
4. statt wiederholen = getch() nimm lieber std::cin

Deine Formatierung des Quelltextes irritiert mich ein wenig -
wann rückst du deinen Quelltext weiter ein und wann nicht?

MfG f.-th.

if (text == text2
{
        cout << "Text";
}

Was genau sollte ich anders machen?

Ach ja, das Problem oben konnte ich jetzt lösen, danke!

Zum Einrücken: in deinem Eingangspost besteht das einrücken nach der ersten if bedingung aus 2 tabs und das noch vor der geschweiften klammer. danach ist es nur ein leerzeichen. jetzt postet du code, wo du nach der geschweiften klammer einrückst und zwar auch wieder mit 2 tabs. Bisschen inkonsistent

Ich weiß jetzt nicht wie universell dein Rechner ist, hab aber den Verdacht, wenn du deine Schleifen optimierst das du deinen Quelltext deutlich kürzer und einfacher lesbar für dich und andere schreiben kannst.

Nur geraten, wie viel Klammern öffnest du maximal gleichzeitig?

MfG f.-th.

Hat man hier die Möglichkeit Dateien an einen Comment anzuhängen?

Ich öffne in meinem Code oft mehrere (3 oder 4) geschwungene Klammern miteinander, ist das schlimm?

Naja bei meinem Editor sind irgendwie die Tabs unkonsistent, ich benutze DevC++..
Aber wenn nur das mit der Länge der Einzüge nicht stimmt, bin ich ja noch beruhigt!

Hat man hier die Möglichkeit Dateien an einen Comment anzuhängen?

Nein, aber du kannst sie bei einem Freehoster hochladen

Bei Zeile 12 und 13 hab ich keine Einzüge gemacht, den Code hab ich da auch nur erstmal vorübergängig geschrieben um etwas auszuprobieren.

Ich öffne in meinem Code oft mehrere (3 oder 4) geschwungene Klammern miteinander, ist das schlimm?

So könnte man einrücken. Extrembeispiel:

if (...)
{
    if (...)
    {
        do
        {
            DoSomething1();

            if (...)
            {
                for (...)
                {
                    // ...
                    DoSomething2();
                }
            }
        } while (...);
    }

    DoSomething3();
}

Warum nimmst du den DEV?
Hast du schon mal den Code::Blocks mit z.B. mit dem MinGW oder einem anderen Compiler getestet? Da kannst du deinen Quelltext, auch nachträglich, mit 2 oder 3
Mausklicks formatieren.

MfG f.-th.

Ich muss meine Aussage von vorhin korrigieren, ich habe viel mehr als 3, 4 Klammern gleichzeitig geöffnet!

Vielleicht könnt ihr es mal angucken und sagen, was ihr davon haltet und was ich verbessern kann.

Zudem hab ich noch ein Problem, meine Addition bzw. Subtraktion Funktion im Programm gibt ganz komische Zahlen aus, hat aber vorhin tadellos funktioniert, obwohl ich nichts daran verändert hab.

Kann sich das jemand anschauen? Wär wirklich nett, denn ich weiss nicht mehr weiter

Addition:

Zeile 512

Subtraktion:

Zeile 551

MfG Shivan

Wobei ich mehr zu Code::Blocks tendiere:

Code::Blocks: http://www.chip.de/downloads/Code-Blocks_32146961.html

VC++: http://www.microsoft.com/germany/express/download/downloaddetails.aspx?p=vcpp

Lg Mentras

Danke nochmals!

Warum dein Code nicht so funktioniert, wie du das willst, lässt sich wegen des sehr unübersichtlichen Codes auch nicht so einfach sagen.

Addition

if(rechnerauswahl == 1)
            {
             system("cls");
             cout << " Addition"<<endl;
             cout << "----------"<<endl;
             cout << " Bitte geben Sie die Anzahl der Summanden ein: ";
             cin  >> anzahl;
             cout <<endl;

             for (int i=1; i<=anzahl; i++)
             {
              cout<<" "<< i <<". Summand eingeben: ";
              cin>>eingabezahl;
              loesung+=eingabezahl;
             }

             cout <<endl;
             cout<<" Resultat: "<<loesung<<endl;
             cout <<endl;
             do{
             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.";
             cin >> wiederholen;
             cout << endl;
             if(wiederholen == 'j' || wiederholen == 'J')
             {
              goto berechnungen;         //Sprung zu berechnungen
             }
             if(wiederholen == 'h' || wiederholen == 'H')
             {
              goto hauptmenue;
             }
             else if(wiederholen == 'n' || wiederholen == 'N')
             {
              danke();
             }
            }while(wiederholen != 'h' && wiederholen != 'H' && wiederholen != 'j' && wiederholen != 'J' && wiederholen != 'n' && wiederholen != 'N');
         }

Das einzig Neue ist die do while Schleife in Zeile 20 bzw. 37, doch das sollte ja die Funktionsweise des Rechners nicht beeinflussen.

Beispiel einer Ausgabe des Programms:

Addition
----------
Bitte geben Sie die Anzahl der Summanden ein: 5

1. Summand eingeben: 100
2. Summand eingeben: 100
3. Summand eingeben: 100
4. Summand eingeben: 100
5. Summand eingeben: 100

Resultat: 540

Möchten Sie noch noch eine Umrechnung durchführen? (j/n)
Drücken Sie H um zum Hauptmenü zurück zu kehren.

Manchmal funktioniert er, manchmal nicht.

Du solltest dir unbedingt Funktionen genauer anschauen. In deinem Code ist so viel Code, der sich wiederholt. Das macht ihn natürlich unnötig lang.
Die Gotos machen den Code schließlich noch unübersichtlicher.

Warum dein Code nicht so funktioniert, wie du das willst, lässt sich wegen des sehr unübersichtlichen Codes auch nicht so einfach sagen.

Danke für deine Tipps, doch wie kann ich schlussendlich den Code kürzer fassen? Die einzige Möglichkeit, die mir so spontan einfällt ist Structs, doch die hab ich integriert wos mir möglich war.

Wo wird die Variable loesung initalisiert? Ohne Initalisierung kann diese Variable jeden Wert haben ==> Ergebnis undefiniert.

Sie ist eine globale Variable und wird am Anfang der Funktion main() definiert.