为什么会出现Segmentation故障?

时间:2015-08-06 15:41:18

标签: c++ segmentation-fault

所以,我正在解决一个类似这样的问题:

给定n个整数的列表,A={a1,a2,…,an},和另一个整数,k代表预期的总和。从A中选择零个或多个数字,使这些数字的总和尽可能接近预期sum (k)但不超过预期的A

请注意

0的每个元素都可以多次选择。 如果未选择任何元素,则总和为n k,

输入格式

第一行包含T测试用例的数量。 每个测试用例包括两行。第一行包含两个整数,A分别代表列表a1,a2,…,an,的长度和预期的总和。第二行由n个空格分隔的整数组成,A表示列表1 ≤ T ≤ 10 1 ≤ n ≤ 2000 1 ≤ k ≤ 2000 1 ≤ ai ≤ 2000, where i∈[1,n] 的元素。

约束:

T

输出格式

输出(k)行,每个测试用例的最大总和尽可能但不超过预期的总和 int knapsack(int arr[],int n, int Sum, int dp[][1000]) { if ( n < 0 || Sum < 0 ) return 0; if(n==0||Sum==0) { dp[Sum][n] = 0; return 0; } if (arr[n-1] == Sum) { dp[Sum][n-1] = arr[n-1]; return arr[n-1]; } else if (dp[Sum][n] != -1) return dp[Sum][n]; else if(arr[n-1] > Sum) { dp[Sum][n] = knapsack(arr,n-1,Sum,dp); return dp[Sum][n]; } else //gets selected or doesn't get selected { dp[Sum][n] = max( arr[n-1] + knapsack(arr,n,(Sum-arr[n-1]),dp) , knapsack(arr,n-1,Sum,dp) ); return dp[Sum][n]; } }

以下是问题链接:https://www.hackerrank.com/challenges/unbounded-knapsack

现在,我开发了一种自上而下的方法,如下所示:

1
5 9
3 4 4 4 8

但是,当输入为:

时,上面给出了Seg故障
ROLLUP

我试过调试它但是在很多递归调用之后它在函数开头显示了一个Seg-fault。我错过了任何条件吗?

2 个答案:

答案 0 :(得分:1)

else

dp[Sum][n] = max( arr[n-1] + knapsack(arr,n,(Sum-arr[n-1]),dp) , knapsack(arr,n-1,Sum,dp) );

它也应该是n - 1,因为无论如何你都完成了当前的元素。就像现在一样,它会进行比必要更多的递归调用。使用此修复程序,我的PC上的段错误消失,函数返回0

dp[Sum][n] = max( arr[n-1] + knapsack(arr,n-1,(Sum-arr[n-1]),dp) , knapsack(arr,n-1,Sum,dp) );

此程序正确返回8作为您示例的答案:

int knapsack(int arr[],int n, int Sum, int dp[][1000]);



int main()
{
    int t;
    int n,k;
    cin>>t;
    int i,j;
    int dp[1000][1000];
    for(int i=0;i<t;i++)
    {
        for ( i = 0; i < 1000; i++ )
            for ( j = 0; j < 1000; j++ )
                dp[i][j] = -1;
        int a[2000];

        cin>>n>>k;

        for(int j=0;j<n;j++)
            cin>>a[j]; // you had i here

        while (knapsack(a,n - 1,k,dp) == 0) // lower k until we can build it exactly
            --k;

        cout << k << endl;

    }
    return 0;
}

// knapsack(n, Sum) = true if we can use the first n elements to build Sum exactly
int knapsack(int arr[],int n, int Sum, int dp[][1000])
{
    if (Sum < 0 )
        return 0;
    if(n < 0)
    {
        return Sum == 0;
    }
    else if (dp[Sum][n] != -1)
        return dp[Sum][n];
    else //gets selected or doesn't get selected
    {
        dp[Sum][n] = knapsack(arr,n-1,(Sum-arr[n]),dp) || knapsack(arr,n-1,Sum,dp);
    }

    return dp[Sum][n];
}

如果您可以多次使用相同的元素,我建议使用简单的一维数组进行以下迭代方法:

dp[0] = true
s = 0
for i = 0 to number of elements:
  s += elements[i]
  for j = elements[i] to s:
    dp[j] = dp[j] || dp[j - elements[i]]

dp[x] = true if we can build sum x

答案 1 :(得分:0)

你的另一个错误是:

/dev/zero

注意你的意思是j