spaiel Ответов: 2

Как мне решить это исключение indexoutofrangeexception ? !

for (int i = 0; i < phrases.Length; i++)
    {
        int x = 0;
        int y = 0;
        // using file input to populate grid
        for (int j = 0; j < phrases[i].trueString.Length; j++)
        {
            numbers[phrases[i].beginXcoor + x, phrases[i].beginYcoor +
            y] = phrases[i].trueString[j];
            phrases[i].endXcoor = phrases[i].beginXcoor + x;
            //    ^ ^  (EXCEPTION BREAKS HERE)
            phrases[i].endYcoor = phrases[i].beginYcoor + y;

            switch (phrases[i].orientation)
            {
                case Orientation.Up:
                    y = y - 1;
                    break;
                case Orientation.Down:
                    y = y + 1;


Что я уже пробовал:

Я понятия не имею , как исправить эту проблему, пожалуйста, помогите, любые предложения приветствуются

2 Ответов

Рейтинг:
2

#realJSOP

Ну, я бы начал с уменьшения сложности, чтобы вы могли видеть, с чем работаете:

for (int i = 0; i < phrases.Length; i++)
{
    int x = 0;
    int y = 0;

    // new code - set a variable to the phrase at the indicated index
    MyPhrase phrase = phrases[i];

    // using file input to populate grid
    for (int j = 0; j < phrase.trueString.Length; j++)
    {
        // new code - do the math ahead of time so you can inspect the values
        int newX = phrase.beginXcoor + x;
        int newY = phrase.beginYcoor + y;

        // I suspect THIS is where you're going wrong. I bet either newX or newY 
        // (or both) are outside the range of the numbers array.
        numbers[newX, newY] = phrase.trueString[j];

        phrase.endXcoor = newX; 
        //^ ^  (EXCEPTION BREAKS HERE)

        phrase.endYcoor = newY;

        switch (phrase.orientation)
        {
            case Orientation.Up:   y = y - 1; break;
            case Orientation.Down: y = y + 1;

Stefan_Lang

Я согласен с вашей оценкой, то, что вы назвали newX и newY, похоже, являются координатами и должно быть проверено на их действительный диапазон.

Более того, похоже, что x и y используются как смещения, а не как координаты, и что последние две строки должны фактически присваивать абсолютные значения +1 или -1 y (и, вероятно, x в следующем коде, который не показан), а не изменять текущие значения путем увеличения или уменьшения.

Рейтинг:
2

Patrice T

Цитата:
не знаю , как исправить эту проблему, пожалуйста, помогите, любые предложения приветствуются

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

Отладка кода C# в Visual Studio - YouTube[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.