編集履歴

回答編集履歴

コメントで指摘いただいて間違いを修正

2019/01/05 15:46

投稿

スコア1193

test CHANGED Viewed

@@ -1,66 +1,32 @@
 # 結論
-以下のようにすれば、おそらくご期待どおりの動作をすると思われます。
+いろいろ間違ったこと書いてすいませんでした。
-が、以下のプログラムには明らかな不備があります。
-具体的には、フィールドの初期化がなされていません。
-それを全て修正すると、変更点が多すぎて今回の回答の要点がぼやけてしまうので、
+他の方が指摘しているとおり、Console.ReadKey(); を削除するだけで済みますね。
-あえて修正していません。
-以下のプログラムは、実際上、ほぼ期待通りの動作をすると思いますが、
+# そのほか
-それは「たまたま」であるということをご了承ください。
+今回のご質問とは直接関係ない点になりますが、ご参考までに。
+> endがtrueになったら終了するのでfalseにする必要はありません
+であれば、 End フィールドを廃止して、以下のようにしてしまった方が、
+シンプルかもしれません。
 ```C#
-using System;
-namespace RPG経験値計算
-{
-    class Program
-    {
-        string input;
-        int Lv;
-        int OneUpExp;
-        int AllExp;
-        bool End;
-        bool IntegerIsEntered;
-        static void Main()
-        {
-            Program p = new Program();
-            while (!p.End)
+            while (p.Input())
             {
-                Console.WriteLine("終了したい場合はendを、経験値の計算をしたい場合は整数を入力してください");
-                p.Input();
                 if (p.IntegerIsEntered)
@@ -74,372 +40,32 @@
             }
-        }
+// ～　中略　～
-        void Input()
+        bool Input()
         {
-            IntegerIsEntered = false;
-            End = true;
+            bool End;
             input = Console.ReadLine();
-            if (input.Contains("end")) End = true;
+            IntegerIsEntered = int.TryParse(input, out Lv);
-            else if (int.TryParse(input, out Lv)) IntegerIsEntered = true;
+            End = input.Contains("end");
-            else
-            {
-                Console.WriteLine("endまたは整数を入力してください" );
+            if(!( End || IntegerIsEntered )) Console.WriteLine("endまたは整数を入力してください");
-                IntegerIsEntered = false;
-            }
+            return !End;
         }
-        void Calculation()
-        {
-            AllExp = Lv * Lv * 1000;
-            OneUpExp = Lv * Lv * 1000;
-            Lv--;
-            OneUpExp -= Lv * Lv * 1000;
-        }
-        void Output()
-        {
-            Console.Write("レベルを1つ上げるために必要な経験値:");
-            Console.WriteLine(OneUpExp);
-            Console.Write("このレベルに達するために必要な経験値");
-            Console.WriteLine(AllExp);
-            Console.WriteLine("続けるにはenterを押してください");
-            Console.ReadKey();
-        }
-    }
-}
 ```
-#問題点と変更箇所
-おおざっぱに言うと、二つの問題があります。
-0. メッセージの出力タイミングが不適切
-0. 判定条件にあてはまらなかった場合の処理がない
-それぞれに対応する問題点と修正箇所を書きます
-##問題点1:メッセージの出力タイミングが不適切
-end 以外を入力した場合、再度 Input()関数　を呼んで、入力を読み込むことになりますが、
-入力を促すメッセージがループの外にあるため、最初の一回しか表示されず、
-二回目以降、使っている人は「end か 数字を入力する状態」に入ったことをはっきり知ることができません。
-また、Output() 関数の最後で、ReadKey()関数を呼んでいますが、
-入力を促すメッセージが表示されていません。
-この二つの理由により、
-0.「1」を入力する
-0.「end」を入力する
-といった操作をした場合、Output() 関数の最後で、ReadKey()関数　に endの 「e」
-が読まれてしまい、そのあとでループの先頭に戻ったのちInput()　関数が呼ばれた際には
-「nd」しか読まれなくなってしまいます。
-### 変更箇所1
-「end か 数字を入力する状態」にあることを示すメッセージをループ内に入れて毎回出るようにします。
-#### 変更前
-```C#
-17:             Console.WriteLine("終了したい場合はendを、経験値の計算をしたい場合は整数を入力してください");
-18:             while (!p.End)
-19:             {
-20:                 p.Input();
-21:                 if (p.IntegerIsEntered)
-22:                 {
-23:                     p.Calculation();
-24:                     p.Output();
-25:                 }
-26:             }
-```
-#### 変更後
-```C#
-17:             while (!p.End)
-18:             {
-19:                 Console.WriteLine("終了したい場合はendを、経験値の計算をしたい場合は整数を入力してください");
-20:                 p.Input();
-21:                 if (p.IntegerIsEntered)
-22:                 {
-23:                     p.Calculation();
-24:                     p.Output();
-25:                 }
-26:             }
-```
-### 変更箇所2
-Output() 関数の最後で、ReadKey()関数の読み込み状態（何らかのキーが押されるのを待っている状態）
-にあることを示すメッセージを追加します。
-#### 変更前
-```C#
-53:
-54:             Console.Write("このレベルに達するために必要な経験値");
-55:             Console.WriteLine(AllExp);
-56:             Console.ReadKey();
-```
-#### 変更後
-```C#
-56:
-57:             Console.Write("このレベルに達するために必要な経験値");
-58:             Console.WriteLine(AllExp);
-59:             Console.WriteLine("続けるにはenterを押してください");
-60:             Console.ReadKey();
-```
-##問題点2:判定条件にあてはまらなかった場合の処理がない
-Input() 関数で、
-- 入力は　end か？
-- 入力は数値か？
-の判定が行われていますが、それぞれ、「end でなかった場合」「数値でなかった場合」の
-処理がありません。
-今回の問題の原因となっている IntegerIsEntered に関して言うと、いったん整数が入力された場合に、IntegerIsEntered を true にしていますが、
-入力が数値でなかった場合に IntegerIsEntered を false にする処理が無いため
-一度 ture になった　IntegerIsEntered　はずっと true のままです。
-このため、一度整数を入力するとそのあとは「end」 を入力しても、いちいち Output() が呼ばれるので、
-意図しない表示が行われ ReadKey() が呼ばれるのですぐには終わりません。
-この動きを回避するためには、欠落している処理を追加する必要があります。
-### 変更箇所
-IntegerIsEntered　と　End にいったん false を入れる処理を追加し、
-入力がそれぞれの条件を満たさない場合は対応するフィールドが false になるようにしました。
-#### 変更前
-```C#
-29:         void Input()
-30:         {
-31:             input = Console.ReadLine();
-32:             if (input.Contains("end")) End = true;
-33:             else if (int.TryParse(input, out Lv)) IntegerIsEntered = true;
-34:             else
-35:             {
-36:                 Console.WriteLine("endまたは整数を入力してください");
-37:                 IntegerIsEntered = false;
-38:             }
-39:         }
-```
-#### 変更後
-```C#
-28:
-29:         void Input()
-30:         {
-31:             IntegerIsEntered = false;
-32:             End = true;
-33:
-34:             input = Console.ReadLine();
-35:             if (input.Contains("end")) End = true;
-36:             else if (int.TryParse(input, out Lv)) IntegerIsEntered = true;
-37:             else
-38:             {
-39:                 Console.WriteLine("endまたは整数を入力してください" );
-40:                 IntegerIsEntered = false;
-41:             }
-42:         }
-```
-## そのほかの問題点
-> この問題に関しては上記コードでは対策できていませんので、
-> ご自分でお調べになって対応してください。
-> きわめて基本的なことなので、調べればすぐにたくさんの情報が得られると思います。
-フィールドの初期化が全く行われていません。
-このため、起動直後の
-```C#
-while (!p.End)
-```
-がどう動くかは厳密には不定（どう動くかは正確には予測できない）です。
-フィールドの初期化はクラスのインスタンスを生成した時点(new した時点)でコンストラクタでなされるべきですが、
-実際の動作を見る限り、 bool　型のフィールドには false が初期値として入っているようですので、
-上記コードはとりあえず動くと思いますが、動作に確実を期すためと、今後の機能追加などでの問題の原因とならないようにするために
-初期化の処理を追加することを強くお勧めします。

記述ミスの訂正

2019/01/05 15:46

投稿

kozuchi

スコア1193

test CHANGED Viewed

@@ -160,7 +160,7 @@
 0. メッセージの出力タイミングが不適切
-0. 変数の初期化がなされていない
+0. 判定条件にあてはまらなかった場合の処理がない
@@ -306,7 +306,7 @@
-##問題点2:フィールドの初期化がなされていない
+##問題点2:判定条件にあてはまらなかった場合の処理がない
 Input() 関数で、