Question

我正在尝试建立一个简单的日志记录系统。

这是我的Log.h文件

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

using namespace std;

class Log{

private:
    const static string ERROR;
    const static string WARNING; 
    const static string NOTICE;
    const static string DEBUG;
    const static string DEFAULT_FILENAME;

    static string filename;
    static ofstream* file;

public:
    Log();
    Log(string filename);
    ~Log();
    void init(string filename);

    static void log(string level, string msg);
    static void error(string msg);
    static void warning(string msg);
    static void notice(string msg);
    static void debug(string msg);
    static Log* getInstance();
};

Log.cpp中的实际代码

#include <stdlib.h>
#include <string>
#include "Log.h"
#include <fstream>
#include <iostream>

const string Log::ERROR = "ERROR";
const string Log::WARNING = "WARNING";
const string Log::NOTICE = "NOTICE";
const string Log::DEBUG = "DEBUG";
const string Log::DEFAULT_FILENAME = "log.txt";

string Log::filename;
ofstream* Log::file;

Log::Log(){
    this->init(DEFAULT_FILENAME);
}       

Log::Log(string filename){
    this->init(filename);
}       

Log::~Log(){ 
    this->file->close();
}       

void Log::init(string filename){
    Log::filename = filename; 
    cout << Log::file << " foo " << Log::filename.c_str() << endl;
    Log::file->open(filename.c_str(), ios::out | ios::app);
    cout << "bar" << endl;
    if(!Log::file->is_open()){
            throw 10;
    }
}

void Log::log(string level, string msg){
    if(Log::file == NULL)
            Log();
    cout << level << " : " << msg << endl;
    *Log::file << level << " : " << msg << endl;
}

void Log::error(string msg){
    log(ERROR, msg);
}

void Log::warning(string msg){
    log(WARNING, msg);
}

void Log::notice(string msg){
    log(NOTICE, msg);
}

void Log::debug(string msg){
    log(DEBUG, msg);
}

我的主要内容包括：

Log::debug("Starting the server");

我编译：

g++  -Wall -std=c++11  -c -o main.o main.cpp
g++  -Wall -std=c++11  -c -o Log.o Log.cpp
g++ -lfcgi++ -lfcgi main.o Log.o -o main

当我执行时，我得到：

0 foo log.txt
make: *** [exec] Segmentation fault

打开文件的代码段错误。不知何故，从这段代码开始，这不是正确的问题：

ofstream myfile;
myfile.open ("log.txt");
myfile << "Writing this to a file.\n";
myfile.close();

完美地运作。

你知道我为什么会出现这个分段错误错误吗？

谢谢！

Answer 1

我认为问题是你永远不能正确地创建Log::file。必须使用new或某个等效分配器初始化任何指针。您在未初始化的指针上调用方法，并且您的代码在那里崩溃。

你的小例子工作的原因是因为你在堆栈上分配，而不是指向堆对象的指针。无论如何，这是解决这个问题的最好方法。使用堆分配的对象可能会非常麻烦，除非您非常小心地管理所有权。

这是使用流的一种非常奇怪的方式，您已经拥有全局static实例，但您也有一个类。您应该将ofstream实例移动到对象中。

作为一种风格问题，没有必要将this->放在每个方法调用或属性引用之前，这是隐含的。只有在名称冲突的情况下才有必要。

以下是一些想法：

class Log {
private:
    string filename;
    ofstream file;
}

void Log::init(string filename_) {
    filename = filename_; 

    cout << file << " foo " << filename << ends;

    file.open(filename.c_str(), ios::out | ios::app);

    cout << "bar" << std::endl;

    if(!file.is_open()){
        throw 10;
    }
}

分段错误打开文件c ++

1 个答案: