RecyclerView加载图片和滑动时的问题解决方案

1. SwipRefreshLayout刷新的同时Recyclerview快速上滑或者点击item时crash

Error
   java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid view holder adapter positionViewHolder{1ab30d24 position=4 id=-1, oldPos=-1, pLpos:-1 no parent}
   at android.support.v7.widget.RecyclerView$Recycler.validateViewHolderForOffsetPosition(RecyclerView.java:4801)
Reason
...
onRefreshListener = new SwipeRefreshLayout.OnRefreshListener() {
            @Override
            public void onRefresh() {
                if (NetWorkTools.isNetworkConnected(getContext())){
                    mPage = 0;
                    mList.clear();
                    mIsRefreshing=true;
                    mThread = new GetMemberItemThread(memberRefreshHandler);
                    mThread.start();
                } else {
                    Toast.makeText(getContext(),"请检查网络连接",Toast.LENGTH_SHORT).show();
                    refreshLayout.setRefreshing(false);
                }
            }
        };
...
    @Override
    public void handleMessage(Message msg) {
        MemberFragment memberFragment = refMemberFragment.get();
        if (memberFragment != null){
            switch (msg.what){
                case Constant.HANDLER_REQUEST_MEMBERS:                        
                    mList = (List<MemberInfo>) msg.obj;
                    mList.addAll(cacheList);
                    adapter.notifyDataSetChanged();
                    isFirstPage = false;
                    canLoadMore = true;
                    refreshLayout.setRefreshing(false);
                    break;
                case Constant.HANDLER_REFRESH_SEARCHLIST:
...
Analyse

SwipeRefreshView刷新时回调onRefresh()方法,上面的代码mList.clear()将数据清空,然后开启线程请求数据,成功时候会通过handler传递过来给handleMessage()处理。由于新的线程请求数据需要一定的时间,如果数据还没请求成功此时点击item或者上滑的话, adapter并没有调用notify**函数通知recyclerview更新数据,造成前后数据不一致,就会出现上面的错误。

Solve:

mList.clear();后面加上notifyDataChanged();方法后问题解决。然而,这样又会造成RecyclerView闪烁一下然后显示数据。因此,可以将mList.clear()方法放到新数据到来之后执行。修改后的代码如下:

...   
onRefreshListener = new SwipeRefreshLayout.OnRefreshListener() {
            @Override
            public void onRefresh() {
                if (NetWorkTools.isNetworkConnected(getContext())){
                    mPage = 0;
                    isFirstPage = true;
                    mThread = new GetMemberItemThread(memberRefreshHandler);
                    mThread.start();
                } else {
                    Toast.makeText(getContext(),"请检查网络连接",Toast.LENGTH_SHORT).show();
                    refreshLayout.setRefreshing(false);
                }
            }
        };
...
@Override
 public void handleMessage(Message msg) {
        MemberFragment memberFragment = refMemberFragment.get();
        if (memberFragment != null){
            switch (msg.what){
                case Constant.HANDLER_REQUEST_MEMBERS:
                    List<MemberInfo> cacheList;
                    cacheList = (List<MemberInfo>) msg.obj;
                    mList.addAll(cacheList);
                    if (isFirstPage){
                        adapter.setData(cacheList);
                        mList.clear();
                        mList.addAll(cacheList);
                    } else {
                        adapter.setData(mList);
                    }
                    adapter.notifyDataSetChanged();
                    isFirstPage = false;
                    canLoadMore = true;
                    refreshLayout.setRefreshing(false);
                    break;
                case Constant.HANDLER_REFRESH_SEARCHLIST:
...

2.Glide加载CircleImageView时的图片时设置了 placeholder 同时设置 crossfade 时抖动闪烁,占用内存超大

图片头像用了开源的圆形 Imageview ( https://github.com/hdodenhof/CircleImageView ),但是用 Recyclerview 加载时会出现错乱的问题,就是快速滑动时在显示正确图片前会在其他图片之间跳动几次最终才显示正确。作者其实已经给出了说明:

  • If you use an image loading library like Picasso or Glide, you need to disable their fade animations to avoid messed up images. For Picasso use the noFade() option, for Glide use dontAnimate(). If you want to keep the fadeIn animation, you have to fetch the image into a Target and apply a custom animation yourself when receiving the Bitmap.
  • Using a TransitionDrawable with CircleImageView doesn't work properly and leads to messed up images.

就是说如果使用 Picasso 或者 Glide 的话,需要去除动画效果来避免图像混乱。如果非要使用动画的话,就要将 image 转化成 Target (在 Glide 中,Target 是 GlideDrawableImageViewTarget,BitmapImageViewTarget,DrawableImageViewTarget之一)并且自定义动画。

解决方案
  1. 按照作者给的方案,使用 dontAnimate(),但是这样没有淡入动画效果了。
  2. 不用 placeholder,这样的话就没有占位符。。。
  3. 最终方案是不用 crossfade(),使用自定义动画 anim(R.anim.fade_in),其实 crossfade() 函数就是创建了一个淡入动画
    private static class DefaultAnimationFactory implements ViewAnimation.AnimationFactory {
    private final int duration;
    DefaultAnimationFactory(int duration) {
    this.duration = duration;
    }
    @Override
    public Animation build() {
    AlphaAnimation animation = new AlphaAnimation(0f, 1f);
    animation.setDuration(duration);
    return animation;
    }
    }
    为啥 crossfade() 就有问题呢?通过源码得知 crossfade 最终动画的执行是在 DrawableCrossFadeViewAnimation
    public boolean animate(T current, ViewAdapter adapter) {
    Drawable previous = adapter.getCurrentDrawable();
    if (previous != null) {
    TransitionDrawable transitionDrawable = new TransitionDrawable(new Drawable[] { previous, current });
    transitionDrawable.setCrossFadeEnabled(true);
    transitionDrawable.startTransition(duration);
    adapter.setDrawable(transitionDrawable);
    return true;
    } else {
    defaultAnimation.animate(current, adapter);
    return false;
    }
    }而自定义动画的执行是在ViewAnimation`
    @Override
    public boolean animate(R current, ViewAdapter adapter) {
    View view = adapter.getView();
    if (view != null) {
    view.clearAnimation();
    Animation animation = animationFactory.build();
    view.startAnimation(animation);
    }
    return false;
    }
    作者说了使用 TransitionDrawable 会混乱,具体原因没有深入去研究,如果有谁知道还望赐教。
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,684评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,143评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,214评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,788评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,796评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,665评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,027评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,679评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 41,346评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,664评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,766评论 1 331
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,412评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,015评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,974评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,203评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,073评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,501评论 2 343

推荐阅读更多精彩内容