anton9088 Jun 20 2014 at 02:15

64 миллисекунды после нажатия

3 min

30K

Development for Android*

+41

Comments 23

Zibx Jun 20 2014 at 05:43

Отличный макаронный код в ядре, а 64мс — прекрасная цифра! Порой бывает думаешь что в крупных компаниях код пишут как-то иначе, даже при приближении дедлайна.

lNevermore Jun 20 2014 at 06:22

Исходный код Android OS вообще часто радует глаз. Посмотрите, советую. Особенно наследников класса View.

demand Jun 20 2014 at 11:02

А я больше люблю сам класс View. Еще ни разу до конца не дочитал )
А вообще, качество кода местами удручает и читается плохо. Но он по крайне мере есть и можно в него залезть и увидеть, что там происходит и как это обойти.

anton9088 Jun 20 2014 at 13:47

в некоторых версиях android она 128)

anton9088 Jun 20 2014 at 15:58

в 4.4 его начали рефакторить)

akira Jun 20 2014 at 06:07

Зачем создавать новый адаптер каждый раз при обновлении данных?

anton9088 Jun 20 2014 at 12:57

Из-за реализации адаптера секций другого пути не было

akira Jun 20 2014 at 13:03

А причем тут реализация секций? С ними работаю через getViewType в getView и выводят нужный шаблон.
Или просто github.com/emilsjolander/StickyListHeaders

recompileme Jun 20 2014 at 09:54

1. Перед тем как обновлять адаптер необходимо вызвать метод
adapterMain.notifyDataSetInvalidated();
2. После окончания работы с адаптером:
adapterMain.notifyDataSetChanged();
3. Прочитайте про viewHolder pattern, да и вобще, ваш код адаптера — ужасен. См. пункт 5
4. Цифра 64мс не случайна. Именно за этот временной интервал должны быть выполнены изменения выполняемые в UI потоке. В противном случае у пользователя создастся ощущение лага. Я не первый раз слышу эту цифру от Гугла.
5. Посмотрите как нужно правильно делать адаптер с секциями, например тут: github.com/recoilme/freemp/blob/master/src/ru/recoilme/freeamp/player/AdpPlayer.java
6. У гугла куча косяков, но в данном конкретном случае — косячите Вы.

bimeg Jun 20 2014 at 10:56

Цифра 64мс не случайна. Именно за этот временной интервал должны быть выполнены изменения выполняемые в UI потоке.

В данном случае это не о том. Чтобы не было лага надо уложиться в 16мс (60fps). Тут это задержка между тачем и перекраской итема в pressed-state. Чтобы при скроле не было морганий итема в прессед и обратно.

recompileme Jun 20 2014 at 11:18

Согласен. Но все таки 16 мс это совсем жесткач, для приложений требования менее строги: developer.android.com/training/articles/perf-anr.html, последний абзац.

bimeg Jun 20 2014 at 11:31

16мс это про плавные анимации. Первый абзац.
http://developer.android.com/training/custom-views/optimizing-view.html

recompileme Jun 20 2014 at 11:38

Чуть чуть поправлю, ANR все же не появится при 100ms

// Default input dispatching timeout if there is no focused application or paused window
// from which to determine an appropriate dispatching timeout.
const nsecs_t DEFAULT_INPUT_DISPATCHING_TIMEOUT = 5000 * 1000000LL; // 5 sec

anton9088 Jun 20 2014 at 12:52

Код адаптера не мой, просто как-то нашел его в исходниках чужого проекта, он там нормально работал и я решил перетащить его себе

recompileme Jun 20 2014 at 13:28

anton9088 Jun 20 2014 at 19:59

минусуют ребята из Mail.ru которые не используют чужие библиотеки и по хардкору пишут весь код сами?

newcommer Jun 22 2014 at 10:42

Использование чужих библиотек — это не то же самое, что и программирование методом копи-паста.

anton9088 Jun 20 2014 at 13:01

1. Мы не обновляем адаптер — мы устанавливаем новый адаптер, соответственно notifyDataSetChanged вызывать не у кого.
3. В этом адаптере ViewHolder не нужен, т.к. он не инфлейтит вьюшки

anton9088 Jun 20 2014 at 13:33

6. Это ошибка гугла т.к. список должен корректно менять свои адаптеры раз такая возможность есть

lNevermore Jun 20 2014 at 14:25

По хорошему, такой возможности нет. Метод setAdapter у ListView ставить адаптер, а не меняет его. Нету метода replaceAdapter или swapAdapter. Адаптер на то и адаптер, что его надо поставить, а потом просто данные ему скармливать, и он будет ими манипулировать.

anton9088 Jun 21 2014 at 19:55

зачем методы replaceAdapter/swapAdapter? Например, класс TextView обходится одним методом setText и он не только устанавливает текст в первый раз, но и меняет в последующие

lNevermore Jun 22 2014 at 09:02

Верно, но TextView-то виджет, а адаптер нет. Это из семантики вещей происходит. Для того, чтобы манипулировать данными в адаптере есть специальные методы, такие как add, addAll, remove, clear и т.д в то время как адаптер должен быть одним. Это просто логично:)

Если ваш адаптер работает таким образом, что его необходимо постоянно пересоздавать, то советую взглянуть на другие альтернативы. Например, как писали выше, StickyListHeaders. Пользовался им, отлично написано и работает правильно.

Lissov Jun 20 2014 at 10:08

Если вы кликнули на ячейку, а в этот момент загрузились новые данные с сервера и установились в список, значит вы кликнули по новым данным…
Всегда проверяйте результат метода ...

Но самой по себе такой проверки не достаточно. Ведь если новые данные в той же ячейке кликабельны, то пользователь получит не то, на то что кликал — тоже плохо. То есть обязательно надо что-то еще — например, как и написано, устанавливать mDataChanged. Я бы в такой ситуации просто ставил Enabled=false на время загрузки новых данных — чтобы пользователь даже не видел анимацию при нажатии и не удивлялся, что ничего не произошло.