2

如何代理 ruby​​ 记录器并保持性能？

所以，我们在工作中有一个要求，很合理。当程序被发送信号 HUP 时，日志被刷新并重新启动。

class LocalObject

  attr_accessor :logger

  def initialize context
    # one less method call! Yea! performance++
    @logger = context.logger
  end

  def something
    @logger.info "Hello world"
  end

end

问题是，如果 context.logger 被重置，那么 @logger 仍然指向旧的。

所以，我想我会代理记录器：

class LoggerProxy
  attr_accessor :logger

  def debug *args
    @logger.send :debug, args
  end

  def info *args
    @logger.send :info, args
  end
end

context.logger = LoggerProxy.new
context.logger.logger = Logger.new 'my_file.log'

Signal.trap('HUP') { 
  context.logger.logger = Logger.new 'my_file.log'
}
...
@logger = context.logger
@logger.info "Hello world"

这很好用，除了我将一个方法调用换成 2 个方法调用（1 个访问器；它返回记录器）。我仍然需要调用 LoggerProxy.:debug, :info, ...，然后调用原始记录器！因此，有 2 个方法调用，其中有一个。

我不想对 Logger 类进行猴子或重载它，因为我想在将来使用其他记录器、syslog、滚动我自己的或类似的。

有没有办法减少方法调用的数量以提高性能？

-丹尼尔

更新：为了回答有关性能的问题，这里是示例测试。

require 'logger'
require 'benchmark';

class MyLogger

  attr_accessor :logger

  def info msg
    @logger.info msg
  end

end

myLogger = Logger.new '/dev/null' # dev null to avoid IO issues
myLoggerProxy = MyLogger.new
myLoggerProxy.logger = myLogger

n = 100000
Benchmark.bm do | benchmarker |
  # plain logger
  benchmarker.report { n.times { myLogger.info 'opps' } }

  # via accessor
  benchmarker.report { n.times { myLoggerProxy.logger.info 'opps' } }

  # via proxy
  benchmarker.report { n.times { myLoggerProxy.info 'opps' } }
end


      user     system      total        real
  1.580000   0.150000   1.730000 (  1.734956)
  1.600000   0.150000   1.750000 (  1.747969)
  1.610000   0.160000   1.770000 (  1.767886)
4

2 回答 2

4

与其重置记录器本身，不如刷新并重新打开其输出：

logfile = File.open 'my_file.log', 'w+'
context.logger = Logger.new logfile

Signal.trap('HUP') {
  logfile.flush
  logfile.reopen 'my_file.log', 'w+'
}
于 2010-12-04T18:00:19.600 回答
2

首先：您的问题闻起来像是过早地优化。如果您知道您的代码太慢，您应该只优化。（你的基准测试只显示了微小的差异）

也就是说，如果记录器有更新，您可以让 Context 通知每个代理：

class ProxyLogger
  attr_accessor :logger

  def initialize(context)
    context.register(self)
  end
end

class Context
  attr_accessor :logger

  def initialize
    @proxies = []
  end

  def logger=(val)
    @logger = val
    @proxies.each { |p| p.logger = val }
  end

  def register(proxy)
    @proxies << proxy
  end
end

但同样，这似乎并不值得额外的复杂性。

（相关：这是一个非常好的演示文稿，展示了@tenderlove 优化 ARel gem）

于 2010-12-10T00:33:10.090 回答