【C++】構造体の変数をpublicからprivateに変えたい

Question

前提・実現したいこと

● 前提
自販機を制御するコードを作成しています。
商品の在庫数を構造体で保存しています。

商品はwater, greentea, coffee で
初期値はすべて'0'です。

● 実現したいこと
商品在庫を保存する構造体の変数を
publicではなく、privateにしたいです。
(※ 他クラスからの変更を制限するためです。)

該当のソースコード

現状は、構造体[ProductStock]を
publicにしている状態でビルドは通っていますが
privateに変える書き方が分かりません。

class CalcVendingMachine {
    public:
        struct ProductStock
        {
            u32_t water;
            u32_t greentea;
            u32_t coffee;
        };
        struct ProductStock list_stock = {0, 0, 0};
    private:
        // privateにする書き方が分からない.
};

試したこと

● 構造体をそのままprivate:に入れる

class CalcVendingMachine {
    public:
    private:
        struct ProductStock
        {
            u32_t water;
            u32_t greentea;
            u32_t coffee;
        };
        struct ProductStock list_stock = {0, 0, 0};
};

試した結果発生している問題・エラーメッセージ

エラーメッセージ
error: ‘struct VendingMachine::CalcVendingMachine::ProductStock’ is private

Answer 1

構造体の定義と、構造体の変数の宣言の関係がわかってないかな...

#include <cstdio>
#include <cstdint>

class CalcVendingMachine {
 public:
  struct ProductStock { //この構造体の定義を公開したいならpublic
    uint32_t water;
    uint32_t greentea;
    uint32_t coffee;
  };
  void setWater(uint32_t w) { 
    list_stock.water = w; 
  }

 private:
  ProductStock list_stock = {0, 0, 0};  // C++ではCと違って「タグ名」だけで型名として成立
};

int main() { 
  CalcVendingMachine machine;
  CalcVendingMachine::ProductStock stock; //ProductStock構造体はpublic
  stock.water = 0; //OK
//  machine.list_stock.water = 0; //list_stock変数はprivateなのでエラー
  machine.setWater(0); //メンバ関数の中でprivateメンバーが使われるのはもちろんOK
}

がご所望の状況になりますでしょうか。

Comments

[howto_o]

thkana様
ご回答いただきありがとうございます。
すみませんが、もう少し教えていただきたいのですが
提示いただいたコード中の、下記の箇所について
ProductStock構造体自体 を privateにして非公開にすることは可能なのでしょうか。
もしくは、list_stock変数をprivateにできていれば、そこまでする必要はないのでしょうか。

struct ProductStock { //この構造体の定義を公開したいならpublic
uint32_t water;
uint32_t greentea;
uint32_t coffee;
};

[thkana]

あなたが自分のやりたいことを見せてないのでなんとも言えませんが、
error: ‘struct VendingMachine::CalcVendingMachine::ProductStock’ is private
というエラーが出るようなことをしたいのなら、つまりそれはProductStock構造体をクラスの外で使いたいのでしょう? だったらpublicにしないと仕方ないのでは?
> そこまでする必要はないのでしょうか。
あなたがどこまでしたいのか、説明して下さい。

少なくとも、
class C{
private:
int num;
}
としたとき、intがアクセス可能じゃないか? と心配する人は多分いないでしょう。intは型なのですから。
同じように、ProductStockをprivateにするかしないかで悩む必要はほとんどないと思うのですけれど?

Answer 2

ProductStockの宣言は別にprivateにする必要がないのでは