構造体ポインタ引数の扱い(代入方法)

Question

mainファイル内でOBJ型のobjグローバル変数を作り、
その変数のアドレスを [OBJ load(OBJ *obj)]関数に引数として渡し、
load関数内で作ったOBJ型のobj変数に引数で受け取ったアドレス先の値を代入し、
load関数で作ったobj変数に値を代入しようとしたのですが、エラーが出てしまいます。
自分でも調べてみたのですが、どこがいけないのかわかりませんでした。
どなたか教えていただければ幸いです。<( _)>

環境 : visual studio 2019 // c++

// main.cpp

#include <vector>
#include "test.h"
OBJ obj;

int main() {
	load(&obj);
	
	return 0;
}

// test.h

#pragma once
#include <vector>

struct TEST {
	int a;
	int b;
	int c;
};

struct OBJ {
	std::vector <int>	key;
	std::vector <TEST>	test;
};

OBJ load(OBJ* _obj);

// test.cpp

#include <stdio.h>

#include "test.h"

OBJ load(OBJ* _obj) {

	OBJ obj = *_obj;

	obj.test[1].a = 120;

	printf("test[1].a=%d\n", obj.test[1].a);

	return obj;
}

Answer 1

グローバル変数objのメンバのvectorに何も値が入っていないのに

    obj.test[1].a = 120;

    printf("test[1].a=%d\n", obj.test[1].a);

としてしまっているからでしょう(しかも添え字が0でもなく1になっていますね)。vectorはあくまでコンテナ(入れ物)なので、中身は用意してやらないといけません。

Comments

[hiru.jp]

koichisugiyamaさんご返答ありがとうございます。
koichisugiyamaさんが教えてくださったことを参考にいろいろな代入方法を考えてみたいと思います
ありがとうございました。(>_<)

Answer 2

objは定義してもその中身はなにも初期化されてません

Comments

[hiru.jp]

y_waiwaiさんご返答ありがとうございます。
y_waiwaiさんのおっしゃる初期化とはコンストラクタのことでしょうか？

struct TEST {
int a;
int b;
int c;
};

struct OBJ {
std::vector <int> key;
std::vector <TEST> test;
};

この二つのオブジェクトのメンバ変数を
int a=0;
int b=0;
int c=0;
にするか、コンストラクタで初期化すればいいのでしょうか？
その場合だと、<vector>型のメンバ変数も初期化したほうがいいのでしょうか？

[y_waiwai]

別回答で述べられてますが、
objのナカミは空なのに
> obj.test[1]
ってどっから来てるんだ？というはなしですね

[hiru.jp]

y_waiwaiさんご返答ありがとうございます
vector<>は配列を扱えるので上のようなソースコードでやってたんですが、
いままでのソースコード見てみたらvector<構造体>=構造体 [配列]というような形だったのに
上のソースコードはいろいろ段階を吹き飛ばしていました。
いろいろとありがとうございました。(>_<)

Answer 3

int load(OBJ* _obj);
OBJ load(OBJ* _obj) {

戻り値の型が違うだけのオーバーロードは出来なかったかと思います。

また、return 0;と書いているので、実装の方もintで良いのではないかと。

修正後の形であれば、あとは既に言われているように、中身の初期化の問題ではないかと。
obj.test[1].a = 120;
としていますが、この時点で、testのサイズは0なので、test[1]にはアクセスできません。

Comments

[hiru.jp]

amiyaさんご返答ありがとうございます。
見落としていました。(>_<)
質問を書いてる時に前回のソースコードをペーストしてしまいました
質問のソースコードを修正させていただきます。<(_ _)>