Класс "Шахматы" C++

[Darklight]

Тип «Позиция на шахматной доске» определить как класс, содержащий:
- закрытые поля с координатами точки, значения которых будут соответствовать нумерации
шахматного поля;
- все функции реализовать в соответствии с общими требованиями.
Тип «Фигура» определить как класс, содержащий:
- закрытые поля с позицией на шахматной доске соответствующего типа и реализовано в виде
символьной строки произвольной длины поле с названием цвета;
- предусмотреть функцию "ход" на одну позицию в одном из четырех направлений;
- остальные функции реализовать соответственно общим требованиям.
В тестовом примере создать объекты типа «Фигура» соответственно общим
требованиям и вывести информацию о них в полном и сокращенном виде.

[Darklight]

Проблема с классом Queen

#include <iostream>
using namespace std;
enum coord1 { a, b, c, d, e, f, g, h };
enum  color { black, white };
class Figure
{
protected:
    coord1 letter;  //a..h
    int number; //1..8
    color colour;   //цвет
public:
    Figure(coord1 x, int y, color z)
    {
        letter = x;
        number = y;
        colour = z;
    }

    virtual int Turn(coord1 new_letter,int new_number) = 0;
};

class Castle : public virtual Figure {
public:
    int Turn(coord1 new_letter,int new_number) {
        if (((new_letter == letter) &&
            (new_number != number)) ||
            ((new_letter != letter) &&
                (new_number == number))) {
            letter = new_letter;
            number = new_number;
            return 1;
        }
        else return 0;
    }
};

class Bishop : public virtual Figure {
public:
    int Turn(coord1 new_letter,int new_number) {
        if ((abs((new_letter - letter) ==
            abs(new_number - number)) &&
            (new_letter != letter))) {
            letter = new_letter;
            number = new_number; return 1;
        }
        else return 0;
    }
};
class Queen : public Bishop, public Castle {
    int Queen::Turn(coord1 new_letter,int new_number) {
        if (Castle::Turn(coord1 new_letter, int new_number) != 0)
            return 1;
        else if (Bishop::Turn(coord1 new_letter, int new_number) != 0)
            return 1;
        else return 0;
    }
};

int main() {
    Queen q(e, 5, white);
    q.Turn(h, 8);
    system("pause");
    return 0;
}

[macomics]

int Queen::Turn(coord1 new_letter,int new_number) { if ((letter != new_letter || number != new_number) && (new_letter == letter || new_number == number || abs(new_letter - letter) == abs(new_number - number))) {
    letter = new_letter;
    number = new_number;
    return 1;
  } else return 0;
}

[Darklight]

macomics,
Queen::Turn (использование полного имени в объявлении члена не допускается)
И в main тоже есть проблемы

[Алексей1153]

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

class Castle : public virtual Figure {

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

class Bishop : public virtual Figure {

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

class Queen : public FigureBishop, public Castle {

не вижу смысла делать виртуальное наследование (то есть, сильно усложнять всю систему) всего лишь из-за того, что ферзь сочетает ходы ладьи и слона. Убираем всю эту мишуру и просто прописываем ферзю свою ходилку

---

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

protected:
coord1 letter; //a..h
int number; //1..8

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

coord1 new_letter,int new_number

эту пару надлежит объединить в класс и прописать операторы и методы. Будет меньше лапши

------

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

enum class coord1 { a, b, c, d, e, f, g, h };
enum class color { black, white };

енум нужно облагородить

[macomics]

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

macomics,
Queen::Turn (использование полного имени в объявлении члена не допускается)
И в main тоже есть проблемы

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

Для шахмат достаточно хранить массив в 64 байта. 1 бит описывает цвет клетки (0 - белая, 16 - черная), 1 бит описывает цвет фигуры (0 - белая, 8 - черная) и 3 бита описывают тип фигуры (0 - нет, 1 - пешка, 2 - слон, 3 - кобыла, 4 - офицер, 5 - ферзь и 6 - король). Даже остается 3 бита на всякие дополнительные фишки вроде выделения фигуры, подсветки клеток для возможного хода выделенной фигуры или индикатор перетаскивания фигуры.

Городить забор из классов - пустая трата памяти и времени.

[Darklight]

А не могли б вы подправить мне код? Я попытался его изменить как вы сказали. У меня не выходит получить решение(

[Darklight]

Проблема в main подчеркнута
#define _USE_MATH_DEFINES
#include <iostream>
#include <iomanip>
#include <cmath>
#
using namespace std;
enum class coord { a, b, c, d, e, f, g, h };
enum class color { black, white };
class Figure
{
protected:
coord letter; //a..h
int number; //1..8
color colour; //цвет
public:
Figure(coord x, int y, color z)
{
letter = x;
number = y;
colour = z;
}

};

class Castle : public Figure {
public:
int Turn(coord new_letter,int new_number,color) {
if (((new_letter == letter) &&
(new_number != number)) ||
((new_letter != letter) &&
(new_number == number))) {
letter = new_letter;
number = new_number;
return 1;
}
else return 0;
}
};

class Bishop : public Figure {
public:
int Turn(coord new_letter,int new_number,color) {
if ((abs((new_letter == letter) ==
abs(new_number != number)) &&
(new_letter != letter))) {
letter = new_letter;
number = new_number;
return 1;
}
else return 0;
}
};


class Queen: public Figure {
public:
int Turn(coord new_letter, int new_number,color) {
if ((letter != new_letter || number != new_number) &&
(new_letter == letter ||
new_number == number ||
abs(new_letter == letter) ==
abs(new_number - number))) {
letter = new_letter;
number = new_number;
return 1;
}
else return 0;
}
};


int main() {
Queen q(coord::e, 5,color::white);// в этой строчке проблема
q.Turn(coord::h, 8, color::white);
return 0;
};

[Алексей1153]

Darklight, конструктор нужно прокинуть

#include <iostream>
#include <iomanip>
#include <cmath>

enum class coord { a, b, c, d, e, f, g, h };
enum class color { black, white };
class Figure
{
protected:
	coord letter; //a..h
	int number; //1..8
	color colour; //цвет
public:
	Figure(coord x, int y, color z)
	{
		letter = x;
		number = y;
		colour = z;
	}
};

class Castle : public Figure
{
public:
    using Figure::Figure;
    
	int Turn(coord new_letter,int new_number,color) {
		if (((new_letter == letter) &&
			(new_number != number)) ||
		    ((new_letter != letter) &&
			(new_number == number))) {
			letter = new_letter;
			number = new_number;
			return 1;
		}
		else return 0;
	}
};

class Bishop : public Figure
{
public:
    using Figure::Figure;
    
	int Turn(coord new_letter,int new_number,color) {
		if ((abs((new_letter == letter) ==
			    abs(new_number != number)) &&
			(new_letter != letter))) {
			letter = new_letter;
			number = new_number;
			return 1;
		}
		else return 0;
	}
};


class Queen: public Figure
{
public:
    using Figure::Figure;
    
	int Turn(coord new_letter, int new_number,color) {
		if ((letter != new_letter || number != new_number) &&
		    (new_letter == letter ||
			new_number == number ||
			abs(new_letter == letter) ==
			abs(new_number - number))) {
			letter = new_letter;
			number = new_number;
			return 1;
		}
		else return 0;
	}
};


int main()
{
	Queen q(coord::e, 5,color::white);
	q.Turn(coord::h, 8, color::white);
	return 0;
};

[Алексей1153]

координату завернул в класс

#include <iostream>
#include <iomanip>
#include <cmath>

enum class coord { a, b, c, d, e, f, g, h };
enum class color { black, white };

struct s_coord
{
	coord x_letter{};
	int y_number{}; //1..8
	
	friend bool operator==(const s_coord& l,const s_coord& r)
	{
		return l.x_letter==r.x_letter && l.y_number==r.y_number;
	}
	
	friend bool operator!=(const s_coord& l,const s_coord& r)
	{
		return !(l==r);
	}
	
	bool ifSomePartsAreEqual(const s_coord& r) const 
	{
		return (x_letter == r.x_letter || y_number == r.y_number);
	}
	
	bool ifAbsDifferencesAreEqual(const s_coord& r) const 
	{
		return std::abs( ((int)x_letter)-((int)r.x_letter)) == std::abs(y_number-r.y_number);
	}
};

class Figure
{
protected:
	s_coord m_coord;
	color m_color; //цвет
public:
	Figure(const s_coord m_coord, color z)
		:m_coord(m_coord)
		,m_color(m_color)
	{
	}
};

class Castle : public Figure
{
public:
	using Figure::Figure;
	
	bool Turn(const s_coord new_coord)
	{
		if(m_coord!=new_coord)
		{
			if (m_coord.ifSomePartsAreEqual(new_coord))
			{
				m_coord=new_coord;
				return true;
			}
		}
		
		return false;
	}
};

class Bishop : public Figure
{
public:
	using Figure::Figure;
	
	bool Turn(const s_coord new_coord)
	{
		if(m_coord!=new_coord)
		{
			if(m_coord.ifAbsDifferencesAreEqual(new_coord))
			{
				m_coord=new_coord;
				return true;
			}
		}
		
		return false;
	}
};


class Queen: public Figure
{
public:
	using Figure::Figure;
	
	bool Turn(const s_coord new_coord)
	{
		if(m_coord!=new_coord)
		{
			if (m_coord.ifSomePartsAreEqual(new_coord))
			{
				m_coord=new_coord;
				return true;
			}
			
			if(m_coord.ifAbsDifferencesAreEqual(new_coord))
			{
				m_coord=new_coord;
				return true;
			}
		}
		
		return false;
	}
};

int main()
{
	Queen q({coord::e, 5},color::white);
	q.Turn({coord::h, 8});
	return 0;
};