创建货币数据类型 C++

Question

我的家庭作业是创建一个名为Currency 的类，使用整数作为美元的基本类型。我必须使用我拥有的这些公共方法...... 现在我删除了“美分”部分和“减法”部分，因为我假设一旦我弄清楚如何对“美元”货币类型进行“加法”，我就可以做到这些。

无论我在“Currency Add(int Dollars);”中输入什么内容我无法让我的代码工作=。 我做过类似totalDollars += Dollars、返回totalDollars 或只是Dollars += Dollars 等操作。

输出总是垃圾：“-8353513636”。 或者有时我在构造函数中输入的任何数字。

我似乎无法让它传递数字并将其保留在那里（最终想让它保存运行总数），并在完成后输出它。我缩短了我的代码，因为我的代码的唯一问题是处理“类”/货币数据类型的部分。它就是行不通。

有一次它告诉我它无法从 int 转换为货币数据类型

这是我到目前为止的代码：

/* Program By: 
    Date: 4/29/2011
    Class: 



*/

#include <iostream>
#include <iomanip>
#include <fstream>
#include <cstring>
#include <cstdlib>
#include <cctype>
#include <cmath>
#include <ostream>

using namespace std;

// Currency Class

class Currency
{
private:

    int Dollars;

public:
    Currency();
    Currency(int Dollars);
    Currency Add(int Dollars);
    int GetDollars();
};

Currency::Currency()
{
    Dollars = 0;
}

Currency::Currency(int Dollars)
{
}

// Add
Currency Currency::Add(int Dollars)
{ 
 // Have put totalDollars += Dollars, etc.
// Have also put a "return" value here.
}

int Currency::GetDollars()
{
    return Dollars;
}


int main()

{
    Currency payroll;

    int  currDollars;

    cout << "Please enter a dollar mount" << endl;
    cin >> currDollars;

    payroll.Add(currDollars);

    cout << "Current Amount is: " << payroll.GetDollars() << endl;

    return 0;

}

我做错了什么？我已经为此花费了几个小时，我需要在明天之前完成=。 帮助非常感谢！！！！

Answer 1

好吧，我们这里有一个范围界定问题。

由于您的参数和成员都命名为 Dollars，因此您需要使用 this-Pointer 来指定要添加到哪个变量。

this->Dollars += Dollars;

如果不这样做，参数将隐藏该成员，因此您实际上只是将新金额添加到自身，然后将其丢弃。

Answer 2

以下是一些可能有所帮助的提示：

在您的货币类中，我建议使用命名约定，以便您可以将成员变量与构造函数和函数中的参数区分开来。似乎在区分 Dollars 成员变量和也名为 Dollars 的参数时可能会出现一些混淆。一种可能的命名约定是将成员变量命名为“my”，因为它们属于该类。还有其他方法可以使用 this 指针来处理这种情况，但您可能还没有了解指针。

class Currency
{
private:

    int myDollars;

public:
    Currency();
    Currency(int dollars);
    void Add(int dollars); // changed this to return void - see below
    int GetDollars();
};

我看到的另一个问题是，采用 int Dollars 参数的函数似乎没有对参数值执行任何操作。我知道您正在尝试弄清楚如何执行此操作，因此这里有一个 Add 函数的示例。对于 Add，我现在只是让它返回 void，这样您就不必处理从函数返回 this 对象的问题。

void Currency::Add(int dollars)
{ 
    myDollars += dollars;
}

int Currency::GetDollars()
{
    return myDollars;
}

我希望这会有所帮助。

Answer 3

每当您使用 C 或 C++、Java 或任何语言编写代码时，您确实需要注意变量名称。

这里发生的情况是，您在类中使用 Dollars 作为变量名称，在函数参数中使用 和 。该参数会覆盖该类，无论您对 Add 内的 Dollars 执行什么操作，都不会对该类内的 Dollars 变量产生任何影响。

尝试将 Dollars 参数重命名为诸如 x 之类的描述性名称。

Answer 4

我认为您不需要在 Add 函数中返回 Currency 。

您的 Add 函数应如下所示：


void 货币::Add(int DollarsToAdd) {
美元 += 美元添加；
}


Answer 5

您当前的代码存在很多问题。

• 当然，您需要两个整数来存储数据。
• 然后为每个值提供一个获取函数。
• 你我认为你通过尝试返回货币而使你的 Add 函数过于复杂。不需要，只需将其设为无效并将输入的金额添加到您的整数值中即可。
• 您需要处理超过 100 美分的情况。例如 56 + 56 = 112 美分...
• 您没有提到数据验证，这可能也需要执行。例如，不允许 alpha 和其他垃圾，并且不允许美分值 > 99 待输入。等等。

尝试一下：

class Currency
{
private:
    int m_dollars;
    int m_cents;

public:
    Currency(int dollars, int cents);
    Currency Add(int dollars, int cents);
    int GetDollars();
    int GetCents();
};

Currency::Currency(int dollars, int cents) 
{ 
    m_dollars = dollars; 
    m_cents = cents;
}

Currency Currency::Add(int dollars, int cents)
{ 
    if (m_cents + cents >= 100) 
        return Currency(m_dollars + dollars + 1, m_cents + cents - 100);

    return Currency(m_dollars + dollars, m_cents + cents);
}

int Currency::GetDollars() { return m_dollars; }
int Currency::GetCents() { return m_cents; }

void main()
{
    int currDollars;
    int currCents;

    cout << "Please enter a whole dollar amount:" << endl;
    cin >> currDollars;

    cout << "Please enter a cents amount:" << endl;
    cin >> currCents;

    Currency payroll(currDollars, currCents);

    cout << "Current Amount is: " 
         << payroll.GetDollars() 
         << "."
         << payroll.GetCents()
         << endl;

    cout << "Please enter a whole dollar amount to add:" << endl;
    cin >> currDollars;

    cout << "Please enter cents to add:" << endl;
    cin >> currCents;

    payroll = payroll.Add(currDollars, currCents);

    cout << "Updated Amount is: " 
         << payroll.GetDollars() 
         << "."
         << payroll.GetCents()
         << endl;
}