Question

我想更快地知道什么。帮助我。

我在一个方法中声明了一个变量：

    public static Regex FindNumber()
{ return new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled); }

如您所见，它会返回正则表达式。

我还有另一个看起来像这样的方法：

    private static string TestOne(string RawData)
{
    Regex rgxFindNumber = FindNumber();
    Regex rgxFindDays = FindDays();
    for (int i = 0; i < mc.Count; i++)
    {
        int days = Convert.ToInt32(rgxFindNumber.Match(rgxFindDays.Match(mc[i].Value).Value).Value);
    }
    return RawData;
}

现在TestOne方法会更快还是TestTwo？

        private static string TestTwo(string RawData)
{
    for (int i = 0; i < mc.Count; i++)
    {
        int days = Convert.ToInt32(FindNumber().Match( FindDays().Match(mc[i].Value).Value).Value);
    }
    return RawData;
}

现在我很好奇，因为TestOne可以在我的代码中调用很多，所以我想知道什么是更好的实现。

谢谢你们。

**编辑：**我使用的代码有一个非常大的类。它是基于文本的策略游戏的文本解析器。我想重构一下，这就是我在这里想的。如果我为Regex创建一个私有变量，那么每次访问该类时都不会运行它吗？那是我的问题。

Answer 1

TestOne将比TestTwo更快，因为您没有为每次循环迭代创建新的正则表达式。

这有两个好处：

用于解析和构造正则表达式对象的时间只执行一次，而不是mc.Count次
减少垃圾收集的压力，因为构建的对象更少。

但是，我会更进一步。如果你总是要返回相同的正则表达式，并且你关心速度，我会将该正则表达式对象缓存在静态字段中。

例如，你可能会考虑这个：

private static Regex _FindNumber;
public static Regex FindNumber()
{
    if (_FindNumber == null)
        _FindNumber = new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled);
    return _FindNumber;
}

这将只创建一个对象，总数，并保持它。

然而，这是我的真实答案。

要真正知道哪一个会是最快的，你将不得不测量你的代码，可选地将我的变量用于测量，然后决定。如果没有硬数据，永远不要决定优化，你最终可能会花时间重写代码，这可能会引入新的错误，这需要修复，你将花费更多的时间，只能获得另外1％的性能。

大优化是通过算法完成的，比如改变排序算法的类型，然后只有在必要之后，你才会继续进行局部优化，如循环调优。

话虽如此，我至少会避免在循环中构造对象，这只是常识。

Answer 2

我相信TestOne会更快，因为在TestTwo中，每次循环时都会创建一个新的Regex对象。如果FindDays的实现与FindNumber相同，则会更糟，因为您将创建两个对象。

Answer 3

从技术上讲，TestOne会更快，因为TestTwo通过调用FindNumber（）来添加堆栈帧。

我不知道它会有多大的不同，我怀疑它是多少。你的方法是静态的，所以真的只是创建对象，这应该非常快。

我的问题是你为什么使用函数调用一遍又一遍地返回相同的字符串？你为什么不宣布一个真正的变量？

像，

private static Regex _findNumber = new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled);

局部变量和从方法调用的变量之间的区别？ C＃

3 个答案: