重载算术运算符C ++

问题描述 投票:-2回答:1

我必须执行以下操作:

Cash.h中,在Money命名空间内添加一个声明,以声明+操作的操作符重载;这应该使用两个Cash操作数,并返回一个新的Cash值。请记住,声明将由结果类型(Cash),操作名称(operator +)和两个参数((Cash a, Cash b))组成,后跟一个分号。

Cash.cpp中,在Money名称空间内为运算符添加一个定义。此定义应创建并返回一个新的Cash值,其总值为a.cents()b.cents()作为其cents()值。如果ab的面额不同,则需要选择一种均等适合于两种面额的面额。 Cash.cpp包含一个函数gcf,您可以使用该函数来确定适合的最大值(对于此分配,您不必保证它代表实际流通的硬币)。

注意:我无法编辑solution.cpp

//Cash.cpp

#include "Cash.h"
#include <cstdlib>
#include <utility>
#include <iostream>

    int gcf(int a, int b)
    {
        using std::swap;

        if (abs(a) < abs(b)) { swap(a, b); }
        if (b == 0) { return a; }
        a %= b;
        return a == 0? b: gcf(b, a);
    }

    namespace Money {
            Cash::Cash(int d, int c)
            : _count{c}, denomination{d} 
        {
            if (_count < 0 || denomination <= 0) {
                std::cerr << "Cash object constructed with invalid currency count.\n";
            }
        }

        // only code I can edit in this file, rest is locked
        const Cash operator + (const Cash& a, const Cash& b){
          Cash cents();
          cents() = a.cents() + b.cents();
          return cents();
        };    
    }
//Cash.h

#pragma once
#ifndef MONEY_CASH
#define MONEY_CASH

    namespace Money {
        class Cash {
        public:
            Cash(): Cash{1, 0} {}
            Cash(int d, int c);
            Cash(const Cash& other): Cash{other.denomination, other._count} {}

            int count() const { return _count; }
            int cents() const { return _count * denomination; }

            const int denomination;
        private:
            int _count;
        };

        // only code I can edit in this file, rest is locked
        const Cash operator + (const Cash& a, const Cash& b);
    }

    #endif
//Solution.cpp

#include <iostream>
#include "Cash.h"

int main(int argc, char* argv[])
{
    using namespace std;
    using namespace Money;

    int D, N;
    cin >> D >> N;
    Cash a {D, N};
    cin >> D >> N;
    Cash b {D, N};
    Cash c = a + b;
    cout << "Result: " << c.cents() << " cents in " << c.denomination << "s.";
    return 0;
}

使用当前代码,我得到the following error

./Cash.cpp: In function 'const Money::Cash Money::operator+(const Money::Cash&, const Money::Cash&)':
./Cash.cpp:28:39: error: no match for 'operator=' (operand types are 'Money::Cash' and 'int')
         cents() = a.cents() + b.cents();
                                       ^
In file included from ./Cash.cpp:1:
./Cash.h:7:11: note: candidate: 'Money::Cash& Money::Cash::operator=(const Money::Cash&)' <deleted>
     class Cash {
           ^~~~
./Cash.h:7:11: note:   no known conversion for argument 1 from 'int' to 'const Money::Cash&'
c++ overloading arithmetic-expressions
1个回答
1
投票

您现在遇到一个问题,一旦解决,您将遇到另一个问题。

第一个问题是

Cash cents();

这没有定义Cash类的对象。相反,它声明一个不带参数的function,并按值返回一个Cash对象。

就做

Cash cents;

定义一个对象。

第二个问题是

cents() = ...

cents是一个函数时很有意义,但不应该这样。当centsCash对象时,则无法调用它。将inta.cents() + b.cents()结果分配给Cash对象没有任何意义。

我想您应该设置_count成员:

cents._count = a.cents() + b.cents();
© www.soinside.com 2019 - 2024. All rights reserved.