c 言語初心者です。

質問

c　言語初心者です。
私は下記の構造体配列をつくりました。
しかしﾊﾞｯﾌｧオーバーランが起きてエラーが起きてしまいます。
ヒープ領域に問題があるのかもしれませんが、プログラム上どこに原因があるのかが良くわかりません。
どなたかよろしければ教えていただけないでしょうか？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
#include<memory.h>

struct s
{ int i;
char name[25];
char huri[25];
char num[23];
};

void touroku(struct s *p);
void hyouji(struct s *p);

int main(void)
{ struct s data;

touroku( &data );
hyouji( &data );

//data.num *= 1; /* dataはポインタではないのでドット演算子 */
hyouji( &data );

return 0;
}

/* 構造体のメンバを設定する */
void touroku(struct s *p)
{ int i=0;

for(i=1;i<3;i++)
{
printf( "25文字以内の名前を入力して下さい\n" );
memset(p[i].name, 0, sizeof(p[i].name));
fgets( p[i].name,sizeof(p[i].name) , stdin );

if(strchr(p[i].name,'\n')==NULL)//ﾊﾞｯﾌｧ処理
{
while(getchar() != '\n');
}
if(p[i].name[strlen(p[i].name)-1]=='\n')//改行解除
{
p[i].name[strlen(p[i].name)-1] = '\0';

}

printf("25文字以内のふりがなを入力してください\n");
memset(p[i].huri, 0, sizeof(p[i].huri));
fgets(p[i].huri,sizeof(p[i].huri),stdin);

if(strchr(p[i].huri,'\n')==NULL)//ﾊﾞｯﾌｧ処理
{
while(getchar() != '\n');
}

if(p[i].huri[strlen(p[i].huri)-1]=='\n')//改行解除
{
p[i].huri[strlen(p[i].huri)-1] = '\0';
}

printf( "整数を入力して下さい\n" );
memset(p[i].num, 0, sizeof(p[i].num));
fgets(p[i].num,sizeof(p[i].num),stdin );

if(strchr(p[i].num,'\n')==NULL)//ﾊﾞｯﾌｧ処理
{
while(getchar() != '\n');
}
if(p[i].num[strlen(p[i].num)-1]=='\n')//改行解除
{
p[i].num[strlen(p[i].num)-1] = '\0';
}
}
}

/* 構造体のメンバを出力する */
void hyouji(struct s *p)
{
int i=0;

for(i=1;i<3;i++)
printf("%-8s %3s %3s %d\n" ,p[i].name , p[i].huri , p[i].num , i);
puts("----------------------------------------------------------------");

return ;
}

回答No.3

和泉 博（@hiroshi09s） · ベストアンサー

もっと簡単に考えて肩の力を抜いてプログラミングされてはいかがでしょうか。
プログラムの違いをチェックすればわかると思います。
頑張って下さい　:-)
（使用OS: Mac OSX、スペースは２バイトですので半角に変更のこと）

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

#define SIZE　25　　　/* 文字の許容バイト数 */
#define MEMBER　3　　/* 登録人数＋α */

struct s {
　// int n;　　　　　意味不明です。
　char name[SIZE];
　char huri[SIZE];
　char num[SIZE];
} data[MEMBER];　　　/* main()の初めに置くと広域変数として使えるよ */

/* プロトタイプ宣言 */
void touroku(void);
void hyouji(void);

int main(void) {

　touroku();　　/* 広域変数としたから引数は考えなく良い */
　hyouji();　　　/*　　　〃　　　　　　*/

　return 0;
}

/* 構造体のメンバを設定する */
void touroku(void) {
　int i, n;

　for(i = 0; i < MEMBER; i++) {

　　fprintf(stderr, "%d文字以内の名前を入力して下さい\n", SIZE);
　　fgets( data[i].name, SIZE, stdin );
　　data[i].name[strlen(data[i].name) - 1] = '?0';

　　fprintf(stderr, "%d文字以内のふりがなを入力して下さい\n", SIZE);
　　fgets( data[i].huri, SIZE, stdin );
　　data[i].huri[strlen(data[i].huri) - 1] = '?0';

　　fprintf(stderr, "整数を入力して下さい\n");
　　fgets( data[i].num, SIZE, stdin );
　　data[i].num[strlen(data[i].num) - 1] = '?0';

　}
}

/* 構造体のメンバを出力する */
void hyouji(void) {
　int i;

　for(i = 0; i < MEMBER; i++) {
　　printf("%-8s %3s %3s %d\n" ,data[i].name , data[i].huri , data[i].num , i);
　　printf("----------------------------------------------------------------?n");
　}
}

お礼

ありがとうございます。
難しく考えすぎというのは自覚しているのですが、基本を理解できていないのでなかなか柔軟に考えてないみたいです。
ご丁寧にわかりやすく書いていただき大変参考になりました。

回答No.4

和泉 博（@hiroshi09s）

解答がズレてしまいました m(_ _)m
#1さんが正解ですよ。
　単一の変数なのに for() を３つの変数枠があるかのようにプログラミングしてあるために不正なアクセスとなってバスエラーが出るということです。
　後はエラーとは関係ありませんが、while(getchar()) を strlen() に換えれることでしょうか。また、変数枠は #define で先に決めて置くのがCの一般的な使い方になっています。理由は、その #define を直すだけでプログラム全体が更新できるからです。

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

#define SIZE 25
#define MEMBER 2

struct s {
char name[SIZE];
char huri[SIZE];
char num[SIZE];
};

/* プロトタイプ宣言 */
void touroku(struct s *);
void hyouji(struct s *);

int main(void) {
struct s data[MEMBER];

touroku(data);
hyouji(data);

return 0;
}

/* 構造体のメンバを設定する */
void touroku(struct s *p) {
int i, n;

for(i = 0; i < MEMBER; i++) {
fprintf(stderr, "%d文字以内の名前を入力して下さい\n", SIZE);
fgets( p[i].name, SIZE, stdin );
p[i].name[strlen(p[i].name) - 1] = '\0';

fprintf(stderr, "%d文字以内のふりがなを入力して下さい\n", SIZE);
fgets( p[i].huri, SIZE, stdin );
p[i].huri[strlen(p[i].huri) - 1] = '\0';

fprintf(stderr, "整数を入力して下さい\n");
fgets( p[i].num, SIZE, stdin );
p[i].num[strlen(p[i].num) - 1] = '\0';
}
}

/* 構造体のメンバを出力する */
void hyouji(struct s *p) {
int i;

for(i = 0; i < MEMBER; i++) {
printf("%-8s %3s %3s %d\n" ,p[i].name , p[i].huri , p[i].num , i);
printf("----------------------------------------------------------------\n");
}
}

お礼

ありがとうございます。
まだまだ分からない事が多くて、書き方など大変参考になります。
今やってる構造体配列やポインタが混ざってくるといろいろ混乱してしまって、
ありがとうございました。

回答No.2

chie65536（@chie65535）

＞私は下記の構造体配列をつくりました。

＞struct s data;
これのどこが「構造体配列」なのか、小一時間問い詰めても宜しいか？

補足

答えていただけないなら書き込みはご遠慮ください。他の方が不快に感じる文章です。

回答No.1

towad

mainの最初で
　struct s data;
と、単一の変数として定義されているのに、tourokuやhyouji関数では
渡された引数を構造体sの配列として使おうとしています。

とりあえずmainの処理を
struct s data[3];

touroku( data );
hyouji( data );
とすれば辻褄が合うのではないでしょうか。

お礼

ありがとうございます。
いろいろ理解できていなかったので、書き方が構造体になってました。
なんとかやってみます。