Ruby 如何使用 if..else 块的结果分配变量

发布于12月13日

我和一位同事就在if中分配变量的最佳方式进行了争论..否则就封锁.他的原始代码是:

@products = if params[:category]
  Category.find(params[:category]).products
else
  Product.all
end

我这样重写:

if params[:category]
  @products = Category.find(params[:category]).products
else
  @products = Product.all
end

这也可以使用ternery操作符(？:)用一行重写，但让我们假设产品分配长度超过100个字符，不能放在一行中.

这两者中哪一个对你来说更清楚？第一个解决方案占用的空间少了一点，但我认为声明一个变量并在三行之后分配它可能更容易出错.我也喜欢看到我的if和else对齐，这让我的大脑更容易解析它！

推荐答案

作为badp's answer中语法的替代方案，我想提出:

@products = 
  if params[:category]
    Category.find(params[:category]).products
  else
    Product.all
  end

我认为这有两个好处:

它确实需要额外的一行代码，这通常是我不喜欢的，但在这种情况下，用垂直极简主义来交换水平极简主义似乎是值得的.

Disclaimer:这是我自己独特的方法，我不知道它在Ruby社区的其他地方被使用到了什么程度.

Edit:我应该提到的是，matsadler's answer也和这个相似.我确实认为有some个缩进是有帮助的.我希望这足以证明这是一个单独的答案.

实用课程推荐