学习C，需要一些“Greedy”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);
}