Question

标题说我遇到了fstream的问题。我创建了一个单例日志记录类，它使用fstream将日志消息保存在文件中。

我使用IDE XCode。在Visual Studio 2013中，代码工作正常。

标题文件：

#ifndef Logger_hpp
#define Logger_hpp

#include <stdio.h>
#include <fstream>
#include <string>

class Logger
{
private:
    static Logger* loggerPtr;
    std::fstream fs;

    //To ensure no one else can instantiate Logger.
    Logger();

public:
    enum MessageType
    {
        ERROR,
        WARNING,
        INFO
    };

    static Logger* Instance();
    void LogFunc(std::string msg, MessageType type);

};

#endif /* Logger_hpp */

CPP文件：

#include "Logger.hpp"
#include <iostream>

Logger* Logger::loggerPtr = NULL;

Logger* Logger::Instance()
{
    if (!loggerPtr)
    {
        loggerPtr = new Logger;
    }

    return loggerPtr;
}

Logger::Logger()
{
    fs.open("docker.log", std::fstream::in | std::fstream::out | std::fstream::app);
}

void Logger::LogFunc(std::string msg, MessageType type)
{
    std::cout << msg;

    switch (type)
    {
        case ERROR:
            fs << msg;
            break;
        case WARNING:
            fs << msg;
            break;
        case INFO:
            fs << msg;
            break;
    }
}

主档案：

int main() {

    Logger::Instance()->LogFunc("Hello.", Logger::INFO);

    std::fstream fs;
    fs.open("docker_test.log", std::fstream::in | std::fstream::out | std::fstream::app);
    fs << "Why does this work?!";

    system("pwd");

    return 0;
}

记录输出：

cat docker.log -> outputs H
cat docker_test.log -> outputs Why does this work?!

我错过了什么？提前谢谢！

Answer 1

这可能与您不flush fstream的事实有关。由于它是缓冲的，因此缓冲区不会写入文件。在代码结束时，Logger不会被破坏（使用原始指针的不便），缓冲区也不会被刷新。

它适用于您在main()中打开的流，因为本地流在离开函数时会被自动销毁（并关闭）。

建议：

fs << msg;   // for every logging output in syour switch 
fs.flush();  // add a flush

Answer 2

问题是，您永远不会delete singleton ，因此std::fstream永远不会被刷新。

我会以不同的方式做你的单身。我不会使用指针而是使用instance()函数本地的静态实例，如下所示：

#include <stdio.h>
#include <fstream>
#include <string>
#include <iostream>

class Logger
{
private:
    std::fstream fs;

    //To ensure no one else can instantiate Logger.
    Logger();

public:
    enum MessageType
    {
        ERROR,
        WARNING,
        INFO
    };

    static Logger& Instance(); // return reference
    void LogFunc(std::string msg, MessageType type);

};

Logger& Logger::Instance()
{
    // put the static in the function to ensure creation time
    static Logger logger; // use a static instance, not pointer
    return logger;
}

Logger::Logger()
{
    fs.open("docker.log", std::fstream::in | std::fstream::out | std::fstream::app);
}

void Logger::LogFunc(std::string msg, MessageType type)
{
    std::cout << msg;

    switch (type)
    {
        case ERROR:
            fs << msg;
            break;
        case WARNING:
            fs << msg;
            break;
        case INFO:
            fs << msg;
            break;
    }
}

int main() {

    Logger::Instance().LogFunc("Hello.", Logger::INFO);

    std::fstream fs;
    fs.open("docker_test.log", std::fstream::in | std::fstream::out | std::fstream::app);
    fs << "Why does this work?!";

    system("pwd");

    return 0;
}

C ++ fstream只保存字符串

2 个答案: