2

ロギングプログラムをコンパイルしていますが、このエラーが発生し、一生理解できません...

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;
4

3 に答える 3

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;
}

等々

于 2011-02-04T20:36:00.727 に答える
1

クラスを再定義しているからです。.h から既にインクルードしている場合、.cpp で「class Logger {」と再度言うことはできません。

于 2011-02-04T20:36:56.627 に答える
0

コンパイラは、クラスが属する名前空間 (またはスコープ) 全体で常に 1 つのクラス定義のみを想定しています。現在、指定したコードには、実際には 2 つのクラス定義があることがわかります。1 つは .h ファイルにあり、もう 1 つは .cpp ファイルにあります。そのため、コンパイラは、許可されていないクラスを再定義していると不平を言っています。

通常、コンパイラ エラーが発生した場合は、コンパイラが示す行を確認することをお勧めします。ほとんどの場合、コンパイラが指摘する行に問題があります。

于 2011-02-04T21:25:40.917 に答える