Ruby实现每处理20%项显示消息：寻求更优方案

你的原逻辑可行，但可以利用Ruby的Enumerable特性和更合理的判断逻辑来优化——既避免手动维护计数器、重复计算百分比的冗余，还能解决原代码可能存在的进度遗漏问题（比如总数量不是5的倍数时，==判断会跳过某些百分比节点）。

优化方案1：提前计算进度节点（高效简洁） #

提前算出每20%对应的处理数量，遍历过程中直接判断是否到达节点，减少循环内的计算量：

size = foo.get_items_size(items)
# 生成20%、40%...100%对应的处理数量节点
progress_milestones = (20..100).step(20).map { |percent| (size * percent / 100).to_i }
current_milestone = 0

items.each_with_index do |item, idx|
  # 这里写你的item处理逻辑

  processed_count = idx + 1
  if processed_count == progress_milestones[current_milestone]
    puts "#{(processed_count * 100 / size)}% items processed"
    current_milestone += 1 unless current_milestone >= progress_milestones.size
  end
end

优化方案2：简化百分比判断逻辑（极简风格） #

用each_with_index替代手动维护计数器，同时把判断条件改为>=，避免因整数除法导致的进度遗漏：

size = foo.get_items_size(items)
next_target_percent = 20

items.each_with_index do |item, idx|
  # 这里写你的item处理逻辑

  processed_percent = (idx + 1) * 100 / size
  if processed_percent >= next_target_percent
    puts "#{next_target_percent}% items processed"
    next_target_percent += 20
    break if next_target_percent > 100 # 避免超过100%后重复判断
  end
end

优化点说明 #

• 用each_with_index替代手动维护count变量，代码更符合Ruby的优雅风格，减少出错概率
• 把原代码的==判断改为>=，解决总数量非5倍数时的进度提示遗漏问题
• 提前计算进度节点的方案减少了循环内的重复计算，更适合处理大量数据的场景
• 去掉冗余变量，逻辑更清晰直观

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