Programm ziegt trotz switch und case alles an

Werbeanzeige

steffen1997

Treue Seele

Beiträge: 200

Beruf: Schüler

13.11.2011, 13:58

Programm ziegt trotz switch und case alles an

Ich habe folgendes Problem ,einfach so aus Spaß habe ich mal gestern angefangen einen Klon
von einem Spiel namens Game Dev Story zu machen,worum es geht muss ich glaube nicht erläutern.
Ich habe verschiedene Probleme z.B. das bei einer switch Anzeige wo ich z.B. case e eingebe
er mir auch von case m die sachen ausführt.
Der Quellcode und das Kompilierte Programm ist in einer .rar Datei und hier:

Ich hoffe ihr könnt mir helfen und mir sagen wie ich den Code sauberer schreiben könnte,wie man sieht
ist der ja ziemlich durcheinander.Ich denke mal ich müßte ne while schleife schrieben damit das Sipe nicht abbricht ,ich weiß
aber nicht wie genau.

C-/C++-Quelltext

#include <iostream>
#include <windows.h>

class Wirtschaft
{
public:
int CompanyName;
char chbuyMitarbeiter;
int Grafik1;
int Sound1;
int Gameplay1;
int money;
int moneywin;
int moneyneu;            //neuer Kontostand nachdem Spielverkaufen
int gamepoints;         //Spielbewertungspunkte
int moneymitarbeiter; //Weil Mitarbeiter gekauft werden
char chInfos;           //Auswählen was für Infos man ansehen will
char chgenre;           //Genre beeinflusst gamepoints(z.B. FPS,RPGS),ist mit Spieltyp verbunden
char chspieltyp;           //z.B. ob im Spiel Mutanten,Soldaten oder sonst was vor kommen
int genre;              //Zum anzeigen des Genres weil char nur einen Buchstaben speichert?
int spieltyp;           //Zum anziegen des Spieltyp weil char nur einen Buchstaben speichert?
void CreateCompanyName()                            //Firma erstellen Funktion
{
    std::cout<<"Geben sie ihren Firmennamen ein"<<std::endl;
std::cin>>CompanyName;

}
void BuyandSellMitarbeiter()               //Mitarbeiter kaufen Funktion ANFANG
{
std::cout<<"Waehlen sie aus welche Mitarbeiter sie kaufen moechten"<<std::endl;
std::cout<<"Waehlen sie ihren ersten Mitarbeiter aus"<<std::endl;
std::cout<<"Sie koennen : "<<std::endl;
std::cout<<"(H)acker kaufen"<<std::endl;
std::cout<<"(G)rafiker kaufen"<<std::endl;
std::cout<<"(M)usiker kaufen"<<std::endl;
std::cout<<"(D)esigner(Gamedesigner) kaufen"<<std::endl;
std::cout<<"(P)rogrammierer kaufen"<<std::endl;
std::cin>>chbuyMitarbeiter;

switch(chbuyMitarbeiter)          
{
case('h'): //Hacker
{
 Grafik1=55;
 Sound1=45;
 Gameplay1=35;
moneymitarbeiter=money-500;
}break;
case('g'): //Grafiker
    {
 Grafik1=80;
 Sound1=30;
 Gameplay1=25;
 moneymitarbeiter=money-400;
    }break;
case('m'): //Musiker
    {
 Grafik1=20;
 Sound1=75;
 Gameplay1=15;
 moneymitarbeiter=money-350;
    }break;
case('d'):  //Gamedesigner
    {
 Grafik1=35;
 Sound1=40;
 Gameplay1=85;
 moneymitarbeiter=money-450;
    }break;
case('p'):  //Programmierer
    {
         Grafik1=50;
     Sound1=30;
         Gameplay1=45;
         moneymitarbeiter=money-280;
    }break;
default:
    {
        std::cout<<"Falsche Eingabe"<<std::endl;
    }
}
std::cout<<Grafik1<<" "<<Sound1<<" "<<Gameplay1<<std::endl;
}                                                 //Mitarbeiter Kaufen Funktion ENDE
void spielentwickeln()                           //Spielentwickeln Funktion  ANFANG
{
std::cout<<"Waehle das Spielgenre aus"<<std::endl;
std::cout<<"(E)go Shooter"<<std::endl;
std::cout<<"(R)ollenspiel"<<std::endl;
std::cout<<"(S)trategie"<<std::endl;
std::cin>>chgenre;
switch(chgenre)
{
case('e'):
    {
        std::cout<<"Du hast als Genre Ego-shooter ausgewählt"<<std::endl;
        
    }break;
case('r'):
    {
        std::cout<<"Du hast als Genre Rollenspiel ausgewählt"<<std::endl;
        
    }break;
case('s'):
    {
        std::cout<<"Du hast als Genre Strategiespiel ausgewählt"<<std::endl;
    
    }break;

}
std::cout<<"Was für einen Spieltyp soll dein Spiel haben?"<<std::endl;
std::cout<<"(M)utanten"<<std::endl;
std::cout<<"(B)löcke"<<std::endl;
std::cout<<"(S)oldaten"<<std::endl;
std::cin>>chspieltyp;
switch(chspieltyp)
{
case('m'):
    {
        std::cout<<"Du hast als Spieltyp Mutanten ausgewählt"<<std::endl;
    }break;
case('b'):
    {
        std::cout<<"Du hast als Spieltyp Blöcke ausgewählt"<<std::endl;
    }break;
case('s'):
    {
        std::cout<<"Du hast als Spieltyp Soldaten ausgewählt"<<std::endl;
}break;
}
std::cout<<"Du hast ein: "<<"genre(Platzhalter)"<<" "<<"spieltyp(Platzhalter)"<<" erschaffen"<<std::endl;
gamepoints=Grafik1*1.5+Sound1*1+Gameplay1*1.8;
std::cout<<gamepoints<<std::endl;
}                                                //Spielentwickeln Funktion ENDE
void spielverkaufen()                           //Spielverkaufen(publishing) Funktion ANFANG
{   
    
    moneywin=gamepoints+(rand()%500)+1;
    moneyneu=moneymitarbeiter+moneywin;
    std::cout<<moneywin;
    
}                                              //Spielverkaufen(publishing) Funktion ENDE
void zeigeInfos()                                //Infos Zeigen Funktion    ANFANG
{   
    std::cout<<"Was für Infos wollen sie sehen?"<<std::endl;
    std::cout<<"(F)irmenname"<<std::endl;
    std::cout<<"(E)ntwickeltes Spiel"<<std::endl;
    std::cout<<"(K)ontostand"<<std::endl;
    std::cout<<"(M)itarbeiter Fähigkeitspunkte"<<std::endl;
    std::cin>>chInfos;
    switch(chInfos)
    {
    case('f'):
        {
            std::cout<<"Ihr Firmenname ist: "<<CompanyName<<std::endl;
        }break;
    case('e'):
        {
std::cout<<"Ihr entwickeltes Spiel hat:  "<<gamepoints<<"Punkte"<<std::endl;
        }break;
    case('k'):
        {
std::cout<<"Ihr Kontostand beträgt:  "<<moneyneu<<"Euro"<<std::endl;
        }break;
    case('m'):
        {
    std::cout<<"Ihr Mitarbeiter hat:  /n"<<Grafik1<<" Grafikpunkte"<<std::endl;
    std::cout<<Sound1<<" Soundpunkte"<<std::endl;
    std::cout<<Gameplay1<<" Gameplaypunkte"<<std::endl;
        }break;
    }

}                                                   //Infos Zeigen Funktion ENDE
void credits()                                     //Credits Funktion ANFANG
{
    std::cout<<"Mein Name"<<std::endl;
}                                                  //Credits Funktion ENDE
}                                                  //Klassen ENDE

;
int main()
{       
    Wirtschaft Mitarbeiter;
    Mitarbeiter.money=5000;
    char chauswahl;

    bool BObeginnen;
    bool BOWirklichbeenden;

    
    std::cout<<"Moechten sie mit dem Spiel beginnen?";
    std::cout<<"1 steht fuer ja und 0 fuer nein"<<std::endl;
    std::cin>>BObeginnen;
    if(BObeginnen==1)
    {
    std::cout<<"Waehlen sie aus was sie machen moechten"<<std::endl;
    std::cout<<"Frima gruenden(f)"<<std::endl;
    std::cout<<"Mitarbeiter kaufen(m)"<<std::endl;
    std::cout<<"Spielentwickeln(s)"<<std::endl;
    
    std::cout<<"Spiel verkaufen(v)"<<std::endl;
        std::cin>>chauswahl;
        
    switch(chauswahl)
    {
        case('f'):
        {
            Mitarbeiter.CreateCompanyName();
        }break;
    case('m'):
        {
            Mitarbeiter.BuyandSellMitarbeiter();
        }break;
    case('s'):
        {
            Mitarbeiter.spielentwickeln();
        }break;
    case('v'):
        {
            Mitarbeiter.spielverkaufen();
        }break;
    default:
        {
            std::cout<<"Falsche Eingabe"<<std::endl;
        }


    
    }

        
    }
    else
    {
        std::cout<<"Möchten sie das Spiel beenden?"<<std::endl;

        std::cout<<"1 steht für Ja und 0 für nein"<<std::endl;
        std::cin>>BOWirklichbeenden;

        if(BOWirklichbeenden==1)
        {
            std::cout<<"Dies ist ein Platzhalter,die Funktion kommt erst spaeter"<<std::endl;

        }
        else
        {
            return BObeginnen;
        }
    
    }

return 0;
    
}

»steffen1997« hat folgende Datei angehängt:

Game Pro story neu.rar (13,15 kB - 79 mal heruntergeladen - zuletzt: 20.04.2024, 16:20)

Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von »steffen1997« (13.11.2011, 14:07)

Zum Seitenanfang

Schorsch

Supermoderator

Beiträge: 5 145

Wohnort: Wickede

Beruf: Softwareentwickler

13.11.2011, 14:03

Ans Ende von jedem Case-Block musst du ein break; setzen. Ansonsten wird ab hier jeder Case-Block ausgeführt.

„Es ist doch so. Zwei und zwei macht irgendwas, und vier und vier macht irgendwas. Leider nicht dasselbe, dann wär's leicht.
Das ist aber auch schon höhere Mathematik.“

Zum Seitenanfang

steffen1997

Treue Seele

Beiträge: 200

Beruf: Schüler

13.11.2011, 14:09

so gut schonmal danke für den Tip, hab gemacht ,das Problem was bleibt ,ich denke mal ich brauche
eine while oder eine for schleife damit das Spiel nicht abbricht ,aber wie genau schreibe ich die "passend" in
meinen code.

Zum Seitenanfang

Schorsch

Supermoderator

Beiträge: 5 145

Wohnort: Wickede

Beruf: Softwareentwickler

13.11.2011, 14:54

Das Switch-Case-Problem ist aber nun gelöst oder nicht? Mit der Schleife liegst du schon richtig. Du könntest dir zum Beispiel eine Variable game_running vom Typ bool machen. Solange das Spiel laufen soll ist diese true. Wenn du beenden möchtest wird sie auf false gesetzt. Dann kannst du diese Variable für deine Schleife benutzen. Das wäre jetzt natürlich nur eine Möglichkeit. Spiel damit ein bisschen rum. Gibt es denn ansonsten noch irgendwelche Probleme?

„Es ist doch so. Zwei und zwei macht irgendwas, und vier und vier macht irgendwas. Leider nicht dasselbe, dann wär's leicht.
Das ist aber auch schon höhere Mathematik.“

Zum Seitenanfang

steffen1997

Treue Seele

Beiträge: 200

Beruf: Schüler

13.11.2011, 17:11

ja das mit switch ist gelöst danke nochmal für den Hinweis mit break,ich werde demnächst dann mal etwas mit den Schleifen
rumprobieren ,wie du es gesagt hast und falls es Probleme gibt schreibe ich es einfach wieder in den Thread hinein.Übrigens sonst sind mir keine
Probleme aufgefallen.

Zum Seitenanfang

steffen1997

Treue Seele

Beiträge: 200

Beruf: Schüler

17.11.2011, 15:22

so im Moment ist das jetzt gelöst mit switch nur es gibt noch 2Probleme erstens das Geld wird nicht mehr und 2. nur beim Punkt Firma-
gründen verhängt er sich und zeigt weiter alles ununterbrochen an hier der Quellcode:

C-/C++-Quelltext

#include <iostream>
#include <windows.h>

class Wirtschaft
{
public:
int CompanyName;
char chbuyMitarbeiter;
int Grafik1;
int Sound1;
int Gameplay1;
int money;
int moneywin;
int moneyneu;            //neuer Kontostand nachdem Spielverkaufen
int gamepoints;         //Spielbewertungspunkte
int moneymitarbeiter; //Weil Mitarbeiter gekauft werden
char chInfos;           //Auswählen was für Infos man ansehen will
char chgenre;           //Genre beeinflusst gamepoints(z.B. FPS,RPGS),ist mit Spieltyp verbunden
char chspieltyp;           //z.B. ob im Spiel Mutanten,Soldaten oder sonst was vor kommen
int genre;              //Zum anzeigen des Genres weil char nur einen Buchstaben speichert?
int spieltyp;           //Zum anziegen des Spieltyp weil char nur einen Buchstaben speichert?
void CreateCompanyName()                            //Firma erstellen Funktion
{
    std::cout<<"Geben sie ihren Firmennamen ein"<<std::endl;
std::cin>>CompanyName;
std::cout<<"Ihr Firmenname ist: "<<CompanyName<<std::endl;

}
void BuyandSellMitarbeiter()               //Mitarbeiter kaufen Funktion ANFANG
{
std::cout<<"Waehlen sie aus welche Mitarbeiter sie kaufen moechten"<<std::endl;
std::cout<<"Waehlen sie ihren ersten Mitarbeiter aus"<<std::endl;
std::cout<<"Sie koennen : "<<std::endl;
std::cout<<"(H)acker kaufen"<<std::endl;
std::cout<<"(G)rafiker kaufen"<<std::endl;
std::cout<<"(M)usiker kaufen"<<std::endl;
std::cout<<"(D)esigner(Gamedesigner) kaufen"<<std::endl;
std::cout<<"(P)rogrammierer kaufen"<<std::endl;
std::cin>>chbuyMitarbeiter;

switch(chbuyMitarbeiter)          
{
case('h'): //Hacker
{
 Grafik1=55;
 Sound1=45;
 Gameplay1=35;
moneymitarbeiter=money-500;
}break;
case('g'): //Grafiker
    {
 Grafik1=80;
 Sound1=30;
 Gameplay1=25;
 moneymitarbeiter=money-400;
    }break;
case('m'): //Musiker
    {
 Grafik1=20;
 Sound1=75;
 Gameplay1=15;
 moneymitarbeiter=money-350;
    }break;
case('d'):  //Gamedesigner
    {
 Grafik1=35;
 Sound1=40;
 Gameplay1=85;
 moneymitarbeiter=money-450;
    }break;
case('p'):  //Programmierer
    {
         Grafik1=50;
     Sound1=30;
         Gameplay1=45;
         moneymitarbeiter=money-280;
    }break;
default:
    {
        std::cout<<"Falsche Eingabe"<<std::endl;
    }
}
std::cout<<"Grafikpunkte des Mitarbeiters: "<<Grafik1<<std::endl;
std::cout<<"Soundpunkte des Mitarbeiters: "<<Sound1<<std::endl;
std::cout<<"Gameplaypunkte des Mitarbeiters: "<<Gameplay1<<std::endl;
}                                                 //Mitarbeiter Kaufen Funktion ENDE
void spielentwickeln()                           //Spielentwickeln Funktion  ANFANG
{
std::cout<<"Waehle das Spielgenre aus"<<std::endl;
std::cout<<"(E)go Shooter"<<std::endl;
std::cout<<"(R)ollenspiel"<<std::endl;
std::cout<<"(S)trategie"<<std::endl;
std::cin>>chgenre;
switch(chgenre)
{
case('e'):
    {
        std::cout<<"Du hast als Genre Ego-shooter ausgewählt"<<std::endl;
        
    }break;
case('r'):
    {
        std::cout<<"Du hast als Genre Rollenspiel ausgewählt"<<std::endl;
        
    }break;
case('s'):
    {
        std::cout<<"Du hast als Genre Strategiespiel ausgewählt"<<std::endl;
    
    }break;

}
std::cout<<"Was für einen Spieltyp soll dein Spiel haben?"<<std::endl;
std::cout<<"(M)utanten"<<std::endl;
std::cout<<"(B)loecke"<<std::endl;
std::cout<<"(S)oldaten"<<std::endl;
std::cin>>chspieltyp;
switch(chspieltyp)
{
case('m'):
    {
        std::cout<<"Du hast als Spieltyp Mutanten ausgewählt"<<std::endl;
    }break;
case('b'):
    {
        std::cout<<"Du hast als Spieltyp Blöcke ausgewählt"<<std::endl;
    }break;
case('s'):
    {
        std::cout<<"Du hast als Spieltyp Soldaten ausgewählt"<<std::endl;
}break;
}
std::cout<<"Du hast ein: "<<"genre(Platzhalter)"<<" "<<"spieltyp(Platzhalter)"<<" erschaffen"<<std::endl;
gamepoints=Grafik1*1.5+Sound1*1+Gameplay1*1.8;
std::cout<<gamepoints<<std::endl;
}                                                //Spielentwickeln Funktion ENDE
void spielverkaufen()                           //Spielverkaufen(publishing) Funktion ANFANG
{   
    
    moneywin=gamepoints+(rand()%5)+1;
    moneyneu=moneymitarbeiter+moneywin;
    std::cout<<moneywin;

    
}                                              //Spielverkaufen(publishing) Funktion ENDE
void zeigeInfos()                                //Infos Zeigen Funktion    ANFANG
{   
    std::cout<<"Was für Infos wollen sie sehen?"<<std::endl;
    std::cout<<"(F)irmenname"<<std::endl;
    std::cout<<"(E)ntwickeltes Spiel"<<std::endl;
    std::cout<<"(K)ontostand"<<std::endl;
    std::cout<<"(M)itarbeiter Fähigkeitspunkte"<<std::endl;
    std::cin>>chInfos;
    switch(chInfos)
    {
    case('f'):
        {
            std::cout<<"Ihr Firmenname ist: "<<CompanyName<<std::endl;
        }break;
    case('e'):
        {
std::cout<<"Ihr entwickeltes Spiel hat:  "<<gamepoints<<"Punkte"<<std::endl;
        }break;
    case('k'):
        {
std::cout<<"Ihr Kontostand beträgt:  "<<moneyneu<<"Euro"<<std::endl;
        }break;
    case('m'):
        {
            std::cout<<"Ihr Mitarbeiter hat:  "<<std::endl<<Grafik1<<" Grafikpunkte"<<std::endl;
    std::cout<<Sound1<<" Soundpunkte"<<std::endl;
    std::cout<<Gameplay1<<" Gameplaypunkte"<<std::endl;
        }break;
    }

}                                                   //Infos Zeigen Funktion ENDE
void credits()                                     //Credits Funktion ANFANG
{
    std::cout<<"von steffen1997"<<std::endl;
}                                                  //Credits Funktion ENDE
}                                                  //Klassen ENDE

;
int main()
{       
    Wirtschaft Mitarbeiter;
    Mitarbeiter.money=5000;
    char chauswahl;

    bool BObeginnen;
    bool BOWirklichbeenden;
    bool beginnen;
    std::cout<<"Geben sie ein ob sie beginnen wollen"<<std::endl;
    std::cout<<"1 steht fuer ja 0 fuer nein"<<std::endl; 
    std::cin>>beginnen;
    while(beginnen==1)
    {

    std::cout<<"Waehlen sie aus was sie machen moechten"<<std::endl;
    std::cout<<"Frima gruenden(f)"<<std::endl;
    std::cout<<"Mitarbeiter kaufen(m)"<<std::endl;
    std::cout<<"Spielentwickeln(s)"<<std::endl;
    
    std::cout<<"Spiel verkaufen(v)"<<std::endl;
    std::cout<<"Infos(i)"<<std::endl;
    std::cout<<"Credits(c)"<<std::endl;
        std::cin>>chauswahl;
        
    switch(chauswahl)
    {
        case('f'):
        {
            Mitarbeiter.CreateCompanyName();
        }break;
    case('m'):
        {
            Mitarbeiter.BuyandSellMitarbeiter();
        }break;
    case('s'):
        {
            Mitarbeiter.spielentwickeln();
        }break;
    case('v'):
        {
            Mitarbeiter.spielverkaufen();
    
        }break;
    case('i'):
        {
            Mitarbeiter.zeigeInfos();
        }break;
    case('c'):
        {
            Mitarbeiter.credits();
        }break;

    default:
        {
            std::cout<<"Falsche Eingabe"<<std::endl;
        }


    
    }
  
        
    }
    
    

        std::cout<<"Möchten sie das Spiel beenden?"<<std::endl;

        std::cout<<"1 steht fuer Ja und 0 fuer nein"<<std::endl;//funkt noch nicht mit nichtbeenden
        std::cin>>BOWirklichbeenden;

    



return 0;
    
}

Zum Seitenanfang

Schorsch

Supermoderator

Beiträge: 5 145

Wohnort: Wickede

Beruf: Softwareentwickler

17.11.2011, 15:37

Ich gebe dir den Tipp, mal von neu auf anzufangen. Fang an mit deiner Schleife an und sorg dafür, dass du sie auch beenden kannst. Dann testest du erst mal. Wenn alles so klappt wie gewollt baust du dein erstes Feature ein. Zum Beispiel Firma gründen. Auch die Funktionen in der Klasse schreibst du jetzt erst. Dann wieder testen. Wenn dabei alles funktioniert dann baust du das nächste Feature ein und testest. Sieht für mich so aus, als wenn du grad zu viel auf einmal machst. Spontan seh ich grad auch keinen Fehler, kann deiner Fehlerbeschreibung aber auch nicht genau entnehmen was nun nicht läuft. Ich könnte jetzt auch nur debuggen, was du aber auch selber schaffst

Setz einfach ein paar Haltepunkte und guck ob diese erreicht werden. Wenn ja, dann guck dir die Werte der Variablen an. Dabei reichen natürlich erst mal die wichtigen. Aber ich denke wenn du noch mal Schritt für Schritt alles zusammen baust, wirst du das Problem vermutlich auch so schon finden.

„Es ist doch so. Zwei und zwei macht irgendwas, und vier und vier macht irgendwas. Leider nicht dasselbe, dann wär's leicht.
Das ist aber auch schon höhere Mathematik.“

Zum Seitenanfang

steffen1997

Treue Seele

Beiträge: 200

Beruf: Schüler

17.11.2011, 15:48

Also so wie ich verstehe komplett neu anfangen und ein Feature nach dem anderen?meinst du das so?

Zum Seitenanfang

Nox

Supermoderator

Beiträge: 5 272

Beruf: Student

17.11.2011, 16:02

Das ist eig das perfekte Beispiel um sich mal mit dem Debugger vertraut zu machen. Hast du schonmal versucht den Debugger zu nutzen? Auch wenn es am Anfang etwas ungewohnt ist, kann man sich eig nie früh genug mit ihm anfreunden. Nutzt du eine IDE und wenn ja welche? Weil dann können wir dir ggf einen passenden Einstieg liefern, oder du bemühst mal google um entsprechende Artikel/kurztuts (ggf. gibt es sogar einen Artikel im Wiki?).

PRO Lernkurs "Wie benutze ich eine Doku richtig"!
CONTRA lasst mal die anderen machen!
networklibbenc - Netzwerklibs im Vergleich | syncsys - Netzwerk lib (MMO-ready) | Schleichfahrt Remake | Firegalaxy | Sammelsurium rund um FPGA&Co.

Zum Seitenanfang

Checkmateing

Community-Fossil

Beiträge: 2 218

17.11.2011, 16:05

Ja. Mach erst einmal das Grundgerüst und danach solltest du die Details ganz langsam hinzufügen.

MfG
Check

Das gibt Übersicht! Stay gold.

Zum Seitenanfang

Werbeanzeige

spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Programm ziegt trotz switch und case alles an

Programm ziegt trotz switch und case alles an

C-/C++-Quelltext

C-/C++-Quelltext

Ähnliche Themen