代码效率的问题

Question

public static int ToInt(this string str, int defValue) { if (string.IsNullOrEmpty(str) || str.Trim().Length >= 11 || !Regex.IsMatch(str.Trim(), @"^([-]|[0-9])[0-9]*(\.\w*)?$")) return defValue; int rv; if (Int32.TryParse(str, out rv)) return rv; return  defValue; }    public static int ToInt(this object obj, int defValue) { if (object.Equals(obj, null) || !Regex.IsMatch(obj.ToString().Trim(), @"^([-]|[0-9])[0-9]*(\.\w*)?$")) return defValue; int rv; if (Int32.TryParse(obj.ToString(), out rv)) return rv; return defValue;} 这两个代码使用起来效率是不是一样的

Answer 1

从绝对意义上来讲效率不一样。取决于执行逻辑、输入值等。
比较 object.Equals(object, null) 和 string.IsNullOrEmpty, object.Equals(object, object) 性能较差。
通过Reflector看反汇编:

[TargetedPatchingOptOut("Performance critical to inline across NGen image boundaries")]
public static bool Equals(object objA, object objB)
{
    return ((objA == objB) || (((objA != null) && (objB != null)) && objA.Equals(objB)));
}

如果用 obj == null 代替 object.Equals(object, null)则性能与string.IsNullOrEmpty相当。

假如用obj == null 直接比较，在 obj = string = "" 的情况下 (obj == null) 性能好于 string.IsNullOrEmpty(string). 原因不难理解，string.IsNullOrEmpty多了一个判断。

同样看一下string.IsNullOrEmpty反汇编的结果·
[TargetedPatchingOptOut("Performance critical to inline across NGen image boundaries")]
public static bool IsNullOrEmpty(string value)
{
    if (value != null)
    {
        return (value.Length == 0);
    }
    return true;
}

it depends:)
 
有了TryParse 就用它吧~~

Answer 2

把你第一个if删掉效率最高

Answer 3

我想不通了。既然你已经知道TryParse这个函数了，干吗还加上那么多多余的？

Answer 4

就是啊，转化为整数就tryparse就可以了，连异常处理也可以不用加