java - Android ArrayIndexOutOfBoundsException 和 AbsListViewRecycleBin.addScrapView

Question

我有一个随机出现的 ArrayIndexOutOfBoundsException。它似乎是在我的 notifyDataSetChanged(); 期间发生的，因为错误是如此随机，所以很难准确指出它发生的位置。

有没有人对自定义 Adaptor 有类似的问题？

FATAL EXCEPTION: main
 java.lang.ArrayIndexOutOfBoundsException: length=12; index=12
    at android.widget.AbsListView$RecycleBin.addScrapView(AbsListView.java:8041)
    at android.widget.ListView.layoutChildren(ListView.java:1604)
    at android.widget.AbsListView.onLayout(AbsListView.java:2444)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1677)
    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1531)
    at android.widget.LinearLayout.onLayout(LinearLayout.java:1440)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1677)
    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1531)
    at android.widget.LinearLayout.onLayout(LinearLayout.java:1440)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
    at android.view.View.layout(View.java:15221)
    at android.view.ViewGroup.layout(ViewGroup.java:4793)
    at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:2260)
    at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2007)
    at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1249)
    at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6364)
    at android.view.Choreographer$CallbackRecord.run(Choreographer.java:791)
    at android.view.Choreographer.doCallbacks(Choreographer.java:591)
    at android.view.Choreographer.doFrame(Choreographer.java:561)
    at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:777)
    at android.os.Handler.handleCallback(Handler.java:730)
    at android.os.Handler.dispatchMessage(Handler.java:92)
    at android.os.Looper.loop(Looper.java:137)
    at android.app.ActivityThread.main(ActivityThread.java:5455)
    at java.lang.reflect.Method.invokeNative(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:525)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1187)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)
    at dalvik.system.NativeStart.main(Native Method)




public class CarUpfitScanvinadapter extends BaseAdapter{
    @SuppressWarnings("unused")
    private final String TAG = this.getClass().getSimpleName();
    private Activity mActivity;
    private ArrayList<CarUpfitModel> mData;
    private static LayoutInflater sInflater = null;
    public int height = 0;
    public int heightSet = 0;
    private CarUpfitModelForm mForm;
    private ScanlistListener mCallback;

    public CarUpfitScanvinadapter(Activity a, ArrayList<CarUpfitModel> d, Resources resLocal){
        mActivity = a;
        mData = d;
        sInflater = (LayoutInflater)mActivity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }

    public void removeItem(int i){
        mData.remove(i);
        this.notifyDataSetChanged();
    }

    @Override
    public int getCount() {
        Log.v(TAG, "getCount");
        if(mData.size()<=0) return 1;
        return mData.size();
    }

    @Override
    public boolean hasStableIds (){
        return false;
    }

    @Override
    public int getViewTypeCount() {                 
        return getCount();
    }

    @Override
    public CarUpfitModel getItem(int position) {
        return mData.get(position);
    }

    @Override
    public int getItemViewType(int position) {
        return position;
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        View vi = convertView;
        if(convertView == null){
            vi = sInflater.inflate(R.layout.subaruupfitscanlistview, parent, false);
            mForm = new CarUpfitModelForm();
            mForm.setllScanlist((LinearLayout) vi.findViewById(R.id.llScanlist));
            mForm.getllScanlist().setLayoutParams(new AbsListView.LayoutParams(LayoutParams.MATCH_PARENT,heightSet));
            mForm.setllheightset((LinearLayout) vi.findViewById(R.id.llheightset));
            mForm.setetModel((EditText) vi.findViewById(R.id.etModel));
            mForm.settvScanlistvin((TextView) vi.findViewById(R.id.tvScanlistvin));
            mForm.settvScanlistdate((TextView) vi.findViewById(R.id.tvScanlistdate));
            mForm.setcbGbkit((CheckBox) vi.findViewById(R.id.cbGbkit));
            mForm.setcbFltmat((CheckBox) vi.findViewById(R.id.cbFltmat));
            mForm.setcbFlmatbrk((CheckBox) vi.findViewById(R.id.cbFlmatbrk));
            mForm.setcbEyesight((CheckBox) vi.findViewById(R.id.cbEyesight));
            mForm.setcbTrnkpan((CheckBox) vi.findViewById(R.id.cbTrnkpan));
            mForm.setcbIntmirror((CheckBox) vi.findViewById(R.id.cbIntmirror));
            mForm.setcbExtmirror((CheckBox) vi.findViewById(R.id.cbExtmirror));
            mForm.setcbPzevbadge((CheckBox) vi.findViewById(R.id.cbPzevbadge));
            mForm.setivDelete((ImageView) vi.findViewById(R.id.ivDelete));
            mForm.setivSave((ImageView) vi.findViewById(R.id.ivSave));
            mForm.getivSave().setTag(mForm);
            vi.setTag(mForm);
        }else{
            mForm = (CarUpfitModelForm) vi.getTag();
        }
        mForm.getllScanlist().setOnClickListener(new OnItemClickListener(position, CarUpfit.ITEM_CLICK));
        if(mData.size()<=0){
            mForm.puttvScanlistvin("No Data");
        }else{
            mForm.puttvScanlistvin(getItem(position).getVin());
            mForm.puttvScanlistdate(getItem(position).getDate());
            mForm.setId(getItem(position).getid());
            mForm.putjson(getItem(position).getData());
            mForm.getivSave().setOnClickListener(new OnItemClickListener(position, CarUpfit.ITEM_UPDATE));
            mForm.getivDelete().setOnClickListener(new OnItemClickListener(position, CarUpfit.ITEM_REMOVE));
        }
        return vi;
    }

    private class OnItemClickListener implements OnClickListener{
        private int mPosition;
        private int mType;

        OnItemClickListener(int position, int type){
            mPosition = position;
            mType = type;
        }
        @Override
        public void onClick(View v) {
            mCallback.scanlistclick(v, mPosition, mType);
        }
    }

    public void setimplements(CarUpfit sl) {
        try {
            mCallback = (ScanlistListener) sl;
        } catch (ClassCastException e) {
            throw new ClassCastException("CarUpfit must implement ScanlistListener");
        }
    }

    public interface ScanlistListener{
        public void scanlistclick(View v, int position, int type);
    }
}

编辑

我有一个带有动画的项目列表，可以展开列表项目。这就是为什么我重写了 getViewTypeCount()。如果我在用户打开一个项目时不覆盖它，它也会打开其他项目。

项目关闭的屏幕截图:

项目打开的屏幕截图:

Answer 1

如果您的所有列表项都是统一的，您不应该覆盖 getViewTypeCount() 和 getItemViewType()，或者至少让它们返回 1 和 0 分别。

通过按照您的方式覆盖它们，首先您会抑制 ListView 的 View 回收机制，其次这可能会导致您描述的崩溃，因为当您从列表，删除后 View 的类型发生变化。

更新: 实际上，这是正在发生的事情:ListView 仅在 setAdapter() 中调用适配器上的 getViewTypeCount() 并创建一个该大小的内部数组。因此，如果您稍后添加 View 并且您的 getItemViewType() 返回大于或等于该大小的索引，您将得到 ArrayIndexOutOfBoundsException。

更新 2:如果您需要可扩展的 ListView ，只需使用 ExpandableListView .

更新 3:好的，我会尽可能详尽地解释。

没有理由在屏幕外打开 View 。 ListView 旨在执行回收并在 99.9% 的时间内禁止它，您的情况不属于剩余的 0.01%。

您的适配器应该记住所有细节并在滚动 ListView 时填充新出现的 View 。

在您的情况下，您有两种类型的 View - 打开和关闭，所以 getViewTypeCount() 应该返回 2 而 getItemViewType() 应该返回 0 或 1 取决于 View 是关闭还是打开。

在 getView() 中，如果 convertView 为 null，则应根据 getItemViewType() 为该位置返回的内容来创建它。如果它不为 null，则可以保证它是您之前为同一类型创建的 View 之一。

如果您的 getItemViewType() 返回任意大的数字(我指的是 API 19 的来源)，就会发生崩溃:

// ListView.java
public void setAdapter(ListAdapter adapter) {
    ...
    // This is the only call to getViewTypeCount()
    mRecycler.setViewTypeCount(mAdapter.getViewTypeCount());
    ...
}

// AbsListView.java
public void setViewTypeCount(int viewTypeCount) {
    ...
    // Here it creates the array of given size
    ArrayList<View>[] scrapViews = new ArrayList[viewTypeCount];
    ...
    mViewTypeCount = viewTypeCount;
    mScrapViews = scrapViews;
}

void addScrapView(View scrap, int position) {
    final int viewType = lp.viewType;
    ...
    // And here you get ArrayIndexOutOfBoundsException
    mScrapViews[viewType].add(scrap);
}