Trinea / android-common

Android common lib, include ImageCache, HttpCache, DropDownListView, DownloadManager, Utils and so on
http://p.codekk.com/
5.01k stars 2.65k forks source link

API design discussion #13

Closed xiaoguaihou closed 9 years ago

xiaoguaihou commented 9 years ago

研究了一下imagecache的code。demo app的用法是: IMAGE_SD_CACHE.get(imageUrlList.get(position), holder.imageView);

一个建议,这个接口是不是不应该关心view呢,包括callback的接口。因为在callback的时候view和url的关系可能变化,或者view已经不可见了。如果用的人把ImageSDCardCache用成singleton的了,那么是不是可能造成view的内存泄漏呢。

可能我的理解有误,供大家讨论吧

Trinea commented 9 years ago

确实可能会有一些内存泄露,暂时没有时间修改,可使用 UIL 代替