有没有比这个 Ruby 习惯更好的替代方案？

Question

我发现自己经常在控制器中编写这段代码：

params[:task][:completed_at] = Time.parse(params[:task][:completed_at]) if params[:task][:completed_at]

不要沉迷于我在这里具体做的事情，因为原因每次都会改变；但在很多情况下，我需要检查 params 中的值并在将其交给 create 或 update_attributes 之前进行更改。

重复 params[:task][:completed_at] 三次感觉非常糟糕。有更好的方法吗？

Answer 1

稍微缩短这一过程的一种方法是：

if c = params[:task][:completed_at]
  params[:task][:completed_at] = Time.parse(c)
end

或者，您可能更喜欢这样：

params[:task][:completed_at] &&= Time.parse(params[:task][:completed_at])

在第二种情况下，仅当左侧为“真实”时才会发生分配。

Answer 2

我想你可以考虑做这样的事情。

在 String 和 NilClass 上实现 #to_time，可能在 extensions.rb 中（如 Ruby 最佳实践，例如，

require 'time'
class String
  def to_time
    Time.parse(self) # add error/exception handling to taste
  end
end

class NilClass
  def to_time
    nil
  end
end

然后您只需调用 params[:task][:created_at].to_time 即可，重复项就消失了。

我完全不确定这是否有效必然构成“最佳实践”，但恕我直言，它满足了问题的目标......

Answer 3

我对 Ruby 并不是非常熟悉，但由于它具有 Perl 根源，因此可能有一个构造允许您像这样编写它：

$_ = Time->parse($_) for params[:task][:completed_at] || ();

基本上利用 for 循环来创建变量的别名（如果存在），

可能类似于：

(params[:task][:completed_at] || ()).each { |i| i = Time.parse(i) }

编辑：

我发现 Ruby 有一个 alias 关键字。我对它不太熟悉，无法给出 Ruby 示例，但在 Perl 中，上面的内容也可以写成：

local *_ = \$params[$task][$completed_at];

$_ = Time->parse($_) if defined;

其中指定 $_ 将是 $params[$task] 的别名[$completed_at]

我尝试在 Ruby 中简单地使用它，但没有找到为标识符别名的方法，只有全局变量。