Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AsyncTask with support LRUCache. Ruslan Murzin #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rusyamurzin
Copy link

Реализация с поддержкой кэширования. Задание от Алексея Никитина.
Руслан Мурзин

Log.w(TAG, "MalformedURL exception");
}
try {
bitmap = downloadImageByUrl(url);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Тут url может быть null-ом?

Выглядит так, что здесь подойдёт один блок try и несколько/один catch

@HighFlyer13
Copy link
Collaborator

Не нашёл кода работы с файлами.

@rusyamurzin
Copy link
Author

Посмотрите внимательней, в классе CityCamActivity с 80-й строчки идет работа с LRUCache, а в методе onPostExecute() добавление в кэш. Сам кэш представлен в виде класса MyCache, который фактически Singletone.

@HighFlyer13
Copy link
Collaborator

LRUCache об оперативной памяти.

@rusyamurzin
Copy link
Author

Я делал задание так, как понял его с ваших слов на занятии. И я понял так, что приложение должно работать при отключении интернета, то есть должно показывать ранее посещенные камеры. И я рассматривал это в рамках одного сеанса ещё и для того, чтобы постоянно не просить новое изображение из сети, а брать закэшированное. Но сейчас я понимаю, что вы хотели немного другое: приложение при включенном интернете должно всегда ходить по сети и класть изображения камер в кэш, а при отключенном интернете пользоваться изображениями из кэша. В любом случае, сейчас я не возьмусь всё переделывать, потому что нужно писать диплом. Смогу переделать после защиты диплома ближе к середине июня.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants