0

我正在尝试从此链接中的记录器代码，但它给了我错误。如何在项目中实现良好的调试/日志记录功能

#ifndef _LOGGER_HPP_
#define _LOGGER_HPP_

#include <iostream>
#include <sstream>

/* consider adding boost thread id since we'll want to know whose writting and
 * won't want to repeat it for every single call */

/* consider adding policy class to allow users to redirect logging to specific
 * files via the command line
 */

enum loglevel_e
    {logERROR, logWARNING, logINFO, logDEBUG, logDEBUG1, logDEBUG2, logDEBUG3, logDEBUG4};

class logIt
{
public:
    logIt(loglevel_e _loglevel = logERROR) {
        _buffer << _loglevel << " :" 
            << std::string(
                _loglevel > logDEBUG 
                ? (_loglevel - logDEBUG) * 4 
                : 1
                , ' ');
    }

    template <typename T>
    logIt & operator<<(T const & value)
    {
        _buffer << value;
        return *this;
    }

    ~logIt()
    {
        _buffer << std::endl;
        // This is atomic according to the POSIX standard
        // http://www.gnu.org/s/libc/manual/html_node/Streams-and-Threads.html
        std::cerr << _buffer.str();
    }

private:
    std::ostringstream _buffer;
};

extern loglevel_e loglevel;

#define log(level) \
if (level > loglevel) ; \
else logIt(level)

#endif

更准确地说，这#define是给出错误：

#define log(level) \
if (level > loglevel) ; \
else logIt(level)

错误是Syntax error: if和Syntax error: else

但后来，我注意到如果我#include "logger.hpp"从移动main.h到main.cpp，问题就消失了。尽管“main.h”在不同的地方多次包含，但它确实包含“#pragma once”。

任何想法？

4

3 回答 3

2

如果loglevel在编译时已知，您可以执行以下操作：

template <bool>
struct LogSystem
{
    template <class T>
    LogSystem& operator << (const T &)
    {
        //ignore the input
        return (*this);
    }
};

template <>
struct LogSystem <true>
{
    template <class T>
    LogSystem& operator << (const T & v)
    {
        cout << v;
        return (*this);
    }
};

template <bool B>
LogSystem<B>    getLog()
{
    return LogSystem<B>();
}

#define log(level) getLog< (level <= loglevel) >()

如果在编译时loglevel未知：

class iLogSystem
{
public:
    virtual iLogSystem& operator << (const int &)
    {
        //empty
        return (*this);
    }
    virtual iLogSystem& operator << (const custom_type &);
    {
        return (*this);
    }
    //make functions for logging all the types you want
};

class LogSystem : public iLogSystem
{
public:
    virtual iLogSystem& operator << (const int & v)
    {
        cout << v;
        return (*this);
    }
    virtual iLogSystem& operator << (const custom_type &  q);
    {
        cout << q.toString();
        return (*this);
    }
    //make functions for logging all the types you want
};
iLogSystem& getLog(const bool l)
{
    static LogSystem actual_log;
    static iLogSystem empty_log;
    if(l)
        return &actual_log;
    return &empty_log;
}

#define log(level) getLog( level <= loglevel )
于 2013-11-05T17:55:42.877 回答
2

任何时候你想定义一个扩展为statement的宏，如果定义包含任何复合语句（包括if/ else），你应该将定义包装在do...中while (0)。包含的代码仍将只执行一次，并且可以在任何需要语句的上下文中使用。

if这是我所知道的在/语句中使用宏时避免语法错误的唯一方法else，因为使用了分号。

所以宁愿这样：

#define log(level) \
    if ((level) > loglevel) ; \
    else logIt(level)

你可以使用这个：

#define log(level) \
    do { \
        if ((level) > loglevel) ; \
        else logIt(level) \
    } while (0)

我在对宏level参数的引用周围添加了括号，以避免任何可能的运算符优先级问题。还要注意末尾缺少分号；分号将由调用者提供。

另一方面，if/else通常可以被条件（三元）?:运算符替换：

#define log(level) \
    ((level) > loglevel ? 0 : logIt(level))

它允许在任何可以使用表达式log(level)的地方使用；如果添加分号，则包括语句上下文。您可能希望用;返回的类型替换。if是一个 void 函数，您可能需要：0logItlogIt

#define log(level) \
    ((level) > loglevel ? (void)0 : logIt(level))

这一切都假设宏是完成这项工作的正确工具。模板（如this answer所建议的那样）或内联函数可能会更好地完成工作并且混淆的可能性较小。

于 2013-11-05T19:31:33.427 回答
2

#define 预处理器指令可以包含 if 和 else 吗？

是的。

关于您的问题：预处理器很笨，只执行简单的文本替换。它不是函数，也不是语言结构，它是简单的、愚蠢的文本替换。结果，这：

#define log(level) \
    if (level > loglevel) ; \
    else logIt(level)

...

log(logINFO) << "foo " << "bar " << "baz";

变成这样：

if (logINFO > loglevel); // << here's your problem.
else 
    logIt(logInfo)
 << "foo " << "bar " << "baz";

你的问题是;。这里，分号表示c++if语句的结束，所以编译器后面遇到时else，不知道怎么处理。

我注意到如果我将 #include "logger.hpp" 从 main.h 移动到 main.cpp，问题就消失了

C++ 具有“对数”功能。这称为log。如果你的其他文件使用对数函数，事情会变得非常有趣，因为它会被你的 if/else 日志代码替换。

例如，如果标题中的某处有内联对数代码，那么如果您首先包含标题，它将变成无稽之谈logger。例如log(6.0) + 1将变成log (if (6.0 > logLevel); else logIt(6.0)) + 1，这不是一个有效的 C++ 语句。

于 2013-11-05T18:19:31.020 回答