C# と VB.NET の質問掲示板

初めまして。プログラミング初心者です。
list内容をテキストファイルに出力したいと思って以下のコードを書きました。
実行してコンソールから適当にカレーやラーメンを入力してもファイルの中が
System.Collections.Generic.List`1[System.String]
と記載されてしまします。

どういった点を修正すればいいでしょうか。
アドバイスをお願いします。

        static void Main(string[] args)
        {
            List<string> lunch = new List<string>();

            while (true)
            {
                string menu;
                Console.WriteLine("終わるにはeを入力");
                menu = Console.ReadLine();
                if (menu == "e")
                {
                    foreach(string e in lunch)
                    {
                        Console.WriteLine(e);
                    }
                    using (StreamWriter wr = new StreamWriter(@"list.text"))
                    {
                        wr.WriteLine(lunch);
                        wr.Close();
                    }
                    break;
                }
                else
                {
                lunch.Add(menu);
                }
             }
        }

もうほとんどできてるので、頑張ってというのは774RRさんも
おっしゃっている通り。
なので、そことは別の部分でアドバイス。
分かりやすいプログラミングみたいな話かな。

今の処理構造はおおざっぱに以下のような感じですよね。
(行数減らすために改行位置変えてます)
while (true)
{
    if (menu == "e") {
        (いろいろ処理1)
        break;
    } else {
        (いろいろ処理2)
    }
}

このうち、(いろいろ処理2)は実際にwhileのループで行いたい
処理であり、(いろいろ処理1)はこの処理を行ったらbreakで
ループを抜けます。

> Console.WriteLine("終わるにはeを入力");
のコメント風に言うなら、
(いろいろ処理1)は終わった後にやりたい処理なわけです。

なので、直感的に書くなら、
while (true)
{
    if (menu == "e") {
        break;
    } else {
        (いろいろ処理2)
    }
}
(いろいろ処理1)

とするだけで、読みやすいコードとなります。

返信ありがとうございます。
いろいろ試したんですが、下記で解決できました。
なんかコードがきれいじゃないんですが、、、

            List<string> lunch = new List<string>();

            while (true)
            {
                string menu;
                Console.WriteLine("終わるにはeを入力");
                menu = Console.ReadLine();

                if (menu == "e")
                {
                    foreach(string e in lunch)
                        Console.WriteLine(e);
                        break;
                    }
                else
                {
                lunch.Add(menu);
                    using (StreamWriter sw = new StreamWriter(@"menu_list.text",true))
                        sw.WriteLine(menu);
                }
            }

オイラが先生なら 86502 のコードは減点ありだ。だって List （の内容）を出力してないもん。
出力しているのは List に追加する前の内容だ。

でもまずは思ったとおりのことが実装できたというのは大きな進歩で、
なんとなくかっこ悪いとか美しくないとか思うのは更なる改善の入口で、良いことだ。

86502 のコードがかっこ悪いのは「入力」と「出力」が混在していることで、分離するときれいになる。
もう「関数化」学んでいる？学んでいるなら

static void Main()
{
    List<string> lunch = new List<string>();
    InputMenu(lunch); // 入力してリストに追加
    OutputMenu(lunch); // リストを出力
}

のように書いてみよう。これで入力と出力は完全に分離されたわけだ。
その上で InputMenu と OutputMenu を実装してみると良いだろう。

期待されている「出力部」はたぶんこういう実装であろうと思われる。

static void OutputMenu(List<string> lunch)
{
    using (StreamWriter sw = new StreamWriter(@"menu_list.text", true))
    {
        foreach (string s in lunch)
        {
            sw.WriteLine(s);
        }
    }
}

入力部を自分で書いてみると良いだろう。