Ruby方法返回不同类型对象是否合理？是否属于不良实践？

这种返回不同类型对象的做法属于不良编程实践，原因主要有两点：

• 破坏返回值一致性，增加调用方负担：调用方必须每次判断返回值的类型才能正确处理，一旦遗漏判断（比如直接对结果执行整数运算），就会触发类型错误，增加了代码出错的概率和维护成本。
• 违背最小惊讶原则：其他开发者看到div方法的命名和功能，预期它返回的是除法运算的数值结果，突然返回一个Error对象，完全不符合直觉，降低了代码的可读性和可维护性。

是否违反单一职责原则？ #

其实并不直接违反单一职责原则。单一职责要求方法只负责一件事，div方法的核心职责是处理除法运算，返回错误信息本质上也是运算结果的一部分（表示运算失败）。问题的核心不是职责过多，而是返回类型不一致导致的接口不清晰。

用单个对象替代的实现方案 #

你提到的返回哈希是一种方式，但更优雅的是用封装结果的类来统一返回类型，既保证单个对象返回，又能清晰区分成功和失败状态。比如：

class DivResult
  attr_reader :value, :error

  def initialize(value: nil, error: nil)
    @value = value
    @error = error
  end

  # 提供便捷方法判断状态
  def success?
    error.nil?
  end

  def failure?
    !success?
  end
end

def div(a, b)
  if b == 0
    DivResult.new(error: "error: division by zero")
  else
    DivResult.new(value: a / b)
  end
end

# 使用示例
result = div(8, 2)
if result.success?
  puts "result of division: #{result.value}"
else
  puts result.error
end

这种方式既保持了返回值类型的一致性，又能清晰传递运算结果的状态，调用方不需要再判断对象类型，只需要通过success?或failure?方法即可处理不同场景，代码更健壮、可读性更强。

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