0 
string s = "abcabcabcabcabc";

var foundIndexes = new List<int>();

问题来自这里的讨论。我只是想知道

这怎么可能：

for (int i = s.IndexOf('a'); i > -1; i = s.IndexOf('a', i + 1))  

      foundIndexes.Add(i);

比这更好：

for (int i = 0; i < s.Length; i++)

      if (s[i] == 'a') foundIndexes.Add(i);

编辑：性能增益来自哪里？

4

2 回答 2

2

我没有观察到 usingIndexOf比直接循环更快。老实说，我不明白这是怎么回事，因为在这两种情况下都需要检查每个字符。我的初步结果是这样的：

Found by loop, 974 ms 
Found by IndexOf 1144 ms

编辑：再运行几次后，我注意到您必须运行发布（即优化）才能看到我上面的结果。如果没有优化，for 循环确实比较慢。

基准代码：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text.RegularExpressions;
using System.Text;
using System.IO;
using System.Diagnostics;

namespace Test
{
    public class Program
    {
        public static void Main(string[] args)
        {
            const string target = "abbbdbsdbsbbdbsabdbsabaababababafhdfhffadfd";

            // Jit methods
            TimeMethod(FoundIndexesLoop, target, 1);
            TimeMethod(FoundIndexesIndexOf, target, 1);            

            Console.WriteLine("Found by loop, {0} ms", TimeMethod(FoundIndexesLoop, target, 2000000));
            Console.WriteLine("Found by IndexOf {0} ms", TimeMethod(FoundIndexesIndexOf, target, 2000000));           
        }

        private static long TimeMethod(Func<string, List<int>> method, string input, int reps)
        {
            var stopwatch = Stopwatch.StartNew();
            List<int> result = null;
            for(int i = 0; i < reps; i++)
            {
                result = method(input);
            }
            stopwatch.Stop();
            TextWriter.Null.Write(result);
            return stopwatch.ElapsedMilliseconds;
        }

        private static List<int> FoundIndexesIndexOf(string s)
        {
            List<int> indexes = new List<int>();

            for (int i = s.IndexOf('a'); i > -1; i = s.IndexOf('a', i + 1))
            {
                 // for loop end when i=-1 ('a' not found)
                indexes.Add(i);
            }

            return indexes;
        }

        private static List<int> FoundIndexesLoop(string s)
        {
            var indexes = new List<int>();
            for (int i = 0; i < s.Length; i++)
            {
                if (s[i] == 'a')
                indexes.Add(i);
            }

            return indexes;
        }
    }
}
于 2012-10-16T15:09:41.313 回答
0

IndexOf(char value, int startIndex)标记有以下属性：[TargetedPatchingOptOut("Performance critical to inline across NGen image boundaries")]。

此外，此方法的实现很可能在许多其他方面进行了优化，可能使用不安全的代码，或者使用更多“本机”技术，例如使用本机FindNLSStringWin32 函数。

于 2012-10-16T16:07:22.773 回答