艺术的Ruby 优雅代码 关于变量初始化和返回值

Chris Wanstrath曾有一篇博文优化初始化变量和返回值。下面是一些进一步的讨论：


原始的丑陋代码如下：

def logger
  unless @logger
    @logger = Logger.new(STDOUT)
    @logger.level = Logger::WARN
  end
  @logger
end

Chris'的改进：

def logger
  return @logger if defined? @logger
  @logger = Logger.new(STDOUT)
  @logger.level = Logger::WARN
  @logger
end

作者本人更倾向：

def logger
  @logger || begin
    @logger = Logger.new(STDOUT)
    @logger.level = Logger::WARN
    @logger
  end
end

值得探讨的是，博主的愿意是希望用@logger ||=的方式表达，如果有一个构造函数返回本身会看起来更漂亮。或者，在加上一个反射，如下：

def logger
  @logger ||= default_logger
end

def default_logger
  logger = Logger.new(STDOUT)
  logger.level = Logger::WARN
  logger
end

当然，如果你有更强的风格，也可写成这样：

def logger
  @logger ||= Logger.new(STDOUT).tap {|l| l.level = Logger::WARN}
end

另外，defined? @logger 和if @logger并不一样，如下：

引用

>> defined?(@logger)
=> nil
>> @logger
=> nil
>> @logger = nil
=> nil
>> defined?(@logger)
=> "instance-variable"

因此，当你试图判断@logger是否为nil的时候，defined?往往不是你需要的，要当心使用。

评论

3 楼 Hooopo 2009-12-17  

还是 ||= 帅

2 楼 夜鸣猪 2009-12-17  

哈哈，常来常往

black_star 写道

打酱油的，路过。

1 楼 black_star 2009-12-17  

打酱油的，路过。