Question

我在这个类Transaction中有一个multimap，我在其中存储了日期的事务（我有另一个Date类）。实例化对象后，它将自动添加到多图中。

问题是在main函数中，在我实例化一个对象之后，事务的数量是2而不是1.

由于多图插入而导致复制构造函数被调用2次，而析构函数只被调用一次。

如何解决这个问题，除了再次减少构造函数中的事务数量？

class Transaction {
private:
    std::string note;
    float value;
    Date date;
    static unsigned int numberOfTransactions;
    static std::multimap<Date, Transaction, Date::Comparator> datedTransactions;

public:
    Transaction( Date date, std::string note, float value )
      {
        std::cout<<"Constructor is called";
        this->date = date;
        this->note = note;
        this->value = value;
        datedTransactions.insert(std::make_pair(date, *this));
    }

    ~Transaction() {
        std::cout<<"Destructor is called";
        numberOfTransactions--;
    }

    Transaction( Transaction const & t ) {
        std::cout<<std::endl<<"Copy constructor is called";
        note = t.note;
        value = t.value;
        numberOfTransactions++;
    }

    Transaction& operator=(Transaction const &t) {
        if (this != &t) {
            note = t.note;
            value = t.value;
        }
    }

    static unsigned int GetNumberOfTransactions() {
        return numberOfTransactions;
    }
};

int main() { // main should return int & not void
    Date date;
    Transaction (date, "dinner", 100);
    std::cout << std::endl << Transaction::GetNumberOfTransactions() << std::endl;
}

Answer 1

您似乎没有复制构造函数，因此通过复制创建的任何临时Transaction对象都不会插入到地图中，但在销毁时仍会减少计数器。

如果你有一个非平凡的析构函数，总是需要考虑如何正确定义复制构造函数和复制赋值运算符。阅读the rule of five。

要解决您的问题，您可以向该类添加bool标志，该标志仅对插入到地图中的对象设置为true，对于任何副本均为false。然后在析构函数中，如果标志为真，则只减少计数器。

Answer 2

如果要跟踪班级中的所有对象，则必须检测其构造函数的所有，包括复制（和移动）构造函数。

发生的事情是：

使用您提供的构造函数创建Transaction对象。
datedTransactions.insert(std::make_pair(date, *this));使用您班级的复制构造函数在对中创建一个临时Transaction对象。
地图中的新Transaction对象由临时移动构造函数初始化。
临时被破坏，这是你看到触发的析构函数。

对象计数复制构造函数比析构函数调用的次数多

2 个答案: