带有嵌套循环问题的 C++ 完美数

Question

我想做的是寻找一个完美的数字。 完美数是其所有除数之和，例如 6 = 1+2+3。

基本上我在这里做的是要求 2 个数字并在这两个数字之间找到一个完美的数字。我有一个测试可分性和 2 个嵌套循环的函数。

我的问题是我没有得到任何结果。我已经对其进行了修改，似乎找不到任何问题。编译器不会抛出任何错误。

有什么问题？

#include <iostream>
using namespace std;

bool isAFactor(int, int);

int main()
{

int startval;
int endval;
int outer_loop;
int inner_loop;
int perfect_number = 0;

cout << "Enter Starting Number: ";
cin >> startval;
cout << "Enter Ending Number: ";
cin >> endval;

for(outer_loop = startval; outer_loop <= endval; outer_loop++)
{
    for(inner_loop = 1; inner_loop <= outer_loop; inner_loop++)
    {
        if (isAFactor(outer_loop, inner_loop) == true)
        {
            inner_loop += perfect_number;
        }
    }

if (perfect_number == outer_loop)
{
    cout << perfect_number << " is a perfect number." << endl;
}

else
{
    cout << "There is no perfect number." << endl;
}

}

system("PAUSE");
return 0;
}

bool isAFactor(int outer, int inner)
{
if (outer % inner == 0)
{
    return true;
}

else
{
    return false;
}

Answer 1

inner_loop += perfect_number; 应该是perfect_number += inner_loop;。

还有其他问题——您需要在每个外循环中将perfect_number 重置为零，如果范围内的数字都不完美，您应该打印消息"There is no perfect number."，而不是每次打印一次范围内的数字不完美。

我建议您将 perfect_number 重命名为 sum_of_factors、outer_loop 重命名为 candidate_perfect_number 和 inner_loop 重命名为 candidate_factor 或类似名称。

Answer 2

if 语句之后：

cout << perfect_number;
cout << outer_loop;

if (perfect_number == outer_loop)
{
    cout << perfect_number << " is a perfect number." << endl;
}

看看他们有什么价值观

更新：

你的 endval 值是多少？是 0？，这就是为什么循环这么早就结束了

Answer 3

哦，问题太多了。

1. 变量perfect_number 永远不会改变。你的编译器标志了吗 这个？
2. 外部循环退出时会比结束值大一； 你知道吗？
3. 您无需将 bool 值与 true 或 false 进行比较。
4. 您可以将isAFactor 函数简化为return (outer % inner) == 0;。
5. 您可以用表达式替换对isAFactor 的调用 ((outer % inner) == 0)。