修补Ruby的基类是否可以接受，例如Fixnum？
我仍然是Ruby的新手（阅读Pickaxe并将大部分时间花在irb ），现在我知道可以在Ruby中修补类，我想知道什么时候这样做是可以接受的，特别是它是否可以接受修补Ruby的基类。 例如：我在这里回答了另一个Ruby问题，海报想知道如何从DateTime减去小时数。 由于DateTime类似乎没有提供此function，我发布了一个答案，将DateTime和Fixnum类修补为可能的解决方案。 这是我提交的代码：
require 'date' # A placeholder class for holding a set number of hours. # Used so we can know when to change the behavior # of DateTime#-() by recognizing when hours are explicitly passed in. class Hours attr_reader :value def initialize(value) @value = value end end # Patch the #-() method to handle subtracting hours # in addition to what it normally does class DateTime alias old_subtract - def -(x) case x when Hours; return DateTime.new(year, month, day, hour-x.value, min, sec) else; return self.old_subtract(x) end end end # Add an #hours attribute to Fixnum that returns an Hours object. # This is for syntactic sugar, allowing you to write "someDate - 4.hours" for example class Fixnum def hours Hours.new(self) end end
我修补了这些类，因为我认为在这种情况下，它会产生一个清晰，简洁的语法，用于从DateTime减去固定的小时数。 具体来说，由于上面的代码，你可以做这样的事情：
five_hours_ago = DateTime.now - 5.hours
这看起来相当不错，易于理解; 但是，我不确定是否应该搞乱DateTime的运算符function。
我能想到的唯一替代方案是：
1.只需动态创建一个新的DateTime对象，计算对new的调用中的新小时值
new_date = DateTime.new(old_date.year, old_date.year, old_date.month, old_date.year.day, old_date.hour - hours_to_subtract, date.min, date.sec)
2.编写一个实用程序方法，接受DateTime和从中减去的小时数
基本上，只是方法（1）的包装：
def subtract_hours(date, hours) return DateTime.new(date.year, date.month, date.day, date.hour - hours, date.min, date.sec) end
3.向DateTime添加新方法，而不是更改#-()的现有行为
也许一个新的DateTime#less方法可以与Fixnum#hours补丁一起使用，以允许这样的语法：
date.less(5.hours)
但是，正如我已经提到的，我采用了修补方法，因为我认为它导致了更具表现力的语法。
我的方法有什么问题，或者我应该使用3种替代品中的一种（或者我没想到的另一种）来做到这一点？ 我觉得补丁正在成为我对Ruby问题的新“锤子”，所以我想得到一些关于我是否采用“Ruby方式”做事的反馈。
简而言之，我的个人答案： 核心级修补锤应位于工具箱的底部 。 您可以使用许多其他技术，几乎在所有情况下，它们都足够，更清洁，更具可持续性 。
但这实际上取决于您编码的环境。 如果这是一个个人项目 – 确定，贴心你的内容！ 当您在很长一段时间内与大量程序员一起工作时，问题就开始出现了。 在我工作的组织中，我有超过100KLOC的Ruby代码库和大约20个左右的开发人员，我们已经开始严厉打击猴子修补，因为我们已经看到它导致头疼，工时浪费行为太常见了。 在这一点上，我们几乎只能容忍它暂时修补第三方代码，这些代码尚未合并或不会合并我们的源补丁。
就个人而言，我认为将方法添加到基类是可以接受的，但是修改现有方法的实现是不可接受的。
最安全的方法是定义自己inheritance自内置类的类，然后将新内容添加到新类中。
class MyDateTime < DateTime alias... def...
但显然现在只有在声明新类的对象时才会获得新的行为。
我认为它是这样的：如果你真的觉得大多数其他程序员会同意你的补丁，那很好。 如果没有，也许您应该改为实现代码库？