Rails HTTParty 的源码

hiveer · 2014年08月18日 · 最后由 hiveer 回复于 2014年08月19日 · 2682 次阅读

lib/httparty.rb

module HTTParty
  module AllowedFormatsDeprecation
    def const_missing(const)
      if const.to_s =~ /AllowedFormats$/
        Kernel.warn("Deprecated: Use HTTParty::Parser::SupportedFormats")
        HTTParty::Parser::SupportedFormats
      else
        super
      end
    end
  end

  extend AllowedFormatsDeprecation

  def self.included(base)
    base.extend ClassMethods
    base.send :include, HTTParty::ModuleInheritableAttributes
    base.send(:mattr_inheritable, :default_options)
    base.send(:mattr_inheritable, :default_cookies)
    base.instance_variable_set("@default_options", {})
    base.instance_variable_set("@default_cookies", CookieHash.new)
  end

  module ClassMethods

    extend AllowedFormatsDeprecation
  ...

代码extend AllowedFormatsDeprecation在这两处的意义分别是什么？特别是第二处，完全感觉不到有什么用处，求大大们解释下

1 个赞

1 楼已删除

outman #2 2014年08月18日

base.extend ClassMethods , 这个地方估计是个 Bug。 module ClassMethods extend AllowedFormatsDeprecation end; 用于捕获模块内部的常量未定义。

hiveer #3 2014年08月18日

@outman 我已经在 github 上提了一个 issue，有回馈这边会更新

kepaning #4 2014年08月18日

我觉得第一个是防止别人用 HTTParty::AllowedFormats，第二个是防止别人用HTTParty::ClassMethods::AllowedFormats

hiveer #5 2014年08月18日

#4 楼 @kepaning 除了在include HTTParty的时候会有base.extend ClassMethods，还有什么时候我们回去调用HTTParty::ClassMethods::XXXX

kepaning #6 2014年08月18日

#5 楼 @hiveer 嗯，我觉得你是对的，是看不出有什么理由用到。

hiveer #7 2014年08月18日

@outman @kepaning 仔细研究了下，发现源码没有问题第一个 extend AllowedFormatsDeprecation是为了处理下面这种情况的 HTTParty::AllowedFormats 的常量调用

第二个extend AllowedFormatsDeprecation是为了处理在ClassMethods里面的方法的常量调用

module ClassMethods
  def test
    AllowedFormats
  end
end

*为了更好的理解上面的解释，还需要特别注意我们在include和extend module的时候，base的 mixing 方法实际上只是对应module方法的一个引用。

pry(main)> ls Test
Out::ClassMethods#methods: test_const

pry(main)> ls Mixing
Mix#methods: b

这也是为什么需要第二个 extend AllowedFormatsDeprecation的原因

kepaning #8 2014年08月19日

#7 楼 @hiveer 我怎么没有看到 ClassMethods 有 test 方法

hiveer #9 2014年08月19日

@kepaning 我上面只是举个例子，test代表了在ClassMethods中定义的实例方法

kepaning #10 2014年08月19日

@hiveer 我也想过这个问题，但是我看不到 ClassMethods 中用了任何的 AllowedFormats。这是相关的 commit，我还是觉得 ClassMethods 中的这个 extend 是多余的 https://github.com/jnunemaker/httparty/commit/d0d88fecfdd7528fa8bd8d12004f686318f3486c

11 楼已删除

hiveer #12 2014年08月19日

@kepaning 从代码上看是没有明显的调用，不过可以起到防患未然的作用

需要登录后方可回复, 如果你还没有账号请注册新账号