Rails 应用程序中的计算逻辑应该放在哪里？

Question

我有一个为房子建模的应用程序。 House has_many Rooms、Rooms has_many Lights 和 Small_appliances 等。我还有一个名为 Calculator 的控制器，这就是访问应用程序的方式。使用计算器控制器将数据添加到房屋（及其房间）。然后生成一个报告，该报告位于app/views/calculator/report.html.erb。

我的问题是报告的所有计算和逻辑应该放在哪里？目前我已经在视图中看到了所有内容，其中一些内容在calculator_helper 中。通常这会出现在模型中，对吗？但计算器没有生成模型。这样做的标准是什么？

这是计算器控制器。

class CalculatorController < ApplicationController
  def index
  end

  def save_house
    @house = House.new(params[:house])
    respond_to do |format|
      if @house.save
        format.html { render :action => 'add_rooms', :id => @house }
        format.xml { render :xml => @house, :status => :created, :location => @house }
      else
        format.html { render :action => 'index' }
        format.xml  { render :xml => @house.errors, :status => :unprocessable_entity }
      end
    end
  end

  def add_rooms
    @house = House.find(params[:id])
    @rooms = Room.find_by_house_id(@house.id)

  rescue ActiveRecord::RecordNotFound
    logger.error("Attempt to access invalid house #{params[:id]}")
    flash[:notice] = "You must create a house before adding rooms"
    redirect_to :action => 'index'
  end

  def add_room
    @room = Room.new(params[:room])
    @house = @room.house

    respond_to do |format|
      if @room.save
        flash[:notice] = "Room \"#{@room.name}\" was successfully added."
        format.html { render :action => 'add_rooms' }
        format.xml { render :xml => @room, :status => :created, :location => @room }
      else
        format.html { render :action => 'add_rooms' }
        format.xml  { render :xml => @room.errors, :status => :unprocessable_entity }
      end
    end
  rescue ActiveRecord::RecordNotFound
    logger.error("Attempt to access invalid house #{params[:id]}")
    flash[:notice] = "You must create a house before adding a room"
    redirect_to :action => 'index'
  end

  def report
    flash[:notice] = nil
    @house = House.find(params[:id])
    @rooms = Room.find_by_house_id(@house.id)
  rescue ActiveRecord::RecordNotFound
    logger.error("Attempt to access invalid house #{params[:id]}")
    flash[:notice] = "You must create a house before generating a report"
    redirect_to :action => 'index'
  end
end

Answer 1

有几种方法可以处理它，但逻辑肯定不属于视图。如果我正确地阅读了您的描述，您可以在清晰的层次结构中相互关联各种模型，层次结构的顶部是 House 模型。在这种情况下，我将向 House 模型添加一组适当的方法，该方法可能由对与给定 House 实例关联的 Room 模型中的计算方法的调用以及关联线下的调用组成。这样，可以在每个级别执行相关计算，并且通过在 House 模型级别编写一个或多个方法，您可以采用一种干净、富有表现力且可维护的方式来处理计算。

要做的一件事是确保数据库可以执行任何计算。例如，如果 Room 模型可以通过简单地查询它自己的数据来完成计算，那么无论如何都要使用 ActiveRecord 调用此类较低级别计算逻辑的能力将该计算负担推给数据库。有关详细信息，请查看 API 文档。

我会非常仔细地查看您想要的逻辑，并了解如何将其推入模型中，因为这可能是它所属的位置，靠近计算的实际数据，并且位于专门表示该数据的类结构中；我不会创建一个模型只是为了处理计算逻辑，除非您确实需要出于某种原因持久存储计算。

Answer 2

我会在 RAILS_ROOT/lib/ 中创建一个名为 Calculator 的类，并将代码放入其中。

/lib/ 中的类应该加载到应用程序中的任何位置。

您还可以在 /app/models/ 中创建一个普通的 ruby​​ 对象。没有理由它们都必须继承 ActiveRecord::Base

Answer 3

好的，现在我可以看到发布的代码了。我可以看到计算器控制器实际上没有计算，它们在视图中吗？尝试这种方法：

1. 编写一个测试来设置一个对象，该对象将返回您需要返回给网页用户的结果（给定房屋、房间或任何其他需要的东西）。
2. 构建一个模型（在模型中）以使测试通过。
3. 修改上面的控制器代码以使用新的计算器模型
4. 修改控制器的测试，以便它们也通过。当然，这些测试不需要测试任何业务逻辑。

我之前的回答：

如果业务逻辑相当简单并且仅在该 Web 应用程序后面使用，那么您可以将其放在 app/models 文件夹中。

class MyCoolClass
  def initialize(clues)
    @other_things = OtherThing.all
  end
  def do_cool_thing; end
  def calculate_coolness
    @other_things.length 
  end
end

然后在控制器中创建模型的实例

def index
  @mcc = MyCoolClass "A clue as to what I want"
  render
end

然后在模板中可以访问它

<%=h @mcc.calculate_coolness %>

请注意，@other_things是MyCoolClass的instance__variable，并且通常无法在未定义访问器方法的情况下访问模板

Answer 4

这一切都取决于您创建的数据类型。计算器控制器是什么样子的？

您可以在 /lib 中创建自己的类并在模型中使用它们，这是将逻辑与控制器/帮助器分离的好方法。您无法将某些逻辑放入模型中是否有原因？