Brauche Hilfe bei einem Tic Tac Toe Spiel

Werbeanzeige

joke1333

Frischling

Beiträge: 4

06.09.2013, 14:08

Brauche Hilfe bei einem Tic Tac Toe Spiel

Guten Tag

Ich lerne zurzeit C++ , mit dem Buch C++ für Spieleprogrammierer von
Heiko Kalista. Dort habe ich gerade Kapitel 5 beendet, bei dem es um
Arrays ging. Nun dachte ich mir, könnte ich doch mit meinen neu
gewonnenen Kenntnissen ein kleines Tic Tac Toe (Mulitplayer, KI zu
programmieren war mir erstmal zu schwierig) Spiel programmieren. Tja,
soweit so gut, ich hab mal angefangen und bin dann auch zu einem Schluss
gekommen.

Nun als allererstes mal meinen Quellcode:

C-/C++-Quelltext

//TIC TAC TOE MULTIPLAYER
//
//
#include <iostream>

using namespace std;

struct S_Feldinfo
{
    bool Besetzt;
    bool Besetzt2;
    
};

struct S_Spieler
{
    char Name[40];

};

struct S_Gewinner
{
    bool Gewonnen;
    char Name[30];
};

const int Breite = 3;
const int Höhe = 3;



S_Spieler Spieler1;
S_Spieler Spieler2;
S_Feldinfo Spielfeld[Breite][Höhe];
S_Gewinner GewinnerErmittlung;

void GewinnerErmitteln ();
void LoescheSpielfeld ();
void Spielen ();
void ZeigeFeld ();
void Spielzug1 ();
void Spielzug2 ();


int main ()
{

    int Auswahl = 0;

    LoescheSpielfeld ();


    do
    {
        // Menü ausgeben
        //
        cout << "Tic Tac Toe Menue" << endl;
        cout << "1 - Spielen" << endl;
        cout << "2 - Beenden" << endl;


        // Eingabe abfragen
        cout << "Auswahl: ";
        cin >> Auswahl;

        // Je nach Eingabe verzweigen
        switch (Auswahl)
        {
            
            case (1):
            {
                  
                Spielen ();


            } break;

            
            case (2):
            {
                
                cout << "Das Spiel wird beendet!\n";

            } break;

            // Falsche Eingabe?
                        default:
            {
                cout << "Falsche Eingabe!\n\n";

            }
        }

    } while (Auswahl != 2 ); // Spiel beenden?

    cout << "Bis zum naechsten Mal" << endl;

    return 0;
} // main

void LoescheSpielfeld ()

{
    for (int y=0;y<Höhe;y++)
    {
        for (int x=0;x<Breite;x++)
        {
            Spielfeld[x][y].Besetzt = false;
        }
    }

}

void Spielen ()
{
    
    //Abfrage der Spielernamen
    cout <<"Willkommen im Tic Tac Toe-Multiplayer!"<<endl;
    cout <<"Spieler 1, waehle deinen Namen: " << endl;
    cin.ignore ();
    cin.get(Spieler1.Name, 19);
    cout <<"Spieler 2, waehle deinen Namen: " << endl;
    cin.ignore ();
    cin.get(Spieler2.Name, 19);
    cout << endl;

    //Matchankündigung
    cout <<"Naechstes Match: "<<endl;
    cout << Spieler1.Name <<" vs. "<< Spieler2.Name << endl;
    cout <<endl;

    cout << endl;

    do
    {
    cout << "Los gehts, "<< Spieler1.Name <<" faengt an!"<<endl;
    {
        ZeigeFeld ();
        Spielzug1 ();
        ZeigeFeld ();

    }

    cout << "Jetzt ist "<< Spieler2.Name << " dran!"<<endl;

    {
        Spielzug2 ();
        cout << endl;
        ZeigeFeld ();

    }

    cout << Spieler1.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug1 ();
        cout << endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        
    
    cout << Spieler2.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug2 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        cout << Spieler1.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug1 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        cout << Spieler2.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug2 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        cout << Spieler1.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug1 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        cout << Spieler2.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug2 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

        cout << Spieler1.Name << " ist an der Reihe!"<<endl;

    {
        Spielzug1 ();
        cout<<endl;
        ZeigeFeld ();
        
        GewinnerErmitteln ();
    }

    } while (GewinnerErmittlung.Gewonnen = true);

    // Spiel im Falle von einem Gewinner beenden

    GewinnerErmitteln ();

        

    }




void Spielzug1 ()

{

    int x = 0;
    int y = 0;

    do
    {
        cout << "x-Position (1-"<< Breite << "): ";
        cin >> x;
    } while (x<1 || x>Breite);

        do
    {
        cout << "y-Position (1-"<< Höhe << "): ";
        cin >> y;
    } while (y<1 || y>Höhe);

    if (Spielfeld[x-1][y-1].Besetzt == true || Spielfeld[x-1][y-1].Besetzt2 == true)
        
    {
        cout <<"Das Feld ist schon besetzt. Bitte waehle ein anderes!"<<endl;
        
        do
    {
        cout << "x-Position (1-"<< Breite << "): ";
        cin >> x;
    } while (x<1 || x>Breite);

        do
    {
        cout << "y-Position (1-"<< Höhe << "): ";
        cin >> y;
    } while (y<1 || y>Höhe);

    }

    else

    Spielfeld[x-1][y-1].Besetzt = true;

}

void Spielzug2 ()

{
    int x = 0;
    int y = 0;

    do
    {
        cout << "x-Position (1-"<< Breite << "): ";
        cin >> x;
    } while (x<1 || x>Breite);

        do
    {
        cout << "y-Position (1-"<< Höhe << "): ";
        cin >> y;
    } while (y<1 || y>Höhe);

        if (Spielfeld[x-1][y-1].Besetzt2 == true || Spielfeld[x-1][y-1].Besetzt == true)
        
    {
        cout <<"Das Feld ist schon besetzt. Bitte waehle ein anderes!"<<endl;
        
        do
    {
        cout << "x-Position (1-"<< Breite << "): ";
        cin >> x;
    } while (x<1 || x>Breite);

        do
    {
        cout << "y-Position (1-"<< Höhe << "): ";
        cin >> y;
    } while (y<1 || y>Höhe);

    }

    else

    Spielfeld[x-1][y-1].Besetzt2 = true;

}


void ZeigeFeld ()

{

    
    if (Spielfeld[0][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << " X |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[0][0].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << " O |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
                if (Spielfeld[1][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   | X |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[1][0].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   | O |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
        if (Spielfeld[2][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   | X \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[2][0].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   | O \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

                if (Spielfeld[0][1].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << " X |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[0][1].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << " O |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

                if (Spielfeld[1][1].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   | X |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[1][1].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   | O |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }


                if (Spielfeld[1][2].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   | X \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[1][2].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   | O \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

                if (Spielfeld[2][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << " X |   |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[2][0].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << " O |   |   \n";
            cout << "   |   |   \n";
            
    }

                if (Spielfeld[2][1].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   | X |   \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[2][1].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << " O |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   | O |   \n";
            cout << "   |   |   \n";
            
    }

            if (Spielfeld[2][2].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   | X \n";
            cout << "   |   |   \n";
            
    }
    
            if (Spielfeld[2][2].Besetzt2 == true)
    {
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   | O \n";
            cout << "   |   |   \n";
            
    }

        

}

void GewinnerErmitteln ()

{
    if ( Spielfeld[0][0].Besetzt == true && Spielfeld[1][0].Besetzt == true && Spielfeld[2][0].Besetzt == true || Spielfeld[0][1].Besetzt == true && Spielfeld[1][1].Besetzt == true && Spielfeld[2][1].Besetzt == true || Spielfeld[0][2].Besetzt == true && Spielfeld[1][2].Besetzt == true && Spielfeld[2][2].Besetzt == true || Spielfeld[0][0].Besetzt == true && Spielfeld[1][1].Besetzt == true && Spielfeld[2][2].Besetzt == true || Spielfeld[2][0].Besetzt == true && Spielfeld[1][1].Besetzt == true && Spielfeld[0][2].Besetzt == true)

    {
        GewinnerErmittlung.Gewonnen = true; 
        cout << Spieler1.Name <<" hat gewonnen! Glueckwunsch!";

        main ();
    }

        if ( Spielfeld[0][0].Besetzt2 == true && Spielfeld[0][0].Besetzt2 == true && Spielfeld[2][0].Besetzt2 == true || Spielfeld[0][1].Besetzt2 == true && Spielfeld[1][1].Besetzt2 == true && Spielfeld[2][1].Besetzt2 == true || Spielfeld[0][2].Besetzt2 == true && Spielfeld[1][2].Besetzt2 == true && Spielfeld[2][2].Besetzt2 == true || Spielfeld[0][0].Besetzt2 == true && Spielfeld[1][1].Besetzt2 == true && Spielfeld[2][2].Besetzt2 == true || Spielfeld[2][0].Besetzt2 == true && Spielfeld[1][1].Besetzt2 == true && Spielfeld[0][2].Besetzt2 == true)

    {
        GewinnerErmittlung.Gewonnen = true; 
        cout << Spieler2.Name <<" hat gewonnen! Glueckwunsch!";

        main ();
    }

}

Erstmal 'tschuldigung für die vielen Zeilen;-) So, mein Problem ist
jetzt, dass ich wenn ich den ersten Zug mache alles super funktioniert,
sobald ich jedoch den zweiten Spielzug mache, der erste gelöscht wird!
Da das ein bisschen verwirrend klingt ein kleines Beispiel:

Spieler 1 macht links oben ein Kreuz.

Spieler 2 macht rechts unten einen Kreis.

Und schon jetzt ist nur noch der Kreis und nicht mehr das Kreuz da. Ich
kann ja wohl unmöglich alle Kombinationsmöglichkeiten in die Funktion
ZeigeFeld () packen! Vorher hatte ich es einfacher geregelt, indem ich
für leere Felder Punkte genommen hatte, mit einer einfachen Schleife:

C-/C++-Quelltext

cout << endl;


 


for (int y=0; y<Höhe;y++)


{


for (int x=0; x<Breite;x++)


{


if (Spielfeld[x][y].Besetzt == true)


cout << "X";


if (Spielfeld[x][y].Besetzt2 == true)


cout << "O";


else


cout << ".";


}


cout << endl;


}


}

Das war aber auch net so das wahre... Kann mir jemand sagen wie ich die
Darstellung wie bei mir oben hinbekomme, aber so dass sie funktioniert??

Danke im voraus!

LG

joke

Zum Seitenanfang

Fireball

Alter Hase

Beiträge: 415

Wohnort: Werne

Beruf: Dipl. Inf.

06.09.2013, 15:08

Schon mal debuged?

Achja und noch was am Rande, versuche dich nicht zu wiederholen beim Programmieren. Kapsel die Logik in einer Funktion, die du dann in deiner Schleife immer wieder aufrufst.
(Siehe Spielen())

Dadurch wird dein Programm nicht nur schlanker, sondern auch wartbarer was Fehler betrifft.

Schöne Grüße

fb

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »Fireball« (06.09.2013, 15:13)

Zum Seitenanfang

Dreat

Frischling

Beiträge: 86

Wohnort: Heilbronn

06.09.2013, 15:21

Da dies dein erster Beitrag ist erstmal Herzlich Willkommen

So zu deinem Problem du machst das alles etwas sehr kompliziert, deine Spielzug1 und Spielzug2 Funktion kannst du in eine zusammenfassen da könntest du z.B. per Parameter den Spieler übergeben also:

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12	Spielzug( int spieler) { //dein code } //beim Aufrufen dann Spielzug(1); Spielzug(2);

oder so ähnlich.

Zu deinem eigentlich beschriebenen Problem.

C-/C++-Quelltext

   if (Spielfeld[0][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << " X |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

Wenn Spielfeld[0][0] true ist wird dieses Spielfeld angezeigt

C-/C++-Quelltext

   if (Spielfeld[0][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << " X |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

     //Was passiert wenn beide true sind?
                if (Spielfeld[1][0].Besetzt == true)
    {
            cout << "   |   |   \n";
            cout << "   | X |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "___|___|___\n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            cout << "   |   |   \n";
            
    }

Wenn beide true sind werden mir beide Felder nacheinander ausgegeben. Da du das nicht willst solltest du die Abfrage überdenken.

Mal als kleiner Denkanstoß:

char[9] Felder;

alternativ auch:

char[3][3] Felder;

und dein Feld so zeichnen:

C-/C++-Quelltext

1
2
3

            cout << Felder[0][0] << " | " <<  Felder[0][1] << " | " <<  Felder[0][2] << "\n";
            cout << Felder[1][0] << " | " <<  Felder[1][1] << " | " <<  Felder[1][2] << "\n";
            cout << Felder[2][0] << " | " <<  Felder[2][1] << " | " <<  Felder[2][2] << "\n";

Vielleicht hilfts dir ja

Zum Seitenanfang

Renegade123

Alter Hase

Beiträge: 494

Wohnort: Berlin

Beruf: Certified Unity Developer

06.09.2013, 17:45

Hier noch ein weitere Denkanstoß für die Ermittlung des Gewinners:
Magic Square

Liebe Grüße,
René

Zum Seitenanfang

joke1333

Frischling

Beiträge: 4

07.09.2013, 11:09

Hi,

vielen Dank erstmal für eure vielen Antworten! Ich weiss es ist ziemlich chaotisch, deshalb will ich eigentlich das auch nur noch zum funktionieren bringen und dann mit etwas neuem starten und dass dann mal strukturierter und übersichtlicher machen;-)
Iwie stehe ich aber was diese Aufgabe betrifft vollkommen auf der Leitung, selbst mit euren Erklärungen kapier ich nicht wie ich das machen soll

Falls es jemandem nicht zu viel Arbeit ist, könnte er mir evtl. Schritt für Schritt aufschreiben was zu tun ist? Dann würde ich das am ehesten verstehen...
Ich weiss, schwieriger Fall...

Zum Seitenanfang

joke1333

Frischling

Beiträge: 4

08.09.2013, 12:46

So, ich habe jetzt nochmal ein bisschen dran getüfftelt und folgendes gemacht:

C-/C++-Quelltext

//TIC TAC TOE MULTIPLAYER
//
//
#include <iostream>

using namespace std;

struct S_Spieler
{
    char Name[40];

};

struct S_Gewinner
{
    bool Gewonnen;
    char Name[30];
};



char chFelder[9] = {0, 1, 2, 3, 4, 5, 6, 7, 8};
S_Spieler Spieler1;
S_Spieler Spieler2;
S_Gewinner GewinnerErmittlung;

int select = 0;
int Weristdran = 1;

void GewinnerErmitteln ();
void LoescheSpielfeld ();
void Spielen ();
void ZeigeFeld ();
void Spielzug ();



int main ()
{

    int Auswahl = 0;

    LoescheSpielfeld ();


    do
    {
        // Menü ausgeben
        //
        cout << "Tic Tac Toe Menue" << endl;
        cout << "1 - Spielen" << endl;
        cout << "2 - Beenden" << endl;


        // Eingabe abfragen
        cout << "Auswahl: ";
        cin >> Auswahl;

        // Je nach Eingabe verzweigen
        switch (Auswahl)
        {
            
            case (1):
            {
                  
                Spielen ();


            } break;

            
            case (2):
            {
                
                cout << "Das Spiel wird beendet!\n";

            } break;

            // Falsche Eingabe?
                        default:
            {
                cout << "Falsche Eingabe!\n\n";

            }
        }

    } while (Auswahl != 2 ); // Spiel beenden?

    cout << "Bis zum naechsten Mal" << endl;

    return 0;
} // main

void LoescheSpielfeld ()

{
  
    chFelder[0] = ' ';
    chFelder[1] = ' ';
    chFelder[2] = ' ';
    chFelder[3] = ' ';
    chFelder[4] = ' ';
    chFelder[5] = ' ';
    chFelder[6] = ' ';
    chFelder[7] = ' ';
    chFelder[8] = ' ';


}

void Spielen ()
{
    
    //Abfrage der Spielernamen
    cout <<"Willkommen im Tic Tac Toe-Multiplayer!"<<endl;
    cout <<"Spieler 1, waehle deinen Namen: " << endl;
    cin.ignore ();
    cin.get(Spieler1.Name, 19);
    cout <<"Spieler 2, waehle deinen Namen: " << endl;
    cin.ignore ();
    cin.get(Spieler2.Name, 19);
    cout << endl;

    //Matchankündigung
    cout <<"Naechstes Match: "<<endl;
    cout << Spieler1.Name <<" vs. "<< Spieler2.Name << endl;
    cout <<endl;

    cout << endl;

    do
    {
        Spielzug ();
        cout << endl;

        Spielzug ();
        cout << endl;

        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

    
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();
    

    } while (GewinnerErmittlung.Gewonnen == true);

    // Spiel im Falle von einem Gewinner beenden

    if (Weristdran == 1)
    {
    cout << Spieler1.Name << " hat gewonnen! Glueckwunsch!";
    cout << endl;
    }

    if (Weristdran == 2)
    {
        cout << Spieler2.Name << " hat gewonnen! Glueckwunsch!";
        cout << endl;
    }

}



void Spielzug ()

{

    ZeigeFeld ();
        cout << "Spieler " << Weristdran << " ist am Zug! Bitte Zahl eingeben (1-9): ";
        cin >> select;

        if (select < 1 || select > 9)
        {
            cout << "Ungueltige Eingabe! Die Zahl muss zwischen 1 und 9 liegen!\n";

            Spielzug ();
        }

        if (Weristdran == 1)
        {
            Weristdran = 2;
        }

        else
        {
            Weristdran = 1;
        }

        if (Weristdran == 1)
        {
            chFelder[select] = 'X';
        }
        else if (Weristdran == 2)
        {
            chFelder[select] = 'O';
        }

}




void ZeigeFeld ()

{

    cout << "                           "; 
    cout << chFelder[1] << " | " << chFelder[2] << " | " << chFelder[3] << endl; 
    cout << endl;
    cout << "                           "; 
    cout << "- + - + - " << endl;
    cout << endl;
    cout << "                           "; 
    cout << chFelder[4] << " | " << chFelder[5] << " | " << chFelder[6] << endl;
    cout << endl;
    cout << "                           "; 
    cout << "- + - + - "<< endl;
    cout << endl;
    cout << "                           "; 
    cout << chFelder[7] << " | " << chFelder[8] << " | " << chFelder[9] << endl;


  

}

void GewinnerErmitteln ()

{

    
    if( chFelder[1] == 'X' && chFelder[2] == 'X' && chFelder[3] == 'X' ||
        chFelder[4] == 'X' && chFelder[5] == 'X' && chFelder[6] == 'X' ||
        chFelder[7] == 'X' && chFelder[8] == 'X' && chFelder[9] == 'X' ||
        chFelder[1] == 'X' && chFelder[4] == 'X' && chFelder[7] == 'X' ||
        chFelder[3] == 'X' && chFelder[5] == 'X' && chFelder[7] == 'X' ||
        chFelder[1] == 'X' && chFelder[5] == 'X' && chFelder[9] == 'X' ||
        chFelder[2] == 'X' && chFelder[5] == 'X' && chFelder[8] == 'X' ||
        chFelder[3] == 'X' && chFelder[6] == 'X' && chFelder[9] == 'X' )
    
    {
    
          GewinnerErmittlung.Gewonnen = true;
          Weristdran = 2;
       
    }



    if(
    chFelder[4] == 'O' && chFelder[5] == 'O' && chFelder[6] == 'O' ||
       chFelder[1] == 'O' && chFelder[2] == 'O' && chFelder[3] == 'O' ||
       chFelder[7] == 'O' && chFelder[8] == 'O' && chFelder[9] == 'O' ||
       chFelder[1] == 'O' && chFelder[4] == 'O' && chFelder[7] == 'O' ||
       chFelder[3] == 'O' && chFelder[5] == 'O' && chFelder[7] == 'O' ||
       chFelder[1] == 'O' && chFelder[5] == 'O' && chFelder[9] == 'O' ||
       chFelder[2] == 'O' && chFelder[5] == 'O' && chFelder[8] == 'O' ||
       chFelder[3] == 'O' && chFelder[6] == 'O' && chFelder[9] == 'O')

    {
        GewinnerErmittlung.Gewonnen = true;
        Weristdran = 1;
    }
     
}

Leider funktioniert jetzt das mit dem Gewinner bestimmen nicht...
Woran liegt das schon wieder???;-(

LG

Zum Seitenanfang

abcd00186

Treue Seele

Beiträge: 109

08.09.2013, 13:04

chFelder fängt bei 0 an und endet bei 8.

Bei dir Fängt es aber bei 1 an:

Quellcode

  if( chFelder[1] == 'X' && chFelder[2] == 'X' && chFelder[3] == 'X' ||
        chFelder[4] == 'X' && chFelder[5] == 'X' && chFelder[6] == 'X' ||
        chFelder[7] == 'X' && chFelder[8] == 'X' && chFelder[9] == 'X' ||
        chFelder[1] == 'X' && chFelder[4] == 'X' && chFelder[7] == 'X' ||
        chFelder[3] == 'X' && chFelder[5] == 'X' && chFelder[7] == 'X' ||
        chFelder[1] == 'X' && chFelder[5] == 'X' && chFelder[9] == 'X' ||
        chFelder[2] == 'X' && chFelder[5] == 'X' && chFelder[8] == 'X' ||
        chFelder[3] == 'X' && chFelder[6] == 'X' && chFelder[9] == 'X' )
    
    {

So müsste es gehen:

Quellcode

  if( chFelder[0] == 'X' && chFelder[1] == 'X' && chFelder[2] == 'X' ||
        chFelder[3] == 'X' && chFelder[4] == 'X' && chFelder[5] == 'X' ||
        chFelder[6] == 'X' && chFelder[7] == 'X' && chFelder[8] == 'X' ||
        chFelder[0] == 'X' && chFelder[3] == 'X' && chFelder[6] == 'X' ||
        chFelder[2] == 'X' && chFelder[4] == 'X' && chFelder[6] == 'X' ||
        chFelder[0] == 'X' && chFelder[4] == 'X' && chFelder[8] == 'X' ||
        chFelder[1] == 'X' && chFelder[4] == 'X' && chFelder[7] == 'X' ||
        chFelder[2] == 'X' && chFelder[5] == 'X' && chFelder[8] == 'X' )
    
    {

Zum Seitenanfang

Dreat

Frischling

Beiträge: 86

Wohnort: Heilbronn

08.09.2013, 13:13

Ja ist doch schonmal nicht schlecht aber überdenk deine Schleife nocheinmal

C-/C++-Quelltext

    do
    {
        Spielzug ();
        cout << endl;

        Spielzug ();
        cout << endl;

        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

    
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();

        
        Spielzug ();
        cout << endl;
        GewinnerErmitteln ();
    

    } while (GewinnerErmittlung.Gewonnen == true);

die Schleife wiederholt sich solange "GewinnerErmittlung.Gewonnen" true ist? es wird doch erst auf true gesetzt wenn das Spiel gewonnen ist...klingelts?

So dann der Sinn einer Schleife ist sich wiederholende Anweisungen nur einmal schreiben zu müssen...somit kannst du deine Schleife auf 3 Zeilen kürzen denk mal drüber nach ;D

Edit: @abcd00186 Stimmt hatte ich gar nicht gesehn aber es kompiliert trotzdem ohne Probleme ich hab den ganzen Code in VS 2012 kopiert, aber eigentlich sollte es bei 0 anfangen

Zum Seitenanfang

joke1333

Frischling

Beiträge: 4

08.09.2013, 13:25

Ok, also die if Bedingung bei GewinnerErmitteln hab ich geändert.
Aber Dreat: Deshalb setzte ich ja nach jedem Spielzug die Funktion GewinnerErmitteln () ein.
Diese gibt den Wert true aus, wenn jemand gewonnen hat...
Trotz der Änderung bei GewinnerErmitteln geht es immer noch nicht

Andere Vorschläge?

LG

Zum Seitenanfang

patrick246

Treue Seele

Beiträge: 328

Wohnort: nahe Heilbronn/BW

Beruf: TG Profil Informatik-Schüler

08.09.2013, 13:47

Er meint, dass die Schleife so lange läuft, wie GewinnerErmittlung.Gewonnen true ist. Das ist aber nie der Fall, da es erst auf true gesetzt wird, wenn jemand gewonnen hat. Deine Schleife sagt also aus:

Führe diesen Teil aus
...
Solange es einen Gewinner gibt

Wenn du die Schleife so lange ausführen würdest, solange niemand gewonnen hat, könntest du dir einige Zeilen sparen. Im Grunde genommen besteht die Schleife aus nichts weiter als

	C-/C++-Quelltext
1 2 3	Spielzug (); cout << endl; GewinnerErmitteln ();

Du kannst das ganze in eine Schleife packen und solange ausführen lassen, solange es keinen Gewinner gibt.

	C-/C++-Quelltext
1 2 3 4 5 6	while(!GewinnerErmittlung.Gewonnen) { Spielzug (); cout << endl; GewinnerErmitteln (); }

Zum Seitenanfang

Werbeanzeige

spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Brauche Hilfe bei einem Tic Tac Toe Spiel

Brauche Hilfe bei einem Tic Tac Toe Spiel

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

Quellcode

Quellcode

C-/C++-Quelltext

C-/C++-Quelltext

C-/C++-Quelltext

Ähnliche Themen