Skip to content

@aleien, Манюхина#15

Open
aleien wants to merge 10 commits intoyamblz-native:masterfrom
aleien:master
Open

@aleien, Манюхина#15
aleien wants to merge 10 commits intoyamblz-native:masterfrom
aleien:master

Conversation

@aleien
Copy link
Copy Markdown

@aleien aleien commented Aug 3, 2016

background-task

Задание 1
Параллельная загрузка картинок реализована средствами rxjava, конкретно - через оператор zip

  • кэширование картинок в памяти через LruCache
    Можно улучшить - скачивать картинки в файл, и организовать кэш на диске.

Задание 2
Реализация запуска тасков через IdleHandler >:D


@Override
public void loadCollage(List<String> urls, ImageTarget imageTarget, CollageStrategy collageStrategy) {
if (subs == null) subs = new CompositeSubscription();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тебе не лень было два одинаковых метода писать?) Можно упаковать ImageView в ImageTarget

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У меня в голове это висит, просто еще руки не дошли это сделать)
А так да, так и собиралась. Правда, не уверена была, что это нормально будет - упаковать в ImageTarget.
Там же так будет?

loadCollage(... ImageView imageview ...) {
    loadCollage(...new ImageTarget(imageView)...)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ага, типа того

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants