Question

这是我第一次在这里发帖，因此，如果我没有遵循正确的礼节，我深表歉意。我也尝试寻找答案，但无济于事。

基本上，我对贪婪的硬币兑换算法具有此功能，该算法将一些int作为输入。在我的函数中，我返回一个包含每个硬币的malloc数组。虽然它在大多数情况下都起作用，但是由于某种原因，任何硬币值会产生5、9，...（+ 4）个硬币作为最佳分布，即9等于（5 +1 + 1 +1 +1），或者650，即13个硬币，每个硬币50个，导致程序中止并显示以下消息：

hello: malloc.c:2401: sysmalloc: Assertion `(old_top == initial_top (av) && 
old_size == 0) || ((unsigned long) (old_size) >= MINSIZE && prev_inuse 
(old_top) && ((unsigned long) old_end & (pagesize - 1)) == 0)' failed.
Aborted (core dumped)

但是，不是5或5 + 4 + ...的每种硬币分配都可以工作。不知道该怎么办。

这是功能：

int* greedyAlg(int value)//computes and returns the optimal (minimum) number of denominations for a given value for US currency
    {
        //denominations
        int one = 1, five = 5, ten = 10, twenty = 20, fifty = 50;
        int x5 = 0, x4 = 0, x3 = 0, x2 = 0, x1 = 0;
        int count = 0;
        //int[] denom;

    while(value != 0)
    {
        if(value >= fifty)
        {
            value -= fifty;
            count++;
            x5++;

            /*while(value >= fifty)
            {
                // int *i = &fifty;
                value-=fifty;
                count++;
                x5++;
            }*/
        }
        else if(value < fifty && value >= twenty)
        {
            value -= twenty;
            count++;
            x4++;
        }
        else if(value < twenty && value >= ten)
        {
            value -= ten;
            count++;
            x3++;
        }
        else if(value < ten && value >= five)
        {
            value -= five;
            count++;
            x2++;
        }
        else if(value < five && value >= one)
        {
            value -= one;
            count++;
            x1++;
        }
    }
    //printf("Optimal denominations: ");
    int* denom = malloc(sizeof(int)*(count + 1));
    //int* denom = (int *)calloc(count + 1,sizeof (int));

    denom[0]=count;
    for(int i = 1; i<= (x5 + 1); i++){
        denom[i] = fifty;
    }
    for(int i= (x5 + 1); i<=(x5 + x4) + 1; i++){
        denom[i] = twenty;
    }
    for(int i = (x5 + x4) + 1; i <= ( x5 + x4 +x3 ) + 1; i++){
        denom[i] = ten;
    }
    for(int i = (x5 + x4 + x3) + 1; i <= (x5 + x4 + x3 + x2) + 1; i++){
        denom[i] = five;
    }
    for(int i = (x5 + x4 + x3 + x2) + 1; i <= (x5 + x4 + x3 + x2 + x1) + 1; i++){
        denom[i]=one;
    }
    return denom;
    free(&denom);
    //return count;
}

这就是我的打印方式：

//prints elements of array created by (greedy) coin change algorithm
void printGreedyArr(int arr[], size_t count)
{
    for(int i = 1; i <= count; i++)
    {
        printf("%s%d%s, ","[",arr[i],"]");
    }
    printf("\n%s %d\n","count was",count);
}

我用第0个索引来称呼它，其长度如下：

printGreedyArr(greedyAlg(x),greedyAlg(x)[0]);

（在我的代码中，我使用x作为用户输入来建立循环以进行测试）

如果需要，我可以发布任何其他相关详细信息。

Answer 1

假设string等于count，您将得到一个错误的错误：

x5+x4+x3+x2+x1

应该是：

for(int i=(x5+x4+x3+x2)+1; i<=(x5+x4+x3+x2+x1)+1; i++){

对于其他for(int i=(x5+x4+x3+x2)+1; i<(x5+x4+x3+x2+x1)+1; i++){循环也是如此。请注意，终止条件已从for更改为<=。

也：

return denom; free(&denom);将永远不会执行，并且如果将free()放在其他地方，也应将其从&之前删除。

使用特定输入的Malloc中止

1 个答案: