我正在编译一个日志记录程序,但是我收到了这个错误并且无法解决这个问题...
logger.cpp:15: error: redefinition of ‘class Logger’
logger.h:20: error: previous definition of ‘class Logger’
当我用编译时,使用gcc
g++ -Wall logger.cpp -o log
logger.h:
#ifndef LOGGER_H
#define LOGGER_H
#include <fstream>
#include <iostream>
#include <string>
using std::string;
class Logger
{
static Logger* m_pInstance;
public:
static Logger* Instance() { return m_pInstance; }
void writeLog(string message);
void openLogFile(string fileName);
void closeLogFile();
void deleteLogger();
};
#endif
logger.cpp
#include "logger.h"
#include <fstream>
#include <iostream>
class Logger
{
static Logger* m_pInstance;
std::ofstream m_pOutputFile;
Logger()
{
}
~Logger()
{
}
public:
static Logger* Instance()
{
if(!m_pInstance)
{
m_pInstance = new Logger;
}
return m_pInstance;
}
void writeLog(std::string message)
{
m_pOutputFile << message << "\n";
std::cout << "you just wrote " << message << " to the log file!\n" << std::endl;
}
void openLogFile(std::string fileName)
{
m_pOutputFile.open(fileName.c_str(),std::ios::out);
}
void closeLogFile()
{
m_pOutputFile.close();
}
void deleteLogger()
{
delete m_pInstance;
}
};
Logger* Logger::m_pInstance = NULL;
答案 0 :(得分:5)
这正是错误消息所说的内容。实现文件不仅可以提供类的重新定义,还可以在任何地方添加新的成员变量和冲突的函数体。相反,为已经声明的函数和静态成员变量提供定义。
#include "logger.h"
#include <fstream>
#include <iostream>
static Logger::Logger* m_pInstance;
Logger::Logger()
{
}
Logger::~Logger()
{
}
// this also is illegal, there's a body provided in the header file
//Logger* Logger::Instance()
//{
// if(!m_pInstance)
// {
// m_pInstance = new Logger;
// }
// return m_pInstance;
//}
void Logger::writeLog(std::string message)
{
m_pOutputFile << message << "\n";
std::cout << "you just wrote " << message << " to the log file!\n" << std::endl;
}
等等
答案 1 :(得分:1)
好吧,因为你正在重新定义课程。当你已经从.h中包含它时,你不能再在.cpp中说'class Logger {'。
答案 2 :(得分:0)
编译器总是期望该类所属的整个命名空间(或范围)中只有一个类定义。目前在您指定的代码中,您会看到有2个类定义:一个在.h文件中,另一个在.cpp文件中。这就是为什么编译器抱怨你正在重新定义一个不允许的类。
通常,每当遇到编译器错误时,最好查看编译器告诉的行。大多数情况下,问题出在编译器指出的行中。