Почему элементы списка подгружаются не так как надо?

Question

[DarkByte2015]

Есть задача - сделать бесконечную ленту дат (календарь не канает). Я делаю ее на основе ListView. Сделал свой адаптер для массива дат и layout для элемента. Все работает, осталось только допилить логику подгрузки элементов при прокрутке. Я думаю это надо сделать как-то так: изначально список прокручен до последнего элемента (текущая дата) - сделал android:stackFromBottom="true" и в него загружены даты за последний год (но для теста сделал пока-что за последние несколько дней чтобы долго не листать), когда юзер начинает прокручивать список и доходит до верхней границы (или даже лучше заранее подгрузить firstVisibleItem < 5 например) то подгружаются элементы еще за год. Я все сделал - должно работать, но получилась хрень какая-то... Сначала все загружается нормально, но когда я начинаю скроллить - даты начинают лететь со страшной скоростью - все проматывается моментально аж на годы назад и вставляется в неправильном порядке. Я залогировал firstVisibleItem - он почему то все время в районе 4-6 находится, хотя по идее должен быть в конце списка (ну почти). Почему так происходит?

код

@BindView(R.id.datesList)
ListView datesList;

final ArrayList<DateTime> dates = new ArrayList<>();

в MainActivity.onCreate

addDatesRange(DateTime.now());
final DatesAdapter adapter = new DatesAdapter(this, R.layout.date_item, dates);
datesList.setAdapter(adapter);

datesList.setOnScrollListener(new AbsListView.OnScrollListener() {
    @Override
    public void onScrollStateChanged(AbsListView view, int scrollState) {
    }

    @Override
    public void onScroll(AbsListView view, int firstVisibleItem, int visibleItemCount, int totalItemCount) {
        String msg = String.format("onScroll: firstVisibleItem = %d, visibleItemCount = %d, totalItemCount = %d", firstVisibleItem, visibleItemCount, totalItemCount);
        System.out.println(msg);

        if (totalItemCount > 5 && firstVisibleItem < 5) {
            addDatesRange(dates.get(0).minusDays(1));
            adapter.notifyDataSetChanged();
        }
    }
});

private void addDatesRange(DateTime end) {
	DateTime start = end.minusDays(6); //end.minusYears(1);

	String msg = String.format("addDatesRange: start = %s, end = %s", start.toString("dd.MM.yyyy"), end.toString("dd.MM.yyyy"));
	System.out.println(msg);

	dates.addAll(0, getDatesRange(start, end));
}

public static ArrayList<DateTime> getDatesRange(DateTime start, DateTime end) {
	ArrayList<DateTime> result = new ArrayList<>();

	while (start.isBefore(end)) {
	    result.add(start);
	    start = start.plusDays(1);
	}

	result.add(start);
	return result;
}

Answer 1

[Денис Загаевский]

Имхо, сделано в лоб, сложно и неоптимально.
Я бы взял RecyclerView , в адаптере количество отдавал бы Integer.MAX_VALUE. В бинде по позиции вычислял бы дату(с помощью календаря).
Плюсы, кажется, очевидны - не нужен пейджинг, не нужно хранить список дат, не нужно сложной логики.

Comments

[DarkByte2015]

Не понял как именно определять дату по позиции? Можно пример?

P.S. Отдавать количество в адаптере - это перегрузить метод getCount? И кстати почему именно RecyclerView? Чем ListView для этих целей не подходит?

[Денис Загаевский]

DarkByte2015: ListView не подходит тем, что он устарел, нужно морочиться с кастомными вьюхолдерами и позволяет он очень мало (костыли для анимаций, декораций и прочего). Так-то дело ваше, можете и его использовать, от этого глобально ничего не изменится.

getCount, да.

Например, хранить GregorianCalendar с некоей базовой датой плюс текущую позицию, на которой календарь находится. При появлении новой позиции считать разницу и звать add.

[DarkByte2015]

Денис Загаевский: Можете назвать конкретно какой метод вызывать у календаря? Или еще лучше у JodaTime - DateTime который я использую. Я просто очень начинающий в java. Еще почти совсем не знаю стандартную библиотеку тут. (перешел с C#) Пожалуйста кодом пример - хотя-бы как именно вы представляете себе это получение даты.

[DarkByte2015]

Денис Загаевский: Что значит базовой датой? Так он будет не бесконечный... Мне надо чтобы по крайней мере до 0000 года был (типа как минимально значение). Правда я совсем не понимаю что вы имеет ввиду под этим получением даты... Что будет представлять собой позиция? Дни? Миллисекунды? Не понимаю хоть убей что вы имеете ввиду...

[Денис Загаевский]

DarkByte2015: У календаря - add, как я сказал. У JodaTime так же, выбирайте базовую дату, у которой position=0, базовый DateTime baseDate; от неё, и затем нужный DateTime можно получить так

final DateTime currentDate = baseDate.plusDays(currentPosition);

[DarkByte2015]

Денис Загаевский: А вы представляете себе что за дата получится на выходе? Вот например я сейчас в отладчике вижу position == 2147483646 (как раз максимальное значение видимо). Если я возьму базовую дату например new DateTime(0, 1, 1, 1, 1) и прибавлю к ней это число дней то получается:



Как-то многовато. Не находите? Мне же надо чтобы верхняя граница была сегодняшним днем.

[Денис Загаевский]

DarkByte2015: базовая дата может быть хоть сегодняшнее число. Можете сделать хоть так - псевдокод

final int basePosition = adapter.getCount()/2;
final DateTime baseDate =  currentDate(); //любая дата, какую хотите
...
recycler.scrollToPosition(basePosition); //где-то вне адаптера
...
//в bindViewHolder
final long delta = currentPosition - basePosition;
final DateTime currentDate = delta > 0 ? baseDate.plusDays(delta) : baseDate.minusDays(-delta); //тут я не знаю, что будет, если отрицательное число отправить, надо проверять, но вот так будет точно ок.

[Денис Загаевский]

DarkByte2015: это всего лишь значит, что сделали что-то не так. Вы всегда должны быть где-то посередине, между MAX_VALUE и 0. До них пользователь доскроллить не должен.

[DarkByte2015]

Денис Загаевский: Не, так тоже не получается. Ну сами подумайте: что толку прибавлять к какой-то рандомной дате позицию? Что вы хотите видеть на выходе я не понимаю? Максимальная дата должна быть - сегодняшний день. Когда пользователь доскролит вниз до сегодняшнего дня - дальше он скроллить не должен - скролл вниз должен заблокироваться. И еще не понял что значит посередине? Пользователь может быть не только в середине, он МОЖЕТ доскроллить до нижней границы в сегодняшнюю дату (фактически он там должен находится в момент старта приложения) и теоретически он может доскроллить до верхней границы 0 года. Именно это мне надо сделать. Мне не нужен набор рандомных дат - эти даты должны быть от 0 года до сегодняшнего дня.

[Денис Загаевский]

DarkByte2015: Я, типа, общий принцип написал. Дальше модифицируй его как хочешь. Если максимальная дата сегодняшний день, и он внизу, то надо сделать так

final int basePosition = Integer.MAX_VALUE;
final DateTime baseDate = currentDate();
...
//в bind
final long delta = basePosition - currentPosition;//всегда будет больше нуля
final DateTime currentDate = baseDate.minusDays(delta); //сегодняшняя дата минус количество дней до той, которую биндим.

//где-то ещё скроллим к текущей позиции(Integer.MAX_VALUE), или играемся с <a href="https://developer.android.com/reference/android/support/v7/widget/LinearLayoutManager.html#setReverseLayout(boolean)">reverseLayout</a> / <a href="https://developer.android.com/reference/android/widget/AbsListView.html#setStackFromBottom(boolean)">stackFromBottom</a>

[DarkByte2015]

Денис Загаевский: Ооо... Ну наконец-то заработало. Спасибо огромное! =)

[Денис Загаевский]

DarkByte2015: не за что. Рекомендую поиграться с этим, попробовать сделать бесконечный в обе стороны список, например и тд. Чтобы понять принцип. У меня сложилось впечатление, что ты его не особо понял.

[DarkByte2015]

Денис Загаевский: А не подскажете нельзя ли обойтись без лайаута элемента? Ну вот например обычному ArrayAdapter же не требуется делать лайаут для элементов базовых типов. Я имею ввиду можно ли сделать что-то вроде этого? (только лайаут и список дат не указывать, ну и чтобы работало)

datesList.setAdapter(new ArrayAdapter<DateTime>(this, R.layout.date_item, dates)
{
    @Override
    public DateTime getItem(int position) {
        final int basePosition = Integer.MAX_VALUE;
        final DateTime baseDate = DateTime.now().plusDays(1);
        final int delta = basePosition - position;
        return baseDate.minusDays(delta);
    }

    @Override
    public int getCount() {
        return Integer.MAX_VALUE;
    }
});

[Денис Загаевский]

Можно использовать лэйауты по умолчанию из android.R.layout.*
Но зачем?

[DarkByte2015]

Денис Загаевский: Чтобы не делать лишний лайаут для элемента. Просто элементы то у меня простые - дата и все. Больше ничего от этих элементов не надо. Поэтому на лайате для элемента у меня лежит всего-лишь один единственный TextView и все. Думал можно было-бы и без него обойтись...

[Денис Загаевский]

DarkByte2015: Это экономия на спичах.