Ruby中替代eval方法消除Rubocop告警的实现方案

为什么不推荐eval实现 #

你当前用eval拼接字符串执行断言的写法，除了触发Rubocop的安全类规则告警外，还存在明确的代码注入风险：如果传入的参数包含恶意构造的Ruby代码片段，会被直接执行，完全绕过安全检查。
你提到的Proc拼接方案不可行：Ruby的Proc不支持{| end_with(arg)}这种语法来拼接匹配逻辑，也完全不需要绕Proc这一层——RSpec的匹配器本身是可组合的一等对象，直接组合匹配器就能实现完全一致的断言逻辑，不需要动态执行字符串代码。

无eval替代实现 #

以下代码和你原有逻辑完全等价，无安全风险，符合Rubocop规范：

def expect_url_to_end_with_string(*args)
  # 按原有逻辑构造所有end_with匹配器
  # 如果你原有代码里的url_str是笔误、所有待匹配后缀都通过args传入，把下一行替换为
  # matchers = args.map { |arg| end_with(arg) } 即可
  matchers = [end_with(url_str), *args.map { |arg| end_with(arg) }]
  # 通过|组合匹配器，实现“匹配任意一个后缀即通过”的逻辑，和你原有拼接的|语义完全一致
  combined_matcher = matchers.reduce(:|)
  expect(driver.url).to combined_matcher
end

实现原理 #

• RSpec所有内置匹配器（包括end_with）都实现了|（逻辑或）和&（逻辑与）运算符，调用后会返回复合匹配器对象，可以直接传给to/not_to等断言方法
• 所有传入的参数只会被当作字符串后缀传入匹配器，不会被解析为Ruby代码，从根源上避免了代码注入问题
• 断言行为和原有eval实现完全一致，不需要修改已有测试用例的调用逻辑

如果你需要的是“URL同时匹配所有传入后缀”的与逻辑，只需要把reduce(:|)替换为reduce(:&)即可。

内容的提问来源于stack exchange，提问作者Piotr Nowakowski