C++構造体の引数を使用した関数を使いたい

Question

//Sc.h

    class Sc{
       public :

       struct kouzou{int hoge;float hoge;int hogege;}
       void mein();
       void set(kouzou k);

       vector<kouzou> ks;
    }

とゆうヘッダがあるとします。

//Sc.cpp

#include "Sc.h"

void Sc::mein(){
for(int i ;i<ks.size();i++){cout<< ks[i]<<endl;}
set();
}

void Sc::set(){
kouzou k;
k={1,12,133};
ks.push_back(k);
}

上のSc.cppようにすると、set関数内のkouzouでポインタの問題が出てきます。
newを使い、kouzouを新しくインスタンスとして作成してもうまくいきません。。

アドバイスをお願いいたします。

Answer 1

コンパイル通るようにテキトーに修正しました。

#include <vector>
#include <iostream>

class Sc {
public :

  struct kouzou{int hoge;float fhoge;int hogege;};
  void mein();
  void set();

  std::vector<kouzou> ks;
};

void Sc::mein(){
  for(int i = 0; i < ks.size(); i++) {
    std::cout << ks[i].hoge << ':' << ks[i].fhoge << ':' << ks[i].hogege << std::endl;
  }
  set();
}

void Sc::set(){
  kouzou k = {1,12,133};
  ks.push_back(k);
}

int main() {
  Sc x;
  for ( int i = 0; i < 5; ++i ) {
    std::cout << "---------- " << i << std::endl; 
    x.mein();
  } 
}

...で、何が問題なのかわかりません。

Comments

[seri]

ありがとうございます。参考にさせていただきます

[episteme]

...で、何が問題だったのですか?

Answer 2

C++の場合、中括弧によるstructの初期化は宣言時にしかできません。あとから代入する形では、コンパイルも通らないと思います。

kouzou k = {1,12,133};

余談ですが、フィールドのhogeが重なっているのは単なるミスでしょうか。