Question

所以我试图让班级“Herder”继承“Mob”班。但我收到的编译器错误如下：

error: invalid use of incomplete type 'struct Mob'
error: forward declaration of 'struct Mob'

这就是Herder.h的样子：

#ifndef HERDER_H_INCLUDED
#define HERDER_H_INCLUDED

#include <SFML/Graphics.hpp>
#include <cstdlib>
#include "Level.h"
#include "Mob.h"

class Mob;

class Herder : public Mob
{
public:
    //Member functions.
    Herder(Level* level, int x, int y);
    void virtual GameCycle(sf::RenderWindow* App);
    void virtual Die();
    void Roar();
protected:
    float m_RoarCountdown;
    float m_RoarTime;
    float m_Speed;
    bool m_Roaring;
};

#endif // HERDER_H_INCLUDED

确定它必须是导致此问题的class Mob;，我将其删除，但随后我收到以下错误，引用大括号打开的行：

error: expected class-name before '{' token

这实际上是我最初添加前向声明的原因 - 我原以为编译器没有在Mob中识别class Herder : public Mob，所以我想我会转发声明

我不认为这是一个周期性依赖的情况，就像我通过谷歌发现的某些情况一样 - “Mob.h”与Herder类无关。

我试图完全删除#include "Mob.h"并坚持使用前向声明，但这也不起作用 - 我只得到一个错误：

error: invalid use of incomplete type 'struct Mob'

这令人困惑。我之前已经成功地获得了继承的类，这段代码在我之前成功尝试的所有相关方面看起来都很相似。

编辑：以下是Mob.h

的内容

#ifndef MOB_H_INCLUDED
#define MOB_H_INCLUDED

#include <SFML/Graphics.hpp>
#include <cstdlib>
#include "Level.h"

class Level;

class Mob
{
public:
    //Member functions.
    Mob(Level* level, int x, int y);
    float GetX();
    float GetY();
    void SetColor(sf::Color color);
    void virtual GameCycle(sf::RenderWindow* App) = 0;
    void virtual Die() = 0;
    void Draw(sf::RenderWindow* App);
protected:
    float m_X;
    float m_Y;
    bool m_Moving;
    int m_Health;
    sf::Sprite m_Sprite;
    Level* pLevel;
};

#endif // MOB_H_INCLUDED

编辑：以下是“Level.h”文件的内容。请注意，Baby是Mob的子类，与Herder的方式非常相似;两者都遇到了同样的错误。

#ifndef LEVEL_H_INCLUDED
#define LEVEL_H_INCLUDED

#include <SFML/Graphics.hpp>
#include <cstdlib>
#include "Tile.h"
#include "Herder.h"
#include "Baby.h"

class Tile;
class Herder;
class Baby;

/// LEVEL
/// This is the collection of all data regarding a level, including layout, objects, mobs, and story elements.
///

class Level
{
public:
    //Constructor
    Level(int height, int width, std::string name);
    //For obtaining positional data
    int GetHeight();
    int GetWidth();
    std::string GetName();
    sf::Image GetTileImage(int image);
    sf::Image GetMobImage(int image);
    std::vector< std::vector<Tile> >& GetGrid();
    void NewHerder(int x, int y);
    void NewBaby(int x, int y);
    void GameCycle(sf::RenderWindow* App);
    void GraphicsCycle(sf::RenderWindow* App);
private:
    //Size
    int m_Height;
    int m_Width;
    //Spatial coords
    std::string m_Name;
    //The grid of tiles.
    std::vector< std::vector<Tile> > m_Grid;
    //A vector of the images to be used for tiles.
    std::vector<sf::Image> m_TileImages;
    //A vector of the images to be used for tiles.
    std::vector<sf::Image> m_MobImages;
    //The herders
    std::vector<Herder> m_Herders;
    //The babies
    std::vector<Baby> m_Babies;
};

#endif // LEVEL_H_INCLUDED

编辑：先发制人，以下是Tile.h：

的内容

#ifndef TILE_H_INCLUDED
#define TILE_H_INCLUDED

#include <SFML/Graphics.hpp>
#include <cstdlib>
#include "Level.h"

class Level;

/// TILE
/// This is the basic environmental unit in the game
///

class Tile
{
public:
    //Constructor
    Tile(int col, int row, int size, int type, Level* level);
    //For obtaining positional data
    int GetX();
    int GetY();
    int GetRow();
    int GetCol();
    //For obtaining type data
    int GetType();
    //For obtaining string type data
    std::string GetStringType();
    //For changing type data
    void SetType(int type);
    void SetStringType(std::string character);
    //For activities that regularly take place
    void GameCycle();
    //Draws the tile.
    void Draw(sf::RenderWindow* App);
private:
    //The level this tile belongs to.
    Level* m_Level;
    //Size (it's a square!)
    int m_Size;
    //Spatial coords
    int m_X;
    int m_Y;
    //Grid coords
    int m_Row;
    int m_Col;
    //Type
    int m_Type;
    //Visual data
    sf::Sprite m_Tile;
};

#endif // TILE_H_INCLUDED

Answer 1

它是循环依赖（Herder.h包括Level.h，其中包括Herder.h等）。

在Herder.h中，只需替换它：

#include "Level.h"

with：

class Level;

并在Mob.h中执行相同的操作

一般规则是尽可能少地包含头文件（即只包含您需要的头文件）。如果你可以使用前向声明，例如，那么使用它而不是完整的包含。

Answer 2

你遇到的问题是循环依赖，这是一种代码气味。一方面，为了能够从类型派生，编译器必须可以使用基本定义（即编译器需要从中继承的完全定义的类型）。另一方面，您的基类取决于派生类的。

技术答案是转发声明派生类型（以便您可以定义基础），然后定义基础。但你应该真正考虑你在做什么：为什么这两个独立的类型通过继承相关？为什么不一个？或者三个（分担责任）？如果基础依赖于它自己的接口的派生，那似乎表明它们耦合得太高了。重新思考设计。

Answer 3

"Herder.h"和"Level.h"彼此为#include。所以，我认为这个错误来自首先包含的"Herder.h"。它正在变得循环。删除它，看看错误是否消失。

继承错误困境：“无效使用不完整类型”VS“预期类名”

3 个答案: