angeriffen werden sondern wieder zu dem Auswahl Menü da wo man Auswählen kann

ob man in den Dungeon oder ins Haus reingehen will. Das Spiel is aber auch noch nicht ganz fertig aber an der Stelle mit dem Haus da krieg ich es irgendwie net geregelt hab schon alles mögliche versucht

Hier ist mein Source Code:

//---------------------------------------------------------------------------
#include <iostream>
using namespace std;

int start;
int wohin;
int verlassen;
int auswahl;
int durchsuchen;
int UserAuswahl;
int PcAuswahl;

int links;
int Angriff;
int Flucht;

int main()
{
cout<<"Konsolen Spiel\n";
cout<<"\n";
cout<<"1 Spiel starten:\n";
cin>>start;

do
{

system("CLS");
cout<<"Wohin willst du gehen ?\n";
cout<<"1=Ins Haus gehen:, 2=In den Dungeon gehen:\n";
cin>>wohin;

if (wohin==1)

{
cout<<"Du stehst in einem Haus\n";
cout<<"1 = Haus Verlassen\n";
cout<<"2 = Haus Durchsuchen\n";
cin>>verlassen;

}

if (verlassen==2)
{
cout<<"Du hast 100 Gold muenzen gefunden! Druecke die (1) um zurueck zu kehren\n";
cin>>verlassen;
}

if (wohin==2)
{
cout<<"Du bist jetzt im Dungeon\n";
cout<<"Woher willst du gehen? 1= links, 4=Dungeon Verlassen\n";
cin>>verlassen;
}

if (verlassen==1)

{
cout<<"Du wirst angegriffen!!!!\n";
cout<<"1 = Angriff, 2 = Flucht\n";
cin>>links;
}

}
while(verlassen==1);

if (verlassen==1)
{

cout<<"Waehle deine Angriffsfertigkeit:\n"
<<" 1. Feuerball \n"
<<" 2. Truemmerstein \n"
<<" 3. Kugelblitz \n"
<<" 4. Obsidian Flamme \n";
cin>>UserAuswahl;

srand( time (NULL) );
PcAuswahl = rand()%4+1;

cout << "Du hast " << UserAuswahl << " geweahlt\n";
cout<<"\n";
cout << "Der Gegner hat " << PcAuswahl << " gewaehlt\n\n";

system("PAUSE");

if (UserAuswahl==1 && PcAuswahl==1)
{

cout<<"Du hast den Gegner zwar besiegt aber konntest leider seinem Feuerball"
<<"nicht mehr rechtzeitig ausweichen. Druecke 1 um wiederbelebt zu werden.\n";
cin>>verlassen;
system("PAUSE");

}

else if (UserAuswahl==2 && PcAuswahl==2)
{
cout<<"Unentschieden\n";
cin>>verlassen;
system("PAUSE");

}
}
}

do
{
  system("CLS");
  cout<<"Wohin willst du gehen ?\n";
  cout<<"1=Ins Haus gehen:,  2=In den Dungeon gehen:\n";
  cin>>wohin;
  ...
  
  if (verlassen==1)
  {
    cout<<"Du wirst angegriffen!!!!\n";
    ...
  }
}

Also da ist das Ergebnis nicht verwunderlich. Eventuell solltest du dir nochmal genauer überlegen, wann was passieren soll und dein Programm passend strukturieren.

PS: sfds

PPS: Ich stelle mir gerade vor, wie das aussieht, wenn du dein Programm weiter entwickelst - Spagetticode in Reinform

#include <iostream>
#include <stdlib.h>

using namespace std;

int main()

{

int start;
int wohin;

int verlassen;
int links;

int wahl;
int angriff;

int userfertigkeit;
int pc;

int rezz;
int flucht;

cout<<"(1) Spiel Starten\n";
cin>>start;

switch(1)

do
{//do klammer

{
case 1: 
system("CLS");     
cout<<"Wohin willst du gehen ? 1= Haus ,3= Dungeon\n";
cin>>wohin; 

//--------------------BEREICH FÜR DAS HAUS !!!!!-------------------------------------------------//          
}
if (wohin==1)
{
cout<<"Du bist jetzt im Haus\n"; 
cout<<"1 Haus durchsuchen\n";
cout<<"2 Haus Verlassen\n";
cin>>verlassen;
}

if (1==verlassen)
{
system("CLS"); 
cout<<"Du hast 100 Goldmuenzen gefunden =)  druecke (2) um das Haus wieder zu verlassen\n";
cin>>verlassen;
}
//-----------------ENDE VOM BEREICH FÜR DAS HAUS -----------------------------------------------------//      

//--------------------BEREICH FÜR DEN DUNGEON!!!!!-------------------------------------------------//    
if (wohin==3)
{
system("CLS"); 
cout<<"Du bist jetzt im Dungeon\n";
cout<<"Woher willst du gehen 4 = Links\n";
cin>>links;
}
//-----------------ENDE VOM BEREICH FÜR DEN DUNGEON ----------------------------------------------------//      

//-----------------------------Bereich für Angriff und Flucht ----------------------------------------//
if (links==4)
{
system("CLS"); 
cout<<"Du wirst angegriffen !!! 5 = Angriff, 6 = Flucht\n";
cin>>wahl;
}

//-----------------------------Bereich für Angriff ------------------------------------------------------//
if (wahl==5)
{
system("CLS");
cout<<"Waehle deine Angriffsfertigkeit aus:\n";
cout<<"1 Kugelblitz\n";
cout<<"2 Feuerball\n";
cin>>userfertigkeit;

srand ( time (NULL) );
pc = rand()%1+1;

cout<<"Du hast die Fertigkeit"<<userfertigkeit<<"gewaehlt\n";
cout<<"Der Gegner hat die Fertigkeit"<<pc<<"gewaehlt\n\n";
}

//------------------------------------Bereich für Kampf-------------------------------------------------// 
if (userfertigkeit==1 && pc==1)
{
cout<<"Du hast den Gegner zwar besiegt konntest aber leider seinem\n"
    <<"Kugelblitz nicht mehr ausweichen. Druecke 2 und bestaetige mit enter\n"
    <<"um wieder belebt zu werden\n";
cin>>rezz;

}//if klammer

//------------------------------------ENDE VON BEREICH FÜR KAMPF ------------------------------------------------// 

//---------------------------- BEREICH FÜR FLUCHT ---------------------------------------------------------------------//

if (wahl==6)
{
cout<<"Druecke die 7 und bestaetige mit Enter um zu fliehen\n";
cin>>flucht;
}
//---------------------------- ENDE VON BEREICH FÜR FLUCHT ---------------------------------------------------------------------//

}//while klammer
while(verlassen==2);

  system("PAUSE");	

}

Ansonsten beschäftige dich erstmal mit ein paar Grundlagen von C++, bevor du sowas zusammenschusterst

PS:

switch(1)

do
{//do klammer

Was soll denn das darstellen?

PPS: "nicht ganz fertig" ist ja wohl die Untertreibung des Jahrhunderts

switch(1) 

do 
{//do klammer

war nur so eine makierung für mich weil ich ab und zu ausversehen zu viel lösche oder mal auf eine falsche taste komme^^ Die Kommentare zu dem Source Code müssen anderen net verstehen die sind nur für mich da ^^ Und die Grundlagen habe ich mir schon angesehen.

Ich finde das so übersichtlicher als das Einzurücken ^^
Also mit diesen Strichen wo ich die Bereiche makiert habe.

Na wenn du meinst. Ich find's mit einrückungen auf jeden Fall klarer zu erkennen, welcher (Unter)Block wo aufhört.

Und das hier:
[...]
war nur so eine makierung für mich weil ich ab und zu ausversehen zu viel lösche oder mal auf eine falsche taste komme^^

Es ist auf jeden Fall die ungewöhnlichste Anwendung von switch(), die mir bisher untergekommen ist

Ich will doch nur wissen wa da an meinem Source Code falsch ist.

Zweitens: Die bereits erwähnte switch()-Konstruktion ist zumindest fragwürdig

Drittens: Der gesamte Code hat keine erkennbare Struktur - wenn du so weitermachst, verlierst du selber den Überblick.

Hier die Frage nochmal: Was ist an dem Source Code falsch, wieso kommen da so blöde Bugs.

Ps. Ja ich weiss es ist schlecht formatiert ....................

Wenn ihr es auch nicht wisst dann schreibt einfach nichts dazu. Thx

Aber irgendwie is das noch etwas verbuggt^^

ist leider keine Fehlerbeschreibung.

Und bei deinem Kommentar kannst du froh sein, wenn überhaupt noch jemand antwortet.

(btw, hast du daran gedacht, daß die einzelnen Variablen am Ende der do-while-Schleife ihren jeweiligen Wert behalten?)

Wenn ich das Spiel starte und ins Haus reingehe. Dann gehe ich auf Haus Durchsuchen finde die 100 Goldmünzen dann drücke 1 und dann werd ich angegriffen -.- und das soll eben nicht passieren.

Hab das Programm schon ein paar mal verändert dann ging es beim ersten versuch alles gut ich konnte alles machen.

Z.B.

Ich geh ins Haus hol die 100 Goldmünzen dann nach geh ich in den Dungeon dann geh ich links dann werd ich angeriffen ich werde besiegt.

Danach bin ich dann wieder in meinem Auswahlmenü wo ich Auswählen kann ob ich ins Haus oder in den Dungeon will.

Das is ja auch alles soweit richtig. Aber wenn ich dann wieder in das Haus gehe und die 100 Goldmünzen hole dann steht dort das ich angegriffen werden Ich versuch schon den ganzen Tag das hinzukriegen das es richtig funktioniert.

Btw, ich würde ja etwas Struktur in dein Spiel reinbringen und den zentralen Teil (Auswahl der Handlungsoption und richtige Reaktion) in eine Funktion auslagern:

struct situation
{
  string beschreibung;
  vector<pair<string,situation*> > folgende;
};

situation* aktion(const situation& aktuell)
{
  int sel;
  cout<<aktuell.beschreibung<<endl;
  for(int i=0;i<aktuell.folgende.size();++i)
    cout<<i+1<<" - "<<aktuell.folgende[i].first<<endl;
  cin>>sel;
  return aktuell.folgende[sel-1].second;
}

Wie gesagt, am Ende der Schleife behalten alle Variablen ihren Wert - auch 'links' (das im vorigen Schleifendurchlauf während des 'if(wohin==3)'-Blocks auf 4 gesetzt wurde).

Was meinst du damit das die Variaböen ihren Wert behalten

#include <iostream>
#include <stdlib.h>

using namespace std;

int main()

{

int start;
int wohin;

int verlassen;
int links;

int wahl;
int angriff;

int userfertigkeit;
int pc;

int rezz;
int flucht;

cout<<"(1) Spiel Starten\n";
cin>>start;

switch(1)

do
{

{

case 1: 
system("CLS");     
cout<<"Wohin willst du gehen ? 1= Haus ,3= Dungeon\n";
cin>>wohin; 

//--------------------BEREICH FÜR DAS HAUS !!!!!-------------------------------------------------//          
                }
                 if (wohin==1)
                 {
                 cout<<"Du bist jetzt im Haus\n";
                 cout<<"1 Haus durchsuchen\n";
                 cout<<"2 Haus Verlassen\n";
                 cin>>verlassen;
                 }

                 if (1==verlassen)
                 {
                 system("CLS");
                 cout<<"Du hast 100 Goldmuenzen gefunden =)  druecke (2) um das Haus wieder zu verlassen\n";
                 cin>>verlassen;
}
//-----------------ENDE VOM BEREICH FÜR DAS HAUS -----------------------------------------------------//      

//--------------------BEREICH FÜR DEN DUNGEON!!!!!-------------------------------------------------//    
if (wohin==3)
{
system("CLS"); 
cout<<"Du bist jetzt im Dungeon\n";
cout<<"Woher willst du gehen 4 = Links\n";
cin>>links;
}
//-----------------ENDE VOM BEREICH FÜR DEN DUNGEON ----------------------------------------------------//      

//-----------------------------Bereich für Angriff und Flucht ----------------------------------------//
                if (links==4)
                {
                system("CLS");
                cout<<"Du wirst angegriffen !!! 5 = Angriff, 6 = Flucht\n";
                cin>>wahl;
                }

//-----------------------------Bereich für Angriff ------------------------------------------------------//
if (wahl==5)
{
system("CLS");
cout<<"Waehle deine Angriffsfertigkeit aus:\n";
cout<<"1 Kugelblitz\n";
cout<<"2 Feuerball\n";
cin>>userfertigkeit;

srand ( time (NULL) );
pc = rand()%1+1;

cout<<"Du hast die Fertigkeit"<<userfertigkeit<<"gewaehlt\n";
cout<<"Der Gegner hat die Fertigkeit"<<pc<<"gewaehlt\n\n";
}

//------------------------------------Bereich für Kampf-------------------------------------------------// 
                if (userfertigkeit==1 && pc==1)
                {
                cout<<"Du hast den Gegner zwar besiegt konntest aber leider seinem\n"
                <<"Kugelblitz nicht mehr ausweichen. Druecke 2 und bestaetige mit enter\n"
                <<"um wieder belebt zu werden\n";
                cin>>rezz;
                }

//------------------------------------ENDE VON BEREICH FÜR KAMPF ------------------------------------------------// 

//---------------------------- BEREICH FÜR FLUCHT ---------------------------------------------------------------------//

if (wahl==6)
{
cout<<"Druecke die 7 und bestaetige mit Enter um zu fliehen\n";
cin>>flucht;
}
//---------------------------- ENDE VON BEREICH FÜR FLUCHT ---------------------------------------------------------------------//

        }
        while(verlassen==2);

  system("PAUSE");

}

Ich hab es jetzt mal eingerückt aber was genau müsste ich denn dann da jetzt verändern

Mal davon abgesehen das es ziemlich grausam eingerückt wurde, bedeutet anders strukturieren nicht nur das Erscheinungsbild zu verändern sondern den Programmfluß anzupassen (Ausgliederung in unterschiedliche Funktionen/Methoden...).

Ganz davon abgesehen: Was soll das bitte sein?

switch(1)
do
{
{
case 1:

Sorry, aber ich habe grad nicht die Zeit um dir hier große Beispiele zu liefern, aber ein paar Regeln zur besseren Lesbarkeit kann ich dir kurz mitgeben:

Ein paar übliche Einrückformen:

int main
{
    // ...
    int wohin;
    // ...
    switch(wohin)
    {
        case 1:
          //...
          break;
        default:
          //...
          break;
    }
}

int main {
    // ...
    int wohin;
    // ...
    switch(wohin) {
        case 1:
          //...
          break;
        default:
          //...
          break;
    }
}

int main
    {
    // ...
    int wohin;
    // ...
    switch(wohin)
        {
        case 1:
          //...
          break;
        default:
          //...
          break;
        }
    }

Unabhängig davon wie du dein Programm schreibst solltest du einen Stil finden, der das Lesen erleichtert. Den damit findet man auch Fehler besser (Hier kann man sehr leicht sehen welcher Code zusammen gehört).

cu André

Ich hab es jetzt mal eingerückt aber was genau müsste ich denn dann da jetzt verändern

Erstens: Die Einrückung ist ja grauenhaft (ja, man könnte stundenlang darüber diskutieren, wie die optimale Einrückung aussieht - aber deine ist alles andere als optimal).

Zweitens: Fang am besten nochmal ganz bei Null an. Die Funktion "aktion()", die ich vorhin gepostet habe, wäre eventuell ein Ansatzpunkt.

struct situation 
{ 
  string beschreibung; 
  vector<pair<string,situation*> > folgende; 
}; 

situation* aktion(const situation& aktuell) 
{ 
  int sel; 
  cout<<aktuell.beschreibung<<endl; 
  for(int i=0;i<aktuell.folgende.size();++i) 
    cout<<i+1<<" - "<<aktuell.folgende[i].first<<endl; 
  cin>>sel; 
  return aktuell.folgende[sel-1].second; 
}

is doch net so schwer!

Das kannst du dann im Hauptprogramm verwenden, indem du für jede Raum des Spiels eine situation anlegst und geeignet verlinkst (NULL-Zeiger stehen dabei für "du bist tot"-Situationenund) anschließend per

situation* akt = /* insert start */;
while(akt != NULL)
  akt = aktion(*akt);

das Spiel loslaufen lässt.

(zufällige Ereignisse umzusetzen ist komplizierter, aber zunächst solltest du mal das Grundprinzip verstanden haben)

Bestandteile der STL

Aber mit den Sachen da konnte ich irgendwie nicht anfangen weil da stand das auch nicht beschrieben wie das mit diesem struct und so geht. Aber bei den Tutorials da sind ja viele drin ich lad mir da einfach mal alle runter und guck ob ich das was finde.