swapCursor()导致CursorIndexOutOfBoundsException,行为异常

时间:2017-09-02 05:33:37

标签: android listview simplecursoradapter android-cursoradapter android-cursor

几天前我遇到了这个问题,我正在开发一个程序,其中ListView通过自定义SimpleCursorAdapter从数据库中获取数据,我以前使用listView.setAdapter(adapter)来更新每次数据更改时ListView中的数据。但这会导致一个问题,即每次更新数据时,listView都会自动滚动到顶部,所以我决定改用swapCursor(),如下所示:

Cursor cursor = database.getData();
    if(listView.getAdapter() == null){
        //Toast.makeText(getApplicationContext(),"null",Toast.LENGTH_SHORT).show();
        adapter = (new CustomSimpleCursorAdapter(this,R.layout.todolist,cursor,new String[] {LIST_TEXT},new int[]{R.id.text}));
        listView.setAdapter(adapter);
    }else {
        Runnable runnable =new Runnable() {
            @Override
            public void run() {
                Cursor finalCs = database.getData();
                todoListAdapter.swapCursor(finalCs);
                todoListAdapter.notifyDataSetChanged();
            }
        };
        runOnUiThread(runnable);
    }

但是每次我尝试将数据添加到数据库并在listView中显示时出现错误:

**CursorIndexOutOfBoundsException: Index 12 requested, with a size of 12**

(请注意,数据已添加到数据库中,之前未使用listView.setAdapter(adapter)发现任何问题,我尝试删除todoListAdapter.notifyDataSetChanged();,但这根本没有帮助。我尝试从数据库中删除数据并使用更改更新listView,无论我选择了哪一个,它始终是listView中的最后一项(现在数据实际上已被正确删除)从数据库中),当我再次尝试添加数据时,我现在可以添加等量的数据,如先前删除的数据量(如果我删除了4项,那么我之后只能添加4项,否则会出现错误无论如何,swapCursor()在使用它时似乎很糟糕,我做错了什么?或者我需要其他东西让swapCursor()起作用?提前谢谢!

更新:错误追溯上下文:

 android.database.CursorIndexOutOfBoundsException: Index 13 requested, with a size of 13
                                                                   at android.database.AbstractCursor.checkPosition(AbstractCursor.java:460)
                                                                   at android.database.AbstractWindowedCursor.checkPosition(AbstractWindowedCursor.java:136)
                                                                   at android.database.AbstractWindowedCursor.getString(AbstractWindowedCursor.java:50)
                                                                   at android.support.v4.widget.SimpleCursorAdapter.bindView(SimpleCursorAdapter.java:139)

更新:自定义SimpleCursorAdapter

import android.content.Context;
import android.database.Cursor;
import android.support.v4.widget.SimpleCursorAdapter;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.CheckBox;
import android.widget.TextView;

import java.util.ArrayList;
public class TodoListAdapter extends SimpleCursorAdapter {
private Cursor c;
private Context context;
private ArrayList<Long> itemChecked = new ArrayList<>();
protected int[] mFrom;
protected int[] mTo;

LayoutInflater inflater;
private int mStringConversionColumn = -1;
private CursorToStringConverter mCursorToStringConverter;
private ViewBinder mViewBinder;
MainActivity main;
String[] mOriginalFrom;

public TodoListAdapter(Context context, int layout, Cursor c, String[] from, int[] to) {
    super(context, layout, c, from, to,1);
    this.c = c;
    this.context = context;
    mTo = to;
    mOriginalFrom = from;
    inflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
}

public ViewBinder getViewBinder() {
    return mViewBinder;
}

public ArrayList returnSelected(){
    return itemChecked;
}

public interface MyInterface{
    public void foo();
}

public void setViewBinder(ViewBinder viewBinder) {
    mViewBinder = viewBinder;
}

public View getView(final int pos, View inView, ViewGroup parent) {
    c.moveToPosition(pos);
    if (inView == null) {
        LayoutInflater inflater = (LayoutInflater) context
                .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        inView = inflater.inflate(R.layout.todolist, null);
    }
    final TextView todoText = (TextView) inView.findViewById(R.id.titleText);
    final CheckBox cBox = (CheckBox) inView.findViewById(R.id.multiSelectionBox); 
    System.out.println(c.getCount());
    newView(context,c,parent);
    bindView(inView,context,c);
    final long id = getItemId(pos);
    cBox.setOnClickListener(new View.OnClickListener() {
        public void onClick(View v) {
            CheckBox cb = (CheckBox) v.findViewById(R.id.multiSelectionBox);
            if (cb.isChecked()) {
                if(context.toString().contains("MainActivity")){
                    ((MainActivity)context).addSelectedId(id);
                }else if(context.toString().contains("HistoryActivity")){
                    ((HistoryActivity)context).addSelectedId(id);
                }
                System.out.println("checked " + id);
            } else if (!cb.isChecked()) {
                System.out.println("unchecked " + id);
                if(context.toString().contains("MainActivity")){
                    ((MainActivity)context).removeSelectedId(id);
                }else if(context.toString().contains("HistoryActivity")){
                    ((HistoryActivity)context).removeSelectedId(id);
                }                    
            }
        }
    });
    return inView;
}

}

1 个答案:

答案 0 :(得分:0)

我终于明白了,这只是因为我没有将适配器中的光标更改为传入的游标,覆盖bindView方法并将this.c = cursor添加到原始代码中,然后调用{ {1}},光标将是新光标。这只是我自己犯的一个愚蠢的错误。

特别感谢@pskink帮助我意识到这一点,毕竟我不需要覆盖getViewbindView,但你提到newView让我意识到这个错误