Чаркин Константин(@kostya_charkin), HW background #4
Conversation
|
Держи |
| return genres; | ||
| } | ||
|
|
||
| public Artist(JSONObject jsonObject) { |
There was a problem hiding this comment.
Вообще очень плохая практика парсить джейсон внутри конструктора
Ты ведь даже не сможешь понять нормально обьект создался или с ошибкой, придется создать обьект в любом случае, плюс это еще возможно тяжелая операция, конструктор должен быть легким
| Log.d("onBindViewHolder,pos = ", Integer.toString(position)); | ||
| Genre genre = genres.get(position); | ||
| holder.textInList.setText(genre.getDescription()); | ||
| new Thread(() -> { |
There was a problem hiding this comment.
На каждый onBindViewHolder новый поток? не надо так
Хочется чтобы было также легко использовать как пикассо или подобное. А тут ты все делаешь на потоке из которого запустили функцию loadCollage, да и еще количество этих потоков никак не контролируешь. Операции не отменяешь (Как минимум не вижу этого) на новый onBindViewHolder
2 participants
add task 2 (handler).
upd:
добавил задание с loader-ом, коллажи делает, но периодически картинки сетятся не в те вьюхи, почему - не знаю😕.
upd:
Добавил отмену старого таска, если на туже вьюху приходит новый.
upd:
добавил
ExecutorServiceдля ограничения кол-ва потоков, убрал создание потоков в реализациюCollageLoader