log4cppのログ出力内容について(オーバーライド)

Question

C++初心者です。
ほぼ触ったことがないので、細かい部分も教えていただきたいです。

C++を使用して試しにログを出力するプログラムを作成しています。
以下のサイトを参考にlog4Cppで作成しております。

http://qiita.com/hsagae/items/d7b0cf6ee876b636b3b9

以下のソースを実行して「Storm is coming」「This is Base」「This is Derived」
というログを出力させたいです。

理解が乏しく自身をもって言えてないですが、オーバーラードしているので、
問題ないと思っていたのですが、「This is Derived」のメッセージが出力されず、
「This is Base」が2回連続で出力されてしまいます。

オーバーライドの実装の仕方が間違えているのでしょうか。
どの箇所を直せばいいか修正箇所と修正サンプルを教えて
いただけないでしょうか。

以下ソースです。

■ConsoleApplication4.cpp

#include "stdafx.h"
#include "Log.h"
#include "Base.h"
#include "Derived.h"

int _tmain(int argc, _TCHAR* argv[])
{
	log4cpp::Category& logger = Log::getLogger();

	logger.warn("Storm is coming");

	Base* pBase = new Base();
	Base* pDerived = new Derived();

	pBase->printLog();
	pDerived->printLog();

	logger.shutdown();

	delete pDerived;
	delete pBase;

	return 0;
}

■Log.h

#pragma once
#include<log4cpp/Category.hh>
#include<log4cpp/PropertyConfigurator.hh>

class Log
{
private:
	Log();
public:
	~Log();
	static log4cpp::Category& getLogger();
};

■Log.cpp

#include "stdafx.h"
#include "Log.h"
Log::Log()
{
}
Log::~Log()
{
}
log4cpp::Category& Log::getLogger()
{

	std::string initFileName = "log4cpp.properties";
	log4cpp::PropertyConfigurator::configure(initFileName);
	return log4cpp::Category::getRoot();
}

■Base.h

#pragma once

class Base
{
public:
	Base();
	~Base();
	void printLog();
};

■Base.cpp

#include "stdafx.h"
#include "Log.h"
#include "Base.h"

Base::Base()
{
}

Base::~Base()
{
}

void Base::printLog()
{
	log4cpp::Category& logger = Log::getLogger();

	logger.info("This is Base");
}

■Derived.h

#pragma once
#include "Base.h"

class Derived : public Base
{
public:
	Derived();
	~Derived();
	void printLog();
};

■Derived.cpp

#include "stdafx.h"
#include "Log.h"
#include "Derived.h"

Derived::Derived()
{
}


Derived::~Derived()
{
}

void Derived::printLog()
{
	log4cpp::Category& logger = Log::getLogger();

	logger.info("This is Derived");
}

Answer 1

こんにちは。

オーバーライドの実装の仕方が間違えているのでしょうか。

ですね。
printLog()関数が通常の関数ですので、基底クラス経由で呼び出すと普通に基底クラスのprintLog()関数が呼ばれます。
仮想関数にして下さい。Base.hのvoid printLog();をvirtual void printLog();とすれば良いです。
基底クラス側で仮想関数にしておけば、派生クラス側の同じメンバ関数はvirtualを付けても付けなくても仮想関数になります。

Comments

[KTMEI]

理解不足を実感し、今回教えていただいた内容を元にネット検索でも
オーバーライドについて理解がさらに深めることが出来ました。

ありがとうございました。

Answer 2

class Base {
public:
    Base();
    ~Base();
    void printLog(); // ココが間違い。正しくは virtual void printLog();
};

Comments

[KTMEI]

理解不足を実感し、今回教えていただいた内容を元にネット検索でも
オーバーライドについて理解がさらに深めることが出来ました。

ありがとうございました。