Hallo,
Ich bin heute Angefangen mit C++ und habe ein Problem bzw. suche etwas.

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71

#include "stdafx.h" #include <iostream>     double Rechnen(int wahl, double zahl1, double zahl2);   int main() {     int wahl;     double ergebnis, zahl1, zahl2;       std::cout << "Wählen sie eine Rechenmothode aus: \n" << std::endl               << "(1)Addieren\n" << "(2)Subdrahieren\n" << "(3)Multi\n" << "(4)Divi\n";     std::cin >> wahl;     std::cout << "\n\n";     std::cout << "\n\n";           std::cout << "\Gebe Zahl 1 ein: ";     std::cin >> zahl1;     std::cout << "\Gebe Zahl 2 ein: ";     std::cin >> zahl2;     std::cout << "\n\n";     std::cout << "\n\n";       ergebnis = Rechnen(wahl, zahl1, zahl2);       std::cout << "Das Ergebnis ist: " << ergebnis << std::endl;       std::cin.ignore();     std::cin.get();     return 0; }   double Rechnen(int wahl, double zahl1, double zahl2) {     double ergebnis;       if ( wahl == 1)     {         ergebnis = zahl1 + zahl2;     }      else if ( wahl == 2)     {         ergebnis = zahl1 - zahl2;     }          else if ( wahl == 3)     {         ergebnis = zahl1 * zahl2;     }       else if ( wahl == 4)     {         ergebnis = zahl1 / zahl2;     }     else     {         std::cout << "Falsche Eingabe!\n";             }               return ergebnis;     }

Möchte ich es so haben das ich eine Rechen Methode wähle die nicht 1-4 ist kommt falsche Eingabe und wenn ich enter drück schließst sich das Programm und fragt nicht noch nach Zahl1 und Zahl2.
Schreibt mir jemand den Code um****?
Oder sagt mir eine Möglichkeit****?

mfG Jonas

Schreibt mir jemand den Code um****?

Nein, das ist ein Hilfe-Forum und kein "Wer kann mir was programmieren" Forum.

Oder sagt mir eine Möglichkeit****

Das jederzeit:
Wie ich das verstanden hab möchtest du das dein Programm nicht nach Zahlen fragt wenn eine ungültige Rechenart gewählt wurde:
Dann musst du zwischen den Zeilen 14 und 15 überprüfen ob "wahl" zwischen 1 und 4 liegt, wenn nicht dann gleich beenden, sonst nach Zahlen fragen:

Code :

1 2 3 4 5 6

wenn (zahl < 1 || zahl > 4)    //Fehlermeldung sonst {   //Zeile 15-32 }

Das verstehe ich irgendwie nicht.

Was konkret verstehst du nicht?

zahl > 0 && zahl < 0

Hoppla, da war wohl die Hand schneller als der Kopf. Es muss natürlich zahl < 1 || zahl > 4 heißen. Ich habs oben ausgebessert.

Ok jetzt macht das auch sinn. Gibt es auch ne andere möglichkeit?

Was tut diese Variante denn nicht, das du unbedingt haben möchtest? Das ist auf jeden Fall die sauberste Lösung.

Ja in diesen Fall aber wenn ich mehrerer Zeilen hab und mehrere solcher Abbruch schleifen oder wie man so was nennen soll ist das ganz schnell unverständlich und unübersichtlich.

Naja, man kann stattdessen folgendes verwenden (statt dem von mir oben geposteten Code)

Code cpp:

1 2 3 4 5

if (zahl < 1 || zahl > 4) {   //Fehlermeldung   return 1; }

Raten würde ich aber keinem dazu, da der Code für jemanden anderen dadurch schwer nachvollziehbar wird.

Zitat von Steiner_B

Hoppla, da war wohl die Hand schneller als der Kopf. Es muss natürlich zahl <= 1 || zahl >= 4 heißen. Ich habs oben ausgebessert.

Immer noch nicht richtig. Es muss z.B.

Code cpp:

1 2 3

if (zahl < 1 || zahl > 4) {   // Fehlermeldung }

heißen.

Zitat von Steiner_B

Naja, man kann stattdessen folgendes verwenden (statt dem von mir oben geposteten Code)

Code cpp:

1 2 3 4 5

if (zahl <= 0 || zahl >= 1) {   //Fehlermeldung   return 1; }

Die Bedingung ist ebenso falsch, da sie immer erfüllt ist.

Zitat von Steiner_B

Raten würde ich aber keinem dazu, da der Code für jemanden anderen dadurch schwer nachvollziehbar wird.

Gegen ein return innerhalb einer Funktion ist grundsätzlich nichts einzuwenden. Das ist allemal übersichtlicher als viele ineinander geschachtelte if-else-Blöcke.

Grüße,
Matthias

Oh mann, das kommt davon wenn man während des Programmierens Fragen beantwortet. Danke für die Hinweise, ich hoff ich habs jetzt überall ausgebessert.