1

我正在制作一个rails json api，它在控制器操作中使用服务对象，并根据服务中发生的情况来呈现正确的json。该示例如下所示。

星服务.rb

class Place::StarService
  def initialize(params, user)
    @place_id = params[:place_id]
    @user = user
  end

  def call
    if UserStaredPlace.find_by(user: user, place_id: place_id)
      return #star was already given
    end

    begin
      ActiveRecord::Base.transaction do
        Place.increment_counter(:stars, place_id)
        UserStaredPlace.create(user: user, place_id: place_id)
      end
    rescue
      return #didn't work
    end

    return #gave a star
  end

  private

  attr_reader :place_id, :user
end

places_controller.rb

def star
  foo_bar = Place::Star.new(params, current_user).call

  if foo_bar == #sth
    render json: {status: 200, message: "sth"}
  elsif foo_bar == #sth
    render json: {status: 200, message: "sth"}
  else
    render json: {status: 400, message: "sth"}
end

我的问题是，如果我应该从服务对象返回纯文本还是有更好的方法？

4

1 回答 1

1

当然会是固执己见，但仍然...

用数据渲染视图、返回数据、重定向等是控制器的职责。因此，您必须在控制器中处理任何数据、纯文本和其他内容。

服务对象必须为执行任何巨大的复杂操作提供一个单一的公共方法。显然，该方法必须返回简单的值，告诉控制器操作是否成功完成。所以它必须是trueor false。也许是一些可识别的结果（对象、简单值）或errors散列。这当然是理想的用例，但这是重点。

至于您的用例，您的服务可能会返回消息或false. 然后控制器将该消息呈现为json.

而且您的star方法必须存在于您的控制器中，可能是私有的，并且看起来像这样：

def star
  foo_bar = Place::Star.new(params, current_user).call

  if foo_bar
    render json: {status: 200, message: foobar} 
  else
    render json: {status: 400, message: "Failed"}
  end
end

您的服务：

class Place::StarService
  def initialize(params, user)
    @place_id = params[:place_id]
    @user = user
  end

  def call
    if UserStaredPlace.find_by(user: user, place_id: place_id)
      return "Message when star is already given"
    end

    begin
      ActiveRecord::Base.transaction do
        Place.increment_counter(:stars, place_id)
        UserStaredPlace.create(user: user, place_id: place_id)
      end
    rescue
      return false
    end

    return "Message if gave a star"
  end

  private

  attr_reader :place_id, :user
end
于 2017-02-03T20:20:52.817 回答