Почему вывелся 5 а не 0?

[Мaкс]

Вот как у меня получилось:
Примечание: красным выделил то самое число которые вывелось не то.
-5 -4 -3 -2 -1 0 1 2 3 4 0

Я компилируется так:
Тут выделил синим то что скомпилировала программа:
-5 -4 -3 -2 -1 0 1 2 3 4 5

С какого перепуга у меня получилось другое? Давайте рассуждать! Сначала выполняется

set
        {
            if (ok(index))
            {
                a[index - lowerBound] = value;
            }
        }

set
        {
            if (ok(index))
            {
                a[index - lowerBound] = value;
            }
        }

сначала вызывается ok, ну ок:

 private bool ok(int index)
    {
        if (index >= lowerBound & index <= upperBound) return true;
        return false;
    }

при этом upperBound равен 4, а index равен 5, результат метода ok равен false, даллее смотрим дальше, false и 5 это будет false, не так ли? Поэтому ничто не возвращается, но потом когда читает значение и выполняется аксессор get то возвращаться по идею должен 0, а возвращается 5, почему?

Код программы:

using System;
class RangeArray {
    // Закрытые данные.
    int[] a; // ссылка на базовый массив
    int lowerBound; // наименьший индекс
    int upperBound; // наибольший индекс
                    // Автоматически реализуемое и доступное только для чтения свойство Length.
    public int Length
    {
        get; private set;
    }
    // Построить массив по заданному размеру.
    public RangeArray(int low, int high)
    {
        high++;
        if (high <= low)
        {
            Console.WriteLine("Неверные индексы");
            high = 1; // создать для надежности минимально допустимый массив
            low = 0;
        }
        a = new int[high - low];
        Length = high - low;
        lowerBound = low;
        upperBound = --high;
    }
    // Это индексатор для класса RangeArray.
    public int this[int index]
    {
        // Это аксессор get.
        get
        {
            if (ok(index))
            {
                return a[index - lowerBound];
            }
            else
            {
                return 0;
            }
        }
        // Это аксессор set.
        set
        {
            if (ok(index))
            {
                a[index - lowerBound] = value;
            }
        }
    }
    // Возвратить логическое значение true, если
    // индекс находится в установленных границах.
    private bool ok(int index)
    {
        if (index >= lowerBound & index <= upperBound) return true;
        return false;
    }
}
// Продемонстрировать применение массива с произвольно
// задаваемыми пределами индексирования.
class RangeArrayDemo {
    static void Main()
    {
        RangeArray ra = new RangeArray(-5, 5);

        // Использовать объект ra в качестве массива.

        Console.WriteLine("Длина массива ra: " + ra.Length);

        for (int i = -5; i <= 5; i++)
            ra[i] = i;

        Console.Write("Содержимое массива ra: ");

        for (int i = -5; i <= 5; i++)
            Console.Write(ra[i] + " ");

        Console.ReadKey();
    }
}

Сразу благодарю за ответ!

[Мaкс]

Сообщение от Serge_Bliznykov

вопрос номер 1. чему равен размер массива a ?

Ну... наверно равен столько же сколько и ra массиву, массив "a" это же ссылка на массив ra, не так ли? Поэтому длина будет ровна 11.

Сообщение от Serge_Bliznykov

вопрос номер 2.
сколько тут элементов

11, везде 11.

Сообщение от Serge_Bliznykov

а мусор вывелся, потому что Вы тут написали неверно

А смысл переделывания <= на < если только переделывали лимит меньше на 1?
Похоже Вы полностью не читали код, поэтому не знаете, что в методе есть такое действие:

upperBound = --high;

так что upperBound будет равен 4, а не 5, а если переделывает <= на < то -- в --high необходимо убрать.

[Мaкс]

Сообщение от Мaкс

А смысл переделывания <= на < если только переделывали лимит меньше на 1?

И теперь такую же ошибку выводит компилятор какую и я сделал в уме:
-5 -4 -3 -2 -1 0 1 2 3 4 0

[Serge_Bliznykov]

Вы меня своими

       high++;
...
        upperBound = --high;

всё таки запутали.
зачем Вы это делаете?

далее.

Сообщение от Мaкс

при этом upperBound равен 4,

с чего это вдруг??

    // Построить массив по заданному размеру.
    public RangeArray(int low, int high)
    {
        high++;
        if (high <= low)
        {
            Console.WriteLine("Неверные индексы");
            high = 1; // создать для надежности минимально допустимый массив
            low = 0;
        }
        a = new int[high - low];
        Length = high - low;
        lowerBound = low;
        upperBound = --high;
        Console.WriteLine(" lowerBound = {0}", lowerBound);
        Console.WriteLine(" upperBound = {0}", upperBound);
    }

результат

 lowerBound = -5
 upperBound = 5
Длина массива ra: 11
Содержимое массива ra: -5 -4 -3 -2 -1 0 1 2 3 4 5

после увеличения на 1 верхней границы,
массив a становится длиной 11 элементов.


в чём вопрос?

[Serge_Bliznykov]

Сообщение от Мaкс

И теперь такую же ошибку выводит компилятор какую и я сделал в уме:

ЗАБУДЬТЕ ВСЁ, что написал раньше!! я всё это удалил.

смотрите пост #4

[Мaкс]

Сообщение от Serge_Bliznykov

в чём вопрос?

Вопрос решён, спасибо!

[Мaкс]

Напомнили мне про ++

[Serge_Bliznykov]

Сообщение от Мaкс

Похоже Вы полностью не читали код, поэтому не знаете, что в методе есть такое действие:

upperBound = --high;

похоже, что Вы тоже не читали код
выше есть

Сообщение от Мaкс

high++;

[Мaкс]

Сообщение от Serge_Bliznykov

похоже, что Вы тоже не читали код

[off]Жарко, ни черта не соображаю.[/off]

[Serge_Bliznykov]

кстати, не надо менять переменные. во-первых, это лишние строчки кода, во-вторых, шанс запутаться выше.

проще так:

    // Построить массив по заданному размеру.
    public RangeArray(int low, int high)
    {
        if (high < low)
        {
            Console.WriteLine("Неверные индексы");
            high = 1; // создать для надежности минимально допустимый массив
            low = 0;
        }
        a = new int[high + 1 - low];
        Length = a.Length;
        lowerBound = low;
        upperBound = high;
    }