2

所以我有一个 Piece 类应该代表板上的棋子，我计划从中继承另外两个类。但是，我在这样做时遇到了很多问题。这是到目前为止的相关代码。

///
/// PIECE CLASS HERE
/// this is an abstract class from which Barrier and Pawn inherit.

class Piece
{
public:
    Piece(Space* start);
    sf::Shape m_Circle;
protected:
    int m_X;
    int m_Y;
    int m_radius;
    Space* CurrentSpace;
};

Piece::Piece(Space* start):
    m_X(start->GetX()),
    m_Y(start->GetY()),
    m_radius(14),
    CurrentSpace(start)
{}

///
/// BARRIER CLASS HERE
/// these are the white stones that block a player's path

class Barrier : public Piece
{
public:
    Barrier(Space* initial);
    void Move(Space* target, bool isCapturing);
};

Barrier::Barrier(Space* initial)
{
    Piece(initial);
    m_Circle = sf::Shape::Circle((float)m_X, (float)m_Y, (float)m_radius, sf::Color(255, 255, 255));
    Move(initial);
}

void Barrier::Move(Space* target, bool isCapturing)
{
    int xChange = abs(target->GetX() - m_X);
    int yChange = abs(target->GetY() - m_Y);
    m_Circle.Move((float)xChange, (float)yChange);
    CurrentSpace.ToggleOccupied();
    if(!isCapturing)
    {
        (*target).ToggleOccupied();
    }
    CurrentSpace = target;
}

我收到很多我不明白的错误，特别是：

no matching function for call to Piece::Piece()
declaration of 'Piece initial' shadows a parameter
no matching function for call to 'Barrier::Move(Piece&)'
request for member 'ToggleOccupied' in '((Barrier*)this)->Barrier::<anonymous>.Piece::CurrentSpace', which is of non-class type 'Space*'|

作为 C++ 的新手，我不明白这其中有什么问题。我试图构建我的代码，类似于我在学习 C++ 的书中找到的代码，但显然我忽略了一些微妙之处。我尝试调用的所有函数似乎都存在于适当的位置，我认为我用与原型中相同的值来定义它们。

4

4 回答 4

12

第一个错误是由这个引起的：

Barrier::Barrier(Space* initial)
{
    Piece(initial);
    m_Circle = sf::Shape::Circle((float)m_X, (float)m_Y, (float)m_radius, sf::Color(255, 255, 255));
    Move(initial);
}

需要看起来像：

Barrier::Barrier(Space* initial) : Piece(initial)
{
    m_Circle = sf::Shape::Circle((float)m_X, (float)m_Y, (float)m_radius, sf::Color(255, 255, 255));
    Move(initial);
}

基类的构造函数在你的构造函数之前运行（无论如何）——如果它需要初始化列表中的参数，你需要将参数传递给它。

我不确定其他错误，因为我不知道它们发生在哪一行。

于 2011-06-14T17:38:34.040 回答
4

行中的括号

Piece(initial);

被编译器忽略。您正在声明一个与参数同名的变量：

Piece initial;

要使用 初始化Piece基础对象initial，您必须使用成员初始化程序列表：

Barrier::Barrier(Space* initial) : Piece(initial)

此外，该Move函数需要两个参数，但您只传递一个。你忘记了布尔值。

于 2011-06-14T17:39:47.437 回答
1

如果你想初始化超类，你应该这样做：

Barrier(Space* initial): 
    Piece(initial) {
    ...
}

如果您没有显式初始化基类型，编译器将尝试通过调用具有空参数列表的构造函数来初始化它，如下所示：

Barrier(Space* initial):
    Piece() {
    ... 
}

但是由于您没有为 Piece 定义零参数的构造函数，因此您会得到您描述的编译器错误。

于 2011-06-14T17:39:23.363 回答
1

很好的答案，所有。加上最后令人生畏的线

request for member 'ToggleOccupied' in '((Barrier*)this)->Barrier::<anonymous>.Piece::CurrentSpace', which is of non-class type 'Space*'|

是由

CurrentSpace.ToggleOccupied();

你已经声明 CurrentSpace 是一个指针，所以这应该是

CurrentSpace->ToggleOccupied();

于 2011-06-14T17:45:51.960 回答