Не могу найти две ошибки.

[Lunex.08]

Здравствуйте!
Проблема в названии темы.Программа определения цвета шахматного поля.
Код:

#include <iostream.h>

class doska 
{
	private:
		int simvol;
		int number;  
		int polya [7] [7];
	public:
		int vvod ();
		int obrabotka (int q,int w);
		doska ();
		~doska ();
};

doska:: doska ()
{
	int pr_one=0;
	int pr_two=0;
	int cwet=0;
	int cvet [2];
	cvet [0]=1;
	cvet [1]=2;
	for (int x=0;x<63;x++)
	{
		polya [pr_one] [pr_two]= cvet [cwet];
		pr_two++,cvet [cwet++];
		if (cwet >=1)
		{
			cwet=0;	
		}
		if (pr_two>=7)
		{
			pr_one++;
			pr_two=0;
		}
		if (pr_one==7)
		{
			break;
		}
	}
	cout << "Ready!";
}
doska:: ~doska ()
{
 cout << "Delete"; 	   
}

int doska:: vvod ()
{
	cout << "Enter simvol and number:";
	cin >> simvol;
	cin >> number;
	return simvol,number;
}

int doska:: obrabotka (int q,int w)
{
	int color;
	color=polya [q] [w];
	return color;
}

void main ()
{
	doska ww;
	int s,n;
	cout << "Enter simvol and number:";
	cin >> s,n;
	ww.vvod();
	ww.obrabotka(s,n);
	cout << "";
}

[Lunex.08]

Уже с ошибками разобрался)Хоть прога и не корректна.Буду исправлять.Спасибо за внимание!

[Lunex.08]

#include <iostream.h>

class doska 
{
	private:
		int simvol;	 
		int number; 
		int polya [7] [7];
	public:
		int vvod (int one,int two);
		int obrabotka ();
		doska ();
	
};

doska:: doska ()
{
	int pr_one=0;
	int pr_two=0;
	int cwet=0;
	int cvet [2];
	cvet [0]=1;
	cvet [1]=2;
	for (int x=0;x<63;x++)
	{
		polya [pr_one] [pr_two]= cvet [cwet];
		pr_two++,cvet [cwet++];
		if (cwet >=1)
		{
			cwet=0;	
		}
		if (pr_two>=7)
		{
			pr_one++;
			pr_two=0;
		}
		if (pr_one==7)
		{
			break;
		}
	}
	cout << "One second.....";
}

int doska:: vvod (int one,int two)
{
	simvol=one;
	number=two;
	return 0;
}

int doska:: obrabotka ()
{
	int color;
	color=polya [simvol] [number];
	return color;
}

void main ()
{
	doska ww;
	int s,n;
	cout << "Enter simvol and number:";
	cin >> s,n;
	ww.vvod (s,n);
	cout << ww.obrabotka ();
	cout << "";
}

Вот ошибок нет.Но так и не нашел причину некорректности работы программы.Помогите пожалста.

[Sweta]

Это работает. но выводит код цвета 2 или 1, для поля 7х7

#include <iostream.h>
#include <conio.h>
class doska
{
	private:
		int simvol;
		int number;
		int polya [7] [7];
	public:
		int vvod (int one,int two);
		int obrabotka ();
		doska ();
	        void setdoska();
};

doska:: doska ()
{
	//int pr_one=0;
	//int pr_two=0;
	//int cwet=0;
	int cvet [2]={1,2};
       //	cvet [0]=1;
       //	cvet [1]=2;
	//for (int x=0;x<63;x++)
        for (int i=0;i<7;i++)
        for (int j=0;j<7;j++)
         {
         if(((i%2==0)&&(j%2==0))||((i%2!=0)&&(j%2!=0)))
          polya[i][j]= cvet[0];
         else
          polya[i][j]= cvet[1];
         }



	/*{
		polya [pr_one] [pr_two]= cvet [cwet];
		pr_two++,cvet [cwet++];
		if (cwet >=1)
		{
			cwet=0;
		}
		if (pr_two>=7)
		{
			pr_one++;
			pr_two=0;
		}
		if (pr_one==7)
		{
			break;
		}
	}
	cout << "One second....."; */
}

int doska:: vvod (int one,int two)
{
	simvol=one;
	number=two;
	return 0;
}

int doska:: obrabotka ()
{
	int color;
	color=polya [simvol] [number];
	return color;
}

void main ()
{
	doska ww;
	int s,n;
	cout << "Enter simvol and number:";
	cin >> s>>n;
	ww.vvod (s,n);
	cout << ww.obrabotka ();
        getch();
}

[Lunex.08]

Объясните пожалста как работает следующий блок кода:

for (int i=0;i<7;i++)
for (int j=0;j<7;j++)
{
if(((i%2==0)&&(j%2==0))||((i%2!=0)& &(j%2!=0)))
polya[i][j]= cvet[0];
else
polya[i][j]= cvet[1];
}

[Sweta]

for (int i=0;i<7;i++)

-задаем перебор полей по строкам (0, 1 ..6)
т.е сначала установили строку 0
далее

for (int j=0;j<7;j++)

--задаем перебор полей по столбцам данной строки, т.е просматриваем 0, 1 ..6-ю клетку в 0-й строке, при этом анализируем

if(((i%2==0)&&(j%2==0))||((i%2!=0)& &(j%2!=0)))

, т.е если индекс клетки (строка-i делится без остатка на 2 и столбец j делится без остатка на 2), т.е в четных строках выбираем четные клетки, или для нечетных строк выбираем нечетные клетки(строка-i не делится без остатка на 2 и столбец j не делится без остатка на 2), цвету присваиваем значение 1, иначе присваиваем значение 2.
После этого значение i увеличивается на 1, т.е. устанавливается 1-я строка и все повторяется сначала и так будет продолжаться пока i не станет равным 7.

[Chea]

Как то сложновато.

По поводу кода lunex

Ошибка скорей всего в том что вы массив объявляли 7*7=49
а в цикле обрабатывали 64 значения - переполнение ведет к ошибкам


По поводу заполнения поля

h=1;
for (i=0;i<8;i++)
{
for (j=0;j<8;j++)
{
polya[i][j]= cvet[h];
h=1-h;
}
h=1-h;
}

при условии что массив 8х8

[Lunex.08]

Как то сложновато.

По поводу кода lunex

Ошибка скорей всего в том что вы массив объявляли 7*7=49
а в цикле обрабатывали 64 значения - переполнение ведет к ошибкам


По поводу заполнения поля
h=1;
for (i=0;i<8;i++)
{
for (j=0;j<8;j++)
{
polya[i][j]= cvet[h];
h=1-h;
}
h=1-h;
}

при условии что массив 8х8

Как раз с индексом все правильно.Если бы инициализация начиналась с 1,тогда вы были бы правы.Но щас инициализация идет с 0,включая его самого.В итоге получаем 8.Все элементарно ватцен

[Lunex.08]

Sweta,спасибо за помощь и дохотчивость.Все понятно и кратко)+ к репутации

[netrino]

Сообщение от Lunex.08

Как раз с индексом все правильно.Если бы инициализация начиналась с 1,тогда вы были бы правы.Но щас инициализация идет с 0,включая его самого.В итоге получаем 8.Все элементарно ватцен

Между прочим Chea дал верное замечание ) У Вас двумерный массив размерностью 7х7, то есть в сумме 49 ячеек. Единственная причина, по которой не происходит переполнение - это break при pr_one == 7. Счётчик в цикле никогда не достигнет значения 63, более того, если хотите заполнять массив 8х8, то счётчик должен быть ограничен не как x < 63, а как x < 64 или x <= 63.