C Jogo da Velha em C erro na marcação

Leandro Arnold · 11 de maio de 2018

Olá amigos do fórum

Estou desenvolvendo um jogo da velha em c, e o único problema que tenho é que a condição que criei para verificar se determinada posição do tabuleiro já foi marcada não funciona. Por exemplo se eu marquei a posição 1,1 com o X e se o outro jogador marcar ela, o programa substitui o X pelo O.

Como eu poderia resolver isso? Já tentei colocar a função na int main, mas não funcionou.

Ficarei grato se puderem me dar uma luz.

#include<stdio.h>
#include<stdlib.h>
#include<string.h>

void jogadap1();
void jogadap2();
void checajogada1();
void checajogada2();
void cvitoriap1();
void cvitoriap2();
void exibitab();
void menu();

int linha, coluna, jogadas, nivel, vencedor=0;
char jogodaveia[3][3];
  

void criartab(){//função para criar o tabuleiro
	for(linha=0;linha<=3;linha++){
		for(coluna=0;coluna<=3;coluna++){
			jogodaveia[linha][coluna]='0';
		}
	}
}



void exibitab(){//função para exibir o tabuleiro
	 printf("\n\n");	
	 printf(" %c | %c | %c\n",jogodaveia[0][0],jogodaveia[0][1],jogodaveia[0][2]);
	 printf("---+---+---\n");
     printf(" %c | %c | %c\n",jogodaveia[1][0],jogodaveia[1][1],jogodaveia[1][2]);
     printf("---+---+---\n");
     printf(" %c | %c | %c\n",jogodaveia[2][0],jogodaveia[2][1],jogodaveia[2][2]);
}

void jogadap1(){//jogador 1 faz a jogada
	 printf("\nO jogador 1 faz a jogada");
     printf("\nInforme a linha que voce quer marcar: ");
     scanf("%d",&linha);
                      
     printf("Informe a coluna que voce quer marcar: ");
     scanf("%d",&coluna);
     checajogada1();
     jogodaveia[linha-1][coluna-1]='X';
     cvitoriap1();
     jogadas++;             
}

void jogadap2(){//jogador 2 faz a jogada
	 printf("\nO jogador 2 faz a jogada");
     printf("\nInforme a linha que voce quer marcar: ");
    scanf("%d",&linha);
  printf("Informe a coluna que voce quer marcar: ");
     scanf("%d",&coluna);
     checajogada2();
     jogodaveia[linha-1][coluna-1]='O';
     cvitoriap2();
     jogadas++;
}


void checajogada1(){//checar o tabuleiro para não marcar em uma posição que já foi marcada
	if (jogodaveia[linha][coluna]!='0'){
		printf("\nJogada invalida");
		jogadap1();
	}
}

void checajogada2(){
	if (jogodaveia[linha][coluna]!='0'){
		printf("\nJogada invalida");
		jogadap2();
	}
}

void cvitoriap1(){//Condições de vitória do jogador 1
	if(jogodaveia[0][0]==jogodaveia[0][1]&&jogodaveia[0][1]==jogodaveia[0][2]&&jogodaveia[0][0]=='X'){
      vencedor=1;
    }
    else if(jogodaveia[1][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[1][2]&&jogodaveia[1][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[2][0]==jogodaveia[2][1]&&jogodaveia[2][1]==jogodaveia[2][2]&&jogodaveia[2][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[0][0]==jogodaveia[1][0]&&jogodaveia[1][0]==jogodaveia[2][0]&&jogodaveia[0][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[0][1]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][1]&&jogodaveia[0][1]=='X'){
        vencedor=1;
    }
    else if (jogodaveia[0][2]==jogodaveia[1][2]&&jogodaveia[1][2]==jogodaveia[2][2]&&jogodaveia[0][2]=='X'){
        vencedor=1;
    }
    else if(jogodaveia[0][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][2]&&jogodaveia[0][0]=='X'){
        vencedor=1;
    }
    else if(jogodaveia[0][2]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][0]&&jogodaveia[0][2]=='X'){
        vencedor=1;
    }
    if(vencedor!=0){
     printf("\n\n---------------O vencedor foi o jogador %d !!!---------------\n\n",vencedor);
     printf("\n\n----------------------Parabens!---------------\n\n");
     exit(0);
	}      
}

void cvitoriap2(){//condições de vitória do jogador 2
	if(jogodaveia[0][0]==jogodaveia[0][1]&&jogodaveia[0][1]==jogodaveia[0][2]&&jogodaveia[0][0]=='O'){
        vencedor=2;
                                                                                              
    }
    else if(jogodaveia[1][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[1][2]&&jogodaveia[1][0]=='O'){
	  vencedor=2;
                                                                                              
    }
    else if(jogodaveia[2][0]==jogodaveia[2][1]&&jogodaveia[2][1]==jogodaveia[2][2]&&jogodaveia[2][0]=='O'){
	 vencedor=2;
    } 
    else if(jogodaveia[0][0]==jogodaveia[1][0]&&jogodaveia[1][0]==jogodaveia[2][0]&&jogodaveia[0][0]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if(jogodaveia[0][1]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][1]&&jogodaveia[0][1]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if (jogodaveia[0][2]==jogodaveia[1][2]&&jogodaveia[1][2]==jogodaveia[2][2]&&jogodaveia[0][2]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if(jogodaveia[0][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][2]&&jogodaveia[0][0]=='O'){
	 vencedor=2;
                                                                                            
    }
     else if(jogodaveia[0][2]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][0]&&jogodaveia[0][2]=='O'){
	 vencedor=2;                                                                                   
    }
	 if(vencedor!=0){
     printf("\n\n---------------O vencedor foi o jogador %d !!!---------------\n\n",vencedor);
     printf("\n\n----------------------Parabens!---------------\n\n");
     exit(0);
   }
}
void menu(){
	 printf("\n\n---------------------BEM VINDO AO JOGO DA VEIA 2.0---------------------");
     printf("\n\nEscolha um nivel de dificuldade:\n\n(1) - Jogador vs Jogador \n(2) - Jogador vs PC \n(3) - Jogador vs IPC: \n\n ");
	 scanf("%d",&nivel);
}

int main(){
	menu();
	
	if (nivel == 1) {
		criartab();
		for(jogadas=1; jogadas<=9; jogadas++){
	 		exibitab();
	 		jogadap1();
	 		exibitab();
	 		jogadap2();
	 
   		}
   }
   return 0;
}

Leonardo0308 · 11 de maio de 2018

Bom dia @Leandro Arnold

void checajogada1(){//checar o tabuleiro para não marcar em uma posição que já foi marcada
	if (jogodaveia[linha-1][coluna-1]!='0'){
		printf("\nJogada invalida");
		jogadap1();
	}
}

void checajogada2(){
	if (jogodaveia[linha-1][coluna-1]!='0'){
		printf("\nJogada invalida");
		jogadap2();
	}

Um pequeno erro de logica, você estava verificando a linha errada

Espero ter ajudado.

Leandro Arnold · 11 de maio de 2018

Muito obrigado amigo, nem tinha me dado por conta.

Agora funciona certinho, e vou fazer as condições para o pc jogar.

Abração amigo!

isrnick · 11 de maio de 2018

O problema é que está fazendo uma recursividade maluca, jogadap1() chama checajogada1() que chama jogadap1() que chama checajogada1()... E no fim não altera nada porque quando voltar no retorno das funções ele vai voltar para o primeiro valor de linha e coluna que foram digitados.

Essas funções checajogada1 e checajogada2 deveriam ser do tipo int ao invés de void, e retornar 1 ou 0 (verdadeiro ou falso) dependendo do resultado. E no jogadap1 e jogadap2 deveria ter um if para checar o retorno, e fazer um ciclo para repetir a entrada de dados e fazer o jogador digitar coluna e linha novamente.

Edit: Não vi que as variáveis coluna e linha são globais. Mas a variável jogadas será incrementada (jogadas++) 2 ou mais vezes, dependendo de quantas vezes o jogador digitar uma posição inválida, ao invés de apenas 1.

Leonardo0308 · 11 de maio de 2018

De nada @Leandro Arnold :D

11 de maio de 2018

@Leandro Arnold Olá. Seu jogo está muito bom, mas, como comentado acima, se um jogador fizer uma jogada inválida a sua variável jogada, q é incrementada a cada jogada, vai resultar em um valor errôneo, q não é o esperado.

Outro ponto q mais chama a atenção é a duplicidade de código. Note: Não estou aqui p corrigir ninguém, mas essa é apenas uma sugestão. Perceba q você cria uma função relativamente grande p verificar se o jogador 1 ganhou e depois praticamente faz outra função igual para verificar se o jogador 2 ganhou. Com a solicitação da jogada é a mesma coisa. Creio q, se você quiser, você pode resolver sozinho essa questão

OBS: É apenas uma sugestão para deixar seu código mais organizado.

Longe d mim corrigir alguém, inda mais com relação ao jogo da velha. Lembro do sofrimento q foi p mim criar o 1º jogo da velha. Misericórdia. Seu código está bom demais se comparar com o 1º jogo da velha q fiz!

Como disse: é apenas uma sugestão procurar resolver a duplicidade de código.

Se porventura interessar posso passar um jogo da velha q tenho aqui q usa uma lógica com relação a verificar se ouve ganhador e a solicitação da jogada muito parecida com a sua.

Se acaso interessar posso postar aqui, aí, quem sabe, você vai poder perceber como é fácil resolver a questão da duplicidade d código. Mas, em geral, seu jogo da velha está ótimo!

Leandro Arnold · 15 de maio de 2018

Muito obrigado pela observação Giu, não tinha me dado por conta disso, e o contador segue incrementando as jogadas inválidas, como eu posso arrumar isso no código?

Outro problema que enfrento agora é com relação ao modo jogador x computador. O computador marca o tabuleiro de forma randômica, mas do nada o programa dá erro e trava. Não sei o que está acontecendo.

Agradeço desde já pela atenção amigos.

Abraço!

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<time.h>

void jogadap1();
void jogadap2();
void checajogada1();
void checajogada2();
void cvitoriap1();
void cvitoriap2();
void cempate();
void exibitab();
void computadorjoga();
void menu();

int linha, coluna, jogadas, nivel, vencedor=0;
char jogodaveia[3][3];

void computadorjoga(){//função para criar as jogadas randomicas do pc
	 srand(time(NULL));
	 linha= rand()%3;
	 coluna= rand()%3;
	 
	 if(jogodaveia[linha][coluna]=='0'){//se o espaço do tabuleiro estiver em branco, o computador marca
	 	jogodaveia[linha][coluna]='O';
	 	jogadas++;
	 }
	 else{
	 	computadorjoga();
	 }
}
void criartab(){//função para criar o tabuleiro
	for(linha=0;linha<=3;linha++){
		for(coluna=0;coluna<=3;coluna++){
			jogodaveia[linha][coluna]='0';
		}
	}
}

void exibitab(){//função para exibir o tabuleiro
	 printf("\n\n");	
	 printf(" %c | %c | %c\n",jogodaveia[0][0],jogodaveia[0][1],jogodaveia[0][2]);
	 printf("---+---+---\n");
     printf(" %c | %c | %c\n",jogodaveia[1][0],jogodaveia[1][1],jogodaveia[1][2]);
     printf("---+---+---\n");
     printf(" %c | %c | %c\n",jogodaveia[2][0],jogodaveia[2][1],jogodaveia[2][2]);
}

void jogadap1(){//jogador 1 faz a jogada
	 printf("\nO jogador 1 faz a jogada");
     printf("\nInforme a linha que voce quer marcar: ");
     scanf("%d",&linha);
                      
     printf("Informe a coluna que voce quer marcar: ");
     scanf("%d",&coluna);
     checajogada1();
     jogodaveia[linha][coluna]='X';
     cvitoriap1();
     jogadas++;
	 cempate();             
}

void jogadap2(){//jogador 2 faz a jogada
	 printf("\nO jogador 2 faz a jogada");
     printf("\nInforme a linha que voce quer marcar: ");
     scanf("%d",&linha);
                       
     printf("Informe a coluna que voce quer marcar: ");
     scanf("%d",&coluna);
     checajogada2();
  jogodaveia[linha][coluna]='O';
     cvitoriap2();
     jogadas++;
     cempate();
}


void checajogada1(){//checar o tabuleiro para não marcar em uma posição que já foi marcada
	if (jogodaveia[linha][coluna]!='0'){
		printf("\nJogada invalida");
		jogadap1();
	}
}

void checajogada2(){
	if (jogodaveia[linha][coluna]!='0'){
		printf("\nJogada invalida");
		jogadap2();
	}
}

void cvitoriap1(){//Condições de vitória do jogador 1
	if(jogodaveia[0][0]==jogodaveia[0][1]&&jogodaveia[0][1]==jogodaveia[0][2]&&jogodaveia[0][0]=='X'){
      vencedor=1;
    }
    else if(jogodaveia[1][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[1][2]&&jogodaveia[1][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[2][0]==jogodaveia[2][1]&&jogodaveia[2][1]==jogodaveia[2][2]&&jogodaveia[2][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[0][0]==jogodaveia[1][0]&&jogodaveia[1][0]==jogodaveia[2][0]&&jogodaveia[0][0]=='X'){
       vencedor=1;
    }
    else if(jogodaveia[0][1]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][1]&&jogodaveia[0][1]=='X'){
        vencedor=1;
    }
    else if (jogodaveia[0][2]==jogodaveia[1][2]&&jogodaveia[1][2]==jogodaveia[2][2]&&jogodaveia[0][2]=='X'){
        vencedor=1;
    }
    else if(jogodaveia[0][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][2]&&jogodaveia[0][0]=='X'){
        vencedor=1;
    }
    else if(jogodaveia[0][2]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][0]&&jogodaveia[0][2]=='X'){
        vencedor=1;
    }
    if(vencedor!=0){
     printf("\n\n---------------O vencedor foi o jogador %d !!!---------------\n\n",vencedor);
     printf("\n\n----------------------Parabens!---------------\n\n");
     exit(0);
	}     
}

void cvitoriap2(){//condições de vitória do jogador 2
	if(jogodaveia[0][0]==jogodaveia[0][1]&&jogodaveia[0][1]==jogodaveia[0][2]&&jogodaveia[0][0]=='O'){
        vencedor=2;
                                                                                              
    }
    else if(jogodaveia[1][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[1][2]&&jogodaveia[1][0]=='O'){
	  vencedor=2;
                                                                                              
    }
    else if(jogodaveia[2][0]==jogodaveia[2][1]&&jogodaveia[2][1]==jogodaveia[2][2]&&jogodaveia[2][0]=='O'){
	 vencedor=2;
    } 
    else if(jogodaveia[0][0]==jogodaveia[1][0]&&jogodaveia[1][0]==jogodaveia[2][0]&&jogodaveia[0][0]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if(jogodaveia[0][1]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][1]&&jogodaveia[0][1]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if (jogodaveia[0][2]==jogodaveia[1][2]&&jogodaveia[1][2]==jogodaveia[2][2]&&jogodaveia[0][2]=='O'){
	 vencedor=2;
                                                                                              
    }
     else if(jogodaveia[0][0]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][2]&&jogodaveia[0][0]=='O'){
	 vencedor=2;
                                                                                            
    }
     else if(jogodaveia[0][2]==jogodaveia[1][1]&&jogodaveia[1][1]==jogodaveia[2][0]&&jogodaveia[0][2]=='O'){
	 vencedor=2;                                                                                   
    }
	 if(vencedor!=0){
     printf("\n\n---------------O vencedor foi o jogador %d !!!---------------\n\n",vencedor);
     printf("\n\n----------------------Parabens!---------------\n\n");
     exit(0);
   }
}

void cempate(){
	if((jogadas==9)&&(vencedor==0)){
		printf("\n\n--------------DEU VEIA!---------------\n\n");
		exit(0);
	}
}
void menu(){
	 printf("\n\n---------------------BEM VINDO AO JOGO DA VEIA 2.0---------------------");
     printf("\n\nEscolha um nivel de dificuldade:\n\n(1) - Jogador vs Jogador \n(2) - Jogador vs PC \n(3) - Jogador vs PC Imbativel: \n\n ");
	 scanf("%d",&nivel);
}

int main(){
	menu();

	if (nivel == 1) {
		criartab();
		do{
	 		exibitab();
	 		jogadap1();
	 		exibitab();
	 		jogadap2();
	 		
   		}while(jogadas<=9); 
   }
   if  (nivel==2){
   	    criartab();
   	    do{
   		    exibitab();jogadap1();
   		    exibitab();
   		    computadorjoga();
   		 
   	}while(jogadas<=9);
   	
   }
   return 0;
}

15 de maio de 2018

@Leandro Arnold Olá. Vou procurar primeiro passar um exemplo de um jogo da velha q creio q você vai entender bem.

A meta, com isso, é você ver como é simples resolver o problema de duplicidade de código.

Perceba nesse exemplo q vou passar, q a mesma função para verificar se há ganhador serve tanto para um jogador como para outro. Espero q com esse exemplo possa, de alguma forma, ajudar.

#include <stdio.h>
#include <stdbool.h> // para trabalhar com variáveis do tipo booleana, true e false
#include <windows.h>
#include <string.h>

char matriz[3][3];

void showMatriz();
void initMatriz();
void requestPlay(char simbolo);
bool isComplete();
bool isSequence();
void iniciar();
void readString(char dados[], int tam);

struct jogador
{
    char nome[50];
    char simbolo;
};


void readString(char dados[], int tam)
{
    char ch;

    fgets(dados, tam, stdin);
    strtok(dados, "\n");
    if (strlen(dados) == tam - 1)
    {
        while ((ch = getchar()) != '\n' && ch != EOF);
    }
}

void iniciar()
{
    int indexAtual, qtdeJogadores = 2, i;
    struct jogador j[2];

    system("cls");

    printf("=====================\n");
    printf("==| JOGO DA VELHA |==\n");
    printf("=====================\n\n");

    indexAtual = 0;

    printf("DIGITE O NOME DO JOGADOR 1: ");
    readString(j[indexAtual].nome, 50); // aqui eu criei uma função para ler uma string (nome), pode também fazer uso do printf se preferir
    j[indexAtual].simbolo = 'X'; // aqui atribu0 o símbolo para o jogador

    indexAtual++;

    printf("DIGITE O NOME DO JOGADOR 2: ");
    readString(j[indexAtual].nome, 50);
    j[indexAtual].simbolo = 'O';

    bool finalizado = false;

    char *vencedor = NULL;

    indexAtual = 0;

    initMatriz(); // esvaziar o tabuleiro

    while (!finalizado)
    {
        system("cls"); // para limpar a tela

        showMatriz(); // imprimir o tabuleiro

        printf("E a vez do jogador %s\n", j[indexAtual].nome);

        requestPlay(j[indexAtual].simbolo); // aqui é a jogada, onde passo o símbolo referente a cada jogador como parâmetro

        if (isSequence()) // aqui equivale a fazer isso (isSequence() == true)
        {
            vencedor = j[indexAtual].nome;
            finalizado = true;
        }
        else if (isComplete())
            finalizado = true;

        // aqui o valor de indexAtual fica sendo intercalado entre 0 e 1
        indexAtual = (indexAtual + 1) % qtdeJogadores;
    }

    system("cls");
    showMatriz();

    if (vencedor == NULL) // se vencedor continua como NULL, significa que não houve vencedor, (empate), senão um jogador ganhou
        printf("    O Jogo terminou empatado!\n\n");
    else
        printf("    O Jogador %s ganhou!\n\n", vencedor);

    vencedor = NULL;

    printf("\nFIM DE JOGO!\n\n");
}


int main()
{
    system("title Jogo da Velha");

    char sim[5];

    do
    {
        iniciar();

        printf("\nDeseja jogar novamente? [s/n]: ");
        readString(sim, 5);

    } while (!strcmp("s", sim));

    system("cls");

    printf("\n\n\tObrigado por usar esse programa!\n\n");

    return 0;
}

// aqui é pra limpar o tabuleiro
void initMatriz()
{
    int i, j;

    for (i = 0; i < 3; i++)
    {
        for (j = 0; j < 3; j++)
        {
            matriz[i][j] = ' ';
        }
    }
}

// tabuleiro usado no jogo
void showMatriz()
{
    int i, j, cont;

    printf("\n\n\t");
    for (i = 0; i < 3; i++)
    {
        for (j = 0; j < 3; j++)
        {
            printf("  %c  ", matriz[i][j]);
            if (j < 2)
            {
                printf("%c", -77); // colunas
            }
        }
        printf("\n\t");

        for (cont = 1; cont <= 15; cont++)
        {
            if (i < 2)
            {
                printf("%c", -60); // linhas
                if (cont == 5 || cont == 10)
                {
                    printf("%c", -59); // '+'
                }
            }
        }
        printf("\n\t");
    }

    printf("\n\n");
}

// exemplo de uma tabuleiro mais simples

void ShowTab()
{
    int i;

	printf("\n\n\n");

	for (i = 0; i < 3; i++)
	{
		printf("\t %c | %c | %c ", matriz[i][0], matriz[i][1], matriz[i][2]);

		if (i < 2)
		{
			printf("\n\t---|---|---\n");
		}
	}

	printf("\n\n\n");
}

// aqui é solicitada a jogada para o jogador
void requestPlay(char simbolo)
{
	int jogada = 0, i, j;

	do
	{
		printf("\nLINHA: ");
		scanf("%d", &i);
		fflush(stdin);

		printf("COLUNA: ");
		scanf("%d", &j);
		fflush(stdin);

		if ((i > 3 || i < 1) || (j < 1 || j > 3))
		{
			printf("\nERRO: Jogada invalida!\n");
			jogada = 0;
			continue;
		}
		else
			jogada = 1;

		if (matriz[i - 1][j - 1] != ' ')
		{
			printf("\nERRO: Essa jogada ja foi efetuada!\n");
			jogada = 0;
			continue;
		}
		else
			jogada = 1;

	} while (!jogada);

	matriz[i - 1][j - 1] = simbolo;
}

// aqui é verificado se houve empate
bool isComplete()
{
    int i, j;

    for (i = 0; i < 3; i++)
    {
        for (j = 0; j < 3; j++)
        {
            if (matriz[i][j] == ' ')
            {
                return false;
            }
        }
    }
    return true;
}

// aqui é verificado se algum jogador ganhou
bool isSequence()
{
	// Linhas
	if (matriz[0][0] == matriz[0][1] && matriz[0][1] == matriz[0][2] && matriz[0][0] != ' ')
	{
		return true;
	}
	if (matriz[1][0] == matriz[1][1] && matriz[1][1] == matriz[1][2] && matriz[1][0] != ' ')
	{
		return true;
	}
	if (matriz[2][0] == matriz[2][1] && matriz[2][1] == matriz[2][2] && matriz[2][0] != ' ')
	{
		return true;
	}
	// Colunas
	if (matriz[0][0] == matriz[1][0] && matriz[1][0] == matriz[2][0] && matriz[0][0] != ' ')
	{
		return true;
	}
	if (matriz[0][1] == matriz[1][1] && matriz[1][1] == matriz[2][1] && matriz[0][1] != ' ')
	{
		return true;
	}
	if (matriz[0][2] == matriz[1][2] && matriz[1][2] == matriz[2][2] && matriz[0][2] != ' ')
	{
		return true;
	}
	// Diagonais
	if (matriz[0][0] == matriz[1][1] && matriz[1][1] == matriz[2][2] && matriz[0][0] != ' ')
	{
		return true;
	}
	if (matriz[2][0] == matriz[1][1] && matriz[1][1] == matriz[0][2] && matriz[2][0] != ' ')
	{
		return true;
	}

	return false;
}

Procure entender bem o que está sendo feito no jogo. Caso tiver alguma dúvida fique a vontade para perguntar, ok?

adicionado 3 minutos depois

Perceba q eu faço uso do tipo de dado bool inserindo o arquivo de cabeçalho "stdbool.h". Caso preferir, usar os valores 0 para false e 1 para true

Leandro Arnold · 15 de maio de 2018

Entendi seu código, obrigado por compartilhar.

O que não consigo resolver no meu, é o erro que aparece quando executo o programa para jogar contra o computador.

Às vezes trava, às vezes funciona.

Não sei o que está acontecendo.

isrnick · 15 de maio de 2018

Sua função da jogada do computador sempre calcula uma posição aleatória entre todas as posições possíveis na velha, e quando a posição escolhida já está ocupada ele chama a função novamente, recursivamente.

Mas quanto mais jogadas se passarem menos posições livres existem, logo é menos provável que a posição sorteada seja uma das que ainda estão livres.

O que significa que existe uma grande chance do seu programa ficar chamando a função de jogada do computador recursivamente muitas e muitas vezes sem dar sorte de cair numa posição livre, até que atinge o limite de chamadas recursivas e o programa trava.

Leandro Arnold · 15 de maio de 2018

8 horas atrás, isrnick disse:

Sua função da jogada do computador sempre calcula uma posição aleatória entre todas as posições possíveis na velha, e quando a posição escolhida já está ocupada ele chama a função novamente, recursivamente.

Mas quanto mais jogadas se passarem menos posições livres existem, logo é menos provável que a posição sorteada seja uma das que ainda estão livres.

O que significa que existe uma grande chance do seu programa ficar chamando a função de jogada do computador recursivamente muitas e muitas vezes sem dar sorte de cair numa posição livre, até que atinge o limite de chamadas recursivas e o programa trava.

Então eu devo retirar a recursividade para o problema parar e fazer outra função para o computador não marcar em uma posição que já está marcada?

isrnick · 15 de maio de 2018

Recursividade funciona bem quando se tem um limite finito as chamadas a própria função, mas sua função pode ficar rodando recursivamente indefinidamente, pois depende de valores aleatórios.

Mas eu diria que o problema está na lógica, o modo que eu faria seria sortear apenas as posições livres, tal que nunca sorteia uma posição já ocupada, assim nunca há conflito.

Por exemplo, poderia obter o número de posições ainda vazias na velha, depois obter um número n aleatório entre 1 e o número de posições vazias, aí você preenche a n-ésima posição vazia da matriz (percorre a matriz até contar n posições não preenchidas e põe o 'O' na posição final).

Leandro Arnold · 16 de maio de 2018

Então eu devo criar uma outra matriz para armazenar as posições vazias e marcar o aleatório em cima delas?

isrnick · 16 de maio de 2018

13 minutos atrás, Leandro Arnold disse:

Então eu devo criar uma outra matriz para armazenar as posições vazias e marcar o aleatório em cima delas?

Não, sua matriz já indica casas vazias com o caractere '0', então a posição está "vazia" se tiver o caractere '0' nela.

count = 0;
for(i = 0; i < 3; i++) {
    for(j = 0; j < 3; j++) {
        if (matriz[i][j] == '0') //Se estiver "vazia"
            count++;  //Incrementa o contador
      
        if (count == n)  //Na n-ésima posição "vazia"
            matriz[i][j] = 'O'; //Jogada do Computador
    }
}

Onde n seria um número aleatório entre 1 e a quantidade de posições ainda vazias na matriz ( = 9 - jogadas se o número de jogadas for contado corretamente).

Leandro Arnold · 18 de maio de 2018

Então o código ficaria assim amigo?

void computadorjoga(){//função para criar as jogadas randomicas do pc
int count=0, n;
	 srand(time(NULL));
	 linha= rand()%3;
	 coluna= rand()%3;
	 
	 for(linha=0;linha<=3;linha++){
	 	for(coluna=0;coluna<=3;coluna++){
	      if(jogodaveia[linha][coluna]=='0'){//se o espaço do tabuleiro estiver em branco, o computador marca
	 	     count++;
	       }
	      if(count==n){
	      	jogodaveia[linha][coluna]=='O';
		  }
	 }
   }
}

isrnick · 19 de maio de 2018

Supondo que o número de jogadas esteja correto:

void computadorjoga(){//função para criar as jogadas randomicas do pc
    int i, j, count, n;
    
    n = rand() % (9-jogadas) + 1;
    
    count = 0;
    for(i = 0; i < 3; i++) {
        for(j = 0; j < 3; j++) {
            if (matriz[i][j] == '0') //Se estiver "vazia"
                count++;  //Incrementa o contador
      
            if (count == n){  //Na n-ésima posição "vazia"
                matriz[i][j] = 'O'; //Jogada do Computador
                jogadas++;
                return;
            }
        }
    }
}

Leandro Arnold · 19 de maio de 2018

17 horas atrás, isrnick disse:

Supondo que o número de jogadas esteja correto:


void computadorjoga(){//função para criar as jogadas randomicas do pc
    int i, j, count, n;
    
    n = rand() % (9-jogadas) + 1;
    
    count = 0;
    for(i = 0; i < 3; i++) {
        for(j = 0; j < 3; j++) {
            if (matriz[i][j] == '0') //Se estiver "vazia"
                count++;  //Incrementa o contador
      
            if (count == n){  //Na n-ésima posição "vazia"
                matriz[i][j] = 'O'; //Jogada do Computador
                jogadas++;
                return;
            }
        }
    }
}