回答編集履歴

改訂

2018/09/28 15:15

投稿

papinianus

スコア12705

test CHANGED Viewed

@@ -291,3 +291,179 @@
 ```
+---追記
+やろうとしていることはオーバーロードってより、ジェネリクスで実現したほうがいいように思います(学習ってことを考えると継承でもいいのかも？どっちにしてもオーバーロードだと課題にあってない気がする)。随所にisInt(旧integre)が出てくるのがその原因。そのせいで、インデントが多すぎる、良くないコードになってしまっている。
+とりあえず着想をかえない範囲で修正しました。
+intかどうかの質問を前にだし、変数を減らして、数のチェックをやって、変数名を整えました。
+今の学習段階でもこれくらいになるといいのかなと思います。
+```csharp
+using System;
+namespace ConsoleApp3
+{
+    internal class Overload
+    {
+        private readonly double ans;
+        public void Show() => Console.WriteLine($"ans = {ans}");
+        public Overload(int x, int y) => ans = x + y;
+        public Overload(double x, double y) => ans = x + y;
+    }
+    class Start
+    {
+        public static void Main()
+        {
+            var isInt = false;
+            var isDouble = false;
+            var intArray = new int[2];
+            var doubleArray = new double[2];
+            var x = 0;
+            var xd = 0.0;
+            while (!isInt && !isDouble)
+            {
+                Console.WriteLine("int or double ?(1 or 2)");
+                var select = Console.ReadLine();
+                switch (select)
+                {
+                    case "1":
+                        Console.WriteLine("You selected 1");
+                        isInt = true;
+                        break;
+                    case "2":
+                        Console.WriteLine("You selected 2");
+                        isDouble = true;
+                        break;
+                    default:
+                        Console.WriteLine("Incorrect input");
+                        break;
+                }
+            }
+            for (var i = 0; i < 2; i++)
+            {
+                Console.WriteLine("Please enter the digit");
+                var numStr = Console.ReadLine();
+                if (isInt ? !int.TryParse(numStr, out x) : !double.TryParse(numStr, out xd))
+                {
+                    Console.WriteLine("Incorrect input");
+                    i--;
+                    continue;
+                }
+                if (isInt)
+                {
+                    intArray[i] = x;
+                }
+                if (isDouble)
+                {
+                    doubleArray[i] = xd;
+                }
+            }
+            if (isInt)
+            {
+                new Overload(intArray[0], intArray[1]).Show();
+            }
+            if (isDouble)
+            {
+                new Overload(doubleArray[0], doubleArray[1]).Show();
+            }
+            Console.ReadKey();
+        }
+    }
+}
+```