Rails «помощники» очень специфично смотрят на помощников

Question

Я читал о хороших манерах Rails, и есть много подобных статей:

http://codefol.io/posts/Where-Do-I-Put-My-Code/

и похоже, что помощники должны быть только используется для просмотров. Я не очень хорошо понимаю, почему, но если это так, то это мой session_helper.rb файл:

module SessionsHelper
  def log_in(user)
    session[:user_id] = user.id
  end

  def logged_in?
    !current_user.nil?
  end

  def log_off
    forget(current_user)
    session.delete(:user_id)
    @current_user = nil
  end

  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif (user_id = cookies.signed[:user_id])
      user = User.find_by(id: user_id)
      if user&.authenticated?(cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end

  def remember(user)
    user.remember
    cookies.permanent.signed[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_digest
  end

  def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_token)
  end
end

, а это моя модель user.rb

class User < ApplicationRecord
  has_many :post
  attr_accessor :remember_token, :activation_token
  before_save { self.email = email.downcase }
  before_create :create_activation_digest
  validates :name, presence: true, length: { maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i.freeze
  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 }

  # Returns the hash digest of the given string.
  def self.digest(string)
    Digest::SHA1.hexdigest(string.to_s)
  end

  # Returns a random token.
  def self.new_token
    SecureRandom.urlsafe_base64
  end

  # Remembers a user in the database for use in persistent sessions.
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  # Returns true if the given token matches the digest.
  def authenticated?(remember_token)
    return false if remember_digest.nil?

    BCrypt::Password.new(remember_digest).is_password?(remember_token)
  end

  # Forgets a user.
  def forget
    update_attribute(:remember_digest, nil)
  end

  # Creates and assigns the activation token and digest.
  def create_activation_digest
    self.activation_token = User.new_token
    self.activation_digest = User.digest(activation_token)
  end
end

что я должен сделать, чтобы правильно его реорганизовать, чтобы быть правильным?

извините, если я прошу слишком много помощи, но я просто учусь Rails и я не очень хорошо понимаем эту часть.

Answer 1

Эта статья действительно только высказывает мнение авторов. AbstractController::Helpers на самом деле имеет helper метод, который используется специально для включения помощников в контроллер, так что это вряд ли официально одобренное мнение.

Можно бесконечно обсуждать, где именно находится этот кусок кода, но так как вы собираетесь использовать его как в представлении, так и в контроллере, я бы сказал, просто поместите его в app/helpers и покончите с этим , Это имеет гораздо больший смысл, чем размещение его где-то еще и последующее ручное включение его в контекст представления.

Однако этот код, вероятно, следует разбить на два модуля, поскольку только часть его имеет смысл использовать из представления.

module SessionsHelper
  def logged_in?
    !current_user.nil?
  end

  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif (user_id = cookies.signed[:user_id])
      user = User.find_by(id: user_id)
      if user&.authenticated?(cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end
end

---

# app/controllers/concerns/authenticable.rb
module Authenticable
  def log_in(user)
    session[:user_id] = user.id
  end

  def log_off
    forget(current_user)
    session.delete(:user_id)
    @current_user = nil
  end

  def remember(user)
    user.remember
    cookies.permanent.signed[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_digest
  end

  def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_token)
  end
end

Answer 2

Чтобы удовлетворить моего супервизора (или рецензента кода), мне пришлось скопировать / вставить весь код помощника сеанса в каждый отдельный файл контроллера, так как его копирования в application_controller было недостаточно.