ruby-on-rails - 什么是 rubocop 中的阴影外部局部变量，我该如何解决？

Question

我在 Rails 上运行 rubocop，它给了我下面的信息。

W: Shadowing outer local variable - user.
  where(provider: auth.provider, uid: auth.uid).first_or_create do |user|
                                                                    ^^^^

这是代码。

def self.from_omniauth(auth)
user = User.where(email: auth.info.email).first

if user
  return user
else
  where(provider: auth.provider, uid: auth.uid).first_or_create do |user|
    user.fullname = auth.info.name
    user.provider = auth.provider
    user.uid = auth.uid
    user.email = auth.info.email
    user.image = auth.info.image
    user.password = Devise.friendly_token[0, 20]
  end
end

结尾

Answer 1

这意味着user作为块提供的参数将覆盖user此处定义的变量user = User.where(email: auth.info.email).first

要克服它，您需要更改其中一个变量的名称。要么是这样的：

result = User.where...
return result if result    

或者：

where(provider: auth.provider, uid: auth.uid).first_or_create do |u|
  u.fullname = auth.info.name
  ...
end

更多信息：https ://github.com/bbatsov/ruby-style-guide#no-shadowing