滚动包含无线电组的列表视图时出现 NPE

NPE when scrolling listview containing radiogroups

我有以下问题。我有一个显示测试的 ListView。在每个 ListView 行中有一个问题和一个 RadioGroup 有答案。我遇到的问题是每次向下滚动 ListView 时都会收到此错误,我不知道这种行为的原因:

LogCat:

02-05 11:22:56.712  27296-27296/com.fragon.testjson E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.fragon.testjson, PID: 27296
    java.lang.NullPointerException
            at com.fragon.testjson.ListAdapter.getView(ListAdapter.java:38)
            at android.widget.AbsListView.obtainView(AbsListView.java:2255)
            at android.widget.ListView.makeAndAddView(ListView.java:1790)
            at android.widget.ListView.fillDown(ListView.java:691)
            at android.widget.ListView.fillGap(ListView.java:655)
            at android.widget.AbsListView.trackMotionScroll(AbsListView.java:5143)
            at android.widget.AbsListView.scrollIfNeeded(AbsListView.java:3243)
            at android.widget.AbsListView.onTouchMove(AbsListView.java:3587)
            at android.widget.AbsListView.onTouchEvent(AbsListView.java:3431)
            at android.view.View.dispatchTouchEvent(View.java:7837)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2210)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1945)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1959)
            at com.android.internal.policy.impl.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:2075)
            at com.android.internal.policy.impl.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1522)
            at android.app.Activity.dispatchTouchEvent(Activity.java:2458)
            at com.android.internal.policy.impl.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:2023)
            at android.view.View.dispatchPointerEvent(View.java:8017)
            at android.view.ViewRootImpl$ViewPostImeInputStage.processPointerEvent(ViewRootImpl.java:3966)
            at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:3845)
            at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3405)
            at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3455)
            at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3424)
            at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3531)
            at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3432)
            at android.view.ViewRootImpl$AsyncInputStage.apply(ViewRootImpl.java:3588)
            at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3405)
            at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3455)
            at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3424)
            at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3432)
            at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3405)
            at android.view.ViewRootImpl.deliverInputEvent(ViewRootImpl.java:5554)
            at android.view.ViewRootImpl.doProcessInputEvents(ViewRootImpl.java:5534)
            at android.view.ViewRootImpl.enqueueInputEvent(ViewRootImpl.java:5505)
            at android.view.ViewRootImpl$WindowInputEventReceiver.onInputEvent(ViewRootImpl.java:5634)
            at android.view.InputEventReceiver.dispatchInputEvent(InputEventReceiver.java:185)
            at android.view.InputEventReceiver.nativeConsumeBatchedInputEvents(Native Method)
            at android.view.InputEventReceiver.consumeBatchedInputEvents(InputEventReceiver.java:176)
            at android.view.ViewRootImpl.doConsumeBatchedInput(ViewRootImpl.java:5607)
            at android.view.ViewRootImpl$ConsumeBatchedInputRunnable.run(ViewRootImpl.java:5653)
            at andr

我的列表适配器:

public class ListAdapter extends ArrayAdapter<Question> {


    public ListAdapter(Context context, ArrayList<Question> questions) {
        super(context, R.layout.list_item, questions);
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        if (convertView == null) {
            convertView = LayoutInflater.from(getContext()).inflate(R.layout.list_item, parent, false);
        }
        final Question question = getItem(position);
        TextView id = (TextView) convertView.findViewById(R.id.question_id);
        TextView text = (TextView) convertView.findViewById(R.id.question_text);
        id.setText(Integer.toString(question.getId()));
        text.setText(question.getName());

        RadioGroup rg = (RadioGroup) convertView.findViewById(R.id.question);
        rg.removeAllViews();//Line 38. Whithout his line I also get the same error.
        rg.setId(position);
        int i = 0;
        for (String qA : question.answers) {
            RadioButton rb = new RadioButton(getContext());
            rb.setText(qA.toString());
            rg.addView(rb);
            rb.setId(i);
            i++;
        }

        rg.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {

            @Override
            public void onCheckedChanged(RadioGroup group, int checkedId) {
            question.setUserAnswer(checkedId);
            Log.d("User answer", Integer.toString(checkedId));
        }
    });

    return convertView;
}

}

rg.setId(position); 正在将 RadioGroup 的 ID 更改为 R.id.question 以外的其他内容,因此当 View 被回收时 - 即,当您滚动时 - findViewById(R.id.question) 正在返回 null。

如果需要缓存位置,可以使用setTag()getTag()方法。您也可以只缓存 Question 对象。例如:

//rg.setId(position);
rg.setTag(question);
...
rg.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(RadioGroup group, int checkedId)
    {
        Question q = (Question) group.getTag();
        q.setUserAnswer(checkedId);
        Log.d("User answer", Integer.toString(checkedId));
    }
});

您也可以只定义一个 RadioGroup.OnCheckedChangeListener 而不是为每个组创建一个新的。

//rg.setId(position);
rg.setTag(question);
...
rg.setOnCheckedChangeListener(checkListener);
...
...
private RadioGroup.OnCheckedChangeListener checkListener =
    new RadioGroup.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(RadioGroup group, int checkedId)
        {
            Question q = (Question) group.getTag();
            q.setUserAnswer(checkedId);
            Log.d("User answer", Integer.toString(checkedId));
        }
    };

您更改广播组的 ID:rg.setId(position);

如果您向下滚动,ListView 会重复使用这些视图。因为您更改了 ID findViewById() return null.

解决方案

  1. 实施自定义 OnCheckedChangeListener 知道其在 ListView.

    中的位置
    private static class QuestionCheckedListener implements RadioGroup.OnCheckedChangeListener{
    
    private final int mPosition;
    
    QuestionCheckedListener(int position){
        mPosition = position;
    }
    
    @Override
    public void onCheckedChanged(RadioGroup group, int checkedId) {
        question.setUserAnswer(mPosition);
        Log.d("User answer", Integer.toString(checkedId));
    }
    

    }

  2. 实例化在getView():

    rg.setOnCheckedChangeListener(new QuestionCheckedListener(position));
    
  3. 并且不要忘记删除所有 setId(i) 调用