このコードをまとめたいです

Question

このようなプログラムをうったのですが、
int price = Shop("薬草");
money -= price;
price = Shop("万能薬");
money -= price;
このShopの部分を一つにまとめることはできないでしょうか？？
薬草と万能薬とをコンマで区切ると
引数 2 を指定するメソッド 'Shop1' のオーバーロードはありません
とでてきてエラーになります

using System;

namespace ConsoleApp2
{
    class Program
    {
        static void Main(string[] args)
        {
            // Plactise3-21
            static int Shop(string itemName)
            {
                int price = 0;
                Console.WriteLine("いらっしゃいませ！");

                if (itemName == "薬草")
                {
                    Console.WriteLine(itemName + "は１００円です");
                    price = 100;
                }
                else if (itemName == "棍棒")
                {
                    Console.WriteLine(itemName + "は１５００円です");
                    price = 1500;
                }
                else if (itemName == "万能薬")
                {
                    Console.WriteLine(itemName + "は５００円です");
                    price = 500;
                }
                return price;
            }

            int money = 2500;
            Console.WriteLine("所持金は" + money + "円です");
            int price = Shop("薬草");
            money -= price;
            price = Shop("万能薬");
            money -= price;
            Console.WriteLine("所持金は" + money + "円です");
        }
    }
}

Answer 1

まとめるという意味が良く判らないんですが、一回の呼び出しで複数の商品名を渡したいって事ですか？Shopの引数を配列かListにすればいいと思いますが。

とりあえずShopの宣言をこんな感じにして、

static int Shop(string[] itemNames)

こんな感じで渡せます。

int price = Shop(new string[] { "薬草", "万能薬"});

配列にした場合のShopの処理は、自分で考えてみてください。

Comments

[退会済みユーザー]

あとはどうでもいいですけど、× Plactise 〇 practice

[BizarreWanderer]

ありがとうございます！
配列にすればいいんですね
下にかいてあるnewをつける意味はどういったものがあるのでしょうか？

[退会済みユーザー]

とりあえずこちらを読んでください。（"C# new" でググッてトップに出てきます）
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/operators/new-operator

言語仕様に関する事は、基本的にMSのドキュメントを読めば詳細にサンプル付きで記載されていますので、言語仕様の疑問はまずMSのドキュメントを確認する事をお勧めします。

[BizarreWanderer]

わかりました！
ありがとうございます！

Answer 2

radianさんの回答とほぼ同じなんですが、コンマ区切りで引数を複数指定したい場合の例として書いておきます。

using System;

namespace ConsoleApp2 {
    class Program {
        static void Main(string[] args) {
            int money = 2500;
            Console.WriteLine("所持金は" + money + "円です");
            int price = Shop("薬草", "万能薬");
            money -= price;
            Console.WriteLine("所持金は" + money + "円です");
            Console.Read();
        }
        static int Shop(params string[] itemNames) {//このparamsが肝
            int price = 0;
            Console.WriteLine("いらっしゃいませ！");

            foreach (var itemName in itemNames) {
                if (itemName == "薬草") {
                    Console.WriteLine(itemName + "は１００円です");
                    price += 100;
                } else if (itemName == "棍棒") {
                    Console.WriteLine(itemName + "は１５００円です");
                    price += 1500;
                } else if (itemName == "万能薬") {
                    Console.WriteLine(itemName + "は５００円です");
                    price += 500;
                }
            }
            return price;
        }
    }
}

Comments

[退会済みユーザー]

課題か練習問題っぽいのをやってるのを処理の実装まで書くのは本人の為にならないのでは？

[Hey_CH]

課題なんて書いてないですし、teratailって質問に対する回答を書く場で、質問者さんを育てる場と思ってないので、質問者さんが言及していることに対して、こういうのもあるよって事で書きました。
お気に障ったのなら申し訳ないです。

[BizarreWanderer]

ありがとうございます！
まだそこまで学んでいないので分からないことがおおいんですけど
var itemName in itemNameとは普通に代入しているのとわけが違うのでしょうか？

[Hey_CH]

foreach (var itemName in itemNames)の部分でしょうか？
itemNames（このコードでは配列）から１つずつ取り出し、itemNameに代入するという感じの意味になります。
配列から要素を取り出す場合のfor文との違いは、for文は要素の数や位置を指定できるのに対し、foreachは全要素に対して処理を行うって事でしょうか。
よくわからない場合は「C# foreach」で検索してみてください。

[BizarreWanderer]

そうです！
検索してみます！ありがとうございます

Answer 3

まさにエラーに表示されているオーバーロードという機能を使えばできると思いますよ。
オーバーロードというのは、同じメソッド名で引数の並びや数、型が異なるもののことです。
詳しくは調べてみてください。

どういう処理を想定していて、戻り値どうしたいのはわかりませんが、以下のような
定義をすればいけると思います。

static int Shop(string itemName,string itemName2){
        // 処理
}

Comments

[BizarreWanderer]

ありがとうございます！
このメソッドを呼び出すときは必ず、この引数をふたつ呼び出さないといけないんでしょうか？？

[Crimson_Tide]

今後のためにもオーバーロードなんぞやというものを調べてみてください。
簡単に言えば、引数のパターンを引数の型や並びの違いで同じメソッド名で複数定義できる機能です。
同じような処理なのに、メソッド名を ShopKai(string itemName,string itemName2) とメソッドを
増やしていくとメンテナンス性が損なわれますよね。

既に定義されているstatic int Shop(string itemName) ではstring型引き数が１つ受け取ることしかできません。
そのためShop("薬草") という使い方しかできません。２つ受け取ることは想定していないので当然エラーがでます。
私の回答にある static int Shop(string itemName,string itemName2){ も”追加”で定義すると、
Shop("薬草","万能薬") 場合は 追加した場合の処理を。
Shop("薬草") の場合は、BizarreWandererさんが既に作成されている処理が実行されます。
どちらか一方ではなく、どちらも使えます。
他の方が回答されているように配列を受け取るようにして、配列を渡すようにすれば何の処理をしたいかによりますが、引数の数を意識しなくてよくはなりますね。

[Crimson_Tide]

結局のところShopメソッドで何をしたいか、どのように使うか次第です。

追加した場合、引数１つと引数２つの両方同時に使えますが、両方使う必要がなければ追加ではなく単に修正すればいいだけです。その場合オーバーロードを使用する必要はありませんね。
質問では Shop("薬草"); とShop("万能薬");をまとめたい、「薬草と万能薬とをコンマで区切る」 とあるのでその場合どうしたら実現できるかだけを回答しました。
まとめることでShopメソッドをどうしたいのかまでは私にはわかりません。
Shopメソッドの引数に1以上の商品を複数指定して、その値段の合計値を返したいということなら、Hey_CHさんの回答のようにすればよいでしょう。

[Zuishin]

いやさすがにこういうオーバーロードの使い方はしないです。

[Crimson_Tide]

すみません、経験不足で できること と できるけど一般的にこういった使い方はしない(という意味かと推察しますが) ということまではわかりません。
認識に誤りがあるようでしたら、どのように誤りで、どういう使い方をするのかご指摘頂ければ幸いです。

[Crimson_Tide]

オーバーロードして同じメソッドなのに全く異なる処理をする ということであればご指摘の通りかと思います。その場合はオーバーロードではなく、別メソッドにすべき・・ですかね。

[BizarreWanderer]

なるほど、オーバーロードのべんきょうをしてみます！

[Zuishin]

同種のデータが複数ある場合は配列やリストを使います。
わざわざ変数をデータの数だけ作ったりはしません。

同様に、渡すデータが item1 item2 item3 のようなものであれば、その数だけオーバーロードを作ることはせず、配列や IEnumerble<> を引数にします。
この時、Hey_CH さんのように params を使うと非常に使い勝手が良くなります。

オーバーロードを使うのは引数の種類が違う場合で、その場合は省略可能引数も選択肢に入ります。

[Crimson_Tide]

Zuishinさん ご丁寧な説明ありがとうございます。不勉強でした、とても参考になります。
BizarreWandererさん 私の回答や説明は適切ではありませんでした。申し訳ありません。