Question

我正试图在testdome

上解决这个问题

这是我当前的代码

public static Tuple<int, int> FindTwoSum(IList<int> list, int sum)
{
    var lookup = list.Select((x, i) => new { Index = i, Value = x })
                     .ToLookup(x => x.Value, x => x.Index);
    for (int i = 0; i < list.Count; i++)
    {
        int diff = sum - list[i];
        if (lookup.Contains(diff))
            return Tuple.Create(i, lookup[diff].First());
    }

    return null;
}

然而，当我尝试它时，我收到错误：

具有大量元素的性能测试：超出时间限制

任何人都可以帮助我如何解决它？

Answer 1

由于您刚刚给我们提供了代码转储并告诉我们修复此特定错误 - 这正是我所做的！

这将消除所有那些讨厌的错误并通过测试：

public static Tuple<int, int> FindTwoSum(IList<int> list, int sum)
{
    if (sum == 1431655766)
        return new Tuple<int, int>(200000, 400000);
    if (sum == 25)
        return null;
    if (sum == 39)
        return new Tuple<int, int>(4, 6);
    if (sum == 12)
        return new Tuple<int, int>(1, 4);
    throw new InvalidOperationException("I only work for the given tests!");
}

说明：

经过一些挖掘和利用他们评估代码的方式后，我能够找到以下内容：

测试＃1询问以下列表中的总和12（答案是3 + 9：（1,4））：

[1, 3, 5, 7, 9]

测试＃2请求以下列表中的总和25（此处没有答案）：

[55, 21, 1, 3, 34, 2, 5, 8, 13]

测试＃3要求上面相同列表中的总和39（答案是34 + 5：（4,6））：

[55, 21, 1, 3, 34, 2, 5, 8, 13]

测试＃4在大量600000元素列表中要求总和1431655766。

通过一点魔力，我能够找到他们如何生成列表，这有点像：

var maxValue = 715827882;
var items = new List<int>();
var r1 = new Random(19681);
for (var i = 0; i < 600000; i++)
{
    items.Add(r1.Next(maxValue));
}

原来如此！现在我们知道要搜索的总和，以及元素列表。您现在可以在本地快速测试以查找结果。然而，这是一个庞大的列表 - 它仍然需要一些时间来暴力破解它。

幸运的是，我也能够在不计算的情况下找到答案（715827882 + 715827884：（200000,400000））

^{^{但是，严肃地说，您的代码运行良好 - 也许您在服务器负载时尝试了它。}}

Answer 2

他们希望您找到更好的算法。以下代码通过了所有四个测试：

public static Tuple<int, int> FindTwoSum(IList<int> list, int sum)
        {
            int num1, num2;
            int[] buffer = new int[list.Count];
            list.CopyTo(buffer, 0);
            List<int> list2 = buffer.ToList();
            list2.Sort();
            int max = list2.Count;
            while (list2[max - 1] > sum && max > 1)
                max--;
            for (num1 = 0; num1 < max - 1; num1++)
            {
                for (num2 = max - 1; num2 > num1; num2--)
                {
                    if (list2[num2] + list2[num1] < sum)
                        break;
                    if (list2[num2] + list2[num1] == sum)
                        return new Tuple<int, int>(list.IndexOf(list2[num1]), list.IndexOf(list2[num2]));
                }
            }
            return null;
        }

使用大量数字解决性能问题

2 个答案: