1

我正在学习 SOLID 并尝试将 SRP 引入我的 rails 应用程序。我有以下具有基本身份验证的用户模型：

class User < ActiveRecord::Base
  attr_accessible :password, :password_confirmation
  attr_accessor :password

  before_save :encrypt_password

  validates_confirmation_of :password
  validates_presence_of     :password, :on => :create

  def self.authenticate(email, password)
    user = find_by_email(email)
    if user && user.password_hash == BCrypt::Engine.hash_secret(password, user.password_salt)
      user
    else
      nil
    end
  end

  def encrypt_password
    if password.present?
      self.password_salt = BCrypt::Engine.generate_salt
      self.password_hash = BCrypt::Engine.hash_secret(password, password_salt)
    end
  end

  def self.generate_random_password
    return ActiveSupport::SecureRandom.hex(12)
  end
end

我想将所有身份验证逻辑移动到一个模块，如下所示：

module Authentication

  attr_accessible :password, :password_confirmation
  attr_accessor :password

  before_save :encrypt_password

  validates_confirmation_of :password
  validates_presence_of     :password, :on => :create

  def self.authenticate(email, password)
    user = find_by_email(email)
    if user && user.password_hash == BCrypt::Engine.hash_secret(password, user.password_salt)
      user
    else
      nil
    end
  end

  def encrypt_password
    if password.present?
      self.password_salt = BCrypt::Engine.generate_salt
      self.password_hash = BCrypt::Engine.hash_secret(password, password_salt)
    end
  end

  def self.generate_random_password
    return ActiveSupport::SecureRandom.hex(12)
  end
end

我的用户模型是这样的：

class User < ActiveRecord::Base
  include Authentication #SRP in action! :P
end

现在错误开始了：

身份验证的未定义方法“attr_accessible”：模块

我将如何解决此错误？我确信这是将 SRP 引入我的 Rails 应用程序的最佳开始。

谢谢

4

1 回答 1

2

在错误的范围内调用 attr_accessible 方法。看看问题来解决这个问题：

http://api.rubyonrails.org/classes/ActiveSupport/Concern.html

这将导致：

module Authentication
  extend ActiveSupport::Concern
  included do
    attr_accessible :password, :password_confirmation
  end
  ...
end

这也将照顾你的类和实例方法定义。

注意：具体来说，这并不能完全实现 SRP，因为多个职责仍然在同一个类中共享，即使它们被分成模块。通过引用或修饰的类组合会是一个更严格的解决方案，但我更喜欢模块的务实方法。

于 2012-07-16T08:24:30.217 回答