如何避免以下代码的SEGMENTATION ERROR?

时间:2015-06-02 17:55:19

标签: c++ segmentation-fault maps dynamic-programming

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

我经历了How to avoid SIGSEGV?并且我确保不使用未初始化的指针,而不是访问内存不足等(给定n≤10 9 )。

我知道数组a[1000000000]会导致堆栈溢出,所以我使用了std::mapstd::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;
    }
}

1 个答案:

答案 0 :(得分: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位类型足以存储任何可能的答案。