设置适配器时,RecyclerView会阻止UI

时间:2017-09-09 02:44:20

标签: android android-recyclerview

我有一个要求,我有一个超过300k行的填充数据库。我已成功实现了一个基于此问题的CursorAdapter,其中混合了两个最常投票的答案HERE

我已经为后台服务实现了一个AsyncTask来执行对数据库的查询,速度非常快,不需要超过2-3秒。来自AsyncTask的My ProgressDialog有时难以检测。

我的问题是,当任务完成并且我检索Cursor时,当我将Adapter设置为RecyclerView时,该过程会冻结我的UI几秒钟,直到设置数据。当我执行搜索时(新查询,与获取所有行但行数较少的相同过程)也会发生这种情况,并替换Cursor以更新数据。

以下是一些相关代码:

的AsyncTask

@Override
protected Void doInBackground(Void... Void) {
    if(type==Constants.GET_ZIP_CODES)
        cursor = db.getAllZipCodes();
    else
        cursor = db.searchZipCodes(text);

    return null;
}

@Override
protected void onPostExecute(Void Void) {
    setAdapter();
    mProgressDialog.dismiss();
    super.onPostExecute(Void);
}

方法

private void setAdapter(){
    if(myAdapter == null){
        myAdapter = new MyAdapter(getActivity(), cursor);
        search_rv.setAdapter(myAdapter);
    } else
        myAdapter.swapCursor(cursor);
}

由于这是一次搜索,除了notifyDataSetChanged()之外我没有太多事情要做,因为所有数据在每次搜索中都会发生很大的变化。

这是正常的吗?由于RecyclerView只渲染可见视图,为什么它会冻结并且需要很长时间才能更新,因为Cursor已经从AsyncTask准备就绪了?

修改

我已经更改了我的适配器以避免使用CursorAdapter,因为@ cricket_007指出在适配器中有一个适配器是不好的设计。

这是我的适配器:

public class SearchListAdapter extends RecyclerView.Adapter<SearchListAdapter.ViewHolder> {


private Context mContext;

private Cursor mCursor;

private boolean mDataValid;
private int mRowIdColumn;

private DataSetObserver mDataSetObserver;

public SearchListAdapter(Context context, Cursor c) {

    mContext = context;
    mCursor=c;

    mDataValid = c != null;
    mRowIdColumn = mDataValid ? mCursor.getColumnIndex("_id") : -1;
    mDataSetObserver = new NotifyingDataSetObserver();
    if (mCursor != null) {
        mCursor.registerDataSetObserver(mDataSetObserver);
    }

}

static class ViewHolder extends RecyclerView.ViewHolder {
    TextView itemTV;

    ViewHolder(View itemView) {
        super(itemView);
        itemTV = (TextView) itemView.findViewById(R.id.itemTV);
    }
}


@Override
public void setHasStableIds(boolean hasStableIds) {
    super.setHasStableIds(true);
}

@Override
public int getItemCount() {
    if (mDataValid && mCursor != null) {
        return mCursor.getCount();
    }
    return 0;
}

@Override
public void onBindViewHolder(ViewHolder holder, int position) {
    // Passing the binding operation to cursor loader
    mCursor.moveToPosition(position);
    String town = mCursor.getString(mCursor.getColumnIndex(Constants.COLUMN_TOWN));
    String zipcode = mCursor.getString(mCursor.getColumnIndex(Constants.COLUMN_ZIPCODE));
    String zipcode_etx = mCursor.getString(mCursor.getColumnIndex(Constants.COLUMN_ZIPCODE_EXTENSION));
    holder.itemTV.setText(zipcode+"-"+zipcode_etx+", "+town);

}

@Override
public SearchListAdapter.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    View itemView = LayoutInflater.from(parent.getContext()).inflate(R.layout.fragment_search_list_item,parent,false);
    // Passing the inflater job to the cursor-adapter
    return new SearchListAdapter.ViewHolder(itemView);
}

public void swapCursor(Cursor cursor) {
    Cursor old = changeCursor(cursor);
    if (old != null) {
        old.close();
    }
}

private Cursor changeCursor(Cursor newCursor) {
    if (newCursor == mCursor) {
        return null;
    }
    final Cursor oldCursor = mCursor;
    if (oldCursor != null && mDataSetObserver != null) {
        oldCursor.unregisterDataSetObserver(mDataSetObserver);
    }
    mCursor = newCursor;
    if (mCursor != null) {
        if (mDataSetObserver != null) {
            mCursor.registerDataSetObserver(mDataSetObserver);
        }
        mRowIdColumn = newCursor.getColumnIndexOrThrow("_id");
        mDataValid = true;
        notifyDataSetChanged();
    } else {
        mRowIdColumn = -1;
        mDataValid = false;
        notifyDataSetChanged();
    }
    return oldCursor;
}

private class NotifyingDataSetObserver extends DataSetObserver {
    @Override
    public void onChanged() {
        super.onChanged();
        mDataValid = true;
        notifyDataSetChanged();
    }

    @Override
    public void onInvalidated() {
        super.onInvalidated();
        mDataValid = false;
        notifyDataSetChanged();
    }
}


}

2 个答案:

答案 0 :(得分:0)

实际上,这不是一个答案(如果我有足够的声誉点,可以将其置于评论中)只是我在将数据从数据库加载到recyclerView时遇到的建议/案例研究。而不是直接将光标发送到适配器我发送它作为arraylist,但那是除了点之外。

当我需要将超过700-800个字符的文本加载到卡片中时,我似乎得到了冻结的地方。因此,当我将文本裁剪为低于600时,冻结消失。

因此,只需检查您是否有任何具有大字符集的数据,如果是这样,请尝试删除并测试它。

希望它对你有用,建议把更多的选项放在像whatsapp这样的大文本上!

答案 1 :(得分:0)

好吧,我发现了为什么会这样,原因很奇怪。这个问题与RecyclerView无关,但与数据获取的方式有关。

在我获取数据的AsyncTask中,我编写了一个Log.d来打印Cursor大小,如下所示:

@Override
protected Void doInBackground(Void... Void) {
    if(type==Constants.GET_ZIP_CODES)
        cursor = db.getAllZipCodes();
    else
        cursor = db.searchZipCodes(text);

    Log.d("DATABASE","SIZE   "+cursor.getCount());
    return null;
}

这使得AsyncTask需要更长的时间,ProgressDialog需要更长的时间才能完成。我理解的是,不知何故,数据库查询被执行,代码继续编译,但数据只在一段时间后在Cursor中准备好了。一旦我在查询之后打印结果,它就没有经过该行,直到光标完全加载为止。