静的メンバを使ったトータルカウント数の求め方

Question

c1のカウント数とc2のカウント数を足したものをトータルのカウント数で出力したいのですが、c2のカウント数が出力されます。足し方が分かりません。トータルのカウント数:4になってほしいです。
現状で以下のように出力されます。
m_count:1
m_totalCount:1
m_count:1
m_totalCount:1
m_count:2
m_totalCount:2
reset count:0
m_count:2
m_totalCount:2
m_count:3
m_totalCount:3
m_count:1
m_totalCount:1
c1のカウント数：3
c2のカウント数：1
トータルのカウント数:1

#include <iostream>
using namespace std;
class Counter {
private:
// 回数
int m_count;
static int m_totalCount;
public:
// コンストラクタ
Counter();
// リセット
void reset();
// カウント
void count();
// カウントの数
int getCount();

static int getTotalCount();

};

int Counter::m_totalCount = 0;

// コンストラクタ（カウント回数を0で初期化）
Counter::Counter() : m_count(0)
{

}
void Counter::reset() {

m_count = 0;

cout << "reset count:" << m_count << endl;

}
void Counter::count() {

m_count++;
m_totalCount = m_count;

cout << "m_count:" << m_count << endl;
cout << "m_totalCount:" << m_totalCount << endl;

}
int Counter::getCount() {

return m_count;

}
int Counter::getTotalCount() {

return m_totalCount ;

}
int main() {
Counter c1, c2;
c1.count();
c2.count();
c2.count();
c2.reset();
c1.count();
c1.count();
c2.count();
cout << "c1のカウント数：" << c1.getCount() << endl;
cout << "c2のカウント数：" << c2.getCount() << endl;

//Counter::getTotalCount()  =  c1.getCount() + c2.getCount();

cout << "トータルのカウント数:" << Counter::getTotalCount() << endl;
return 0;

}

Answer 1

別解をこしらえてみた

#include <iostream>
#include <set>

using namespace std;

class Counter {
private:
  //  回数
  int m_count;
  
  // この世に存在する全Counter
  static std::set<const Counter*> counters;

public:
  Counter();
  ~Counter();
  void reset();
  void count();
  int getCount() const; 
  static int getTotalCount();
}; 

std::set<const Counter*> Counter::counters;

Counter::Counter() : m_count(0) {
  // Counter誕生
  counters.insert(this); 
}

Counter::~Counter() { 
  // Counter死去
  counters.erase(this); 
}

void Counter::reset() { m_count = 0; }
void Counter::count() { m_count++; }
int Counter::getCount() const { return m_count; }

int Counter::getTotalCount() { 
  // この世の全Counterのカウントを集計する
  int result = 0;
  for ( const Counter* ctr : counters ) {
    result += ctr->getCount();
  }
  return result;
}

int main() {
  Counter c1;
  c1.count();
  {
    Counter c2;
    c2.count();
    c2.count();
    c2.reset();
    c1.count();
    c1.count();
    c2.count();
    cout << "c1のカウント数：" << c1.getCount() << endl;
    cout << "c2のカウント数：" << c2.getCount() << endl; 
    cout << "トータルのカウント数:" << Counter::getTotalCount() << endl;
  }
  cout << "c1のカウント数：" << c1.getCount() << endl;
  cout << "トータルのカウント数:" << Counter::getTotalCount() << endl;
} 

Answer 2

staticなクラス変数は、すべてのインスタンスに共通で一つしかありません。現状のコードだと、最後にcount()呼び出したインスタンスのカウント数でm_totalCountを上書きしてしまっています。おそらく、やりたいのはこんな感じのことでしょうか。

void Counter::reset() {

    m_totalCount -= m_count; // ここを追加
    m_count = 0;

    cout << "reset count:" << m_count << endl;
}

void Counter::count() {

    m_count++;
    m_totalCount++; // ここを変更

    cout << "m_count:" << m_count << endl;
    cout << "m_totalCount:" << m_totalCount << endl;

}

Comments

[sattsun]

実行できました。ありがとうございます。