Question

我的编程课中有一个挑战，我们必须使用void函数来计算可能的硬币组合，给定的变化值从1到99美分。

到目前为止，我的代码看起来像这样：

    #include <iostream>

using namespace std;

void computeCoins(int coinValue, int& num, int& amount_left);

int main()
{
    //Define varibles then get user input for change
    int leftOver, quarters=0, dimes=0, nickels=0, pennies=0, coins=0, originalAmount;
    do
    {
    cout << "Enter change amount: ";
    cin >> originalAmount;
    leftOver = originalAmount;
    } while ((leftOver > 99) || (leftOver < 1)); //Larger than 99 or smaller than 1? If yes, then try again.

    //Quarters
    computeCoins(25, coins, leftOver);
    quarters = coins;

    //Dimes
    computeCoins(10, coins, leftOver);
    dimes = coins;

    //Nickels
    computeCoins(5, coins, leftOver);
    nickels = coins;
    pennies = leftOver;

    cout << originalAmount << " cent/s could be given as " << quarters << " quarter/s, " << dimes << " dime/s, " << nickels << " nickel/s, " << " and " << pennies << " pennies.";
    cout << endl;
    system("PAUSE");
    return 0;
}

void computeCoins(int coinValue, int& num, int& amount_left)
{
    //Using the specified coin value, find how many can go into it then subtract it
    while (amount_left % coinValue == 0)
    {
        // Still dividable by the coin value
        num += 1;
        amount_left -= coinValue;
    }
}

现在我的问题是当我运行程序时，它会返回一个非常大的负值，包括四分之一，硬币和镍币。我很肯定它与我的循环条件设置有关，有没有人知道为什么会发生这种情况？

Answer 1

两个问题：一个未定义的硬币初始值。两个amount_left % coinValue == 0部分 - 我认为你的意思是amount_left >= coinValue

虽然没有必要继续在该函数中进行迭代

void computeCoins(int coinValue, int& num, int& amount_left)
{
    // add as many coinValues as possible.    
    num += amount_left / coinValue;
    // the modulus must be what is left.
    amount_left = amount_left % coinValue;
}

请注意（除其他事项外），您最好使用unsigned ints来处理大量事情。

Answer 2

当我读到你的问题时，你似乎应该寻找一种方法来获得所有可能的组合。奥利弗马修斯＆＃39;答案处理第一部分（计算出你可以在变化中适应多少特定类型的硬币），但是你必须在一个检查各种其他组合的循环中这样做（例如所有的便士，所有镍币，硬币，硬币，硬币，硬币，硬币等等，并且需要一种方法来返回组合（例如，通过输出参数返回一些处理硬币计数的结构/类的向量 - 即，参考文献的载体）。

使用while循环的C ++可能的硬币组合

2 个答案: