0

（这不是作业，只是我正在使用的书中的一个练习）

“如果整数的因数（包括一个（但不包括数字本身））与该数字相加，则称其为完美数。例如，6 是一个完美数，因为 6 = 1 + 2 + 3。编写方法 Perfect判断参数值是否为完美数。在判断并显示2到1000之间的所有完美数的应用中使用此方法。显示每个完美数的因数，以确认该数字确实是完美的。

问题是它显示完美数字两次而不是一次。为什么要这样做？

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Perfect_Numbers2
{
class Program
{
static bool IsItPerfect(int value)
{
    int x = 0;

    bool IsPerfect = false;

    List<int> myList = new List<int>();

    for (int i = value; i == value; i++)
    {
        for (int j = 1; j < i; j++)
        {
            if (i % j == 0)  // if the remainder of i divided by j is zero, then j     is a factor of i
            {
                myList.Add(j); //add j to the list

            }

    }
        x = myList.Sum();
        // test if the sum of the factors equals the number itself (in which     case it is a perfect number)
        if (x == i)    
        {
            IsPerfect = true;

            foreach (int z in myList)
            {
                Console.Write("{0} ",z);

            }

            Console.WriteLine(".  {0} is a perfect number", i);
        }            

    }
    return IsPerfect;
}

static void Main(string[] args)
{
    bool IsItAPerfectNum = false;



    for (int i = 2; i < 1001; i++)
    {
        IsItAPerfectNum = IsItPerfect(i);

        if (IsItPerfect(i) == true)
        {

            Console.ReadKey(true);
        }


    }
}
}
}
4

2 回答 2

9

您调用IsItPerfect了两次，这会导致它两次评估该方法中的代码。该方法将数字写入控制台，因此它将数字显示两次。

您可以按如下方式重写代码，这将消除问题并阻止您执行相同的逻辑两次：

static void Main(string[] args)
{
    for (int i = 2; i < 1001; i++)
    {
        bool IsItAPerfectNum = IsItPerfect(i);

        if (IsItAPerfectNum)
        {
            Console.WriteLine("{0} is a perfect number", i);
            Console.ReadKey(true);
        }
    }
}

当然，Console.WriteLine从您的ItIsPerfect方法中删除相应的内容。

于 2013-08-30T19:35:31.227 回答
4

你打IsItPerfect(i)了两次电话，它包含一个Console.WriteLine(). 您需要删除IsItPerfect(i). if我还建议从您的方法中完全删除 UI - 这是不好的做法。

于 2013-08-30T19:38:47.147 回答