创建适用于System.Object的扩展方法的好习惯？

我想知道是否应该创建适用于对象级别的扩展方法,或者它们是否应该位于类层次结构中的较低点.我的意思是:

public static string SafeToString(this Object o) {
    if (o == null || o is System.DBNull)
        return "";
    else {
        if (o is string)
            return (string)o;
        else
            return "";
    }
}

public static int SafeToInt(this Object o) {
    if (o == null || o is System.DBNull)
        return 0;
    else {
        if (o.IsNumeric())
            return Convert.ToInt32(o);
        else
            return 0;
    }
}
//same for double.. etc

我编写了这些方法,因为我必须处理很多数据库数据(来自OleDbDataReader),这些数据可以为null(不应该),因为底层数据库很可能非常宽松,列可能为null.为了让我的生活更轻松,我想出了那些扩展方法.

我想知道的是这是好风格,可接受的风格还是坏风格.我有点担心它,因为它有点"污染"对象类.

提前谢谢你和最好的问候:)

基督教

PS我没有故意将其标记为"主观".

不,这不是好的做法.您希望在尽可能低的点应用扩展方法.我相信(几乎)所有东西都有时间和地点,但扩展方法System.Object几乎永远不合适.您应该能够在继承堆栈中进一步应用扩展方法.否则它会混乱你的智能感知,并可能最终被其他开发人员错误地使用/依赖.

但是,用于处理Null值的数据对象的扩展方法是扩展方法的一个很好的用法.考虑将它们放在OleDbDataReader上.我有一个名为ValueOrDefault的通用扩展方法...好吧,我会告诉你:

 _
Public Function ValueOrDefault(Of T)(ByVal r As DataRow, ByVal fieldName As String) As T
    If r.IsNull(fieldName) Then
        If GetType(T) Is GetType(String) Then
            Return CType(CType("", Object), T)
        Else
            Return Nothing
        End If
    Else
        Return CType(r.Item(fieldName), T)
    End If
End Function

这是VB,但你得到了图片.这个吸盘节省了我很多时间,并且在读出数据流时真正能够获得干净的代码. 你是在正确的轨道上,但你的咒语感是正确的:你的扩展方法太高了.

将扩展方法放在一个单独的命名空间中总比没有好(这是命名空间的完全有效使用; Linq使用它),但你不应该这样做.要使这些方法适用于各种db对象,请将扩展方法应用于IDataRecord.

该框架设计指南建议你不要这样做。但是，这些准则特别适用于框架，因此，如果您发现它在您的业务应用程序中非常有用，请这样做。

但是请注意，在对象上添加这些扩展方法可能会使IntelliSense混乱，并可能使其他开发人员感到困惑。他们可能不希望看到这些方法。在这种情况下，只需使用老式的静态方法即可：-)

public static class SomeExtensions{
     static void Foo(this object o){…} } … Object o = … o.Foo();