Крестики нолики на с++

[Гриб_под_зонтом]

Всем привет
Приступил к написанию крестиков ноликов и столкнулся с двумя проблемами: в функциях win_X() и win_O() программа не остонавливаеться, а в функциях igrok_1() и igrok_2() вообще чёт не то твориться. Код вот:

Код: (cpp) [Выделить]

#include <iostream>
using namespace std;

 char a [3] [3] = {{' ',' ',' '},
                   {' ',' ',' '},
                   {' ',' ',' '}};
   
   void print_a()
     {
for (int i=0; i<3; i++)
  {
              cout<<a[i][0]<<" | "<<a[i][1]<<" | "<<a[i][2];
              if (i<2) {cout<<endl<<"----------";}
              cout<<endl;
           }
     }
   void igrok_1()
   {
  int l;
  cout<<"Ход первого игрока: ";
  L:cin>>l;
  cout<<endl;
  if (l==1 && a[0][0]==' ') {a[0][0]='X';}
  if (l==2 && a[0][1]==' ') {a[0][1]='X';}
  if (l==3 && a[0][2]==' ') {a[0][2]='X';}
  if (l==4 && a[1][0]==' ') {a[1][0]='X';}
  if (l==5 && a[1][1]==' ') {a[1][1]='X';}
  if (l==6 && a[1][2]==' ') {a[1][2]='X';}
  if (l==7 && a[2][0]==' ') {a[2][0]='X';}
  if (l==8 && a[2][1]==' ') {a[2][1]='X';}
  if (l==9 && a[2][2]==' ') {a[2][2]='X';}
  if (l==1 && a[0][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==2 && a[0][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==3 && a[0][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==4 && a[1][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==5 && a[1][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==6 && a[1][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==7 && a[2][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==8 && a[2][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
  if (l==9 && a[2][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto L;}
   }
   void igrok_2()
   {
     int k;
  cout<<"Ход второго игрока: ";
  K:cin>>k;
  cout<<endl;
  if (k==1 && a[0][0]==' ') {a[0][0]='O';}
  if (k==2 && a[0][1]==' ') {a[0][1]='O';}
  if (k==3 && a[0][2]==' ') {a[0][2]='O';}
  if (k==4 && a[1][0]==' ') {a[1][0]='O';}
  if (k==5 && a[1][1]==' ') {a[1][1]='O';}
  if (k==6 && a[1][2]==' ') {a[1][2]='O';}
  if (k==7 && a[2][0]==' ') {a[2][0]='O';}
  if (k==8 && a[2][1]==' ') {a[2][1]='O';}
  if (k==9 && a[2][2]==' ') {a[2][2]='O';}
  if (k==1 && a[0][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==2 && a[0][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==3 && a[0][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==4 && a[1][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==5 && a[1][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==6 && a[1][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==7 && a[2][0]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==8 && a[2][1]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
  if (k==9 && a[2][2]!=' ') {cout<<"Ошибка: поле занято попробуйте ещё раз: "; goto K;}
   }
   int win_X()
   {
   if (a[0][0]=='X' && a[0][1]=='X' && a[0][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[1][0]=='X' && a[1][1]=='X' && a[1][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[2][0]=='X' && a[2][1]=='X' && a[2][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[0][0]=='X' && a[1][0]=='X' && a[2][0]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[0][1]=='X' && a[1][1]=='X' && a[2][1]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[0][2]=='X' && a[1][2]=='X' && a[2][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[0][0]=='X' && a[1][1]=='X' && a[2][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   if (a[2][0]=='X' && a[1][1]=='X' && a[0][2]=='X') {cout<<"Первый игрок выиграл!"<<endl; return 0;}
   }
   int win_O()
   {
   if (a[0][0]=='O' && a[0][1]=='O' && a[0][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[1][0]=='O' && a[1][1]=='O' && a[1][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[2][0]=='O' && a[2][1]=='O' && a[2][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[0][0]=='O' && a[1][0]=='O' && a[2][0]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[0][1]=='O' && a[1][1]=='O' && a[2][1]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[0][2]=='O' && a[1][2]=='O' && a[2][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[0][0]=='O' && a[1][1]=='O' && a[2][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   if (a[2][0]=='O' && a[1][1]=='O' && a[0][2]=='O') {cout<<"Второй игрок выиграл!"<<endl; return 0;}
   }
   int main()
     {
print_a();
igrok_1();
print_a();
win_X();
igrok_2();
print_a();
win_O();
igrok_1();
print_a();
win_X();
igrok_2();
print_a();
win_O();
igrok_1();
print_a();
win_X();
igrok_2();
print_a();
win_O();
igrok_1();
print_a();
win_X();
igrok_2();
print_a();
win_O();
igrok_1();
print_a();
win_X();
cout<<"Ничья!"<<endl;
return 0;
}Буду благодарен за помощь!
P.S извиняюсь за goto ничего лучше придумать не смог

[.ubuntufan]

ты где так писать научился?
может для начала попробуешь почитать литературу по С например?

[alsoijw]

Гриб_под_зонтом, про циклы почитай. Всех типов.

(Нажмите, чтобы показать/скрыть)

Да, жуткий вид у c++ кода

[Гриб_под_зонтом]

С goto разберусь сам, а что насчёт других ошибок?

[alsoijw]

Гриб_под_зонтом, разберись с goto, а потом поговорим. Кстати как программа должна останавливаться, а?

[Гриб_под_зонтом]

ладно сдаюсь: пробовал do while но не знаю как сделать "или" в данном цикле

[alsoijw]

Гриб_под_зонтом,

Код: (vala) [Выделить]

for(var x=0; x<3; x++)
{
    for(var y=0; y<3; y++)
    {
        //Перебор значений
    }
}Не оно?

[peregrine]

(Нажмите, чтобы показать/скрыть)

Гриб_под_зонтом, а если бы у тебя крестики нолики были на поле 25x25 и по 5 в линию ты бы как условия проверки писал?

[Гриб_под_зонтом]

Гриб_под_зонтом,

Код: (vala) [Выделить]

for(var x=0; x<3; x++)
{
    for(var y=0; y<3; y++)
    {
        //Перебор значений
    }
}Не оно?

что означает var?

(Нажмите, чтобы показать/скрыть)

Гриб_под_зонтом, а если бы у тебя крестики нолики были на поле 25x25 и по 5 в линию ты бы как условия проверки писал?

Согласен, код не очень красивый, но что поделать: я новичок)

[peregrine]

Гриб_под_зонтом, там не var, а int надо писать. А var это в общем случае сокращение от слова variable - переменная.

[Lifewalker]

Согласен, код не очень красивый, но что поделать: я новичок)

Новичок - это не оправдание, а повод учиться. В далёком 198забытом году мой преподаватель информатики, глядя как я «лабаю прогу» сказала: «сначала программу нужно придумать в голове, затем записать на бумаге, и только после этого подходить к компьютеру». Этот универсальный рецепт нужно использовать всегда и везде.

[alsoijw]

Гриб_под_зонтом, я пишу на vala, на нем и пример. Делай так как в плюсах. var позволяет не указывать тип переменной.

[.ubuntufan]

лол, вот в каждом треде свою валу пропихнуть пытается.
ты запутываешь человека больше, неужели сложно вместо var int было написать.

[Гриб_под_зонтом]

Гриб_под_зонтом,

Код: (vala) [Выделить]

for(var x=0; x<3; x++)
{
    for(var y=0; y<3; y++)
    {
        //Перебор значений
    }
}Не оно?

Насколько я понимаю нет: на вашем примере мы 3 раза прогоняем перебор вариантов и этим ничего не добиваемся. Возможно я неправильно всё понял,но если так объясните пожалуйста.

[alsoijw]

Гриб_под_зонтом, это всего лишь пример цикла. Для проверки нам нужно просмотреть весь массив на совпадение по вертикали, горизонтали, диагоналям. Например можно так

Код: (vala) [Выделить]

for(var x = 0; x < 3; x++)
{
if(pole[x, 0] != ' ' && pole[x, 0] == pole[x, 1] && pole[x, 1] == pole[x, 2])
{
// Кто то выиграл Кто можно узнать по pole[x, 0]
}
}Сейчас мы просмотрели по горизонтали. Как сделать для вертикали подумай, выложи сюда код. В чем суть - код должен быть понятным. Твой код может работать в разы быстрее, но будет там 1 ошибка - и ты не поймешь где она, а код работать не будет. Если у нас поле будет 4*4, то нужно будет поправить код так

Код: (vala) [Выделить]

for(var x = 0; x < 4; x++)
{
if(pole[x, 0] != ' ' && pole[x, 0] == pole[x, 1] && pole[x, 1] == pole[x, 2] && pole[x, 2] == pole[x, 3])
{
// Кто то выиграл Кто можно узнать по pole[x, 0]
}
}Дальше менять - как кому хочется..ubuntufan, кроме холивара какой ЯП лучше ты ни на что не способен?
Пользователь решил продолжить мысль 05 Октября 2014, 23:13:14:Гриб_под_зонтом, задача - массив 25*25 заполнен случайными символами. Создать алгоритм, который скажет - есть ли в массиве строка состоящая только из одного символа. Код выложить.