ruby-on-rails - 在助手中使用实例变量是一种不好的做法吗？

Question

我一直在重构一个旧项目，我发现一些辅助方法很难测试，因为辅助方法中存在实例变量。

实现此目标的最佳做法是什么？将这些实例变量作为参数传递给助手...

在测试中我可以做这样的事情，但看起来很奇怪

  it 'returns family and categoy names' do
    @category = instance_double(Category, name: 'category_name')
    expect(helper.meta_description_home_products).to eq(
      'blabla - categoria'
    )
  end

作为 Controller View 和助手的示例:

class HomeController < BaseController
  @family = ...
  @category = ...
  @products = ...
end

在 View 中

<% @products.each ...
  ...
  my_helper
...

在助手中:

  module ApplicationHelper
   def my_helper
     ...
     desctiption = 'blabla' if @category.name == 'blabla'

谢谢

Answer 1

我不会的。它紧密耦合了你的 Controller 和 View 层，使得 future 的功能、重写和重构变得困难。

它还使您的 Controller 和 View 难以阅读、测试和错误修复。

关注点和责任分离是需要考虑的问题。保留这些确实有助于长期项目。