while(cFeld1,cFeld2,cFeld3 != 'X')
{}

so allerdings nicht mehr :

while(cFeld1,cFeld2,cFeld3 != 'X' || cFeld4,cFeld5,cFeld6 != 'X')
{}

Könnt ihr mir bitte helfen?
Danke im Vorhinein.

z.B. ist das das gleiche :

while(cFeld1,cFeld2 != 'X')
//und
while(cFeld1 && cFeld2 != 'X')

Ich Schreib ma eben den kompletten Quelltext, villeicht versteht ihr mich dann. :

/*Tic-Tac-Toe*/
#include <iostream>
#include <conio.h>

using namespace std;

int main()
{
	char cFeld1 = '1',
			 cFeld2 = '2',
			 cFeld3 = '3',
			 cFeld4 = '4',
			 cFeld5 = '5',
			 cFeld6 = '6',
			 cFeld7 = '7',
			 cFeld8 = '8',
			 cFeld9 = '9';

		char cPlayer1 = 'X',
			 cPlayer2 = 'O';

		int iEingabe = 0 ;

	while(cFeld1,cFeld2,cFeld3 != 'X' && cFeld4,cFeld5,cFeld6 != 'X' )		//while(cFeld1,cFeld2,cFeld3 != 'X' || cFeld4,cFeld5,cFeld6 != 'X' || cFeld7,cFeld8,cFeld9 != 'X' || cFeld1,cFeld4,cFeld7 != 'X' || cFeld2,cFeld5,cFeld8 != 'X' || cFeld3,cFeld6,cFeld9 != 'X'|| cFeld1,cFeld5,cFeld9 != 'X' || cFeld3,cFeld5,cFeld7 != 'X'				 ||					cFeld1,cFeld2,cFeld3 != 'O' || cFeld4,cFeld5,cFeld6 != 'O' || cFeld7,cFeld8,cFeld9 != 'O' || cFeld1,cFeld4,cFeld7 != 'O' || cFeld2,cFeld5,cFeld8 != 'O' || cFeld3,cFeld6,cFeld9 != 'O'|| cFeld1,cFeld5,cFeld9 != 'O' || cFeld3,cFeld5,cFeld7 != 'O' )
	{

		/* Zu Player 1 */
		do
		{
			iEingabe = 0 ;

			cout << cFeld1 << " | " << cFeld2 << " | " << cFeld3 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld4 << " | " << cFeld5 << " | " << cFeld6 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld7 << " | " << cFeld8 << " | " << cFeld9 << endl
				 << endl
				 << "Player 1 ist an der Reihe." << endl;

		cin >> iEingabe;
			//	iEingabe = getch();
			system("cls");
		}
		while(iEingabe > 9 || iEingabe < 1 );

		switch(iEingabe)
		{
			case 1:
				cFeld1 = cPlayer1;
				break;

			case 2:
				cFeld2 = cPlayer1;
				break;

			case 3:
				cFeld3 = cPlayer1;
				break;

			case 4:
				cFeld4 = cPlayer1;
				break;

			case 5:
				cFeld5 = cPlayer1;
				break;

			case 6:
				cFeld6 = cPlayer1;
				break;

			case 7:
				cFeld7 = cPlayer1;
				break;

			case 8:
				cFeld8 = cPlayer1;
				break;

			case 9:
				cFeld9 = cPlayer1;
				break;
		}

		/* Zu Player 2 */
		do
		{
			iEingabe = 0 ;

			cout << cFeld1 << " | " << cFeld2 << " | " << cFeld3 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld4 << " | " << cFeld5 << " | " << cFeld6 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld7 << " | " << cFeld8 << " | " << cFeld9 << endl
				 << endl
				 << "Player 2 ist an der Reihe." << endl;

			cin >> iEingabe;
			//iEingabe = getch();
			system("cls");
		}
		while(iEingabe > 9 || iEingabe < 1 );

		switch(iEingabe)
		{
			case 1:
				cFeld1 = cPlayer2;
				break;

			case 2:
				cFeld2 = cPlayer2;
				break;

			case 3:
				cFeld3 = cPlayer2;
				break;

			case 4:
				cFeld4 = cPlayer2;
				break;

			case 5:
				cFeld5 = cPlayer2;
				break;

			case 6:
				cFeld6 = cPlayer2;
				break;

			case 7:
				cFeld7 = cPlayer2;
				break;

			case 8:
				cFeld8 = cPlayer2;
				break;

			case 9:
				cFeld9 = cPlayer2;
				break;
		}

	}
	system("pause>nul");
	return 0;
}

Geht so nicht.

Ist zwar jetzt nur Fleißarbeit, aber es geht so:

while(cFeld != 'X' && cFeld2 != 'X')

while(cFeld1,cFeld2 != 'X') 
//und 
while(cFeld1 && cFeld2 != 'X')

Geht so nicht.

Ist zwar jetzt nur Fleißarbeit, aber es geht so:

while(cFeld != 'X' && cFeld2 != 'X')

Nichts anderes wurde dir oben gesagt: Du sollst && verwenden und Komma funktioniert nicht, weil du jeden Wert einzeln prüfen musst.

while(cFeld1 != 'X' && cFeld2 != 'X')

okay... aber warum geht es trozdem nicht wenn ich es so mache???

while(cFeld1 != 'X' && cFeld2 != 'X' && cFeld3 != 'X' || cFeld4 != 'X' && cFeld5 != 'X' && cFeld6 != 'X' )

dann beendet er nur, wenn entweder in feld 1,2, und 3 ein 'x' ist UND in 4,5 und 6 kein 'O' oder wenn in 4,5 oder 6 ein 'X' ist aber KEINS in 1,2 oder 3 ...

Wenn nirgends ein x steht, dann ist feld1 != 'X' -> true
folglich steht im while(true || true || true) -> true das programm läuft weiter

nehmen wir an feld 1 ist nun X dann ist feld1 != 'X' -> false
folglich steht im while(false||true||true) -> true das programm läuft weiter

nehmen wir an feld 1 und feld 2 ist nun X dann ist feld1 != 'X' -> false
folglich steht im while(false||false||true) -> true das programm läuft weiter

wenn jetzt feld 1, feld 2 und feld 3 = 'X' sind, dann sind sie alle false
folglich steht im while(false || false || false) -> false schleife endet

while((cFeld1 != cPlayer1 || cFeld2 != cPlayer1 || cFeld3 != cPlayer1) && (cFeld4 != cPlayer1 || cFeld5 != cPlayer1 || cFeld6 != cPlayer1))

/*Tic-Tac-Toe*/
#include <iostream>
#include <conio.h>

using namespace std;

int main()
{
	char cFeld1 = '1',
			 cFeld2 = '2',
			 cFeld3 = '3',
			 cFeld4 = '4',
			 cFeld5 = '5',
			 cFeld6 = '6',
			 cFeld7 = '7',
			 cFeld8 = '8',
			 cFeld9 = '9';

		char cPlayer1 = 'X',
			 cPlayer2 = 'O';

		char cjn;

		int iEingabe = 0 ;

		/* Anleitung */
		cout << "Bitte gebe die Zahl des Feldes ein, und bestätige mit der [Enter]-Taste!";
		system("pause>nul");

	do
	{

		/* Zu Player 1 */
		wieder1:
		system("cls");
		do
		{
			iEingabe = 0 ;

			cout << cFeld1 << " | " << cFeld2 << " | " << cFeld3 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld4 << " | " << cFeld5 << " | " << cFeld6 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld7 << " | " << cFeld8 << " | " << cFeld9 << endl
				 << endl
				 << "Player 1 ist an der Reihe." << endl;

		cin >> iEingabe;
			//	iEingabe = getch();
			system("cls");
		}
		while(iEingabe > 9 || iEingabe < 1 );

		switch(iEingabe)
		{
			case 1:
				if(cFeld1 == '1')
				{
					cFeld1 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
					break;

			case 2:
				if(cFeld2 == '2')
				{
					cFeld2 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 3:
				if(cFeld3 == '3')
				{
					cFeld3 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 4:
				if(cFeld4 == '4')
				{
					cFeld4 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 5:
				if(cFeld5 == '5')
				{
					cFeld5 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 6:
				if(cFeld6 == '6')
				{
					cFeld6 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 7:
				if(cFeld7 == '7')
				{
					cFeld7 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 8:
				if(cFeld8 == '8')
				{
					cFeld8 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;

			case 9:
				if(cFeld9 == '9')
				{
					cFeld9 = cPlayer1;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder1;
				}
				break;
		}

		/* Zu Player 2 */
		wieder2:
		system("cls");
		do
		{
			iEingabe = 0 ;

			cout << cFeld1 << " | " << cFeld2 << " | " << cFeld3 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld4 << " | " << cFeld5 << " | " << cFeld6 << endl
				 <<          "--+-" <<          "--+--" << endl
				 << cFeld7 << " | " << cFeld8 << " | " << cFeld9 << endl
				 << endl
				 << "Player 2 ist an der Reihe." << endl;

			cin >> iEingabe;
			//iEingabe = getch();
			system("cls");
		}
		while(iEingabe > 9 || iEingabe < 1 );

		switch(iEingabe)
		{
			case 1:
				if(cFeld1 == '1')
				{
					cFeld1 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 2:
				if(cFeld2 == '2')
				{
					cFeld2 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 3:
				if(cFeld3 == '3')
				{
					cFeld3 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 4:
				if(cFeld4 == '4')
				{
					cFeld4 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 5:
				if(cFeld5 == '5')
				{
					cFeld5 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 6:
				if(cFeld6 == '6')
				{
					cFeld6 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 7:
				if(cFeld7 == '7')
				{
					cFeld7 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 8:
				if(cFeld8 == '8')
				{
					cFeld8 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;

			case 9:
				if(cFeld9 == '9')
				{
					cFeld9 = cPlayer2;
				}
				else
				{
					cout << "Feld schon belegt!";
					system("pause>nul");
					goto wieder2;
				}
				break;
		}

	}
	while((cFeld1 != cPlayer1 || cFeld2 != cPlayer1 || cFeld3 != cPlayer1) && (cFeld4 != cPlayer1 || cFeld5 != cPlayer1 || cFeld6 != cPlayer1) && (cFeld7 != cPlayer1 || cFeld8 != cPlayer1 || cFeld9 != cPlayer1)&& (cFeld1 != cPlayer1 || cFeld4 != cPlayer1 || cFeld7 != cPlayer1)&& (cFeld2 != cPlayer1 || cFeld5 != cPlayer1 || cFeld8 != cPlayer1)&& (cFeld3 != cPlayer1 || cFeld6 != cPlayer1 || cFeld9 != cPlayer1)&& (cFeld3 != cPlayer1 || cFeld5 != cPlayer1 || cFeld7 != cPlayer1)&& (cFeld1 != cPlayer1 || cFeld5 != cPlayer1 || cFeld9 != cPlayer1)&&//Möglichkeiten zu Player 1
		  (cFeld1 != cPlayer2 || cFeld2 != cPlayer2 || cFeld3 != cPlayer2) && (cFeld4 != cPlayer2 || cFeld5 != cPlayer2 || cFeld6 != cPlayer2) && (cFeld7 != cPlayer2 || cFeld8 != cPlayer2 || cFeld9 != cPlayer2)&& (cFeld1 != cPlayer2 || cFeld4 != cPlayer2 || cFeld7 != cPlayer2)&& (cFeld2 != cPlayer2 || cFeld5 != cPlayer2 || cFeld8 != cPlayer2)&& (cFeld3 != cPlayer2 || cFeld6 != cPlayer2 || cFeld9 != cPlayer2)&& (cFeld3 != cPlayer2 || cFeld5 != cPlayer2 || cFeld7 != cPlayer2)&& (cFeld1 != cPlayer2 || cFeld5 != cPlayer2 || cFeld9 != cPlayer2));//Möglichkeiten zu Player 2
	cout << "Player wins" << endl
		 << "Nocheinmal? (j/n)";
	cin >> cjn;
	if(cjn == 'j')
	{
		goto wieder1;
		system("cls");
	}
	else
	{
		return 0;
	}
}

Und ich möchte mich noch einmal bedanken!

Ja ich weiss das der Befehl goto als Schlampig gilt wussta aber nicht direkt, wie ich es vermeiden soll : D

goto könntest du mit Schleifen eliminieren. Du verwendest goto ja immer, wenn ein Feld schon belegt ist. Stattdessen könntest du um die Eingabe für einen Spieler einfach ein while(bFeldBelegt) packen. bFeldBelegt ist ein bool und wird erst false, wenn der Spieler eine gültige Eingabe macht.

Gastschorry schrieb:

arrys wollte ich als name nutzen statt diesem player1 und player2...

Meinst du jetzt, du willst char-Arrays nutzen, um Spielernamen festzuhalten? Das ginge natürlich, du könntest aber auch den wesentlich komfortableren C++-Typen string verwenden. HighLigerBiMBam meinte eher was anderes. Ich habe deinen Code nur ganz kurz überflogen, aber schon die ganzen cFeldx-Variablen könnte man einfach in ein Array packen. Das macht die Sache übersichtlicher und erlaubt z.B. auch das Verarbeiten aller Elemente des Arrays in Schleifen. Mit deinen Einzel-Variablen könntest du sowas nicht.

Gastschorry schrieb:

und funktionen hab ich noch nicht so verstanden : D ...

Dann solltest du dich besser heute als morgen damit beschäftigen. Funktionen sind essentiell wichtig. Sie erlauben, wiederkehrende Funktionalität ohne Redundanz (doppelten Code) zu implementieren. Außerdem behältst du nur die Übersicht, wenn du Funktionaltät in Funktionen (und auch Klassen) unterbringst. Stell dir vor, du hast irgendwann ein Programm von 20000 Zeilen. Wenn die alle untereinandergeklatscht sind, blickst du höchstwahrscheinlich nicht mehr durch. Das Suchen von bestimmten Stellen wird zur Qual. Wenn aber alles in kleinen, handlichen Funktionen von nur wenigen Zeilen verpackt ist, lässt sich das alles wesentlich leichter handhaben.

Kleines Beispiel:

Sagen wir mal, du willst (warum auch immer ) mehrfach Multiplikationen durchführen und das Ergebnis auf der Konsole ausgeben.

So sieht es ohne Funktion aus (jede Menge redundanter Code, unschön):

#include <iostream>

using namespace std;

int main() {
  cout << "Das Produkt von " << 2 << " und " << 3 << " ist " << 2*3 << ".";
  cout << "Das Produkt von " << 6 << " und " << 8 << " ist " << 6*8 << ".";
  cout << "Das Produkt von " << 9 << " und " << 5 << " ist " << 9*5 << ".";
  cout << "Das Produkt von " << 3 << " und " << 6 << " ist " << 3*6 << ".";
  cout << "Das Produkt von " << 4 << " und " << 1 << " ist " << 4*1 << ".";
  cout << "Das Produkt von " << 0 << " und " << 9 << " ist " << 0*9 << ".";
  cout << "Das Produkt von " << 5 << " und " << 4 << " ist " << 5*4 << ".";
  cout << "Das Produkt von " << 5 << " und " << 4 << " ist " << 5*4 << ".";
  cout << "Das Produkt von " << 2 << " und " << 6 << " ist " << 2*6 << ".";
  cout << "Das Produkt von " << 7 << " und " << 7 << " ist " << 7*7 << ".";
}

Und so könnte es mit einer Funktion aussehen:

#include <iostream>

using namespace std;

void druckeProdukt(int a,int b) {
  cout << "Das Produkt von " << a << " und " << b << " ist " << a*b << ".";
}

int main() {
  druckeProdukt(2,3);  
  druckeProdukt(6,8);  
  druckeProdukt(9,5);  
  druckeProdukt(3,6);  
  druckeProdukt(4,1);  
  druckeProdukt(0,9);  
  druckeProdukt(5,4);  
  druckeProdukt(5,4);  
  druckeProdukt(2,6);  
  druckeProdukt(7,7);  
}

Und nun bedenke noch, dass du im letzteren Fall ganz einfach die Funktionalität deines Programms ändern kannst. Wenn z.B. der Text anders sein soll, musst du nur eine Stelle (nämlich die Funktion) ändern, anstatt in 10 Zeilen rumzupfuschen.

Gastschorry schrieb:

naja ich bin froh, das ich es überhaupt hinbekommen hab... zwar nicht ohne hilfe aber egal

Ist doch schon mal was. Übung macht den Meister!