如何避免以下代码的分段错误？

Question

我收到以下代码的分段错误。这是the SPOJ problem "Coins"的解决方案。

我检查了How to avoid SIGSEGV?，并确保不要使用未初始化的指针，不要访问内存不足等（给定 n ≤ 10⁹）。

我知道数组a[1000000000] 会导致堆栈溢出，所以我使用了std::map。 std::map 会导致堆栈溢出吗？我的代码有什么问题？

#include <iostream>
#include <cstring>
#include <algorithm>
#include <cstdio>
#include <cmath>
#include <map>

using namespace std;

map<unsigned long long int, unsigned long long int> a;

unsigned long long int dp(unsigned long long int n)
{   
   if (a.find(n) == a.end())
      a[n] = dp(n/2) + dp(n/3) + dp(n/4);

   return a[n];
}

int main()
{
    for (unsigned long long int i = 1; i <= 24; i++) {
        a[i] = i;
        if (i == 12 || i == 24)
           a[i] = i + 1;
    }

    unsigned long long int n = 0;
    cin >> n;

    while (!feof(stdin)) {
        printf("%llu\n", dp(n));
        cin >> n;
    }
}

Answer 1

您在dp(0) 通话中获得 SIGSEGV。它会导致无限递归。

顺便说一句，你的解决方案是错误的，例如24的答案不是25。尽量避免使用魔法常数，设置a[0] = 0并制作更准确的dp函数就足够了：

uint32_t dp(uint32_t n) {
    if (a.find(n) == a.end())
        a[n] = max(n, dp(n / 2) + dp(n / 3) + dp(n / 4));

    return a[n];
}

从上面可以看出，32位类型足以存储任何可能的答案。