Крестики-нолики

[Dima-War]

Доброго времени суток. Хочу написать игру крестики-нолики. Вроде все работает, но есть одна ошибка. Условие конца игры работает неправильно. На скринах показаны ходы компьютера. На первом 3 нолика и в столбце, и в строке. На втором скрине трех ноликов нету вообще. Подскажите, в чем ошибка?
P.S. Условие конца игры полностью переписал из книги, и все равно где то косяк.

#pragma warning(disable:4996);
#include <iostream>
#include <conio.h>
#include <time.h>

using namespace std;

char matrix[3][3]; //игровое поле

void init_matrix(); // инициализируем матрицу
void display_matrix(); // выводим матрицу на экран
void get_player_move(); // ход игрока
void get_comp_move(); // ход компьютера
bool is_nothing(int, int); // проверка правильности хода
char is_over(); // проверка конца игры

int main()
{
	srand(time(NULL));
	setlocale(LC_ALL, "Russian");
	char over = ' ';
	init_matrix();
	do
	{
		system("cls");
		display_matrix();
		/*get_player_move();
		over = is_over();
		if (over != ' ')
		{
			break;
		}*/
		get_comp_move();
		over = is_over();
	}
	while (over == ' ');
	system("cls");
	display_matrix(); // конечное отображение матрицы
	if (over == 'X') // поиск победителя
	{
		cout << "Ты выиграл. " << endl;
	}
	else if (over == 'O')
	{
		cout << "Ты проиграл. " << endl;
	}
	system("pause");
	return 0;
}

void init_matrix()
{
	for (int i = 0; i < 3; i++)
	{
		for (int j = 0; j < 3; j++)
		{
			matrix[i][j] = ' ';
		}
	}
}

void display_matrix()
{
	for (int i = 0; i < 3; i++)
	{
		for (int j = 0; j < 3; j++)
		{
			cout << matrix[i][j] << ' ';
		}
		cout << endl;
	}
}
void get_player_move()
{
	int x, y;
	cout << "Введите координаты: ";
	cin >> x >> y;
	while (!is_nothing(x, y))
	{
		cout << "Неверный ход. " << endl;
		cout << "Введите координаты: ";
		cin >> x >> y;
	}
	matrix[--x][--y] = 'X';
}

void get_comp_move() // ставит О в случайную клетку
{
	int x, y;
	x = rand() % 3 + 1;
	y = rand() % 3 + 1;
	while (!is_nothing(x, y))
	{
		x = rand() % 3 + 1;
		y = rand() % 3 + 1;
	}
	matrix[--x][--y] = 'O';
}

bool is_nothing(int x, int y)
{
	if (x <= 0 || x > 3 || y <= 0 || y > 3) // если введены неверные координаты
	{
		return false;
	}
	return (matrix[--x][--y] == ' ') ? true : false; // если клетка уже занята
}

char is_over()
{
	int i;
	for (i = 0; i < 3; i++) // проверяем строки
	{
		if (matrix[i][0] == matrix[i][1] && matrix[i][0] == matrix[i][2])
		{
			return matrix[i][0];
		}
	}
	for (i = 0; i < 3; i++) // проверяем столбцы
	{
		if (matrix[0][i] == matrix[1][i] && matrix[0][i] == matrix[2][i])
		{
			return matrix[0][i];
		}
	}
	if (matrix[0][0] == matrix[1][1] && matrix[1][1] == matrix[2][2]) // проверяем главную диагональ
	{
		return matrix[0][0];
	}
	if (matrix[0][2] == matrix[1][1] && matrix[1][1] == matrix[2][0]) // проверяем побочную диагональ
	{
		return matrix[0][2];
	}
}

[Igor95]

	for (i = 0; i < 3; i++) // проверяем строки
	{
		if (matrix[i][0] == matrix[i][1] && matrix[i][0] == matrix[i][2])
		{
			return matrix[i][0];
		}
	}
	for (i = 0; i < 3; i++) // проверяем столбцы
	{
		if (matrix[0][i] == matrix[1][i] && matrix[0][i] == matrix[2][i])
		{
			return matrix[0][i];
		}
	}
	if (matrix[0][0] == matrix[1][1] && matrix[1][1] == matrix[2][2]) // проверяем главную диагональ
	{
		return matrix[0][0];
	}
	if (matrix[0][2] == matrix[1][1] && matrix[1][1] == matrix[2][0]) // проверяем побочную диагональ
	{
		return matrix[0][2];
	}

Скобки правильно в логических условиях расставьте.
Например,

if ((matrix[0][2] == matrix[1][1]) && (matrix[1][1] == matrix[2][0])) // проверяем побочную диагональ

[Dima-War]

Сообщение от Igor95

Скобки правильно в логических условиях расставьте.

Проблему это не решило.

[frommars]

http://kde-look.org/content/download...nt=123968&id=2

[Dima-War]

Слишком тяжело для меня. Просто надо, чтобы кто то подсказал что не так с функцией проверки конца игры и все.

[Кащей]

На. как увидиш ошибку отпишись.
(подсказка - смотри на уловие победы, а конкретно на оператор "==" и то как инициализируется матрица)

#include <stdio.h>
char matrix[3][3];

char is_over(void)
{
int i;
for (i = 0; i < 3; i++)
if (matrix[i][0] == matrix[i][1] && matrix[i][0] == matrix[i][2])
return matrix[i][0];

for (i = 0; i < 3; i++)
if (matrix[0][i] == matrix[1][i] && matrix[0][i] == matrix[2][i])
return matrix[0][i];

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

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

return -1;
}

int main(void){
int i, n;char s[2];

for(i = 0; i < 3; i++){
for(n = 0; n < 3; n++){
matrix[i][n] = 'Z';
if(i == 1) matrix[i][n] = 'X';
printf("%s ", &matrix[i][n]);
}
puts("\n");
}
if((s[0] = is_over()) != -1){
s[1] = '\0';
printf("Win = %s\n", s);
}

return 0;
}

ещё подсказка - исключи символ которым иницализируеш матрицу из уловия победы

[Dima-War]

Такс... Ок, я исключил пробел из условия победы, а дальше непонятно что за массив s и какое отношение он имеет к полю игры. В общем не вдупляю что то..

[Кащей]

S - он только для отображения символа как строки с нультерминатором, без него пёрло XXXYYY чтото такое.
Наверно хватило бы и putchar().

[Dima-War]

В общем подскажи, пожалуйста, что мне исправить в моем коде. Уже 3ий день сижу и никак не могу дописать игру, задолбался..

[Кащей]

Ну так ты уловие исправил, и какие изменения?