メソッドが複数回呼ばれる

Question

現在独学でやっているプログラミング初心者です。
C#(.Netframework)について質問があります。

前提・実現したいこと

現在、初心者向けの本を読んでいて、その本の「ポイント計算プログラム」を作る課題で行き詰っています。
①商品の金額を入力
②商品の金額の1％がポイントになる（四捨五入で整数に）
③日付を入力して5の倍数の日はポイントを5倍に

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

商品の金額を入力してくださいが3回出てしまいます。

該当のソースコード

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace PointSystem
{
    class Program
    {
        static void Main(string[] args)
        {
            ItemValue();
            Point();
            SaleDay();
        }



        public static int Point()
        {
            //ポイント計算
            int itemValue = ItemValue();
            return (int)(itemValue * 0.01);

        }

        public static int ItemValue()
        {
            Console.WriteLine("商品の値段を入力してください：");
            var item = Console.ReadLine();
            var itemValue = int.Parse(item);
            return itemValue;
        } 

        public static void SaleDay()
        {
            Console.WriteLine("今日は何日ですか？：");
            var day = Console.ReadLine();
            var date = int.Parse(day);
            var point = Point();

            //ポイント5倍の日か判定
            if (date % 5 == 0)
            {
                Console.WriteLine("今日はポイント5倍の日です");
                point *= 5;
                Console.WriteLine("獲得ポイントは{0}です",point);
            }
            else
            {
                Console.WriteLine("獲得ポイントは{0}です。",point);
            }
        }
    }
}

試したこと

MainメソッドにてSaleDay以外のメソッドを呼び出さなかった場合、金額を入力する回数が1回だけで済むのですが、どうしてなのでしょうか？
すべてのメソッドを呼び出す必要がないのでしょうか？

補足情報（FW/ツールのバージョンなど）

独学の初心者なので、見やすいコードの書き方なども教えていただけるとありがたいです。

Comments

[m.ts10806]

書籍にはヒントが何も載ってないのでしょうか

[m.ts10806]

あと念のため「どういう操作をしたら3回出てきたか」
手順まで記載してもらえると。

[Nanachi]

書籍には課題だけで、ヒントは載っていませんでした。
実行してみると、金額を入力してくださいと出て、入力するとまた再度入力してくださいとでます。その後入力すると日付の入力を求められて、その後また金額の入力を求められます。

この回答で合っていますでしょうか？

[m.ts10806]

とりあえず、回答してます（ほぼ同じタイミングで3名同じ指摘ですね）
書籍の内容をどの程度理解できているか、が見えてきます。
復習しましょう。

Answer 1

出るように書かれてます。3回。

        static void Main(string[] args)
        {
            ItemValue();
            Point();
            SaleDay();
        }

ItemValue();

ここで1回目。

Point();

の中でItemValue()呼び出してるので2回目

SaleDay();

の中でPoint()呼び出してるのでその中でItemValue()呼び出して3回目。

Mainで呼んでる最初の２つ書かなくていいんでは。
返却値あるのに、受け取ってないですし。

独学の初心者なので、見やすいコードの書き方なども教えていただけるとありがたいです。

これは「要件による」のですけど、
コード書いてから考えるのではなく、考えてからコード書いたらいいのではないでしょうか。
クラスもそうですけど、メソッドも一定の役割を与えられています。
「どこまでの責任を持たせるのか」を考えてから、
お絵描きレベルでも書いてから（つまるところ「設計書」です）
実装にうつると、今回のようなことは避けられるのではないかと思います。

あとは「リーダブルコード」のような書籍を読んでみること。
Githubなどで公開されているプロジェクトから着想を得ること。
そして、書くこと。

可能なら「自分が数か月、数年見なくなっても、久しぶりに読んだときに読めるコード」を目指すと良いでしょうね。コメントもきちんと書くことです。

Comments

[Nanachi]

ありがとうございます。

[Nanachi]

なるほど、次から試してみたいと思います。　確かにコメントも書いた方が後から見やすくなりますね。　わざわざありがとうございました！

[m.ts10806]

「考えながら書く」のではなく「考えてから書く」です。
コードを書きながら悩んでも整合性取れないのは当然なので、
書き始めたときには最後まで書き上げること。
「考える」ときには書籍を頼らないようにすると良いと思います。

[Nanachi]

ありがとうございます！
自分のしたい処理をノートにまとめてから書いてみようと思います。

Answer 2

SaleDayの中でPointを呼び出しています。
またPointの中でItemValueを呼び出しています。
なのでMainからSaleDay以外を呼び出すと、それぞれが複数回呼び出されることになります。

デバッガの使い方を学習して、ステップ実行など行いながらご自身のプログラムがどのような順番で実行されているのか確認してください。

また、関数の戻り値の使い方と変数、変数のスコープについて学習されるといいかと思います。

Comments

[Nanachi]

ありがとうございます。戻り値の使い方についてもう少し勉強してみます。

Answer 3

ItemValue関数が実行されると、"商品の値段を入力してください："がでます。これはいいですね？

public static int Point()

    {
        //ポイント計算
        int itemValue = ItemValue();

これをみると、Point関数を実行しても、ItemValue関数が実行されますんで、これがでます。

    public static void SaleDay()

    {
        Console.WriteLine("今日は何日ですか？：");
        var day = Console.ReadLine();
        var date = int.Parse(day);
        var point = Point();

SaleDay関数を実行すると、Point関数が実行されるんで、こいつもまたでてきますね

Comments

[Nanachi]

ありがとうございます。