ポインター構造体について？

Question

お世話になっております。
C言語についての質問になります。
現在体重と身長を入力して肥満度を表すコードを作成しています。
男性のみになっておりますがscanf関数から自身の性別,年齢,身長,体重を入力すると
肥満度を表すコードをなっておりますが実行すると
・性別を入力した後にエンタキーを押すと空欄のみが入力される
・全項目入力後の計算結果で肥満度がおかしな数字になっている
・あなたは○○ですと診断するhikaku関数にたどり着いていない？
・*datab->のないようをみると性別、空欄に入力した数値,年齢,体重の数値が入っている
・・・と色々エラーが発生しております。
以上の問題に対してアドバイスよろしくお願いします。

コード
// ConsoleApplication18.cpp : コンソール アプリケーションのエントリ ポイントを定義します。
//

#include "stdafx.h"
#include<stdio.h>
#include<stdlib.h>

/*入力者のデータを入れる構造体*/
struct yourbmi {
	/*性別*/
	int sex;
	/*年齢*/
	int age;
	/*体重*/
	double weight;
	/*身長*/
	double height;
};
/*入力されたデータをもとにBMI＝体重（Kg）÷身長（ｍ）÷身長（ｍ）を計算する関数*/
double calculator(struct yourbmi *data);
/*入力したデータを表示する関数*/
void typedata(struct yourbmi* data);
/*入力したデータを比較して性別と年齢で結果を変える*/
void hikaku(struct yourbmi* hikaku , double result);

int main()
{
	//BMIを入れる変数
	double sum = 0;
	struct yourbmi data;
	typedata(&data);
	sum = calculator(&data);
	hikaku(&data , sum);

	return 0;
}

void typedata(struct yourbmi* data) {
	printf("性別 男:1 女:2 ");
	scanf_s("%d\n", &(data->sex));
	printf("年齢:");
	scanf_s("%d\n", &(data->age));
	printf("体重:");
	scanf_s("%lf\n", &(data->weight));
	printf("身長:");
	scanf_s("%lf\n", &(data->height));
}

double calculator(struct yourbmi *data) {
	double result = 0;
	result = (data->weight / data->height) / (data->height);
	printf("%f", result);
	return result;
}

void hikaku(struct yourbmi* data , double result) {
	//性別が男だった場合
	if (data->sex = 1 ) {
		//15～17歳だった場合
		if (data ->age >= 15 &&data ->age <= 17) {
			//BMIが7以下だった場合
			if(result >= 0 && result <= 7) {
				printf("あなたはやせ型です。");
			}else
				//BMIが8以上23以下だった場合
					if (result >= 8 && result <= 23) {
						printf("あなたは標準型です。");
					}
				}else
			        //BMIが24以上27以下だった場合
					if (result >= 24 && result <= 27) {
							printf("あなたは軽肥満型です。");
						}
						else
							//BMIが28以上だった場合
							if (result >= 28) {
								printf("あなたは肥満型です。");
		}
	    //18～39歳だった場合
		if (data->age >= 18 && data->age <= 39) {
			        //BMIが10以下だった場合
					if (result >= 0 && result <= 10) {
						printf("あなたはやせ型です。");
					}
					else
						//BMIが11以上21以下だった場合
						if (result >= 11 && result <= 21) {
							printf("あなたは標準型です。");
						}
				}
				else
					//BMIが22以上26以下だった場合
					if (result >= 22 && result <= 26) {
						printf("あなたは軽肥満型です。");
					}
					else
						//BMIが27以上だった場合
						if (result >= 27) {
							printf("あなたは肥満型です。");
						}
		//40～59歳だった場合
		if (data->age >= 40 && data->age <= 59) {
			//BMIが0以上11以下だった場合
			if (result >= 0 && result <= 11) {
				printf("あなたはやせ型です。");
			}
			else
				//BMIが12以上22以下だった場合
				if (result >= 12 && result <= 22) {
					printf("あなたは標準型です。");
				}
		}
		else
			//BMIが23以上27以下だった場合
			if (result >= 23 && result <= 27) {
				printf("あなたは軽肥満型です。");
			}
			else
				//BMIが28以上だった場合
				if (result >= 28) {
					printf("あなたは肥満型です。");
				}
		//60歳以上だった場合
		if (data->age >= 60) {
			//BMIが0以上13以下だった場合
			if (result >= 0 && result <= 13) {
				printf("あなたはやせ型です。");
			}
			else
				//BMIが14以上24以下だった場合
				if (result >= 14 && result <= 24) {
					printf("あなたは標準型です。");
				}
		}
		else
			//BMIが25以上29以下だった場合
			if (result >= 25 && result <= 29) {
				printf("あなたは軽肥満型です。");
			}
			else
				//BMIが30以上だった場合
				if (result >= 30) {
					printf("あなたは肥満型です。");
				}
	}
}

Answer 1

性別を入力した後にエンタキーを押すと空欄のみが入力される

scanf_s("%d", &(data->sex));に書き換えれば解消されます。

全項目入力後の計算結果で肥満度がおかしな数字になっている

単位が違うのだと思います。身長をcmで入力していませんか？
BMI = 体重[kg] / 身長[m] / 身長[m]です。

他の問題は、少なくとも上記の改変後は再現できません。
なぜhikaku関数が呼び出されていないと判断したのでしょうか？

---

蛇足かもしれないですが、コードの良くない点をいくつか挙げます。

構造体の使い方
まず名前の記法。構造体の名前が全部小文字ってことはあんまりないかと思います。
C言語っぽくUPPER_SNAKE_CASEで書くか、C++を見据えてPascalCaseじゃないかと。
ぱっと見て普通の変数と見分けられる方が何かと便利です。

また、typedefを適宜用いると、毎回毎回structと書かずに済みます。

typedef struct {
    int sex;
    int age;
    double height;
    double weight;
} Human;

勝手にタグ名も変えちゃいました。

値自体に意味がない定数
ご提示のコードでは、男性を1、女性を0として区別しています。
このようなときは、列挙体を用いるときれいに書けます。

enum SEX {
    MAN, WOMAN,
}

bool isMan(Human *human) {
    return human->sex == MAN;
}

なお、boolを使うにはstdbool.hをインクルードしてください。
...cppとしてコンパイルしているならそのままでも使えますが。

排他的な条件が何度も評価されている
else if使いましょう。

if(inAge(human, 15, 17)) {
    ...
}
else if(inAge(human, 18, 39)) {
    ...
}
...

煩雑な条件式
例えば次のような関数を定義するだけで、一気に見通しが良くなります。

bool inAge(Human *human, int lower, int upper) {
    return lower <= human->age && human->age <= upper;
}

『〇〇世代』と表現できる世代なら、専用の関数を作ってもいいでしょう。

bool isTeenager(Human *human) {
    return inAge(human, 13, 19);
}

BMIに関しても同様です。
ネットで見つけた表を参考にすると、こんな感じかな？

bool isUnderweight(Human *);
bool isHealthyWeight(Human *);
bool isOverweight(Human *);
bool isObese(Human *);
bool isSeverelyObese(Human *);
bool isMorbidlyObese(Human *);

同じことを何度も書きすぎ
配列を用いて、世代ごとに評価の境目を管理すると良いでしょう。

Answer 2

身長をcmで入力していますが、cmをmに治す計算が見えません。

height /= 100;

もし自分が見落としていたらすいません。
それと、2人の人が回答してくれています。自分にとってわかりやすい回答をベストアンサーに選びましょう。

Answer 3

scanf関数の使い方、yourbmi構造体/hikaku関数/calculator関数の定義が無いのでアドバイスができません。

Comments

[LouiS0616]

見落としてるんじゃないですか？
質問に編集履歴はなく、しかもコードは全文が載せられ、プロトタイプ宣言には充分な(むしろ過剰な)コメントが入っています。

[diracpaul]

失礼しました。見落としていようです。
なお、scanfの関数のフォーマット文字列には、%で始まる型format指定子で指定しますが、これは、printfの書式指定とは異なり任意の文字列や改行を指定するようにはなっていないようです。
このため、scanfのフォーマット文字列内の\nによって誤動作しているようです。