Question

我正在制作一个rails json api，它在控制器动作中使用服务对象，并基于服务中发生的事情，我必须渲染正确的json。这个例子看起来像这样。

star_service.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

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

Answer 1

它当然会顽固但仍然......

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

服务对象必须为任何巨大的复杂操作执行提供单一的公共方法。显然，该方法必须返回简单的值，告诉控制器操作是否成功完成。因此，它必须是true或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

服务对象返回状态

1 个答案: