在Rails中重构视图逻辑

Question

这是我需要做的。 我有一个Tournament模型，该模型通过Signup （N：N）连接到User 。

Signup唯一添加的是Signup状态。 比赛有开始时间，用户只能注册，直到比赛开始前60分钟为止。 之后，注册用户可以签入。所以基本上，我有两种状态选择

简而言之，模型看起来像这样

class Signup < ActiveRecord::Base
  REGISTERED = 0
  CHECKED = 1

  belongs_to :tournament
  belongs_to :user
end

class Tournament < ActiveRecord::Base
  has_many :signups
  has_many :users, :through => :signups
end

class User < ActiveRecord::Base  
  has_many :signups
  has_many :tournaments, :through => :signups
end

为了使内容简短，我跳过了一些代码。 问题在于视图，因为我要记住很多条件。 这是我的实际代码（ 使用Slim作为模板引擎 ）

- if logged_in?
  - if current_user.registered_for?(@tournament)
    - if @tournament.starts_at < 60.minutes.from_now
      p Signups are closed, only registered users can now check in
      - if current_user.registered_for?(@tournament)
        = button_to "Checkin!", { :controller => :signups, :action => :update, :id => @tournament.id }, :method => :put
    - else
      = button_to "Cancel your registration for the tournament", { :controller => :signups, :action => :destroy, :id => @tournament.id }, :method => :delete
  - elsif current_user.checked_in?(@tournament)
    p You have already checked in.            
  - elsif @tournament.starts_at > 60.minutes.from_now
    = button_to "Sign up for the tournament", :controller => :signups, :action => :create, :method => :post, :id => @tournament.id
  - else
    p
      | The tournament starts in less than 60 minutes, you can't sign in
- else
  p 
    | You need to 
    |  
    = link_to "log in", login_path
    |  to play

问题是，我不知道如何使它更清洁。 我的意思是，我可以为按钮添加助手，但是if if else else丑陋的if if else else ，那将无济于事，因为有许多不同的组合。 这是一个简短的清单：

• 用户未登录
• 直到比赛开始且用户尚未注册才超过60
• 在比赛开始之前已经超过60，并且用户已经注册
• 不到60分钟，但用户尚未注册
• 不到60分钟且用户已注册但尚未签到
• 不到60分钟，用户已经签到

这只是冰山一角，因为管理员应该比普通用户看到更多的信息，但是我不想使这个问题复杂化。

主要问题是，我应如何处理此类案件？ 在视图中这样做似乎太糟糕了，但是我看不到任何其他更简单的方法。

Answer 1

一种更干净的方法是在模型上创建有意义的方法。 例如，在锦标赛模型中，添加以下内容：

def can_register?( user )
  !user.registered_for?(self) && self.starts_at > 60.minutes.from_now
end

然后在您看来，您可以检查can_register? 在展示东西之前。 像您一样向视图中添加逻辑并不是MVC应用程序想要的。

Answer 2

您应该使用一个对象来封装逻辑。 也许是这样的：

class UserSignup

  def initialize(user, tournament)
    @user, @tournament = user, tournament
  end

  def registered?
    @user.registered_for?(@tournament)
  end

  def signups_closed?
    @tournament.start_at < 1.hour.from_now
  end

  def checked_in?
    @user.checked_in?(@tournament)
  end

end

这使得视图更加简单，并且不需要太多工作。 您会看到许多重复项将通过这种方式被删除，并且您可以独立于视图来测试注册逻辑。

您还可以创建一个演示者，这涉及更多，但可以更清晰地整理视图。 看看像draper这样的宝石可以帮助您。

class SignupPresenter

  def initialize(user_signup)
    @user_signup = user_signup
  end

  def register_button
    view.button_to("sign up") if @user_signup.registered?
  end

  # etc ...

end

另外，我会考虑为不同的用户使用不同的模板甚至控制器。 因此，完全没有登录的用户甚至无法访问该页面，并且管理员拥有一个不同的控制器（甚至是名称空间）。

我不会只是将其分成多个部分，因为这只会隐藏逻辑。 除了将其放入模型之外，我还更喜欢一个单独的对象，因为这样一来，模型就不会变得混乱不堪，并且所有逻辑都保持在一起，并且集中精力。