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

もうほとんどできてるので、頑張ってというのは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)

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