C程序错误 - 输出0

我一直试图让这个工作几天，我仍然无法找出错误。当我输出代码时，它会打印，但它不会找到友好的对（第一个==秒的除数，反之亦然）。C程序错误 - 输出0

#include <stdio.h> 
#include <stdlib.h> 
#define _USE_MATH_DEFINES 
#include <math.h> 

int sumDivisors(int num); 

int sumDivisors(int num) 
{ 
    int counter, total; 
    for(counter = 1; counter < num; counter++) 
    { 
     if(num % counter == 0) 
     { 
      total += counter; 
     } 
    } 
    return (total); 
} 

int main(void) 
{ 
    int higher, lower, lowx, lowy, x, y, numOfPairs = 0; 

    printf("This program finds all amicable numbers within a range. \n"); 
    printf("Please enter a lower limit: \n"); 
    scanf("%d", &lower); 
    printf("Please enter a higher limit: \n"); 
    scanf("%d", &higher); 


    for(lowx = lower; lowx <= higher; lowx++) 
    { 
     for(lowy = lower; lowy <= higher; lowy++) 
     { 
      if(sumDivisors(lowx) == sumDivisors(lowy)) 
      { 
       numOfPairs++; 
       printf("Pair #%d: (%d, %d)\n", numOfPairs, lowx, lowy); 
      } 
     } 
    } 

    printf("There are %d amicable pairs from %d to %d\n", numOfPairs, lower, higher); 
    system("pause"); 
    return (0); 
}

来源

2016-09-20 Submersed24

考虑写内环为'为（洛伊= lowx + 1;洛伊<=更高;洛伊++）'，以避免重复。 –

好点，它使它更有效率。谢谢！ – Submersed24

在进一步思考时，你可能误解了友好数字的定义，它是[两个不同的数字，因此每个数字的正确除数的和等于另一个数字]（https://en.wikipedia.org/维基/ Amicable_numbers）。您可以在[OEIS - A063990]（http://oeis.org/A063990）上找到第一对清单。看看这个[Q＆A]（http://stackoverflow.com/a/28268160/4944425）进行更有趣的优化。 –

您还没有给予任何价值在你的代码以总价：

int sumDivisors(int num) 
{ 
    int counter, total; 
    for(counter = 1; counter < num; counter++) 
    { 
     if(num % counter == 0) 
     { 
      total += counter; 
     } 
    } 
    return (total); 
}

使其包含垃圾不可预测的价值！

它应该是这样的：int counter, total = 0;

来源

2016-09-20 21:54:17

哦，天哪......谢谢.. 。你救了我，我被用来归零0 – Submersed24

@ Submersed24它与null为零有什么关系？ – immibis

我曾经认为变量默认为0，所以你不需要将它们设置为零 – Submersed24

C程序错误 - 输出0

回答

相关问题