Как создать кнопку, которая циклически перебирает элементы массива?

Question

[Денис Каракчиев]

Имеются следующие элементы и их назначения(весь код приводить не буду, думаю и так все понятно):

private Button next_button;
 private TextView mQuestionTextView;
 private int mCurrentIndex = 0;
 private TrueFalse[] mQuestionBank = new TrueFalse[] {
            new TrueFalse(R.string.question_ocean, true),
            new TrueFalse(R.string.question_mideast, false),
            new TrueFalse(R.string.question_africa, false),
            new TrueFalse(R.string.question_americas, true),
            new TrueFalse(R.string.question_asia, true)
    };

    <b>private void updateQuestion() {
        if (mCurrentIndex != -1) {
            int question = mQuestionBank[mCurrentIndex].getQuestion();
            mQuestionTextView.setText(question);
        } else {
            throw new ArrayIndexOutOfBoundsException("ArrayIndexOutOfBoundException");
            mQuestionTextView.setText(arrayExc);</b>

Далее в методе onCreate это происходит так:
Button mPrevButton = (Button)findViewById(R.id.prevButton);
        mPrevButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v)  {
                    mCurrentIndex = (mCurrentIndex - 1) % mQuestionBank.length;
                    updateQuestion();
                }
        });

Все работает только для элемента массива > 0, иначе, разумеется, появляется ArrayIndexOutOfBoundsException.
(Жирный код до моих попыток исправить ситуацию выглядел как

int question = mQuestionBank[mCurrentIndex].getQuestion();
            mQuestionTextView.setText(question);

, без if/else. в методе updateQuestion.

При этом кнопка "Next", перебирающая элементы массива работает циклически, и не возникает исключения выходы за пределы массива:

mNextButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mCurrentIndex = (mCurrentIndex + 1) % mQuestionBank.length;
                updateQuestion();

Суть проблемы: две кнопки, пользующие одинаковые методы, но одна работает в положительную сторону нормально

mCurrentIndex = (mCurrentIndex + 1) % mQuestionBank.length

;, а другая

mCurrentIndex = (mCurrentIndex - 1) % mQuestionBank.length;

не работает.

Answer 1

[Алексей Сергей]

Все работает только для элемента массива > 0, иначе, разумеется, появляется ArrayIndexOutOfBoundsException

Вы же сами и указали причину из-за которой все валится.
(mCurrentIndex - 1) % mQuestionBank.length = -1 , при mCurrentIndex = 0

исправить prevButton можно, например, так

public void onClick(View v)  {
  mCurrentIndex--;
  if (mCurrentIndex == -1) {
    mCurrentIndex = mQuestionBank.length - 1;
  }
  updateQuestion();
}

Comments

[Денис Каракчиев]

"Этот оператор возвращает значение остатка при делении первого операнда на второй. Результат будет иметь тот же знак, что и первый операнд.", т.е. мы даем отрицательный знак для mQuestionBank, тем самымм делая положительным индекс?

[Денис Каракчиев]

Ииии нет, теперь по нажатии prevButton просто ничего не происходит.

[Алексей Сергей]

Денис Каракчиев: так, еще раз смотрим на это выражение: mCurrentIndex = (mCurrentIndex - 1) % mQuestionBank.length

рассмотрим случай, когда на входе mCurrentIndex равняется нулю:
mCurrentIndex = (0 - 1) % mQuestionBank.length; т.е. mCurrentIndex = -1;

последующий вызов updateQuestion() кинет исключение.
теперь, меняем Ваш код:
public void onClick(View v) {
mCurrentIndex = (mCurrentIndex - 1) % mQuestionBank.length;
updateQuestion();
}
на
public void onClick(View v) {
mCurrentIndex--; // уменьшаем текущий индекс на 1
if (mCurrentIndex == -1) {
// если перешли нижнюю границу
// переходим к последнему элементу
mCurrentIndex = mQuestionBank.length - 1;
}
updateQuestion();
}

[Денис Каракчиев]

Алексей Сергей: я новичек в программировании, логика на нуле еще, извиняюсь =) Не подумал, что updateQuestion нужно поставить после { в if-statement'е. Спасибо огромное, заработало =)

Вот только на понимание: почему mQuestionBank.length-1 вообще дает последний элемент массива, а не уменьшает его length на еденицу?

[Алексей Сергей]

хм, как же объяснить то)
вот смотри, допустип есть простой массив:
char [] arr = {'a', 'b', 'c', 'd'};
Его длина (arr.length) равна 4
В то же время индексация начинается с нуля ->

System.out.println(arr[0]); // печатает 'a'
System.out.println(arr[1]); // печатает 'b'
System.out.println(arr[2]); // печатает 'c'
System.out.println(arr[3]); // печатает 'd'

Индекс первого элемента равен 0, последнего (он же (arr.length - 1)) -- 3
Попытка выйти за рамки -- получаем исключение.

к вопросу, почему не меняется length...
точно так же как и не поменяется x в коде ниже:
int x = 5;
int y = x + 1;
System.out.println(x); // печатает 5
System.out.println(y); // печатает 6

по сути мы читаем значение из переменной x; точно также читаем и количество записей в массиве.
кстати, java не даст скомпилировать решение, которое пытается явно изменить поле length в массиве

[Денис Каракчиев]

Алексей Сергей: Спасибо, эти вещи вроде понятны. Но вот конкретно мучает вопрос: почему мы получаем именно ПРЕДЫДУЩИЙ элемент массива, от того элемента, в котором находимся, а не элемент массива по .length-1, т.е. при 4-х элементах должен был получиться 3-й. Ох не варит мой котелок )

[Алексей Сергей]

снова простой массив
char [] arr = {'a', 'b', 'c', 'd'};
допустимые индексы: 0, 1, 2, 3
arr.length == 4

гоняем циклично,
в случае кнопки next должны получить такую последовательность индексов:
0, 1, 2, 3, 0, 1, 2, 3, 0, 1 ..
для кнопки prev: 0, 3, 2, 1, 0, 3, 2, 1, 0, 3, 2, 1

допустим mCurrentIndex = 1
жмем на prevButton

mCurrentIndex--; // уменьшаем текущий индекс на один
if (mCurrentIndex == -1) // условие не выполняется, т.к. mCurrentIndex = 0
...

теперь снова жмем на prevButton
mCurrentIndex--; // еще раз уменьшили текущий индекс на единицу
if (mCurrentIndex == -1) { // условие выполняется, т.к. mCurrentIndex теперь равно -1
// переводим mCurrentIndex на правый конец массива
mCurrentIndex = arr.length - 1; // mCurrentLength = 4 - 1 = 3
}

[Денис Каракчиев]

Алексей Сергей: понял! Не сразу, но понял :D Осталось донять кого-нибудь вопросом: почему так же работает nextButton, при отсутствии if-statement, но это уже не так важно. Спасибо Вам огромное! =)