Сравнение переменных типа char

[ArmanPrestige]

Добрый день. Пишу консольные крестики и нолики
Делаю функцию, которая проверяет наличие победителя. Возникла проблема с сравнением.
То есть если в ряд Х то функция останавливает игру, и выводит соответсвующее сообщение. Это понятно как бы)))
Но, мое ветвление почему напрочь пропускает такие случаи. то есть даже если игрок выграл, игра продолжается. Может я неверно сравниваю? Подскажите пожалуйста.

if(    (status[1-1]==status[2-1] && status[2-1]==status[3-1]) ||
		(status[1-1]==status[5-1] && status[5-1]==status[9-1]) ||
		(status[1-1]==status[4-1] && status[4-1]==status[7-1]) ||
		(status[4-1]==status[5-1] && status[5-1]==status[6-1]) ||
		(status[7-1]==status[5-1] && status[5-1]==status[3-1]) ||
		(status[7-1]==status[8-1] && status[8-1]==status[9-1]) ||
		(status[2-1]==status[5-1] && status[5-1]==status[8-1]) ||
		(status[3-1]==status[6-1] && status[6-1]==status[9-1])	)

[Stilet]

Может стоит пересмотреть способ проверки?
Например если известны координаты хода (x,y) то:

bool winner(int x,int y){
 for(int i=0;i<3 &&  status[i][y]!=status[x][y],i++) return false;
 for(int i=0;i<3 &&  status[x][i]!=status[x][y],i++) return false;
 if(x==y) for(int i=0;i<3 &&  status[i][i]!=status[x][y],i++) return false;
 if((3-x)==y) for(int i=0;i<3 &&  status[i][i]!=status[3-x][y],i++) return false;

 return true;
}

Т.е. смысл в проходах по параллелям и меридианам с диагоналями карты и выяснениям все ли символы на пути совпадают с поставленным.
Код не проверял, писал с потолка, но думаю как идея он подходит.

[ArmanPrestige]

Спасибо за идею. Буду переделывать тогда под двухмерный массив. А вообще, к примеру есть такое объявление:

char ch1='y';
char ch2='y';

то сравнение их производится также, как и сравнение чисел?

if(ch1==ch2)
{
....
}

так?

[Stilet]

В общем то да.

[ArmanPrestige]

есть массив

#define SIZE 3
test[SIZE][SIZE];

как через цикл проверить совпадение столбцов, строк и диагонали? ничего не додумаю. вручную прописывать все случаи(их 7) можно, но глуповато. а если поменять SIZE на 9 к примеру, то их еще больше. вообщем..нуждаюсь в мыслях.

for(int i = 0; i < SIZE; i++)
   for(int j = 0; j < SIZE; j++)
   {
      if( test[i][j]==test[i][j+1] && test[i][j+1+1] )
   }

это скажем для столбцов, и то не верно я делаю. так как когда j будет равным 1 уже будет ошибка( test[i][3] не существует, массив 3х3)

вообщем, прошу ваших мыслей..

просто ваш код Stilet мне не много не понятен..) никогда не использовал в цикле фор более 1ого условия продолжения цикла..

[waleri]

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

[lexrova871]

Вообще я полагаю нужны будут еще символы для обозначения незаполненных элементов.
Попробуй так:

bool w(char s[3][3])
{
	//строки
	if((s[0][0] == s[0][1] && s[0][0] == s[0][2] && s[0][0] != 0) ||	
	   (s[1][0] == s[1][1] && s[1][0] == s[1][2] && s[1][0] != 0) || 	
	   (s[2][0] == s[2][1] && s[2][0] == s[2][2] && s[2][0] != 0)) 	return true;
	//столбцы
	if((s[0][0] == s[1][0] && s[0][0] == s[2][0] && s[0][0] != 0) ||	
	  (s[0][1] == s[1][1] && s[0][1] == s[2][1] && s[0][1] != 0) ||	
	  (s[0][2] == s[1][2] && s[0][2] == s[2][2] && s[0][2] != 0))	return true;
	//диагональ
	if((s[0][0] == s[1][1] && s[0][0] == s[2][2] && s[0][0] != 0 ) ||	
	  (s[0][2] == s[1][1] && s[0][2] == s[2][0] && s[0][2] != 0 ))	return true;

	return false;
}

...

char status[3][3] = { { 'O', 'O', 0 },
			     { 'X', 'X', 'X'},
			     { 0, 0, 0 } };
if(w(status))
{
   printf("winner!");
}

здесь символом 'O' - игровой ноль, 0 - незаполненный элемент

[ArmanPrestige]

Это все понятно. Это можно написать вручную, если небольшая матрица. А если матрица из N строк и N столбцов? В ручную все писать опять таки? Переформулирую вопрос.:

Есть матрица размером N на N. Как проверить, одинаковы ли элементы матрицы по столбцу, по строке и диагоналям.

Должен ведь существовать какой-то алгоритм. В принципе как меняются индексы я понял. Чтобы по строке прогнать, мы первый индекс не меняем, а инкремментируем только второй(который указывает на столбец). Чтобы прогнать по столбцу, то наоборот. Я просто не знаю как это запихнуть в цикл, и как в теле цикла проводить проверку.

[waleri]

for (int col=0; col < SIZE; ++col) if (array[const][col] == cond) { ... }

[rrrFer]

как через цикл проверить совпадение столбцов, строк и диагонали? ничего не додумаю. вручную прописывать все случаи(их 7) можно, но глуповато. а если поменять SIZE на 9 к примеру, то их еще больше. вообщем..нуждаюсь в мыслях.

не уверен что понял Вас, но если понял:

for( int i = 0; i < n; ++i ) {
    if( 0 == a[ i ][ 0 ] )
        continue; 
    for( int j = 1; j < n; ++j ) 
        if( a[ i ][ j ] != a[ i ][ 0 ] )
            break;
    if( j != n )
        continue;
    //в i-той строке все элементы одинаковы и не равны нулю
}

for( int i = 0; i < n; ++i ) {
    if( 0 == a[ 0 ][ i ] )
        continue; 
    for( int j = 1; j < n; ++j ) 
        if( a[ j ][ i ] != a[ 0 ][ i ] )
            break;
    if( j != n )
        continue;
    //в i-том столбцевсе элементы одинаковы и не равны нулю
    
}

if( 0 != a[ 0 ][ 0 ] )
    for( i = 1; i < n; ++i ) 
        if( a[ i ][ i ] != a[ 0 ][ 0 ] )
            break;
if( i == n )
    //на главной диагонали все элементы равны и...

int t = n - 1
if( 0 != a[ 0 ][ t ] )
    for( i = 1; i < n; ++i ) 
        if( a[ i ][ t - i ] != a[ 0 ][ t ] )
            break;
if( i == n )
    //на побочной диагонали все элементы равны и...

код не проверял, писал без подсветки в браузере, МБ опечатался

а если поменять SIZE на 9 к примеру, то их еще больше. вообщем..нуждаюсь в мыслях.

на поле 9х9 должны быть другие правила игры, там обычно требуется собрать 5 крестиков или ноликов в ряд