Log error_log("example.txt");
error_log << "An example error message " << 123;
error ( "example.txt", "An example error message 123" );
Why don't you just use a function that will open a file, writes an error message in it and then closes the file.
Then you just call the function like this:
error ( "example.txt", "An example error message 123" );
I would seriously recommend against opening and closing the file for each line unless your logging is VERY sparse. Doing it this way causes immense CPU and I/O usage for each log line as close() usually invokes a flush to disk, and will make verbose logging infeasible for any larger project (been there, done that!)
class Log
{
public:
Log(const char* filename)
{
m_filestream.open(filename, std::fstream::out);
}
~Log(void)
{
m_filestream.close();
}
inline Log& operator<<(const char* text)
{
m_filestream << text;
return *this;
}
inline Log& operator<<(const float text)
{
m_filestream << text;
return *this;
}
private:
std::fstream m_filestream;
};
template <class T>
Logger& operator<<(const T &in)
{
if(m_File) { *m_File << in; }
if(m_ConsoleStream) { *m_ConsoleStream << in; }
return *this;
}
Logger::GetInstance() << "Hello" << " this is a test" << 23 << "new line\n" << 8.01 << "\n" << "blah blah";
class Logger
{
public:
class endl {};
template <class T>
Logger& operator<<(const T &in)
{
if(m_File) { *m_File << in; }
if(m_ConsoleStream) { *m_ConsoleStream << in; }
return *this;
}
Logger& operator<<(const endl&)
{
if(m_File) { *m_File << std::endl; }
if(m_ConsoleStream) { *m_ConsoleStream << std::endl; }
return *this;
}
};
Logger::GetInstance() << "Hello" << " this is a test" << 23 << "new line\n" << 8.01 << Logger::endl() << "blah blah";
@Nanoha
I don't see the benefit of a singleton over a simple global or free function here.
Logger::Logger(LoggerLevel level, bool setAsGlobal)
{
if(setAsGlobal)
{
assert((instance == NULL) && "Global logger object already set");
instance = this;
}
}
