继承 + 使用对象 (this) 作为参数

Question

编辑：从标题文件中添加代码。

现在我正在做某种模拟，但出现了一些问题。 我有一些像下面这样的课程：

#ifndef CARTYLERIA_H
#define CARTYLERIA_H
#include "cjednostka.h"
#include "cnieruchome.h"
#include "battlewindow.h"

class BattleWindow;

class CArtyleria : public CJednostka, public CNieRuchome <------ error line
{
public:
    CArtyleria();
    void rysuj(BattleWindow *okno);
};

#endif // CARTYLERIA_H

当我尝试编译时，标记行出现错误：

“，”标记之前的预期类名

在我添加指向 BattleWindow 对象的指针之前，一切正常。

用

调用 CArtyleria 方法是个好习惯吗

这个

在 BattleWindow 类对象中？

Cjednostka.h：

 #include "battlewindow.h"

    class CJednostka
    {
    public:
        CJednostka();
        virtual void rysuj (BattleWindow *okno) =0;
}

CNieRuchome.h：

class CNieRuchome
{
public:
    CNieRuchome();
protected:
    int PozycjaX;
    int PozycjaY;
};

和battlewindow.h：

#include <coddzial.h>
#include "main.h"
#include "ctimer.h"
#include "cpotyczka.h"
#include "cjednostka.h"

namespace Ui {
class BattleWindow;
}
class CPotyczka;
class BattleWindow : public QMainWindow
{
    Q_OBJECT

public:
    CPotyczka *potyczka;
    void tworz_oddzial();
    explicit BattleWindow(QWidget *parent = 0);
    ~BattleWindow();
    std::vector <QGraphicsPixmapItem*> items_vector;
    void dodaj_jednostki(std::vector<QString> jednostki, QString frakcja);
    void rysuj_jednostke(QString jednostka, QString frakcja, int x, int y);

private slots:

    void on_now();

private:

    QGraphicsScene *scene;
    Ui::BattleWindow *ui;
    CTimer *timer;


    void rysuj_jednostki(QString frakcja);
    void rysuj_plansze ();
    void rysuj_flagi ();
    void usun_jednostke (QGraphicsPixmapItem *item);
    std::list<CJednostka*> jednostki_do_narysowania;
};

而我想做的是：

STL列表中对象的调用方法std::list<CJednostka*> jednostki_do_narysowania;

现在实现如下：

for(std::list <CJednostka*>::iterator it=jednostki_do_narysowania.begin();it!=jednostki_do_narysowania.end();++it){

        (*it)->rysuj(this);

Answer 1

这段代码有几个问题。

首先，您可能在编译时遇到的问题是因为您缺少像 Martin Broadhurst 在他的回答中注意到的分号。

另外，cjednostka.h 和 battlewindow.h 之间存在循环依赖关系。

由于 cjednostka.h 只有一个接口类，因此删除包含并为 class BattleWindow 添加前向声明，就像您在 cartyleria.h 中所做的那样

下一个问题是您为 CJednostka 创建了一个纯虚拟接口，但它没有虚拟析构函数。这很糟糕，最终会导致通过基指针对对象进行不当销毁。

在 cjednostka.h 中试试这个

#ifndef CJEDNOSTKA_H
#define CJEDNOSTKA_H
class CJednostka
{
public:
  CJednostka();
  virtual ~CJednostka() = default;
  virtual void rysuj (BattleWindow *okno) =0;
};
#endif

我还建议在所有头文件中使用头保护或使用#pragma 一次。只需坚持一个选项，并在每个标题中这样做。

标题的另一个问题是，在其中一个标题中，您将自己的标题包含为#include <coddzial.h> 而不是#include "coodzial.h"，这可能会导致问题，因为 与“”之间存在差异。第一个搜索库包含路径，后者查找与您的项目文件相关的文件，这很可能是您想要的。

最后，在 90% 的场景中，向量优于列表的建议，我强烈感觉您不需要列表。我建议再次检查两者之间的差异。