C++ 基底クラスを継承、さらに派生クラスを継承について(エラー)

Question

こんにちは。いま継承をうまいようにつくりたくて試行錯誤しているのですがこのようなエラーがたくさん出てしまいました。

C2504 'Character';定義されていない基底クラスが宣言されています。
C3668　'Player::Update': オーバーライド指定子 'override' を伴うメソッドは、基底クラス メソッドをオーバーライドしませんでした

なにが文法的におかしいでしょうか?

/*--------
Object.h
--------*/


class Object
{
private:

public:

 Object()
 {}

 virtual void Initialize()=0;
 virtual void Update()=0;
 virtual void Finalize()=0;
};

/*--------
Character.h
--------*/

#include "Object.h"

class Character :public Object
{
private:

public:

 Character()
 {}

 virtual void Initialize() override
 {}
 virtual void Update() override
 {}
 virtual void Finalize() override
 {}
};

/*--------
Player.h
--------*/
#include "Character.h"


class Player :public Character
{
public:
Player()
 {
   .....
 }

 virtual void Initialize() override
 {
   .....
 }
 virtual void Update() override
  {
   .....
 }
 virtual void Finalize() override
  {
   .....
 }

Comments

[itagagaki]

2番目のコード、#include "Object のダブルクォートが閉じていませんが。

[yukkuri_55]

エラーコードは全部
cppファイルも書き込んでください。

[fana]

コードが３つほど提示されていますが，各々が何というファイル名なのかを明記すべきでしょう．

[int32_t]

質問文で開示されているコードはとりたてて間違いが見当たりません。ほんとうにこのコードでエラーが出ていますか? エラーがでる完全なコードを開示しましょう。

Answer 1

もう一度コードを打ちなおしてみました。
結果はエラーが収まりました。
わからずじまいで終わってしまい申し訳ありませんが、
一応解決したので終わります。
回答いただいた皆様ありがとうございました。

Answer 2

提示されているコードから、
マスク箇所および欠落箇所(Playerの閉じカッコ不足やmain関数等)を正常に補った場合
文法上のエラーもなく正常に実行できます。

では、なぜエラーが発生するかを発生したエラーメッセージから確認すると
Playerクラスの宣言時にCharacterクラスが宣言されていないように見えます。

推測するに
#include "Player.h"が#include "Character.h"より前に行われている事になります。
その状況を再現する最小限の変更としては

// Character.h
// Character.hから"Player.h"をCharacterクラスより前にincludeする
#include "Player.h"

// Player.h
// Player.hからCharacter.hがincludeされていない
// #include "Character.h"

どちらかじゃないかな、という印象です。

Answer 3

...なんの問題もありませんが。

#pragma once
// Object.h

class Object {
public:
    virtual void Initialize() = 0;
    virtual void Update() = 0;
    virtual void Finalize() = 0;
};

#pragma once
// Character.h

#include "Object.h"

class Character :public Object {
public:
    void Initialize() override {}
    void Update() override {}
    void Finalize() override {}
};

#pragma once
// Player.h

#include "Character.h"

class Player : public Character {
public:

    void Initialize() override {}
    void Update() override {}
    void Finalize() override {}
};

// main.cpp
#include <iostream>
#include "Player.h"

int main() {
//  Object o;
  Character c;
  Player p;
}

Comments

[UE4benkyo-]

ありがとうございます。
epistemeさんのコードは
Character.hの関数が純粋仮想関数になっていないのですが、
そこが私のコードの文法のおかしいところですか？

[episteme]

あなたが質問に挙げたコードも純粋仮想関数になっていませんけど?

[UE4benkyo-]

間違いました。
仮想関数でした。

[fana]

大元の Object で virtual と書いている時点でそれは仮想関数である．
それを継承した先では "virtual" と明記してもよいし，しなくてもよい．

[fana]

Initialize() を例にすれば，以下のどれでもいい．

void Initialize(){} //virtual も override もつけない
virtual void Initialize(){} //virtual だけつける
void Initialize() override {} //override だけつける
virtual void Initialize() override {} //virtual と override の両方つける

[episteme]

フォローありがとです > fana
シグニチャが一致していれば自動的に再定義され、"virtual"は不要です。
僕は(回答のとおり)シグニチャの不整合を検出してくれる"override"だけつけるのが好み。

[UE4benkyo-]

そうなんですか。
知りませんでした。教えていただきありがとうございます。

Answer 4

#include "Object

これおかしいですね

Comments

[UE4benkyo-]

回答ありがとうございます。
すみません。
そこコピペが失敗したところです。
直しておきます。

[y_waiwai]

#include "Object"

って、拡張子ないですが、これであってますか？

[fana]

質問が修正されて，どうやら "Object.h" が正しそうだけども…
だとすれば

> #include "Object"

これ，コンパイル時に「そんなファイルねぇよ」とか怒られそうに思うけど…
仮にその名前のファイルがあったとして内容がコンパイルに耐えるものでなければやはりエラーが起こるハズだしなぁ．

[UE4benkyo-]

たびたびすみません。
直しましたのでよろしくお願いします。