RoR中使用安全导航运算符后比较值触发报错的解决方法

问题复现 #

控制器中最初的判断逻辑用于校验封面图片参数长度，符合要求时才执行附件挂载，原始代码如下：

recipe.cover_image.attach(data: params[:recipe][:cover_image]) if params[:recipe][:cover_image] && params[:recipe][:cover_image].length > 255

为了简化重复的参数判空逻辑，尝试用Ruby安全导航运算符&.改写：

recipe.cover_image.attach(data: params[:recipe][:cover_image]) if params[:recipe][:cover_image]&.length > 255

改写后运行抛出错误：

undefined method `>' for nil:NilClass

根因分析 #

安全导航运算符的逻辑是：如果调用者为nil，直接返回nil，不再执行后续的方法调用。当params[:recipe][:cover_image]不存在（值为nil）时，params[:recipe][:cover_image]&.length的返回值就是nil，此时表达式变成nil > 255，而NilClass没有定义>比较方法，自然会抛出方法不存在的错误。

可行解决方案 #

以下几种写法都可以解决问题，同时保持代码简洁：

• 数值兜底转换：对安全导航返回的可能nil值调用to_i做类型兜底。nil.to_i固定返回0，图片长度不可能为负，0必然小于255，和原始逻辑完全等价，写法最简洁：

recipe.cover_image.attach(data: params[:recipe][:cover_image]) if params[:recipe][:cover_image]&.length.to_i > 255

• 链式安全调用：比较运算符本质也是实例方法，可以在比较运算符前也加上安全导航符。当length为nil时，整个比较表达式直接返回nil，在条件判断中nil为假值，不会执行后续逻辑，也不会触发报错，适合Ruby 2.7及以上版本：

recipe.cover_image.attach(data: params[:recipe][:cover_image]) if params[:recipe][:cover_image]&.length&.> 255

• 语义化前置判断：如果觉得运算符写法可读性差，可以先提取参数做一次存在性判断，Rails环境下用present?判空语义更清晰，也不会增加太多代码量：

cover_image = params[:recipe][:cover_image]
recipe.cover_image.attach(data: cover_image) if cover_image.present? && cover_image.length > 255

注意：不推荐为了简化过度堆叠运算符，优先选择团队成员普遍能快速读懂的写法即可。

内容的提问来源于stack exchange，提问作者Andrés