質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.65%

なぜインデックスの配列の範囲外というエラーが出るのか理解できません

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 4,103

 なぜエラーが出るのか理解したいです。

ある、本の課題をしているときに発生したエラーです。
問題のある箇所は

intarray[i] = x;


の部分で、エラー内容は、インデックスの配列の範囲外という内容でした。
その理由がわからないので教えて欲しいです。

ついでに課題内容は、
int型とint型の値の和とdouble型とdouble型の値の和を求めるメソッドをもつクラスを、オーバーロードを利用して作り、またMainメソッドからこれらのメソッドを利用して和を求めなさい。

という内容でした。
そこで、課題の内容が簡単だったので、工夫して以下の内容に挑戦して見ました。

入力した値をint型とint型の和、もしくはdouble型とdouble型の値を入力し、和を求めるメソッドを持つクラスを、オーバーロードを利用して作り、さらに、thisを利用し、Mainメソッドからこれらのこれらのメソッドを利用して和を求めるというコードに挑戦しました。

が失敗してしまいました。
その上、読みにくいコードにもなってしまいました。
読みやすく、わかりやすいコードにもしたいので、アトバイスを出来れば教えて頂けないでしょうか?
よろしくお願いします。

 発生している問題・エラーメッセージ

System.IndexOutOfRangeException: 
"Index was outside the bounds of the array."

 該当のソースコード

using System;

class Overload
{

    public void Show()
    {
        var ans = this;
        Console.WriteLine("ans = {0}", ans);
    }

    public Overload(int x, int y)
    {
        int ans = x + y;
    }

    public Overload(double x, double y)
    {
       double ans = x + y;
    }
}

class Start
{
    public static void Main()
    {
        bool integre = false;
        bool doublegre = false;
        int[] intarray;
        double[] doublearray;
        int x = 0;
        int y = 0;
        double xd = 0;
        double yd = 0;

        for (int i = 0; i < 2; i++)
        {
            while(true)
            {
                Console.WriteLine("Please enter the digit");
                string num_str = Console.ReadLine();

                if (!Char.IsDigit(num_str[0]) && num_str[0] != '-')
                {
                    Console.WriteLine("Incrrect input");
                    Console.WriteLine();
                    continue;
                }
                if (integre == false && doublegre == false)
                {
                    while (true)
                    {
                        Console.WriteLine("int or double ?(1 or 2)");
                        string select = Console.ReadLine();

                        bool select_roop = false;

                        switch (select)
                        {
                            case "1":
                                Console.WriteLine("You selected 1");
                                integre = true;
                                select_roop = true;
                                break;

                            case "2":
                                Console.WriteLine("You selected 2");
                                doublegre = true;
                                select_roop = true;
                                break;

                            default:
                                Console.WriteLine("Incrrect input");
                                break;
                        }
                        if (select_roop == true)
                        {
                            break;
                        }
                    }
                    if (integre == true)
                    {
                        intarray = new int[i];

                        if (i == 0)
                        {
                            x = int.Parse(num_str);
                           intarray[i] = x; //Error
                        }
                        else if (i == 1)
                        {
                            y = int.Parse(num_str);
                            intarray[i] = y;
                        }
                        else
                        {
                            Console.WriteLine("Error");
                        }
                    }
                    else if (doublegre == true)
                    {
                        doublearray = new double[i];

                        if (i == 0)
                        {
                            xd = double.Parse(num_str);
                            doublearray[i] = xd;
                        }
                        else if (i == 1)
                        {
                            yd = double.Parse(num_str);
                            doublearray[i] = xd;
                        }
                        else
                        {
                            Console.WriteLine("Error");
                        }
                    }
                    else
                    {
                        Console.WriteLine("Error!");
                    }
                }
                break;
            }   
        }

        if (integre == true)
        {
            Overload ov = new Overload(x, y);
            ov.Show();
        }

        if(doublegre == true)
        {
            Overload ov = new Overload(xd, yd);
            ov.Show();
        }
    }
}

      上記のコードを直しても正常に動かないので修正後のコードを追記しました。     

using System;

class Overload
{

    public void Show()
    {
        var ans = this;
        Console.WriteLine("This ans = {0}", ans);//Ovreload display
    }

    public int Over(int x, int y)
    {
        int ans = (x + y);
        return ans;
    }

    public double Over(double x, double y)
    {
        double ans = (x + y);
        return ans;
    }
}

class Start
{
    public static void Main()
    {
        bool integre = false;
        bool doublegre = false;
        int[] intarray;
        double[] doublearray;
        int x = 0;
        int y = 0;
        double xd = 0;
        double yd = 0;

        for (int i = 0; i < 2; i++)
        {
            while (true)
            {
                Console.WriteLine("Please enter the digit");
                string num_str = Console.ReadLine();

                if (!Char.IsDigit(num_str[0]) && num_str[0] != '-')
                {
                    Console.WriteLine("Incrrect input");
                    Console.WriteLine();
                    continue;
                }
                if (integre == false && doublegre == false)
                {
                    while (true)
                    {
                        Console.WriteLine("int or double ?(1 or 2)");
                        string select = Console.ReadLine();

                        bool select_roop = false;

                        switch (select)
                        {
                            case "1":
                                Console.WriteLine("You selected 1");
                                integre = true;
                                select_roop = true;
                                break;

                            case "2":
                                Console.WriteLine("You selected 2");
                                doublegre = true;
                                select_roop = true;
                                break;

                            default:
                                Console.WriteLine("Incrrect input");
                                break;
                        }
                        if (select_roop == true)
                        {
                            break;
                        }
                    }
                }
                    else if (doublegre == true)
                    {
                        doublearray = new double[i];

                        if (i == 0)
                        {
                            xd = double.Parse(num_str);
                            doublearray[i] = xd;
                        }
                        else if (i == 1)
                        {
                            yd = double.Parse(num_str);
                            doublearray[i] = xd;
                        }
                        else
                        {
                            Console.WriteLine("Error");
                        }
                    }
                    else
                    {
                        Console.WriteLine("Error!");
                    }
                if (integre == true)
                {
                    intarray = new int[2];

                    if (i == 0)
                    {
                        x = int.Parse(num_str);
                        intarray[i] = x;
                    }
                    else if (i == 1)
                    {
                        y = int.Parse(num_str);
                        intarray[i] = y;
                    }
                    else
                    {
                        Console.WriteLine("Error");
                    }
                }
                break;
            }
        }

        if (integre == true)
        {
            Overload ov = new Overload();
            ov.Show();
            Console.WriteLine("ans = {0}",ov.Over(x, y));
        }

        if (doublegre == true)
        {
            Overload ov = new Overload();
            ov.Show();
            Console.WriteLine("ans = {0}", ov.Over(x, y));

        }
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2018/09/28 23:04

    エラーが出た際の i の値を確認できれば、なぜエラーになったかの参考になると思います。

    キャンセル

回答 3

checkベストアンサー

+2

なんでこういうコードにしたかについては学習途中と考えて、動くように直しました。
残せるところはコメントアウトで残しましたが、ざっくり書きかえたところもあります。
いちおう、1を3回入力すれば動きますが、コメントにもあるとおり、1aとかにすると落ちますので適宜直してください

using System;

namespace ConsoleApp
{
    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 integre = false;
            var doublegre = false;
            var intarray = new int[2];
            var doublearray = new double[2];
            //int x = 0;
            //int y = 0;
            //double xd = 0.0;
            //double yd = 0.0;

            for (int i = 0; i < 2; i++)
            {
                //while (true) //無意味
                //{
                Console.WriteLine("Please enter the digit");
                string num_str = Console.ReadLine();

                if (!char.IsDigit(num_str[0]) && num_str[0] != '-') // 91a などを弾けない。
                {
                    Console.WriteLine("Incrrect input");
                    Console.WriteLine();
                    continue;
                }
                //if (integre == false && doublegre == false)
                //{
                while (true)
                {
                    if (integre || doublegre)
                    {
                        break;

                    }
                    Console.WriteLine("int or double ?(1 or 2)"); //ユーザにとって分かりにくい、最初にintなのかdoubleなのかの問いかけをすべき。
                    string select = Console.ReadLine();

                    bool select_roop = false;

                    switch (select)
                    {
                        case "1":
                            Console.WriteLine("You selected 1");
                            integre = true;
                            select_roop = true;
                            break;

                        case "2":
                            Console.WriteLine("You selected 2");
                            doublegre = true;
                            select_roop = true;
                            break;

                        default:
                            Console.WriteLine("Incrrect input");
                            break;
                    }
                    if (select_roop)
                    {
                        break;
                    }
                }
                if (integre)
                {
                    //intarray = new int[2];

                    if (i == 0)
                    {
                        //x = int.Parse(num_str);
                        intarray[i] = int.Parse(num_str); //Error
                    }
                    else if (i == 1)
                    {
                        //y = int.Parse(num_str);
                        intarray[i] = int.Parse(num_str);
                    }
                    else
                    {
                        Console.WriteLine("Error");
                    }
                }
                else if (doublegre)
                {
                    //doublearray = new double[2];

                    if (i == 0)
                    {
                        //xd = double.Parse(num_str);
                        doublearray[i] = double.Parse(num_str);
                    }
                    else if (i == 1)
                    {
                        //yd = double.Parse(num_str);
                        doublearray[i] = double.Parse(num_str);
                    }
                    else
                    {
                        Console.WriteLine("Error");
                    }
                }
                else // このelseに到達することはない
                {
                    Console.WriteLine("Error!");
                }
                //}
                //}
            }

            if (integre)
            {
                //Overload ov = new Overload(x, y);
                //ov.Show();
                new Overload(intarray[0], intarray[1]).Show();
            }

            if (doublegre)
            {
                //Overload ov = new Overload(xd, yd);
                //ov.Show();
                new Overload(doublearray[0], doublearray[1]).Show();
            }

            Console.ReadKey();
        }
    }
}

---追記
やろうとしていることはオーバーロードってより、ジェネリクスで実現したほうがいいように思います(学習ってことを考えると継承でもいいのかも?どっちにしてもオーバーロードだと課題にあってない気がする)。随所にisInt(旧integre)が出てくるのがその原因。そのせいで、インデントが多すぎる、良くないコードになってしまっている。
とりあえず着想をかえない範囲で修正しました。
intかどうかの質問を前にだし、変数を減らして、数のチェックをやって、変数名を整えました。
今の学習段階でもこれくらいになるといいのかなと思います。

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();
        }
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/09/28 23:38

    ちなみに、そもそも配列の個数が2個なのがあらかじめ分かっているのに初期化をiでやったのが間違いの原因ですが、そこだけ直しても動作しないので、まあ色々見比べてくださいな。

    キャンセル

  • 2018/09/28 23:53

    ありがとうございます!
    確かに自分で配列を直した時正常に動かずに修正してました。
    その修正後のコードと見比べて、直していきます。

    キャンセル

  • 2018/09/29 00:15

    個人的に許せる限度まで、修正しました、参考になれば幸いです。

    キャンセル

+2

その課題内容から,この長大なコードが生まれた経緯がわかりませんが…

とりあえずエラー要因に関しては,そのエラーが発生する際の配列のサイズを考えてみるとよいかと.

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/09/28 23:54

    確かに、配列の値を[0]と指定してました・・・

    キャンセル

+2

↓ここしか読まずに回答します

                        intarray = new int[i]; // -> 1.

                        if (i == 0)
                        {
                            x = int.Parse(num_str);
                           intarray[i] = x; //Error // -> 2.


2.に進むとき、iは0なのですから、1.において、intarrayの個数は0個と宣言されています。
つまり、intarrayの1個目の要素intarray[0]は確保されていません。

配列の要素(0個)より多い番地である1個目を利用しようとすれば、範囲外のエラーがでても仕方ないです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.65%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る