Question

我已经完成了我的代码并尝试使其尽可能高效。仍然我不能让这个程序一直运行而不冻结我的盒子。我让它在没有冻结的情况下运行的最长时间约为2小时。我无法相信这不会在那段时间内执行。

这是我的电脑的问题还是这种解决这个问题的强力方法效率低下。

在将来编写方法时，有哪些方法可以避免这种低效率？

private void Form1_Load(object sender, EventArgs e)
    {
        ArrayList listOfAbundantNumbers = new ArrayList();
        ArrayList listOfSums = new ArrayList();
        long total = 0;

        for (int i = 1; i < 20162; i++)
        {
            if (isAbundandt(i))
            {
                listOfAbundantNumbers.Add(i);
            }
            total+=i;
        }

        for (int i = 1; i < listOfAbundantNumbers.Count; i++)
        {
            for (int a = 0; a < listOfAbundantNumbers.Count; a++)
            {
                long temp1 = Convert.ToInt64(listOfAbundantNumbers[i]);
                long temp2 = Convert.ToInt64(listOfAbundantNumbers[a]);
                long num = temp1 + temp2;
                if(listOfSums.Contains(num) == false)
                {
                    listOfSums.Add(num);
                }
            }
        }

        for (int i = 1; i < listOfAbundantNumbers.Count; i++)
        {
            long temp1 = Convert.ToInt64(listOfAbundantNumbers[i]);
            total -= temp1;
        }

        printLn(total + "");
    }

    private ArrayList divisorList(long input)
    {
        ArrayList divisors = new ArrayList();
        for (long i = 2; i < Math.Round(Math.Sqrt(input),0,0); i++)
        {
            long temp = input % i;
            if (temp == 0)
            {
                divisors.Add(i);
                divisors.Add(input / i);
            }
        }
        return divisors;
    }

    private Boolean isAbundandt(long input)
    {
        long sum = 0;
        ArrayList divisor = divisorList(input);

        for (int i = 0; i < divisor.Count; i++)
        {
            long temp1 = Convert.ToInt64(divisor[i]);
            sum += temp1;
        }
        sum++;

        if (sum > input)
        {
            return true;
        }
        return false;
    }

Answer 1

避免使用ArrayList，更喜欢List<Int64>这样的通用集合，因为转换可能非常昂贵。

您的程序中有一部分位于 O（n ^ 3）中。它可以很容易地减少到 O（n ^ 2）。只需替换以下代码（ O（n），因为List<T>.Contains）：

if(listOfSums.Contains(num) == false)
{
    listOfSums.Add(num);
}

由：

listOfSums.Add(num);

其中listOfSums现在是HashSet<Int64>（它曾经是一个没有重复的集合）。

此外，只需注意如果您交换listOfSums和i，a就是sum，您可以将填充listOfSums的时间减少2倍同样，只有两个组合中的一个被添加到for (int i = 1; i < listOfAbundantNumbers.Count; i++) for (int a = 0; a < listOfAbundantNumbers.Count; a++)。

您可以替换：

for (int i = 1; i < listOfAbundantNumbers.Count; i++)
    for (int a = 0; a <= i; a++)

通过;

private static void Main()
{
    List<Int64> listOfAbundantNumbers = new List<Int64>();
    HashSet<Int64> listOfSums = new HashSet<Int64>();
    long total = 0;

    for (int i = 1; i < 20162; i++)
    {
        if (isAbundandt(i))
        {
            listOfAbundantNumbers.Add(i);
        }
        total += i;
    }

    for (int i = 0; i < listOfAbundantNumbers.Count; i++)
        for (int a = 0; a <= i; a++)
        {
            long temp1 = Convert.ToInt64(listOfAbundantNumbers[i]);
            long temp2 = Convert.ToInt64(listOfAbundantNumbers[a]);
            long num = temp1 + temp2;
            listOfSums.Add(num);
        }

    for (int i = 1; i < listOfAbundantNumbers.Count; i++)
    {
        long temp1 = Convert.ToInt64(listOfAbundantNumbers[i]);
        total -= temp1;
    }

    Console.WriteLine(total + "");
}

private static List<Int64> divisorList(long input)
{
    List<Int64> divisors = new List<Int64>();
    for (long i = 2; i < Math.Round(Math.Sqrt(input), 0, 0); i++)
    {
        long temp = input % i;
        if (temp == 0)
        {
            divisors.Add(i);
            divisors.Add(input / i);
        }
    }
    return divisors;
}

private static Boolean isAbundandt(long input)
{
    long sum = 0;
    List<Int64> divisor = divisorList(input);

    for (int i = 0; i < divisor.Count; i++)
    {
        long temp1 = divisor[i];
        sum += temp1;
    }
    sum++;

    if (sum > input)
    {
        return true;
    }
    return false;
}

最终代码是：

{{1}}

项目Euler＃23在C＃中执行效率低下的代码

1 个答案: