c言語の課題が解けなくて困っています

Question

大学で出された課題が解けなくて困っています。
課題文は次の通りです。

変数amountには貯金の合計額（最初は0円）が入っている．
以下のような3回貯金できるプログラムを作成せよ．
また①②の2つの処理をそれぞれユーザ定義関数として定義して，これを使うようにせよ．
①一回の貯金の処理
・現在の貯金額の表示
・ 貯金額の問い合わせと入力受付
・入金処理を行い、処理後の貯金額を円単位で整数として返す
②金額の表示（金額を整数で受け取って，?万?円表示．例：64万5120円）
・この金額の表示関数は，1つ目のユーザー定義関数でも使用する．
金額の入力は千円単位とすること(例：1.5 = 1500円)．

現在の貯金高は，0万0円です．
今回はいくら貯金しますか？（単位：千円）：10.5/貯金する金額はscanfを使って入力できるようにする/

現在の貯金額は，1万500円です．
今回はいくら貯金しますか？（単位：千円）：0.03/上と同様に/

現在の貯金額は，1万530円です．
今回はいくら貯金しますか？（単位：千円）：100.0/上と同様に/

全部で11万530円貯まりました．
よくがんばりました．

###自分のコード(うまく動作しない)

#include <stdio.h>
int cal_man(int amount)
{
	int man;
	man = amount / 10000;
	return man;
}

int cal_en(int amount)
{
	int en;
	en = amount % 10000;
	return en;
}

void display_saving(int amount)
{
	int en, man;
	double saving;
	amount = amount + 0;
	man = cal_man(amount);
	en = cal_en(amount);
	printf("現在の貯金額は、%d万%d円です.\n", man, en);
	printf("今回はいくら貯金しますか？　(単位：千円)　：");
	scanf("%lf", &saving);
	amount = amount + (saving * 1000);
}

int main(void)
{
	int i, man, en, amount = 0;

	for (i = 0; i < 3; i++)
	{
		display_saving(amount);
	}
	return 0;
}

Comments

[y_waiwai]

このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください
また、うまく動かないとは、どういうふうに動くんでしょうか

[jimbe]

一応、teratail に課題を質問して良いのかどうかをお調べになっては如何でしょうか。

[fukario]

プログラミングを学びだして1ヶ月しか経っておらず、右も左もわからないので丁寧に説明してもらえるとありがたいです。

[退会済みユーザー]

エンジニアの問題解決のためのサイトで、そういう用途のサイトではないです。
課題の疑問なら、教官なり先生なりに聞きましょう。
[推奨していない質問] https://teratail.com/help/avoid-asking

Answer 1

-void display_saving(int amount)
+int display_saving(int amount)

     amount = amount + (saving * 1000);
+    return amount;

-        display_saving(amount);
+        amount = display_saving(amount);

この 3行を修正して、あとは、「よくがんばりました.」ですね。
なぜこの修正が必要なのかをよく考えてコメントに書いてください。

Comments

[kazuma-s]

なぜ、元のコードが思い通りに動かなかったのか、
なぜ、修正したコードが思い通りに動くのかを理解しないと解決ではありません。
分からないのなら、説明しますから、どこが分からないのかを具体的に書いてください。

[退会済みユーザー]

課題の直接的な回答を示すのは非推奨質問の助長に繋がり悪影響があると考え、低評価しました。

[BeatStar]

同上。

直接的な回答より、曖昧な回答の方がいいかもしれません。(流れで説明するとか)

Answer 2

まず基礎から出来ていない。

関数の基本構造は ( "■" を半角スペースと見立てて )

関数の型■関数名(引数){
    // 何らかの処理
return■戻り値;
}

です。

関数の型 = 戻り値の型

です。

そもそも関数は『呼び出し側(main関数とか)』から「こういう処理をしてくれ」と引数(対象データ)を渡されて処理し、呼び出し側にその結果を返す。

この呼び出し側に返すデータのことを戻り値と言い、これを返すことを、

戻り値を返すと言います。

ですが、質問にあるコードのcal_en関数やcal_man関数は「戻り値を返していない」です。

returnなんちゃらが戻り値です。

---

[追記1]

ご自分のコードを読んでください。

コードを読むときは、「一行レベルで、この行が何をしているのかを考えながら読む」です。

main関数だけに着目したら、

int main(void)
{
　　// 変数宣言
    int i, man, en, amount = 0;

　　// 3回繰り返す
    for (i = 0; i < 3; i++)
    {
　　　　// display_saving関数を呼び出す
        display_saving(amount);
    }
　　// 戻り値を0として返す
    return 0;
}

的な感じで。

これを現実世界で手作業でやってみる。(display***を3回繰り返すとかそういうのを)
脳内でもいいです。

関数を呼び出している所はその関数内にも潜り込む。

C言語の場合はmain関数から始まります。

なので main関数 -> main関数内で呼ばれた関数 -> その関数から呼ばれた関数 ...

のように潜っていきます。

そしてその処理を追う。

変数とかが使われているのならその値を考えながら読む。

i = 0 のときどうなるか。i = 1 のときどうなるか。i = 2 のとき... とかみたいに。

具体的な数字や文字列等のデータを入れて考えてみる。

そうするとロジック違反をしていれば気づくはずです。

つまりデバッグぐらいはしましょうです。

Comments

[fukario]

ありがとうございます！戻り値の入力をすっかり忘れてました。