记忆应该有效

Question

没有记忆这个Euler Project 14 的解决方案可以正常工作！然后通过记忆，它应该工作得更快......但它几乎停在i = 1818 或附近。多么奇怪！很难理解有什么问题！你能帮忙吗？

#include <stdio.h>

#define limit 1000000

int arr[limit];

int fun(long long int i) {
    long long int count = 1;
    long long int num;
    arr[limit];
    num = i;
    while (num > 1) {
        if (arr[num] != NULL) {
            count = count - 1 + arr[num];
            break;
        }
        if (num % 2 == 0) {
            num = num / 2;
            count++;
        } else {
            num = 3 * num + 1;
            count++;
        }
    }
    arr[i] = count;
    return count;
}

int main() {
    long long int i;
    for (i = 2; i < limit; i++) {
        long long int count = fun(i);
        printf("d %lld c: %lld\n", i, count);
    }
    return 0;
}

Answer 1

好的，我认为您的代码的主要问题是 Collat​​z 序列可以为您提供的数字比您在下降到 1 之前开始的数字高很多。根据Project Euler 14，你应该找到低于 1000000 的起始数字，它会在达到零之前产生最长的链。但是从 1819 年开始的 Collat​​z 数列包括大于一百万的数字。因此，您正试图访问超出范围的 arr[] 元素。

此外，正如 cmets 中所指出的，arr[limit]; 函数中的声明 fun() 没有任何用处。如果您在编译器中启用了警告，它可能会标记这一点，以及将void* 指针与整数进行比较的语句if(arr[num]!=NULL)。

如果您将while() 块的第一条语句替换为if (num < limit && arr[num]!=NULL)，那么您至少应该避免分段错误。

您的main() 函数需要重写以找到产生最长链的起始数字，而不是仅仅打印一百万行数据。

如果您愿意，可以尝试运行它：

#include <stdio.h>

#define LIMIT 1000000

int arr[LIMIT] = { 0 };

long fun(long i) {
    long count = 1;
    long num;
    num = i;
    while (num > 1) {
        if (num < LIMIT && arr[num] != 0) {
            count = count - 1 + arr[num];
            break;
        }
        if (num % 2 == 0) {
            num = num / 2;
            count++;
        }
        else {
            num = 3 * num + 1;
            count++;
        }
    }
    arr[i] = count;
    return count;
}

int main(){
    long i, longest=0, maxstart;
    for (i=2; i<LIMIT; i++) {
        long count = fun(i);
        if (count > longest) {
            longest = count;
            maxstart = i;
        }
    }
    printf("%ld\n",maxstart);
    return 0;
}