C言語（構造体とポインタ) 文字列を返す関数が実現できません。

Question

前提・実現したいこと

C言語初心者です。
拙いコードですが回答いただけると幸いです。

氏名、年齢、身長、体重を入力すると、BMIと標準体重、そしてBMIの値に応じて肥満度(Light,Standard,Heavy 1,....)を印字するプログラムを作成中です。

BMIと標準体重を求めて印字するまではできたのですが、肥満度を印字することができていません。(文字化けが起こります）。
このプログラムで間違っている部分はどこでしょうか。

個人的には関数fatnessにおいて、思うようにreturnできていないのが原因だと思っているのですが、デバッグできません。

お力をお貸しいただけると幸いです。
よろしくお願いいたします。

発生している問題・エラーメッセージ

特にエラーメッセージは出ないのですが、出力結果は以下のようになります。

氏名,年齢,身長(m),体重(kg)を入力してください
Satou 22 1.8 70
21.6 71.3 ﾌﾌﾌﾌ向・

実行環境はVisual Studio 2019です。

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

//構造体を定義
typedef struct {
	int age;
	double height, weight, BMI, standardweight;
	char name[20],fat[20];
}Person;

//関数のプロトタイプ宣言
void calc(Person*);
char* fatness(Person);

int main() {
	Person data;
	printf("氏名,年齢,身長(m),体重(kg)を入力してください\n");
	scanf("%s %d %lf %lf", &data.name, &data.age, &data.height, &data.weight);

	calc(&data);
	printf("%.1lf %.1lf ", data.BMI, data.standardweight); //BMIと標準体重を印字

	printf("%s", fatness(data)); //肥満度を印字
}

//BMIと標準体重を計算する関数
void calc(Person* data) {
	data->BMI = data->weight / (data->height * data->height);
	data->standardweight = 22 * data->height * data->height;
}

//BMIに応じて肥満度(文字列)を返す関数
char* fatness(Person data) {
	if (data.BMI < 18.5) {
		strcpy(data.fat, "Light");
		return data.fat; //ここがおかしい..?
	}
	else if (18.5 <= data.BMI < 25) {
		strcpy(data.fat,"Normal");
		return data.fat;
	}
	else if (25 <= data.BMI < 30) {
		strcpy(data.fat,"Heavy 1");
		return data.fat;
	}
	else if (30 <= data.BMI < 35) {
		strcpy(data.fat,"Heavy 2");
		return data.fat;
	}
	else if (35 <= data.BMI < 40) {
		strcpy(data.fat,"Heavy 3");
		return data.fat;
	}
	else {
		strcpy(data.fat,"Heavy 4");
		return data.fat;
	}
}

Comments

[BeatStar]

コードは"<code>"または"<コード>"のボタンを押して出てくるヤツの中に書きましょう。今のままでは非常に読みづらいです。

質問は編集できるので編集しましょう。

[shintoatsu]

ご指摘ありがとうございます。
所定の位置に書き直しました。

[BeatStar]

想定されている結果は何でしょうか?
質問にある入力値であれば

21.6 71.3 Normal

となればいいのでしょうか?

[shintoatsu]

肝心な部分の説明が抜けており、申し訳ございません。

はい、おっしゃる通りです。
BMIの値によって出力される肥満度は以下のようにしたいです。

BMI < 18.5 → Light
18.5 <= BMI <25 → Normal
25 <= BMI < 30 → Heavy 1
30 <= BMI < 35 → Heavy 2
35 <= BMI < 40 → Heavy 3
40 <= BMI → Heavy 4

[BeatStar]

これ、もしかすると「環境」のせいかもしれません。少なくともPaiza IO ( https://paiza.io/en/languages/online-c-compiler )で試すと、上で書いた、「～ Normal」となります。
でもVSCode (Windows)でやると似たような感じに。

文字コードはどれで保存されていますか?
ヒント: https://kazo9.net/win-memo-check

あるいはCMD(コマンドプロンプト)の設定とか。

[hidezzz]

BeatStarさん

文字列が壊れる原因は、関数「char* fatness(Person*);」の戻り値の文字配列領域がreturn後に無効になってしまうことです。
もし正しく「Normal」と表示される環境があったとしても、たまたまその環境では壊れなかったというだけで、プログラムが間違っていることに変わりはないです。

[BeatStar]

あ、確かにそうですね。(細かいところまで見ていなかった…)

[shintoatsu]

ベストアンサーの方にいただいたアドバイスを元に、プログラムを正常に動作させることができました。
初めてのteratail利用で右も左も分からない状態でしたが、非常に参考になりました。
ありがとうございました。

Answer 1

Person dataを値渡しでなくアドレス渡し（ポインター）にしてください。
値渡しにすると、fatnessに渡されるデータはPerson dataのコピーになります。
そのコピーは、fatnessからreturnした瞬間に消滅します。その為、期待した結果が得られなくなります。
以下、アドレス渡しの例です。

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

//構造体を定義
typedef struct {
    int age;
    double height, weight, BMI, standardweight;
    char name[20],fat[20];
}Person;

//関数のプロトタイプ宣言
void calc(Person*);
char* fatness(Person*);

int main() {
    Person data;
    printf("氏名,年齢,身長(m),体重(kg)を入力してください\n");
    scanf("%s %d %lf %lf", &data.name, &data.age, &data.height, &data.weight);

    calc(&data);
    printf("%.1lf %.1lf ", data.BMI, data.standardweight); //BMIと標準体重を印字

    printf("%s", fatness(&data)); //肥満度を印字
}

//BMIと標準体重を計算する関数
void calc(Person* data) {
    data->BMI = data->weight / (data->height * data->height);
    data->standardweight = 22 * data->height * data->height;
}

//BMIに応じて肥満度(文字列)を返す関数
char* fatness(Person* data) {
    if (data->BMI < 18.5) {
        strcpy(data->fat, "Light");
        return data->fat; //ここがおかしい..?
    }
    else if (18.5 <= data->BMI < 25) {
        strcpy(data->fat,"Normal");
        return data->fat;
    }
    else if (25 <= data->BMI < 30) {
        strcpy(data->fat,"Heavy 1");
        return data->fat;
    }
    else if (30 <= data->BMI < 35) {
        strcpy(data->fat,"Heavy 2");
        return data->fat;
    }
    else if (35 <= data->BMI < 40) {
        strcpy(data->fat,"Heavy 3");
        return data->fat;
    }
    else {
        strcpy(data->fat,"Heavy 4");
        return data->fat;
    }
}

Comments

[shintoatsu]

値渡しとアドレス渡しの理解が曖昧だったので、理解できプログラムを正常に動かすことができました。
回答していただき、ありがとうございました。