Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

TIC TAC game C++

chris7522

Bonjour a toutes et a tous ,
J'aurai voulu qu'a chaque debut de partie , la personne qui commence soit différente . J'ai fait une fonction pour que cela soit aleatoire mais j'ai ce message :

chris@chris:~/Bureau/Essai$ g++ -o essai16C -std=c++1z essai16.cpp
essai16.cpp: In function ‘void Input()’:
essai16.cpp:42:16: error: invalid conversion from ‘char (*)()’ to ‘char’ [-fpermissive]
player=gen_random;
^~~~~~~~~~
chris@chris:~/Bureau/Essai$

J'ai les yeux hs pour aujourd'hui et j'ai besoin de votre aide . Ca doit etre simple pour vous , mais je vois pas . Il y avait certainement une autre facon plus simple de le faire .
Merci

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;


char matrix[3][3] = { '1', '2', '3', '4', '5', '6', '7', '8', '9' };
char player;
int count;

char gen_random(){
   char choix[2] = {
      'L', 'T'
   };
   char res;
   for (int i = 0; i < 2; i++)
      res = res + choix[rand() % 2];
   return res;
}

void Draw()                                                                        
{
	cout << "     |     |     " << endl;
	cout << "  " << matrix[0][0]  << "  |  " << matrix[0][1] << "  |  " << matrix[0][2]  << endl;
 
	cout << "_____|_____|_____" << endl;
	cout << "     |     |     " << endl;

	cout << "  " << matrix[1][0]  << "  |  " << matrix[1][1]  << "  |  " <<matrix[1][2]  << endl;

	cout << "_____|_____|_____" << endl;
	cout << "     |     |     " << endl;

	cout << "  " << matrix[2][0] << "  |  " << matrix[2][1] << "  |  " << matrix[2][2] << endl;

	cout << "     |     |     " << endl << endl;
}

void Input()                                                                       //              
 {
        srand(time(NULL));
        player=gen_random;
                                                                                            
	cout << "Its "<<player<<" turn's Enter the number of the field: ";  
	int a;
	cin >> a; 								   
	
	if (a == 1)								   	
		if (matrix[0][0]=='1')                                             
			matrix[0][0] = player;
		else
			{
			cout << "\nInvalid move :-( \n";                           
				count=count-1;                                     
	
			}
	else if (a == 2)
		if (matrix[0][1]=='2')
			matrix[0][1] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
				
			}
	else if (a == 3)
		if (matrix[0][2]=='3')
			matrix[0][2] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
		
	else if (a == 4)
	        if (matrix[1][0]=='4')
			matrix[1][0] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
	else if (a == 5)
		  if (matrix[1][1]=='5')
			matrix[1][1] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
		
	else if (a == 6)
  		if (matrix[1][2]=='6')
			matrix[1][2] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
		
	else if (a == 7)
		  if (matrix[2][0]=='7')
			matrix[2][0] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
		
	else if (a == 8)
		  if (matrix[2][1]=='8')
			matrix[2][1] = player;
		else
			{
			cout << "\nInvalid move :-(  \n";
				count=count-1;
			}
		
	else if (a == 9)
		  if (matrix[2][2]=='9')
			matrix[2][2] = player;
		else
			{
			cout << "\nInvalid move :-( \n";
				count=count-1;
			}
}

void cuser()                                                                      //                                          
{
	if (player == 'L')                                                      //condition 
	{
	 
		player = 'T';                                                   
		
	}

	else
	{	
		player = 'L';
		
	}
}

char Winner()                                                                  
{
	                                                                        
	if (matrix[0][0] == 'L' && matrix[0][1] == 'L' && matrix[0][2] == 'L')
		return 'L';
	if (matrix[1][0] == 'L' && matrix[1][1] == 'L' && matrix[1][2] == 'L')
		return 'L';
	if (matrix[2][0] == 'L' && matrix[2][1] == 'L' && matrix[2][2] == 'L')
		return 'L';

	if (matrix[0][0] == 'L' && matrix[1][0] == 'L' && matrix[2][0] == 'L')
		return 'L';
	if (matrix[0][1] == 'L' && matrix[1][1] == 'L' && matrix[2][1] == 'L')
		return 'L';
	if (matrix[0][2] == 'L' && matrix[1][2] == 'L' && matrix[2][2] == 'L')
		return 'L';

	if (matrix[0][0] == 'L' && matrix[1][1] == 'L' && matrix[2][2] == 'L')
		return 'L';
	if (matrix[2][0] == 'L' && matrix[1][1] == 'L' && matrix[0][2] == 'L')
		return 'L';

	                                                                           
	if (matrix[0][0] == 'T' && matrix[0][1] == 'T' && matrix[0][2] == 'T')
		return 'T';
	if (matrix[1][0] == 'T' && matrix[1][1] == 'T' && matrix[1][2] == 'T')
		return 'T';
	if (matrix[2][0] == 'T' && matrix[2][1] == 'T' && matrix[2][2] == 'T')
		return 'T';

	if (matrix[0][0] == 'T' && matrix[1][0] == 'T' && matrix[2][0] == 'T')
		return 'T';
	if (matrix[0][1] == 'T' && matrix[1][1] == 'T' && matrix[2][1] == 'T')
		return 'T';
	if (matrix[0][2] == 'T' && matrix[1][2] == 'T' && matrix[2][2] == 'T')
		return 'T';

	if (matrix[0][0] == 'T' && matrix[1][1] == 'T' && matrix[2][2] == 'T')
		return 'T';
	if (matrix[2][0] == 'T' && matrix[1][1] == 'T' && matrix[0][2] == 'T')
		return 'T';

	return '/';                                                                  //fun
}

int main()                                                                                     
{
        
	count=0;
	cout << "*****************************************************************************\n";	
	cout << "\n Tic Tac Game  \n\n";
	cout << "*****************************************************************************\n\n";
	Draw();                                                                                 
	while (1)
	{
		count++;                                                                       
 		Input();
		Draw();
		if (Winner() == 'L')                                                      
		{
			cout << "_____________________________RESULT_________________________________________\n\n";
			cout << " Player Lauralee wins! :-)\n\n";
			break;
		}
		else if (Winner() == 'T')                                                       
		{
			cout << "_____________________________RESULT___________________________________________\n\n";
			cout << " Player Théo wins! :-) \n\n";
			break;
		}
		else if (Winner() == '/'&& count==9)                                           
		{
			cout << "_____________________________RESULT___________________________________________\n\n";
			cout << "Draw please try again :-( \n\n";
			break;
		}


		cuser();                                                                      
	}
	
	return 0;

}

pingouinux

Bonjour,
Je crois que c'est :

         player=gen_random();

chris7522

Merci , c'est ca ! je suis aveugle !
Par contre je n'obtiens pas le resultat escompté : Au lieu du 'T' ou du 'L' en retour de ma fonction , j'obtiens un point d'interogation dans un hexagone 🙁

chris7522

Bonjour,
Est ce que ce code gagnerai a etre une classe ?

Nuliel

char gen_random(){
   char choix[2] = {
      'L', 'T'
   };
   char res;
   for (int i = 0; i < 2; i++)
      res = res + choix[rand() % 2];
   return res;
}

res est pas initialisé, et je comprends pas trop le res = res + choix[rand() % 2]; , tu veux pas juste choisir une lettre entre les deux lettres L et T?
Ou alors tu veux utiliser une chaine de caractère mais tu as mis char comme type pour res

chris7522

Je me suis fait aider pour la fonction gen_random . J'ai copié betement ce qu'on m'a proposé sans trop savoir ce qu'il se passait .
Comme je le disais , ce qui m'intéréssait , c'était qu'a chaque nouvelle partie et pour le 1er coup seulement , se soit une personne différente qui commence la partie .
Une fois le 1 er coup terminé , ce serai chacun son tour .

Nuliel

char gen_random(){
   char choix[2] = {
      'L', 'T'
   };
   return choix[rand() % 2];
}

est à mon avis suffisant (pas besoin de boucle, d'ailleurs l'ancienne boucle additionnait les valeurs associées aux caractères, d'où les caractères impossibles à afficher).
Mais cette fonction est appelée à chaque tour alors qu'elle devrait être appelée seulement au début d'une partie, elle devrait donc plutôt être appelée dans le main.
Aussi srand(time(NULL)); ne devrait être appelée qu'une fois, il faut aussi mettre ça dans le main.
Les break à la fin ne sont pas utiles, ils te font sortir de la boucle infinie (ils seraient utiles en cas d'utilisation du case)
La fonction Input() pourrait être très simplifiée, une boucle serait top!

Dis moi quand tu voudras la version avec les modifs

grigouille

Quelque chose comme ceci ?

char gen_random(){
    return rand() % 2 ? 'L' : 'T';
}

chris7522

Il n'existe pas de fonctions aleatoire propre au C++ qui serai plus adapté au lieu de faire du C ?
Du genre random a utiliser avec la bibliotheque du meme nom ?

Nuliel

La fonction rand est largement suffisante, et il n'y aurait pas trop de raison de créer une nouvelle fonction qui fasse exactement la même chose
Après si tu as envie de changer la distribution: https://en.wikipedia.org/wiki/C%2B%2B11#Extensible_random_number_facility (c'est une blague, tu vas sûrement jamais utiliser autre chose qu'une distribution uniforme, ce qui est utilisé par rand)

chris7522

Merci a tous de votre aide !

Nuliel

Tu as réussi à corriger ton code?