[C] gioco del Tris

[Filly95]

Salve a tutti, ho scritto un piccolo gioco del Tris in C, ma ho riscontrato questi problemi:
[RISOLTO] 1. Quando si prova a mettere una pedina su una casella occupata il numero dei turni non diminuisce (pur avendo messo count_turni--).
[RISOLTO]2. A fine partita non funziona la parte in cui si chiede se si vuole iniziare una nuova partita.
[RISOLTO] 3. In caso di parità non viene eseguito il printf relativo, penso che la variabile risultato non venga settata correttamente a 3 perché credo di aver sbagliato qualcosa nel controllo parità nella funzione check_vincita.
Per ora non mi viene altro in mente, ringrazio tutti in anticipo.
Il mio codice è questo: -.
Uso come editor NotePad++ e come compilatore Dev-C++.

EDIT: Ho risolto due problemi, mi manca l'ultimo, ma proprio non ne vengo a capo. Inoltre vorrei chiedervi se c'è qualche modo di snellire il codice, soprattutto nella funzione check_vincita. Pensavo che

if(schema[7] == schema[8] == schema[9] == 'X' || schema[4] == schema[5] == schema[6] == 'X' || schema[1] == schema[2] == schema[3] == 'X') r = 1;
...

ovvero delle eguaglianze multiple funzionasser, ma mi sono sbagliato.
Grazie ancora.

[!fazz]

le eguaglianze devi farle 2 a 2 ovvero cella1 ='X' && cella2=='X' ecc ecc

ma se vuoi snellire il codice puoi fare questo trucchetto

considera i char come intero (cast a int) somma i 3 valori e controlla se la somma vale 264 (per le X che corrispondono in intero a 88)

[Stewino]

Per riavviare ho provato a inserire

nuova=getch();

invece di

scanf("%c", &nuova);

e funziona.
Ovviamente devi aggiungere la libreria conio.h.
Ho notato che dopo il riavvio rimangono le pedine della partita precedente.

Bel programma comunque

[Filly95]

Originariamente inviato da !fazz

le eguaglianze devi farle 2 a 2 ovvero cella1 ='X' && cella2=='X' ecc ecc

ma se vuoi snellire il codice puoi fare questo trucchetto

considera i char come intero (cast a int) somma i 3 valori e controlla se la somma vale 264 (per le X che corrispondono in intero a 88)

Considerando che è un esercizio di terza (che ho fatto per diletto però) e i cast non sono stati ancora affrontati, penso che terrò questo codice. Strano perché la soluzione che avevo ideato l'avevo cercata, e su un altro source l'avevo vista. Comunque grazie per il trucchetto.

Originariamente inviato da Stewino

Per riavviare ho provato a inserire

nuova=getch();

invece di

scanf("%c", &nuova);

e funziona.
Ovviamente devi aggiungere la libreria conio.h.
Ho notato che dopo il riavvio rimangono le pedine della partita precedente.

Bel programma comunque

Grazie, avevo già provato getch, e andava. A casa a sto punto sostituisco scanf con getch, e metto anche l'inizializzazione dell'array schema dentro il do while al posto che fuori dal main, così ogni volta si resetta. Ora però voglio capire perché scanf non va... Qualche idea?

[Stewino]

Praticamente lo scanf legge ogni tipo di carattere digitato da tastiera, quindi anche l'invio.

Per risolvere questo problema basta mettere

scanf("%s", &nuova);

se noti ho sostituito la 'c' con una 's'.
Oppure utilizzi il getch.

[Filly95]

EDIT: Cercando su Google ho trovato che "%c" non dovrebbe tenere conto del \n ("%s" invece sì) ma nel caso del do while lo fa. Come si risolve? Ho trovato qui che mettendo "%c\n" dovrebbe andare a posto.

[Stewino]

Se utilizzi %c verrà letto anche l'invio.
Es. digito 's' e premo invio, il programma legge il carattere 's' e /n, la variabile sarà uguale a 's' con /n.

Se utilizzi \n%c l'invio verrà scartato.
Es. digito 's' e premo invio, il programma legge il carattere 's' e /n, scarta /n, quindi la variabile sarà uguale a 's'.

[Filly95]

Sisi, ora funziona tutto! Stavo spiegando a mio nonno come giocare e nel frattempo mi sono accorto che mancava anche l'azzeramento (o meglio, l'inizializzazione a 1) dei turni quando si ricomincia una partita
Comunque questo è il codice sorgente finale, se a qualcuno mai sarà utile: http://codepad.org/n73qX3nc.
Un'altra cosa carina da implementare sarebbe l'uscita in qualsiasi momento premendo esc ma non so/ho voglia di implementarla

[Stewino]

Sarebbe bello farlo con una veste grafica, solo che non so come si fa, a quest'ora l'avrei già fatto

[Filly95]

Sicuramente il c da solo non basta, ci vuole qualche libreria... Però l'esercizio era realizzarlo in console, ce l'ho fatta in 8 ore e ne sono fiero

[tecno789]

sei grande. Fare giochi in C è davvero complicato!!

[Phantom1989]

Ciao a tutti ragazzi, sono nuovo del forum. Io ho prodotto una mia versione del gioco del tris ed è funzionante in Code::Blocks 10.05. Questo è il codice:

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define DIM 3
#define CPU 1
#define PLAYER 0
void stampa(char matrice[DIM][DIM]); //stampa il tabellone ogni volta che un giocatore inserisce una propria pedina
int inserimento (char matrice[DIM][DIM], int stato); //inserisce la pedina del giocatore corrispondente
void vittoria (char matrice[DIM][DIM], char ); //controlla se si è verificato un tris e decreta il vincitore
int main()
{
    int status=0, i=0;
    char matrix[DIM][DIM]={'.','.','.','.','.','.','.','.','.'}; //inizializzazione di matrice (ho 2 warning per questo)
    printf("Il programma permette di giocare a tris con il computer\n");
    srand(time(NULL));
    status=rand()%2; //testa o croce fra il giocatore e la CPU
    stampa(matrix);
    i=0;
    while(i<10) //si possono occupare solo 9 caselle
    {
        status=inserimento(matrix, status);
        i++;
    }
    printf("La partita contro la CPU e' finita pari\n");
    return 0;
}

void vittoria (char matrice[DIM][DIM], char pedina)
{
    int i=0, j=0, flag=0;
    for(i=0;i<DIM;i++) //controllo per righe della matrice
    {
        flag=0;
        for(j=0;j<DIM;j++)
        {
            if(matrice[i][j]==pedina)
            flag++;
        }
        if(flag==3)
        {
            stampa(matrice);
            printf("Giocatore '%c' HAI FATTO TRIS! SEI IL VINCITORE!\n", pedina);
            exit(EXIT_SUCCESS);
        }
    }
    for(j=0;j<DIM;j++) //controllo per colonne della matrice
    {
        flag=0;
        for(i=0;i<DIM;i++)
        {
            if(matrice[i][j]==pedina)
            flag++;
        }
        if(flag==3)
        {
            stampa(matrice);
            printf("Giocatore '%c' HAI FATTO TRIS! SEI IL VINCITORE!\n", pedina);
            exit(EXIT_SUCCESS);
        }
    }
    if(matrice[0][0]==pedina && matrice[1][1]==pedina && matrice[2][2]==pedina) //controllo delle diagonali della matrice
    {
        stampa(matrice);
        printf("Giocatore '%c' HAI FATTO TRIS! SEI IL VINCITORE!\n", pedina);
        exit(EXIT_SUCCESS);
    }
    if(matrice[0][2]==pedina && matrice[1][1]==pedina && matrice[2][0]==pedina)
    {
        stampa(matrice);
        printf("Giocatore '%c' HAI FATTO TRIS! SEI IL VINCITORE!\n", pedina);
        exit(EXIT_SUCCESS);
    }
    stampa(matrice);
}

void stampa(char matrice[DIM][DIM])
{
    int i=0, j=0;
    char righe[DIM]={'A','B','C'}, colonne[DIM+1]={' ','1','2','3'};
    for(i=0;i<DIM+1;i++)
    {
        printf("%c ", colonne[i]);
    }
    printf("\n");
    for(i=0;i<DIM;i++)
    {
        printf("%c ", righe[i]);
        for(j=0;j<DIM;j++)
        {
            printf("%c ", matrice[i][j]);
        }
        printf("\n");
    }
}

int inserimento (char matrice[DIM][DIM], int stato)
{
    char xy[DIM]={'\0','\0','\0'}, sign='\0';
    int i=0,j=0;
    printf("Mossa?<inserire coordinate>\n");
    if(stato==CPU)
    {
        sign='O'; //assegna un segno distinguibile ad ogni giocatore
        do
        {
            srand(time(NULL));
            i=rand()%3; //la CPU deve scegliere casualmente una riga ed una colonna da 0 a 3
            srand(time(NULL));
            j=rand()%3;
        }
        while(matrice[i][j]!='.'); //controlla che la casella non sia occupata da un altra pedina
        matrice[i][j]=sign; //assegna alla casella libera il segno del giocatore corrente
        vittoria(matrice, sign); //rimanda al controllo del tris
        stato=PLAYER; //passa la mano all'avversario
        return stato; //ritorna il turno del giocatore corrispondente
    }
    else
    {
        sign='X';
        do
        {
            gets(xy); //il giocatore deve inserie un carattere compreso fra A,B,C per le righe e 1,2,3 per le colonne
            i=xy[0]%65; //traduce le lettere in indici della matrice
            j=xy[1]%49; //mette in scala i numeri inseriti per le colonne
        }
        while(matrice[i][j]!='.');
        matrice[i][j]=sign;
        vittoria(matrice, sign);
        stato=CPU;
        return stato;
    }
}

Devo ammettere che però non sono molto soddisfatto del risultato perché nella funzione "vittoria" ci sono troppe 'if' e troppe righe di codice nel complesso (ovviamente dovrei ottimizzare il programma passando i parametri delle funzioni con i puntatori). La funzione 'srand()' mi blocca il pc e non so per quale motivo. Vorrei sapere cosa ne pensate e se mi potete dare qualche dritta per ottimizzarlo. Grazie a tutti!