学习 C，需要一些“贪婪”CS50 解决方案的帮助

Question

我对 C 很陌生。我来自 python 背景。我想知道我的代码哪里出错了。

我正在做 cs50 贪心问题。我的代码有什么问题？它适用于某些数字，但其他数字不起作用。我试图从用户那里得到一个输入，询问要返还多少零钱，然后计算我可以仅使用 $.25、$.10、$.05、$.01 返还的最小硬币数量

#include <cs50.h>
#include <stdio.h>


int main(void)
{
    float n;
    do
    {
        n = get_float("How much change is owed?\n");
    }
    while(n == EOF); 
    int minimumamountofcoins = 0;
    if (n/.25 >=1){
        do 
        {
            n -= .25;
            minimumamountofcoins++;
        }
        while (n/.25 >= 1);
    }
    if (n/.1 >=1){
        do
        {
            n -= .1;
            minimumamountofcoins++;
        }
        while (n/.1 >=1);
    }
    if(n/.05 >=1){
        do
        {
            n -= .05;
            minimumamountofcoins++;
        }
        while (n/.05 >=1);
    }
    if (n/.01 >=1){
        do
        {
            n -= .01;
            minimumamountofcoins++;
        }
        while (n/.01 >=1);
    }
    printf("The minimum amount of coins is %d\n", minimumamountofcoins);
}

新代码：（除了进入 4.2 时可以正常工作）

#include <cs50.h>
#include <stdio.h>


int main(void)
{
    float n;
    do
    {
        n = get_float("How much change is owed?\n");
    }
    while(n == EOF);

    int cents = (int)(n * 100);
    int minimumamountofcoins = 0;
    if (cents/25 >= 1){
        while (cents/25 >= 1)
        {
            cents -= 25;
            minimumamountofcoins++;
        }

    }
    if (cents/10 >= 1){
        while (cents/10 >= 1)
        {
            cents -= 10;
            minimumamountofcoins++;
        }
    }
    if(cents/5 >= 1){
        while (cents/5 >= 1)
        {
            cents -= 5;
            minimumamountofcoins++;
        }
    }
    if (cents/1 >= 1){
        while (cents/1 >= 1)
        {
            cents -= 1;
            minimumamountofcoins++;
        }
    }
    printf("The minimum amount of coins is %d\n", minimumamountofcoins);
}

Answer 1

由于您没有包含测试用例，所以我自己做了。以下是您的算法未返回正确答案的一些情况：

.04、.11、.17、.19、.21、.26、.32 等

这些情况在计算便士的数量时都失败了（在最后的 do-while 循环中），并且它们都返回了比它们应该少的一枚硬币。这是因为浮点数的错误。使用打印语句，我发现在计算最后一分钱的除法时，每次都会发生同样的事情：

n/.01 = 0.99999999

这显然不是故意的，因为这应该等于 1，最后一分钱应该加到总数中。因此，理论上有效的代码由于浮点数而被破坏。

为避免这种情况，您可以做很多事情，例如将美元和美分分别记录为整数，将条件更改为 n/.01 >= .9999 而不是 n/.01 >= 1，处理您正在做的金额计算为整数美分或任何其他数量的东西。

就个人而言，我更喜欢最后一种选择，将金额视为整数美分。这很容易做到，因为将美元转换为美分只需乘以 100。因此，最简单的方法是使用相同的算法，但使用整数除外。

因此，代码如下所示：

int main(){
    float n;
    //code that parses in the amount of money, n now stores that amount
    int cents = (int)(n * 100);
    int minimumamountofcoins = 0;
    if (cents/25 >= 1){
        while (cents/25 >= 1)
        {
            cents -= 25;
            minimumamountofcoins++;
        }

    }
    if (cents/10 >= 1){
        while (cents/10 >= 1)
        {
            cents -= 10;
            minimumamountofcoins++;
        }
    }
    if(cents/5 >= 1){
        while (cents/5 >= 1)
        {
            cents -= 5;
            minimumamountofcoins++;
        }
    }
    if (cents/1 >= 1){
        while (cents/1 >= 1)
        {
            cents -= 1;
            minimumamountofcoins++;
        }
    }
    printf("The minimum amount of coins is %d\n", minimumamountofcoins);
}