如何向类添加错误记录器

Question

我想使用通用错误消息处理程序，这样我就可以为任何类轻松地 GetLastError 和 SetError 。我想出了这个方案。但我有几个问题。请注意，此实现仅用于测试。但我想让基本设计正确。

#include <iostream>
#include <stdarg.h>

class ErrorHandler
{
    public:
        virtual void SetError(const char* zFormat, ...)
        {
            va_list args;
            va_start (args, zFormat);
            vsnprintf (z_ErrorBuf, sz_MaxBufSize, zFormat, args);
            va_end (args);
        }

        const char* GetError() const
        {
            return z_ErrorBuf;
        }

        virtual ~ErrorHandler(){}

        explicit ErrorHandler(const size_t szMaxBufSize = 1024)
        {
            z_ErrorBuf = malloc(sizeof(char)*szMaxBufSize);
            sz_MaxBufSize = szMaxBufSize;
        }

        void ResizeBuffer(const size_t szMaxBufSize)
        {
            z_ErrorBuf = realloc(z_ErrorBuf, szMaxBufSize);
            sz_MaxBufSize = szMaxBufSize;
        }

    protected:
        char* z_ErrorBuf;
        size_t sz_MaxBufSize;
};

class MyClass;

//Worker can be just an interface if needed. So, can't use friend.
class Worker
{
    public:
        void Work(MyClass& oGod);
};

void Worker::Work(MyClass& oGod)
{
    //Work
    //OnError
    oGod.GetErrorHandler().SetError("Save me %s", "not");
}

class RecordingErrors
{
    public:
        const char* GetLastError() const
        {
            return oErrorHandler.GetError();
        }

        //GetErrorHandler is public
        ErrorHandler& GetErrorHandler()
        {
            return oErrorHandler;
        }

    private:
        ErrorHandler oErrorHandler;
};

class MyClass : public RecordingErrors
{
    public:
        bool GetThingsDone(Worker& me)
        {
            me.Work(*this);

            //on Error
            return false;
        }
};

int main()
{
    MyClass oL;
    Worker w;
    if(!oL.GetThingsDone(w))
    {
        std::cout << oL.GetLastError() << std::endl;
    }
}

1. 我可以在子类中重写这个函数吗？ virtual void SetError(const char* zFormat, ...)。
2. 我可以取消 RecordingErrors 类吗？我觉得 MyClass 必须继承 ErrorHandler ，我认为这不好。我是对还是错？这是关于组合而不是继承的另一个问题吗？编辑：我的意思不是名字，而是想法？
3. 这种方法有什么可能失败的情况吗？
4. 有没有更好的方法来实现错误日志记录？（日志记录在这里不是正确的词。它是什么）

Answer 1

错误处理程序和记录器是两个不同的实体。错误处理程序决定如何处理错误。应该立即发送到记录器，应该将其保存在数据库中，还是应该简单地保存在某个缓冲区中直到有人询问。
记录器决定如何记录给定的消息。它应该显示在控制台上还是应该保存在磁盘文件中，应该以什么格式显示。

请记住记录器的功能。
1）它应该是独立的类。它的行为不应该依赖于其他类。
2）记录器最好是单例的。您不需要许多漂浮的物体做同样的事情。但是单例类在多线程方面也有自己的麻烦。所以我知道这一点是有争议的。
3）它必须并且必须具有异步日志记录的能力。这意味着生产者和消费者的实现。 （日志记录是一种 I/O 操作，因此本质上是昂贵的。您不希望主处理占用此操作。但话又说回来，您可能不想使用线程来废弃这个操作。）

在错误记录器的实现中，我可以没有看到任何记录器。此外，您的错误处理程序正在保存一个错误。您可能需要错误向量。将 SetError 保留为固定参数。传递错误 ID、错误消息和错误缓冲区长度等参数。让调用者创建错误消息。