1

我正在尝试通过使用接口来创建一个非常模块化的程序。来自 C# 背景，我会将接口用作变量类型，这样我就可以使用多态性，允许我自己/其他人将从该接口继承的许多不同对象传递给函数/变量。但是，尝试在 C++ 中执行此操作时，我遇到了许多奇怪的错误。我在这里做错了什么？

我希望能够拥有接口类型的变量。但是，以下会产生编译错误。我认为编译器认为我的 ErrorLogger 类是抽象的，因为它继承自抽象类或其他东西。

ILogger * errorLogger = ErrorLogger();

error C2440: 'initializing' : cannot convert from 'automation::ErrorLogger' to 'automation::ILogger *'

如果我以错误的方式解决这个问题，即使是在设计方面，我也在学习，并且很乐意听取任何和所有建议。

ILogger.h：

#ifndef _ILOGGER_H_
#define _ILOGGER_H_

namespace automation
{
    class ILogger
    {
    public:
        virtual void Log(const IError &error) = 0;
    };
}
#endif

错误记录器.h：

#ifndef _ERRORLOGGER_H_
#define _ERRORLOGGER_H_
#include "ILogger.h"
#include "IError.h"

/* Writes unhandled errors to a memory-mapped file.
 * 
**/

namespace automation
{
    class ErrorLogger : public ILogger
    {
    public:
        ErrorLogger(const wchar_t * file = nullptr, const FILE * stream = nullptr);
        ~ErrorLogger(void);
        void Log(const IError &error);
    };
}
#endif

错误记录器.cpp：

#include "stdafx.h"
#include "ErrorLogger.h"
#include "IError.h"

using namespace automation;

ErrorLogger::ErrorLogger(const wchar_t * file, const FILE * stream)
{

}

void ErrorLogger::Log(const IError &error)
{
    wprintf_s(L"ILogger->ErrorLogger.Log()");
}

ErrorLogger::~ErrorLogger(void)
{
}

错误.h：

#ifndef _IERROR_H_
#define _IERROR_H_

namespace automation
{
    class IError
    {
    public:
        virtual const wchar_t *GetErrorMessage() = 0;
        virtual const int &GetLineNumber() = 0;
    };
}
#endif

编译错误：

在此处输入图像描述

谢谢，-弗朗西斯科

4

2 回答 2

3

ILogger * errorLogger = ErrorLogger(); errorLogger是一个指针，你需要用new operator.

定义指向派生类的基指针的正确方法是：

automation::ILogger * errorLogger = new automation::ErrorLogger();
//                                  ^^^^

在现代 C++ 中更好地使用智能指针：

#include <memory>
std::unique_ptr<automation::ILogger> errorLoggerPtr(new automation::ErrorLogger());

您还需要包含IError.h在ILogger.h

#include "IError.h"

其他建议：

1 在 cpp 文件中使用fstream而不是FILE 2 在 cpp 文件中使用std::wstring而不是wchar_t * 2，不要调用

using namespace automation;

而是用命名空间包装函数定义，就像你在头文件中所做的那样：

namespace automation
{
    ErrorLogger::ErrorLogger(const std::wstring& file, std::ofstream& stream)
    {
    }
}

关键是不要将 C++ 代码与 C 代码混合，C++ 类如 string，fstream 提供RAII，它更安全，更易于使用。

于 2013-07-12T23:34:27.830 回答
0

你需要#include "IError.h"在ILogger.h头文件中。

于 2013-07-12T23:09:26.160 回答