[C] odnalezienie błędu

[stachoo0 0]

prosze Was o pomoc

/* jest to gra w kółko i krzyżyk dla 2 graczy jednak nie rozpoznaje od razu wygranej a dopiero po wykonaniu ! 10 ! ruchu */ 

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <conio.h>

#define true 1
#define false 0

typedef int boolean;

char znak;
char pole[3][3];
int x,y,liczba,licznik;
boolean wygrana;

void wyswietl()
{
 printf("  X\n\n");
 printf("0 1 2 \n\n");
 printf("%c|%c|%c 0\n",pole[0][0],pole[1][0],pole[2][0]);
 printf("-+-+-\n");
 printf("%c|%c|%c 1   Y\n",pole[0][1],pole[1][1],pole[2][1]);
 printf("-+-+-\n");
 printf("%c|%c|%c 2\n",pole[0][2],pole[1][2],pole[2][2]);


}

void czyszczenie()
{
 for(x=0; x<=2; x++) pole[x][0]=' ';
 for(x=0; x<=2; x++) pole[x][1]=' ';
 for(x=0; x<=2; x++) pole[x][2]=' ';
}	

void sprawdzenie()
{
 if((pole[0][0]=='O') && (pole[0][1]=='O') && (pole[0][2]=='O')) wygrana=true;		  
 if((pole[1][0]=='O') && (pole[1][1]=='O') && (pole[1][2]=='O')) wygrana=true;
 if((pole[2][0]=='O') && (pole[2][1]=='O') && (pole[2][2]=='O')) wygrana=true;

 if((pole[0][0]=='O') && (pole[1][0]=='O') && (pole[2][0]=='O')) wygrana=true;		  
 if((pole[0][1]=='O') && (pole[1][1]=='O') && (pole[2][1]=='O')) wygrana=true;
 if((pole[0][2]=='O') && (pole[1][2]=='O') && (pole[2][2]=='O')) wygrana=true;

 if((pole[0][0]=='O') && (pole[1][1]=='O') && (pole[2][2]=='O')) wygrana=true;		  
 if((pole[2][0]=='O') && (pole[1][1]=='O') && (pole[0][2]=='O')) wygrana=true;


 if((pole[0][0]=='X') && (pole[0][1]=='X') && (pole[0][2]=='X')) wygrana=true;		  
 if((pole[1][0]=='X') && (pole[1][1]=='X') && (pole[1][2]=='X')) wygrana=true;
 if((pole[2][0]=='X') && (pole[2][1]=='X') && (pole[2][2]=='X')) wygrana=true;

 if((pole[0][0]=='X') && (pole[1][0]=='X') && (pole[2][0]=='X')) wygrana=true;		  
 if((pole[0][1]=='X') && (pole[1][1]=='X') && (pole[2][1]=='X')) wygrana=true;
 if((pole[0][2]=='X') && (pole[1][2]=='X') && (pole[2][2]=='X')) wygrana=true;

 if((pole[0][0]=='X') && (pole[1][1]=='X') && (pole[2][2]=='X')) wygrana=true;		  
 if((pole[2][0]=='X') && (pole[1][1]=='X') && (pole[0][2]=='X')) wygrana=true;
}

void gra(int nr)
{
 /* printf(" KOLKO I KRZYZYK\n\n" \n"); */
/* printf(" GRACZ 1: KOLKO\n GRACZ 2: KRZYZYK\n\n"); */
 wyswietl();
 printf("\n");	
 printf(" Ruch GRACZA %d. Podaj wspolrzedne pola: \n",nr);
 printf(" Podaj X: ");
 scanf("%d",&x);
 printf(" Podaj Y: ");
 scanf("%d",&y);
 if (pole[x][y]==' ') pole[x][y]=znak; else {printf("\n TO POLE JEST JUŻ ZAJĘTE, WYBIERZ INNE");}
}


int main()
{
 znak='O';	
 liczba=1;
 licznik=0;
 wygrana=false;
 czyszczenie();
 do
 {
	 gra(liczba);
	 system("cls");

	 if(liczba==1)
	 {
		 liczba=2;
		 znak='X';
	 }
	 else
	 {
		 liczba=1;
		 znak='O';
	 }  
	 sprawdzenie();
	 licznik++;			  
 }
 while ( (licznik<9) || (wygrana==false) );
 printf(" KOLKO I KRZYZYK\n\n");
 wyswietl();
 if (wygrana==true)
 {
	if(liczba==1) liczba=2; else liczba=1;
	printf("\n WYGRAL GRACZ %d !!!",liczba);
 }
 else printf("\nREMIS!");
 getch();
}

[W2K 0]

prosze Was o pomoc

...

No chyba sobie żartujesz.... to ma być jakies sudoku czy coś ? Sugeruję wywalenie tematu... chyba że autor poda na czym polega problem z kodem i co nie działa albo co wpisuje... A tak swoją drogą co to za programista który nie potrafi znaleźć błedu we własnym kodzie ?

[stachoo0 0]

1 linijka kodu tam jest cały problem opisany ..

 

dopiero zaczynam C i nie znam jeszcze dobrze.

[W2K 0]

dopiero zaczynam C i nie znam jeszcze dobrze.

A co to za tłumaczenie ?

 

To właśnie przez szukanie błędów we własnym kodzie człowiek się uczy najlepiej. Włączaj debuger i przeszukuj wartości zmiennych linijka po linijce w czasie wykonanywania kodu...

[tomb 0]

Zamień || na &&

 

EDIT [15 minut później]: I co? Zamieniłeś?

[stachoo0 0]

Zamień || na &&

 

EDIT [15 minut później]: I co? Zamieniłeś?

tak tak

działa juz poprawnie dzięki :D

[bartek161616 0]

OMG... Ci początkujący....

[Zwolejo 0]

Fajny program szkoda tylko ze jak daje na ta sama pozycje co już była to nic się nie dzieje a kolejka przechodzi dalej

[Dakorn 0]

Nie czytałem kodu, jednak jak chcesz mieć nie powtarzanie ruchu, dodaj sobie pewnie tablice, która będzie trzymała informację co było, a co nie ;p a później to już tylko sprawdzanie.