Question

我尝试编写一个简单的Logger，但是ofstream会将十六进制而不是字符写入文件。定义：

#pragma once
#include <iostream>
#include <fstream>
#include <string>
#define LOG Logger::getInstance()
class Logger
{
public:
    static Logger m_instance;
    static Logger &getInstance()
    {
        if (m_logfile == nullptr)
        {
            m_logfile = new std::ofstream(m_logfile_string.c_str(),
                                          std::ofstream::out | std::ofstream::app);
        }
        return m_instance;
    }

    Logger &operator<<(const std::string &c);
    Logger &operator<<(const char c[]);

private:
    static std::ofstream *m_logfile;
    static std::string m_logfile_string;
    Logger() {};
    Logger(const Logger &) = delete;
    Logger(Logger &&other) = delete;
    Logger &operator=(const Logger &other) = delete;
    Logger &operator=(Logger &&other) = delete;

    ~Logger()
    {
        if (m_logfile != nullptr)
            m_logfile->close();
    };

    std::string currentDateTime() const;
};

默认地将Impl。

#include "Logger.h"
#include <iostream>
#include <ctime>

Logger Logger::m_instance;
std::ofstream *Logger::m_logfile = nullptr;
std::string Logger::m_logfile_string = "log.txt";

Logger &Logger::operator<<(const std::string &c)
{
    this->operator<<(c.c_str());
    return *this;
}

Logger &Logger::operator<<(const char c[])
{
    std::cout << currentDateTime() << " - "
              << c << std::endl;
    m_logfile->operator<<(c);
    return *this;
}

// Get current date/time, format is YYYY-MM-DD.HH:mm:ss
std::string Logger::currentDateTime() const
{
    auto now = time(nullptr);
    struct tm  tstruct;
    char       buf[80];
    tstruct = *localtime(&now);
    strftime(buf, sizeof(buf), "%Y-%m-%d.%X", &tstruct);

    return buf;
}

用法：

#include "Logger.h"
void main()
{
    LOG << "started"; // Logger::getInstance() << "started";
}

结果： 00007FF696EF3C00 log.txt控制台输出正确。

这里出了什么问题？

Answer 1

您使用iostream的成员函数：

m_logfile->operator<<(c);

没有成员函数operator<<(char* c)，但有operator<<(void* p)，因此指针被隐式转换。请参阅ostream上的文档。

char*的运算符不是类成员函数：

 ostream& operator<< (ostream& os, const char* s);

见这里：operator<< (ostream)

所以你需要这个代码：

operator<<(*m_logfile, c);

或者更清洁：

(*m_logfile) << c;

测试示例：

#include <iostream>

int main() {
    std::cout.operator<<("test");
    operator<<(std::cout, "test");
    std::cout << "test";
}

打印0x400944testtest。

Answer 2

如果您使用

(*m_logfile)<<c;

而不是

m_logfile->operator<<(c);

它会起作用。

原因：

您的语法调用的成员函数ostream::operator<<()没有为char*或string定义。它仅为void*定义，它始终以十六进制显示地址。
经典语法在这里调用非成员函数ostream std::operator<<(...)，它具有string和char*的重载

ofstream写十六进制而不是char *

2 个答案: