Question

我发现了一些与此问题相关的主题，我只是想知道为什么我的代码会返回错误的数据。所以我们必须找到第一个三角形数超过500的除数。详情请见：http://projecteuler.net/problem=12 这是我的代码：

Int64 triangularnum = 1;
            for (Int64 num = 2; num > 0; num++)
            {
                if(has501Divisors(triangularnum))
                {
                    MessageBox.Show(triangularnum.ToString());
                    break;
                }
                triangularnum += num;
            }



private bool has501Divisors(Int64 candidate)
        {
            bool has501 = false;
            int count = 0;
            for (int i = 1; i < Math.Sqrt(candidate); i++)
            {
                if (candidate % i == 0) count += 1;
                if (count > 501)
                {
                    return true;
                }
            }
            return has501;
        }

这给了我号码842161320，这显然是不正确的。

Answer 1

您应该将count号码增加2而不是1。

也是你的

if (count > 501)

部分不正确，因为您的边界500不是501。将其更改为count > 500而不是。

static void Main(string[] args)
{
    Console.WriteLine(Find());
}

public static int Find()
{
    int number = 0;
    for (int i = 1; ; i++)
    {
        number += i; // number is triangle number i
        if (CountDivisorsOfNumber(number) > 500)
            return number;
    }
}


private static int CountDivisorsOfNumber(int number)
{
     int count = 0;
     int end = (int)Math.Sqrt(number);
     for (int i = 1; i < end; i++)
     {
         if (number % i == 0)
             count += 2;
     }
     if (end * end == number) // Perfect square
         count++;
     return count;
}

这会打印76576500，看起来像是一个正确的解决方案。

Answer 2

问题是你将你的循环限制在平方根，这很聪明，但这意味着你需要将你的计数增加2而不是1来计算两个除数。

将增量更改为：

if (candidate % i == 0) count += 2;

此外，您的计数检查会检查超过501的除数，而不是500。

Answer 3

快速查看，但检查结果不合适：

if (count > 501)

这将在计数502处停止，而不是501.

for (int i = 1; i < Math.Sqrt(candidate); i++)

9可以被3整除，所以你应该使用＆lt; = here。此外，你将除以1，你应该从i = 2开始。

三角数的除数（Euler 12）

3 个答案: