升压日志：设置属性使日志条目消失

Question

我有示踪剂，应记录进出函数：

class Tracer 
{ 
public: 
    Tracer(Logger logger, std::string function, std::string file) 
    : 
    _logger(logger), 
    _function(function), 
    _file(file) 
    {} 

    ~Tracer() 
    { 
    using namespace boost::log; 
    // attribute_cast<attributes::mutable_constant<int>>(core::get()->get_global_attributes()["Line"]).set(1); 
    attribute_cast<attributes::mutable_constant<std::string>>(core::get()->get_global_attributes()["File"]).set(_file); 
    attribute_cast<attributes::mutable_constant<std::string>>(core::get()->get_global_attributes()["Function"]).set(_function); 
    BOOST_LOG_SEV(_logger, trivial::severity_level::trace) << "Leaving " << _function; 
    } 

private: 
    Logger _logger; 
    std::string _function; 
    std::string _file; 
}; 

这工作得很好，但只要我取消那台线Line attribut我不再收到任何离开消息。

属性被预先赋予这样的：

using namespace boost::log; 
add_common_attributes(); 
core::get()->add_global_attribute("Scope", attributes::named_scope()); 
core::get()->add_global_attribute("Rank", attributes::constant<int>(5)); 
core::get()->add_global_attribute("Line", attributes::mutable_constant<int>(5)); 
core::get()->add_global_attribute("File", attributes::mutable_constant<std::string>("")); 
core::get()->add_global_attribute("Function", attributes::mutable_constant<std::string>("")); 

设置所有其他属性，类同线路工作正常。我很困惑，因为Line似乎不是一个保留名称，它可以在其他地方工作，而boost :: log源不会显示该名称的相关用法。

这里有什么问题？我很高兴地向任何人发送complate代码（总共约150行）

Answer 1

您的代码的主要问题是get_global_attributes()返回全局属性集的副本。无论您对副本所做的更改都不会影响您看到的与日志记录相关的属性。如果你的代码是多线程的，那么首先使用全局属性是错误的，因为执行上下文是线程特定的。另外，假设attribute_cast成功（即不返回空属性）假定该属性在程序中的任何地方都不会改变。您至少应该检查attribute_cast的结果。

更好的方法来实现你想要的是将行和函数属性添加到Tracer构造函数中的线程特定属性集而不是日志初始化代码，并在析构函数中恢复其以前的状态。恢复部分很重要，您计划嵌套使用Tracer标记的通话。

using namespace boost::log; 

class Tracer 
{ 
    enum restore_action 
    { 
    noop, 
    restore_prev, 
    remove 
    }; 

public: 
    Tracer(
    Logger const& logger, 
    std::string const& function, 
    std::string const& file, 
    int line) 
    : 
    _logger(logger), 
    _function_name(function), 
    _prev_line_number(0), 
    _function(set_attribute("Function", function, _prev_function_name)), 
    _file(set_attribute("File", file, _prev_file_name)), 
    _line(set_attribute("Line", line, _prev_line_number)) 
    { 
    BOOST_LOG_SEV(_logger, trivial::severity_level::trace) 
     << "Entering " << _function_name; 
    } 

    ~Tracer() 
    { 
    BOOST_LOG_SEV(_logger, trivial::severity_level::trace) 
     << "Leaving " << _function_name; 

    restore_attribute(_function, _prev_function_name); 
    restore_attribute(_file, _prev_file_name); 
    restore_attribute(_line, _prev_line_number); 
    } 

private: 
    template< typename T > 
    static std::pair< attribute_set::iterator, restore_action > 
    set_attribute(attribute_name name, T const& value, T& prev_value) 
    { 
    typedef attributes::mutable_constant<T> mutable_constant_t; 
    core_ptr p = core::get(); 

    std::pair< attribute_set::iterator, bool > res = 
     p->add_thread_attribute(name, mutable_constant_t(value)); 
    if (res.second) 
     return std::make_pair(res.first, remove); 

    mutable_constant_t prev = 
     attribute_cast<mutable_constant_t>(res.first->second); 
    if (!prev) 
     return std::make_pair(res.first, noop); 

    prev_value = prev.get(); 
    prev.set(value); 
    return std::make_pair(res.first, restore_prev); 
    } 

    template< typename T > 
    static void restore_attribute(
    std::pair< attribute_set::iterator, restore_action > const& state, 
    T const& prev_value) 
    { 
    typedef attributes::mutable_constant<T> mutable_constant_t; 

    switch (state.second) 
    { 
    case restore_prev: 
     { 
     mutable_constant_t attr = 
      attribute_cast<mutable_constant_t>(state.first->second); 
     if (attr) 
      attr.set(prev_value); 
     } 
     break; 

    case remove: 
     core::get()->remove_thread_attribute(state.first); 
     break; 

    case noop: 
    default: 
     break; 
    } 
    } 

private: 
    Logger _logger; 
    std::string _function_name; 
    std::string _prev_function_name, _prev_file_name; 
    int _prev_line_number; 
    std::pair< attribute_set::iterator, restore_action > _function; 
    std::pair< attribute_set::iterator, restore_action > _file; 
    std::pair< attribute_set::iterator, restore_action > _line; 
}; 

注意，该代码将标志着该线程与功能，文件和行所做的所有记录，因为它们是由示踪设置属性。如果这不是你真正想要的，你只需要标记关于进入和离开函数的消息，那么你应该添加函数，文件和行到记录器属性，而不是线程特定的。这将显着简化代码，因为您不必费心恢复以前的属性状态。

我还要补充说，有更好的工具来维护调用堆栈，特别是标记当前的执行上下文。 Named scopes允许您以比上面写的更便宜的方式标记当前范围或功能。该标记不会生成日志记录，但这很容易添加。

class Tracer 
{ 
public: 
    Tracer(
    Logger const& logger, 
    string_literal const& function, 
    string_literal const& file, 
    unsigned int line) 
    : 
    _logger(logger), 
    _function_name(function), 
    _scope_sentry(function, file, line) 
    { 
    BOOST_LOG_SEV(_logger, trivial::severity_level::trace) 
     << "Entering " << _function_name; 
    } 

    ~Tracer() 
    { 
    BOOST_LOG_SEV(_logger, trivial::severity_level::trace) 
     << "Leaving " << _function_name; 
    } 

private: 
    Logger _logger; 
    string_literal _function_name; 
    attributes::named_scope::sentry _scope_sentry; 
}; 

相比原来代码这个版本有额外的限制，即功能和文件名必须是字符串文字，而不仅仅是任意的字符串。您还将不得不设置formatter。