1

Здравствуйте!

Задача такая: нужно создать кнопку, которая при клике возвращает предыдущий элемент массива. Надо, чтобы при значении индекса массива -1 индекс принимал значение максимального индекса, то есть шёл по кругу в обратном порядке. Я захотел организовать кликер с помощью исключений, но у меня ничего не получилось, ошибка выхода за границы массива всё равно вылазила:

public void onClick(View view) {
            try {
                mCurrentIndex = (mCurrentIndex - 1) % mQuestionBank.length;
            }catch(ArrayIndexOutOfBoundsException e){
                mCurrentIndex=mQuestionBank.length;
            }
}

Что я сделал не так?

5
  • mCurrentIndex = (mCurrentIndex<0)? (mQuestionBank.length-1) : (mCurrentIndex - 1); 2 сен 2014 в 11:50
  • Мне почему-то кажется, что массив у нас обычно mQuestionBank[0..mQuestionBank.length-1]. Подозреваю, что дело в этом.
    – Чад
    2 сен 2014 в 11:55
  • Окей, я исправил " mCurrentIndex=mQuestionBank.length;" на "mCurrentIndex=mQuestionBank.length-1;", но не помогло.
    – Nemezis16
    2 сен 2014 в 12:06
  • А, так и не должно, у вас наверное ексепшен то в другом месте возникает? Вам надо try{ mc...=; mQuestionBank[mc...] = something value;/*вот тут ексепшен и возникнет и перейдёт в кетч*/ } catch(
    – Чад
    2 сен 2014 в 12:13
  • Чад, спасибо, ваш комментарий мне помог!
    – Nemezis16
    2 сен 2014 в 13:34

2 ответа 2

3

Ну а почему бы не сделать так:

if (mCurrentIndex<0)mCurrentIndex=mQuestionBank.length-1

И ещё mQuestionBank.length возвращает длину массива, но поскольку нумерация элементов начинается с нуля, последний элемент будет иметь индекс mQuestionBank.length-1

1
  • Сделать так можно, просто меня заинтересовал вариант с исключением. Я был уверен, что оно заработает.
    – Nemezis16
    2 сен 2014 в 12:01
1

Вообще сам механизм исключений медленный. Намного быстрей будет действовать оператор if:

private int[] array = { 1, 2, 3, 4};

private int currIndex;

{
    currIndex = array.length - 1;
}

public void onClick() {
    int result = array[currIndex--];

    if (currIndex == -1) {
        currIndex = array.length - 1;
    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.