面试题：Ruby代码审查之模块与类的设计

Ruby 代码实现

module MathUtils
  def factorial(n)
    return 1 if n <= 1
    n * factorial(n - 1)
  end

  def fibonacci(n)
    return [] if n == 0
    return [0] if n == 1
    result = [0, 1]
    (2...n).each do |i|
      result << result[i - 1] + result[i - 2]
    end
    result
  end
end

class Calculator
  include MathUtils
  attr_accessor :result

  def initialize
    @result = 0
  end
end

代码审查关注点

模块设计
- 单一职责：MathUtils模块应只专注于数学相关的工具方法，确保每个方法都围绕数学计算功能，不包含其他无关逻辑。
- 方法命名：factorial和fibonacci方法名应清晰直观，准确描述其功能，便于其他开发者理解和使用。
- 错误处理：对于factorial方法，应确保输入非正整数时（如负数）有合理的处理逻辑，当前实现仅处理了n <= 1的情况，负数处理可进一步完善。fibonacci方法输入负数或非整数时也应处理。
类设计
- 类的职责：Calculator类应明确其主要职责，当前类使用MathUtils模块，但实例变量@result目前未与MathUtils方法很好地结合使用，应确保类的功能设计合理。
- 模块混入：使用include MathUtils将模块混入类中，需确保模块方法与类的设计理念相契合，不会产生命名冲突等问题。
- 实例变量：@result实例变量的使用应符合类的整体设计，其赋值和更新逻辑应清晰，当前仅初始化未在方法中使用，应进一步完善。
方法实现
- 算法正确性：factorial方法递归实现的正确性，确保边界条件处理正确；fibonacci方法生成斐波那契数列的算法逻辑正确，结果符合预期。
- 性能优化：对于factorial方法，递归实现可能在计算较大数时性能不佳，可考虑使用迭代实现优化。fibonacci方法生成数列时，可探讨是否有更高效的算法（如矩阵快速幂算法用于快速计算斐波那契数）。
- 代码可读性：方法内部的代码结构应清晰，注释应适当，如factorial和fibonacci方法中可添加注释解释算法逻辑和边界条件处理。

面试题：Ruby代码审查之模块与类的设计

知识考点

面试题答案

Ruby 代码实现

代码审查关注点