プログラミング構造体について。

質問

include<stdio.h>
#include<stdlib.h>
#include<string.h>
struct person{
char name[10];
int gender;
int age;
};
void printPersonList(struct person *person_p, int size);
void outputPersonList(struct person *person_p, int size);
double getAverageOfAge(struct person *person_p, int size);
int countMales(struct person *person_p, int size);
int countFemales(struct person *person_p, int size);
int main(void){
struct person *person_p;
int i, count, gender, age, maleCount, femaleCount;
char name[20];
double average;
printf("登録する人数を入力してください。\n");
scanf("%d", &count);
person_p = (struct person*)malloc(sizeof(struct person)* count);
for(i=0; i < count; i++){
printf("名前・性別(男性:0, 女性:1)・年齢をスペース区切りで入力してください。\n");
scanf("%s %d %d", name, &gender, &age);
strcpy((person_p + i) -> name, name);
(person_p + i) -> gender = gender;
(person_p + i) -> age = age;
}
printPersonList(person_p, count);
outputPersonList(person_p, count);

average = getAverageOfAge(person_p, count);
printf("平均年齢:%f\n", average);

maleCount = countMales(person_p, count);
femaleCount = countFemales(person_p, count);
printf("男性:%d名, 女性:%d名\n", maleCount, femaleCount);

free(person_p);
return 1;
}
void printPersonList(struct person *person_p, int size){
int i;
printf("登録リスト\n");
printf(" name | gender | age\n");
printf("----------+--------+-----\n");
for(i=0; i < size; i++){
printf("%10s | %1d | %2d\n",
(person_p + i) -> name, (person_p + i) -> gender, (person_p + i) -> age);
}
}
void outputPersonList(struct person *person_p, int size){
FILE *output;
int i;

if((output = fopen("meibo.c", "w")) == NULL){
printf("meibo.cを開けませんでした。\n");
return;
}
for(i=0; i < size; i++){
fprintf(output, "%s, %d, %d\n",
(person_p + i) -> name, (person_p + i) -> gender, (person_p + i) -> age);
}
fclose(output);
}
残り３つの関数をすべて定義する（それぞれ１０行程度）
getAverageOfAge, countMales, countFemales
どう定義すればいいのか教えてください。お願いします。

回答No.2

D-Matsu · ベストアンサー

・メンバーgenderはintよりenumの方がいい。あるいはMALE、FEMALEという定数定義をdefineする手もある
・動的配列ではなく線形リスト構造なら関数にサイズを渡す必要がなくなり、不正サイズによるメモリアクセスエラーの可能性も配列アクセスよりは低くなる。もっとも今回なら全部に入力値を使うだけなのであまり違いはないとも言える

という「そもそもの部分」は書くだけ書いておいて。

printPersonList、outputPersonListで「入力データの最初から最後までアクセスする」部分は出ています。

AverageOfAge、つまり「年齢の平均」は「全ての入力データの合計／人数」です。
countMale/Femaleは男女それぞれの人数、つまりgenderの値が男(女)を示す要素の総数です。

これだけ書けばここまでプログラムが書けている人なら普通はわかると思いますが、わからなければ「どこがわからないのか」を明確にした上で補足してください。
なお「全部わからない」というのは「自分で考える気がない」というのと同義なので無視します。あしからず。

補足

回答ありがとうございます。

とりあえず書いてみたんですけど、printfのところはどぅやってかけばいいのかよくわからなかったんですが、以下のような書き方は間違ってますか？あと、もし男性なら０を数える、女性なら１を数えるやり方なんですが、gender++はダメですよね？
double getAverageOfAge(struct person *person_p, int size){
int i, count, sum, average, age;
sum = sum + age;
average = sum / count;

for(i=0; i < size; i++){
printf("%f, &average\n",
(person_p + i) -> name, (person_p + i) -> gender, (person_p + i) -> age);
}
}

int countMales(struct person *person_p, int size){
int i, gender;
if(gender == 0){
pintf("gender++\n",
(person_p + i) -> name, (person_p + i) -> gender, (person_p + i) -> age);
}
}
}

int countFemales(struct person *person_p, int size){
int i, gender;
if(gender == 1){
printf("gender++\n",
(person_p + i) -> name, (person_p + i) -> gender, (person_p + i) -> age);
}
}
}

回答No.1

SaKaKashi

それらの関数が何をどうするのかさっぱりわかりませんけど。
例えば、言葉の違う国のレストランに入って、日本語で「ただ腹減った」とだけしか言わないのと全く同じです。
想像はできますけど、プログラムは想像ではできませんので。

まず問題なのは、
>strcpy((person_p + i) -> name, name);
構造体のnameのサイズを無視してコピーしてます。典型的なメモリ破壊と脆弱性と言う弱点です。