Можете ли вы помочь мне понять, почему этот логический метод возвращает false? - PullRequest
Новая
       12

Можете ли вы помочь мне понять, почему этот логический метод возвращает false?

0 голосов
/ 02 июля 2018

У меня есть модель User с атрибутом account_type, который является либо «Student», либо «Partner». Я создал логические методы в своей модели User, чтобы определить, является ли запись пользователя учеником или партнером (см. Ниже). 

 def student?
    self.account_type == "Student"
  end

  def partner?
    self.account_type == "Partner"
  end

В консоли rails, когда я устанавливаю пользователя равным экземпляру User, который имеет тип учетной записи студента, и вводю user.account_type == "Student", я получаю значение true, но когда я вызываю user.student?, Я получаю false. Есть ли проблема с тем, как я настроил эти методы? Они кажутся довольно простыми, поэтому я не понимаю, почему true не возвращается для записи.

Консольный вывод: 

user = User.last
#<User id: 18, first_name: "gjalrgkj", last_name: "kgjrlgakjrl", email: "terajglrkj@gmail.com", password_digest: "$2a$10$WF.Rw3PzlWilH0X.Nbfxfe5aB18WW6J7Rt4SAKQEwI8...", remember_digest: nil, activation_digest: "$2a$10$/bXG4/nKCiiZHWailUPAmOZj7YhCjKhPm4lUW6nPC3N...", activated: nil, activated_at: nil, reset_digest: nil, reset_sent_at: nil, account_type: "Student", created_at: "2018-07-02 04:21:07", updated_at: "2018-07-02 04:21:07">
>> user.account_type
=> "Student"
>> user.account_type = "Student"
=> "Student"
>> user.student?
=> false

Модель пользователя: 

class User < ApplicationRecord
  has_one :personal_information

  attr_accessor :remember_token, :activation_token, :reset_token
  before_save   :downcase_email
  before_create :create_activation_digest
  validates :first_name,  presence: true, length: { maximum: 50 }
  validates :last_name,  presence: true, length: { maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(\.[a-z\d\-]+)*\.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 },
                      format: { with: VALID_EMAIL_REGEX },
                      uniqueness: { case_sensitive: false }
  has_secure_password
  validates :password, presence: true, length: { minimum: 6 }, allow_nil: true
  validates :account_type, presence: true


  def User.new_token
    SecureRandom.urlsafe_base64
  end

  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost  BCrypt::Engine::MIN_COST :
                                              BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  def create_reset_digest   
    self.reset_token = User.new_token
    update_columns(reset_digest: User.digest(reset_token), reset_sent_at: Time.zone.now)
  end

  def authenticated?(attribute, token)
    return false if digest.nil?
 BCrypt::Password.new(digest).is_password?(token)
  end


  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  def forget
    update_attribute(:remember_digest, nil)
  end


  def provide_age
    now = Time.now.utc.to_date
    if self.birthday.nil?
      nil
    else
      self.age = now.year - self.birthday.year - ((now.month > self.birthday.month || (now.month == self.birthday.month && now.day >= self.birthday.day)) ? 0 : 1)
      update_attribute(:age, self.age)
    end
  end


  def send_activation_email
    UserMailer.account_activation(self).deliver_now
  end  

  def activate
    update_columns(activated: true, activated_at: Time.zone.now)
  end

  def send_password_reset_email
    UserMailer.password_reset(self).deliver_now
  end

  def password_reset_expired?
    reset_sent_at < 2.hours.ago
  end


  private 

  def downcase_email
    self.email.downcase!
  end

  def create_activation_digest
    self.activation_token  = User.new_token
    self.activation_digest = User.digest(activation_token)
  end
end

User Helper 

  def account_type
    [
      ['Student'], 
      ['Partner'], 
      ['School Administrator'], 
      ['Philanthropist']
    ]
  end

  def student?
    self.account_type == "Student"
  end

  def partner?
    self.account_type == "Partner"
  end
 end

Ответы [ 2 ]

0 голосов
/ 02 июля 2018

попробуйте вставить этот код в вашу модель пользователя (не в Helper) 

def student?
  self.account_type == "Student"
end

def partner?
  self.account_type == "Partner"
end

и это хорошая идея, чтобы создать сеттер для установки account_type, пример 

def set_account_type(type)
  self.account_type = type
end
0 голосов
/ 02 июля 2018

Ваши 2 метода student? и partner? принадлежат пользовательской модели.

Когда вы делаете user.student?, он ищет метод student? как метод экземпляра в пользовательской модели.

self в помощнике не является экземпляром пользователя, он указывает на ваш вспомогательный модуль.

Надеюсь, это поможет.

...