C (sorry mir ist kein längerer Titel eingefallen)

Ich hätts ja so gemacht:

Zitat

#include <iostream>

using namespace std;

int main() {
char eingabe;
cout << "Was eingeben: " << endl;
cin.get(eingabe);
if(eingabe=='a') {
cout << "a" << endl;
}
else if(eingabe=='b') {
cout << "b" << endl;
}
else {
cout << "kA!?" << endl;
}
system("PAUSE");
}

Alles anzeigen

cout find ich doch irgendwie besser als printf^^.
Statt "cin.get(var);" ginge auch "cin >> var;" .
tolower() kenn ich garnicht, bin aber auch nich so der programmier-crack.

edit:
öhm, ich vergas, hier gehts ja um c.
keine Ahnung ob es in C auch geht, ich hab nämlich nur n bissl c++ gelernt und kenn die unterschiede auch nicht.

Zitat von xmms

Danke, das ändert aber nichts an meinem Problem :wink:
Außerdem ist das doch C++?!

siehe edit oben^^

Komisch, bei mir funzt der Code einwandfrei, nur dass ich iostream mit einbinden muss, weil bei mir anscheinend in stdio tolower() nicht deklariert ist. Benutze dev-cpp! :?

#include <stdio.h>
#include <iostream>


int main(){
    char menu;
    printf ("Gib einen Befehl ein: ");
    menu = tolower(getchar());
    if (menu=='a')  printf ("a\n");
    else if (menu=='b') printf ("b\n");
    else printf ("kA!?\n");
    system("pause");
}

Zitat von xmms

Funktionieren tut er ja es geht nur um den gesamten Aufbau.
Also:

ich habe die Funktion menueausgabe(). Die gibt das Menu aus und wartet auf eine Eingabe, liest sie ein (so wie oben)

Von da werden dann andere Funktionen aufgerufen (das funktioniert auch meistens). Wenn die andere Funktion dann fertig ist, soll das Menu wieder ausgegeben werden

versteh ich das richtig, dass du es also ungefähr so hast?

void menueausgabe() {
    char menu;
    printf ("Gib einen Befehl ein: ");
    menu = tolower(getchar());
    if (menu=='a') {
    blablatudas;
    menueausgabe();
    }
    else if (menu=='q') {
    blablatudas;
    menueausgabe();
    }
}

normal sollte das ja nicht passieren, aber vllt solltest du in jedes if noch ein return (ich ghe mal von einem void aus) schreiben, damit sich die funktion, nachdem sie sich wieder selbst aufgerufen hat, sofort wieder beendet. :?

Poste bitte mal den kompletten Code oder lade ihn irgendwo hoch. Ich muß das ganze mal im Zusammenhang sehen, da der Code so eigentlich korrekt aussieht. Höchstwahrscheinlich gibt es Problem auf Klammerungsebene.

Wobei folgender Abschnitt definitiv Käse ist:

else if (menu=='u');
         blablatujenes
}

Die Abfrage auf 'u' ist wirkungslos, da sofort ein Semikolon = leere Anweisung folgt und deshalb blablatujenes immer ausgeführt wird

Zitat von xmms

Es geht da drum, was passiert, wenn sich das Ding selbst aufruft... Kann es da Probleme geben?

Aber Hallo! Schon mal Stack Overflow gehört?

[edit] Bei Rekursion immer an die Abbruchbedingung und return; denken! Wobei... wozu brauchst Du hier Rekursion, da tut's doch wohl 'ne Schleife :-??

Wie wäre es denn hiermit:

#include <stdio.h>


int main()
{
    char menu = ' ';


    while(menu != 'c')     /* c = cancel */
    {
       printf ("Gib einen Befehl ein: ");
       menu = tolower(getchar());


       if (menu=='a')
       {
          blablatudas();
       }
       else if (menu=='q')
       {
          blablatudies();
       }
       else if (menu=='u')
       {
         blablatujenes();
       }
    }
}

...und so wäre es sogar elegant:

#include <stdio.h>


int main()
{
    char menu = ' ';


    do
    {
       printf ("Gib einen Befehl ein: ");
       menu = tolower(getchar());
       switch(menu)
       {
          case 'a':
             blablatudas();
             break;
          case 'q':
             blablatudies();
             break;
          case 'u':
             blablatujenes();
             break;
          case 'c':
             printf("Bye!");
             break;
          default:
             printf("Häääh?");
       }
    } while(menu != 'c');     /* c = cancel */
}

:mrgreen:
[edit] Semikolon nach while(...) vergessen :x
Aber egal, hätte eh der Compiler gemerkt

Zitat von xmms

//Edit: Uli: In deinem Code ist ein Fehler

Nicht mehr...

hm, sehr interessantes Thema :shock:
:lol:

Schleife *autsch* da hätt ich ja auch drauf kommen können. :roll:
Rekursion hab ich bisher auch nur ein einziges Mal benutzen müssen, und das war das durchsuchen von Unterordnern mit der WinAPI-Funktion FindNextFile().
Sonst hat immer die gute alte Schleife geholfen. :lol: