为何检查哈希中key值为布尔true的if条件未按预期工作？

原始数据与需求 #

你有一个嵌套哈希：

cars = {
  :honda => {year: 2008, is_condition_good?: true},
  :toyota => {year: 2010, is_condition_good?: false}
}

需要返回所有is_condition_good?为true的嵌套哈希，你知道用select方法可行：

cars.select { |car, attributes| attributes[:is_condition_good?] == true }

但你自己写的each迭代代码无法达到预期，想知道问题所在：

cars.each do |car, attributes|
  if attributes[:is_condition_good?] == true then
     return attributes
  end
end

问题分析 #

你的代码核心问题出在return的使用上，还有冗余的判断逻辑：

• return的作用范围错误：
  如果这段代码在顶级作用域（不在任何方法内）运行，return会触发LocalJumpError——因为顶级上下文没有可返回的方法或块，代码会直接报错终止，看起来像是“没进入if语句”。
  如果代码在方法内部，return会直接结束整个方法，只返回第一个符合条件的嵌套哈希，无法实现select那样收集所有符合条件结果的效果。
• 冗余的布尔判断：attributes[:is_condition_good?]本身就是布尔值，没必要额外和true做比较，直接写if attributes[:is_condition_good?]即可。

修正方案 #

1. 实现和select一致的效果（收集所有符合条件的哈希） #

用each_with_object手动收集结果：

# 基础写法
result = {}
cars.each do |car, attributes|
  result[car] = attributes if attributes[:is_condition_good?]
end

# 更简洁的链式写法
result = cars.each_with_object({}) do |(car, attributes), hash|
  hash[car] = attributes if attributes[:is_condition_good?]
end

2. 仅获取第一个符合条件的嵌套哈希 #

用find方法更高效：

first_good_car = cars.find { |car, attributes| attributes[:is_condition_good?] }&.last

注：你提到在each之前打印attributes[:is_condition_good?]有结果，说明哈希的键和取值逻辑是正确的，核心问题还是return的使用场景错误。

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