0

我注意到在 Rails 教程中，作者有时会制作只使用一次的辅助方法。这让我觉得很荒谬，但在学习 ActionCable 时，我注意到 DHH 在 2 年前对 ActionCable 的介绍中做了同样的事情。所以也许有一点我错过了。只是为了说明，这里是来自 DHH 的代码。

# app/jobs/message_broadcast_job.rb
class MessageBroadcastJob < ApplicationJob
  queue_as :default

  def perform(message)
    ActionCable.server.broadcast 'room_channel', message: render_message(message)
  end

  private
    def render_message(message)
      ApplicationController.renderer.render(partial: 'messages/message', locals: { message: message })
    end
end

现在我只需编写类似这样的 perform 方法，并跳过辅助方法：

def perform(message)
  ActionCable.server.broadcast 'room_channel', { 
    message: ApplicationController.renderer.render(
      partial: 'messages/message', 
      locals: { message: message }
    )
  }
end

第一个结构比我的有什么好处吗？我只看到一个毫无意义的抽象。

4

2 回答 2

1

您所写的内容很好，但在 ruby​​ on rails 中，有一些流程需要遵循，您可以选择是否遵循。

我们使用助手的原因如下。

  1. 在应用程序中组织好代码。
  2. 使用 DRY 概念，这样，如果需要相同的辅助方法，可以由多个方法或操作使用。

希望它清楚在 ruby​​ on rails 应用程序中使用帮助程序的目的。

感谢和问候哈伦德

于 2017-10-30T18:59:22.993 回答
0

创建辅助方法（或与此相关的任何方法）有两个正当理由：

  1. 使一段代码可重用
  2. 使任何阅读代码的人都清楚代码的目的

我认为两者本身都是有效的。您应该问自己，创建辅助方法是否会增加应用程序逻辑的清晰度，并使任何人（甚至您）更容易理解正在发生的事情。如果是这样的话，我认为创建一个只会使用一次的助手是很好的。

在大型应用程序中，我发现清晰度通常比几行额外的代码更重要。以后有人甚至可能会发现它的另一种用途。

于 2017-10-30T23:45:34.160 回答