Ruby条件重构咨询：简化含params的条件判断方法

嘿，这个问题我之前也跟Reek较劲过，它对重复调用的警告确实挺严格的，但刚好有几个更符合Ruby风格的简洁方案，比单纯赋值变量还优雅，给你列出来：

1. 提取变量（稳妥且易读） #

这是最基础但永远不会出错的方案，先把重复调用的params[:smth].to_s存到变量里，彻底解决Reek的警告，同时可读性拉满：

smth_str = params[:smth].to_s
a = smth_str == 'foo' ? 'foo2' : smth_str

如果想压缩成一行（不影响可读性的前提下），也可以这么写：

a = (smth_str = params[:smth].to_s) == 'foo' ? 'foo2' : smth_str

2. 使用case表达式（优雅且可扩展） #

如果以后可能要增加更多匹配条件，case比三元运算符更灵活，代码结构也更清晰：

smth_str = params[:smth].to_s
a = case smth_str
    when 'foo' then 'foo2'
    else smth_str
    end

就算现在只有一个条件，这种写法也比三元运算符更符合Ruby的“面向表达”风格。

3. 用then链式调用（现代Ruby简洁写法） #

Ruby 2.6及以上支持的then方法可以让你链式处理值，不需要额外变量，代码流畅又简洁，Reek绝对不会报重复使用的警告：

a = params[:smth].to_s.then { |str| str == 'foo' ? 'foo2' : str }

这种写法把转换和判断逻辑串在一起，没有冗余的变量声明，非常适合简单的条件转换场景。

总结 #

• 如果追求最高可读性，优先选提取变量的方案；
• 如果以后可能扩展条件，case表达式是更好的选择；
• 如果喜欢现代Ruby的简洁风格，then链式调用是最优解。

内容的提问来源于stack exchange，提问作者Robert Rusiecki