C++ Switch mit Char - "case-Ausdruck ist keine Konstante"

TheBang · 15. Juli 2014

Code

cout << "Zeitumwandlung\n" << endl;
int iEingabe, iEndwert;
char chEingabe;
cout << "\n Geben sie eine Zeit in Sekunden an: ";
cin >> iEingabe;
cout << "\n Waehlen sie ihren Umrechnungstyp." <<endl;
cout << "A = Minuten, B = Stunden, C = Tage." << endl;
cin >> chEingabe;
switch(chEingabe)
{
case A:
case a:
iEndwert = iEingabe / 60;
cout << iEingabe << " Sekunden in Minuten betraegt " << iEndwert <<endl;
break;
case B:
case b:
iEndwert = iEingabe / 3600;
cout << iEingabe << " Sekunden in Stunden betraegt " << iEndwert <<endl;
break;
case C:
case c:
iEndwert = iEingabe / 86400;
cout << iEingabe << " Sekunden in Tagen betraegt " << iEndwert <<endl;
break;
default: cout << "Falsche Eingabe" << endl;
break;
}

Alles anzeigen

2 Fehler entstehen:
"case-Ausdruck ist keine Konstante"
und
'a': nicht deklarierter Bezeichner (mit allen, A, a, B, b, C, c)

Was habe ich falsch gemacht?

javaest · 15. Juli 2014

case 'a':
//Anweisung(en)
break;
case 'b':

usw

Character gehören in " ' ".

Mario64 · 15. Juli 2014

Würden da keine einfachen Anführungsstriche ' um die einzelnen Werte (a,A etc.) schon reichen?

edit: Yay, selber Moment...

TheBang · 15. Juli 2014

Ah, danke dir. Hatte ich wohl vergessen :8

Hat gereicht ja. Klappt jetzt

javaest · 15. Juli 2014

Mario64 schrieb:

Würden da keine einfachen Anführungsstriche ' um die einzelnen Werte (a,A etc.) schon reichen?

edit: Yay, selber Moment...

Das muss man erstmal hinkriegen. Post-Skil over 9000....Oder so

TheBang · 15. Juli 2014

So funktioniert das ganze jetzt.
Musste zusätzlich noch das (float) hinzufügen, da sonst ja keine Umwandlung möglich ist.

Code

{
cout << "Zeitumwandlung\n" << endl;
int iEingabe;
float fEndwert;
char chEingabe;
cout << "\n Geben sie eine Zeit in Sekunden an: ";
cin >> iEingabe;
cout << "\n Waehlen sie ihren Umrechnungstyp." <<endl;
cout << "A = Minuten, B = Stunden, C = Tage." << endl;
cin >> chEingabe;
switch(chEingabe)
{
case 'A':
case 'a':
fEndwert = (float)iEingabe/60;
cout << iEingabe << " Sekunden in Minuten betraegt " << fEndwert <<endl;
break;
case 'B':
case 'b':
fEndwert = (float)iEingabe / 3600;
cout << iEingabe << " Sekunden in Stunden betraegt " << fEndwert <<endl;
break;
case 'C':
case 'c':
fEndwert = (float)iEingabe / 86400;
cout << iEingabe << " Sekunden in Tagen betraegt " << fEndwert <<endl;
break;
default: cout << "Falsche Eingabe" << endl;
break;
}
system("pause");
return 0;
}

Alles anzeigen

javaest · 15. Juli 2014

Füge doch noch 2 Methoden hinzu. Die eine soll die Textausgabe mit Hilfe globaler Variablen (Gültigkeitsbereich beachten) übernehmen und die andere um die Werte korrekt umzurechnen (KEINE void Methode :-))

Ps: Mit system("pause") pausierst du das gesamte Programm, hoffe das hast du nachgelesen.
Nimm stattdessen cin.get();.

hangman · 15. Juli 2014

du weißt aber schon, dass das ein ganz schönes gehacke ist?

was passiert, wenn iEingabe "abc" ist? Auch der switch mit Charaktern hat einen ganz faden Beigeschmack. Ich hätte das in ein enum class gepackt, oder gleich in eine extra Klasse ausgelagert.

Und du machst float = (float) / int. Das geht zwar prinzipiell, ist aber Brechreizerzeugend. Sobald du casten musst, ist dein Design falsch. Überlege dir doch lieber eine Lösung, die nur mit floats (besser double, soviel Speicher hast du bestimmt) arbeitet.

Das \n ist auch kritisch, nicht auf allen Systemen wird das so funktionieren. Die Zeilenenden sind nicht immer gleich. Da C++ prinzipiell auf jedem Betriebssystem laufen kann, würde ich die Umbrüche korrekt machen.

Und warum machst du alle cases schön eingerückt und den default so hässlich auf dieselbe Zeile?

Teilen