構造体の中にある配列の初期化の方法が分からず…

Question

###実現したいこと
C言語によるプログラミングを行っていて、構造体の配列を初期化しようとしています。

###発生している問題・エラーメッセージ
初期化ならこうすればいけるんじゃないかと以下のコードを追加したところ、実行不可の状況に陥りました。

###該当のソースコード

/* 細かい所は省略し、追加したコードだけを書いてます。
　また、[std]は構造体（ポインタ）です。 */
int f, g;

for(f = 0; f < 2; f++){
	for(g = 0; g < 4; g++){
		std->cards[f][g] = 0;
	}
}

Comments

[maisumakun]

stdの型と、宣言部分についても追記していただけましたら幸いです。

[yumetodo]

加えて実行不可の状況の詳細も追記してください

Answer 1

stdは構造体ポインタということで、malloc等で確保されたものであればcalloc等に置き換えることで対応が可能かと思いましたが、質問者さんの前回の質問を参照すると、stdの実定義は下記かと推測します。

typedef struct status{
    int hp;
    int cards[2][4];
    int have[5];
} Sta;

初期化処理として全ての値を0とする操作を行うとした場合、単純に下記としましたが、cardsの初期化処理は質問者さんの記述と同様です。
そのため、実行不可の状況は該当の処理とは別の箇所によるものではないかと推測します。

#include <stdio.h>

typedef struct status{
    int hp;
    int cards[2][4];
    int have[5];
} Sta;

void init(Sta* std)
{
   int i, j;
   std->hp = 0;
   for (i = 0; i < 2; i++)
   {
      for (j = 0; j < 4; j++)
      {
         std->cards[i][j] = 0;
      }
   }
   for (i = 0; i < 5; i++)
   {
      std->have[i] = 0;
   }
}

void print(Sta* std)
{
   int i, j;
   printf("hp: %d\n", std->hp);
   for (i = 0; i < 2; i++)
   {
      for (j = 0; j < 4; j++)
      {
         printf("cards[%d][%d]: %d\n", i, j, std->cards[i][j]);
      }
   }
   for (i = 0; i < 5; i++)
   {
      printf("have[%d]: %d\n", i, std->have[i]);
   }
}

int main(void)
{
   Sta std;

   printf("-- 初期化前\n");
   print(&std);

   init(&std);

   printf("\n");
   printf("-- 初期化後\n");
   print(&std);

   return 0;
}

/*
実行結果：

$ ./a.out 
-- 初期化前
hp: 1
cards[0][0]: 0
cards[0][1]: 4196397
cards[0][2]: 0
cards[0][3]: 0
cards[1][0]: 0
cards[1][1]: 0
cards[1][2]: 0
cards[1][3]: 4196320
have[0]: 0
have[1]: 4195632
have[2]: 0
have[3]: -748586032
have[4]: 32767

-- 初期化後
hp: 0
cards[0][0]: 0
cards[0][1]: 0
cards[0][2]: 0
cards[0][3]: 0
cards[1][0]: 0
cards[1][1]: 0
cards[1][2]: 0
cards[1][3]: 0
have[0]: 0
have[1]: 0
have[2]: 0
have[3]: 0
have[4]: 0
*/

また、この構造体の構成であれば、Null0lluNさんのご回答にあるmemsetによる対応でも大きな問題はないものと考えます。

#include <stdio.h>
#include <string.h>

typedef struct status{
    int hp;
    int cards[2][4];
    int have[5];
} Sta;


void print(Sta* std)
{
   int i, j;
   printf("hp: %d\n", std->hp);
   for (i = 0; i < 2; i++)
   {
      for (j = 0; j < 4; j++)
      {
         printf("cards[%d][%d]: %d\n", i, j, std->cards[i][j]);
      }
   }
   for (i = 0; i < 5; i++)
   {
      printf("have[%d]: %d\n", i, std->have[i]);
   }

}

int main(void)
{
   Sta std;

   printf("-- 初期化前\n");
   print(&std);

   memset(&std, 0, sizeof(Sta));

   printf("\n");
   printf("-- 初期化後\n");
   print(&std);

   return 0;
}

/*
実行結果：

$ ./a.out
-- 初期化前
hp: 1
cards[0][0]: 0
cards[0][1]: 4196333
cards[0][2]: 0
cards[0][3]: 0
cards[1][0]: 0
cards[1][1]: 0
cards[1][2]: 0
cards[1][3]: 4196256
have[0]: 0
have[1]: 4195696
have[2]: 0
have[3]: 1855798112
have[4]: 32767

-- 初期化後
hp: 0
cards[0][0]: 0
cards[0][1]: 0
cards[0][2]: 0
cards[0][3]: 0
cards[1][0]: 0
cards[1][1]: 0
cards[1][2]: 0
cards[1][3]: 0
have[0]: 0
have[1]: 0
have[2]: 0
have[3]: 0
have[4]: 0
*/

Answer 2

構造体の宣言部分や型等が書かれていないため、コードそのものの修正方法は提示できませんが、

memset( std, 0, sizeof(std) );

としておけばどのような構造体であっても初期化できるかと思います。

Comments

[LouiS0616]

私が低評価を付けたわけではありませんが、参考になるリンクを貼っておきます。
http://www.kijineko.co.jp/tech/superstitions/initialization-by-memset.html

低評価を付けた人がしっかり理由を書いてくれれば良いんですけどね。

[Null0lluN]

初期化というのは確かに語弊がありましたね。ゼロクリアというべきでした。
回答のトップに書いてある通り、構造体の宣言部分や型等が書かれていないため、コードそのものの修正方法は提示できないのですが、以下の理由から今回の回答をいたしました。

リンク先にもある通り、確かに「構成する全ビットが 0 になったとしても、オブジェクトの値が 0 になるかどうかは分からない」ですが、（特に構造体を動的配列にする場合）コンパイラによっては構造体が初期化されていないと警告を吐く場合がありますので初期化しているっぽくしてあげるのに使えるのではないかと。どうせ、あとから構造体の値をどんどん代入していくわけですし。
あとは、「おまじない」的な意味もなくはないです。一文だけでコードの見た目はきれいですので。

ちなみに、リンク先にあった「ゼロクリアしたい理由は、多くの場合、memcmp を使って一致判定を行いたいことが理由でしょう。」とありましたが、それは考慮していませんでした。