Android:ListView中的按钮未接收onClick事件

时间:2010-12-01 18:35:08

标签: android

我正在制作一个日期选择器活动,看起来像滚动30天的月/日历(想想Outlook日历)。日期选择器包含MonthView视图的ListView(用于滚动),每个视图都是各个日期的TableView。 MonthView中的每一天都是一个按钮。当实例化MonthView时,我每天都会走(按钮)并附加一个点击监听器:

final Button b = getButtonAt(i);
b.setOnClickListener(new OnClickListener()
{
      @Override
      public void onClick(View v) {
            setSelectedDate(buttonDayClosure, b);
      }
});

setSelectedDate可以执行各种操作,但它也会将按钮的背景变为黄色,以表示选择了日期。

在我的模拟器上,一切都按预期工作。活动出现,你按一天,一天变黄。没问题。

然而,在我的一些同行的模拟器和物理设备上,当你触摸一天没有任何事情发生...直到你滚动ListView ...然后突然所选的一天变黄。所以,例如,你触摸“第3个”,然后没有任何反应。等待几秒钟,然后滚动ListView(触摸不是第3个日历的日历区域),一旦ListView滚动,第3个神奇地变为黄色。

在显示此行为的我的对等模拟器上,我可以在onClick的第一行设置断点,我发现在滚动ListView之前,BP实际上没有被击中。

这种行为对我没有任何意义。我希望onClick行为与封装View的滚动工作无关。

有关为什么会出现这种情况以及如何纠正这种情况的任何想法,以便触摸按钮时onClicks会立即发生?

感谢。

发布Scriptus:请求ArrayAdapter和ListView代码:

public class MonthArrayAdapter extends ArrayAdapter<Date> {

    private MonthView[] _views; 

    private Vector<Procedure<Date>> _dateSelectionChangedListeners = new Vector<Procedure<Date>>();

    public MonthArrayAdapter(Context context, int textViewResourceId, Date minSelectableDay, Date maxSelectableDay) {
        super(context, textViewResourceId);
        int zeroBasedMonth = minSelectableDay.getMonth();
        int year = 1900 + minSelectableDay.getYear();

        if(minSelectableDay.after(maxSelectableDay))
        {
            throw new IllegalArgumentException("Min day cannot be after max day.");
        }

        Date prevDay = minSelectableDay;
        int numMonths = 1;
        for(Date i = minSelectableDay; !sameDay(i, maxSelectableDay); i = i.addDays(1) )
        {
            if(i.getMonth() != prevDay.getMonth())
            {
                numMonths++;
            }
            prevDay = i;
        }

        _views = new MonthView[numMonths];

        for(int i = 0; i<numMonths; i++)
        {
            Date monthDate = new Date(new GregorianCalendar(year, zeroBasedMonth, 1, 0, 0).getTimeInMillis());
            Date startSunday = findStartSunday(monthDate);
            this.add(monthDate);
            _views[i] = new MonthView(this.getContext(), startSunday, minSelectableDay, maxSelectableDay);

            zeroBasedMonth++;
            if(zeroBasedMonth == 12)
            {
                year++;
                zeroBasedMonth = 0;
            }
        }

        for(final MonthView a : _views)
        {
            a.addSelectedDateChangedListener(new Procedure<MonthView>()
            {
                        @Override
                    public void execute(MonthView input) {

                        for(final MonthView b: _views)
                        {
                            if(a != b)
                            {
                                b.clearCurrentSelection();
                            }
                        }

                        for(Procedure<Date> listener : _dateSelectionChangedListeners)
                        {
                            listener.execute(a.getSelectedDate());
                        }
                    }
            });
        }




    }

    void addSelectedDateChangedListener(Procedure<Date> listener)
    {
        _dateSelectionChangedListeners.add(listener);
    }

    private boolean sameDay(Date a, Date b)
    {
        return a.getYear() == b.getYear() && a.getMonth() == b.getMonth() &&
               a.getDate() == b.getDate();
    }

    @Override
    public View getView (int position, View convertView, ViewGroup parent)
    {
        return _views[position];
    }


    private Date findStartSunday(Date d)
    {
        return d.subtractDays(d.getDay());
    }

    public void setSelectedDate(Date date)
    {
        for(MonthView mv : _views)
        {
            mv.setSelectedDate(date);
        }
    }



}

public class DatePicker extends ActivityBase {

    public static final String CHOSEN_DATE_RESULT_KEY = "resultKey";

    public static final String MIN_SELECTABLE_DAY = DatePicker.class.getName() + "MIN";

    public static final String MAX_SELECTABLE_DAY = DatePicker.class.getName() + "MAX";

    private static final String SELECTED_DATE = UUID.randomUUID().toString();

    private long _selectedDate = -1;

    private MonthArrayAdapter _monthArrayAdapter;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        Date now = new Date();

        Bundle inputs = this.getIntent().getExtras();

        long min = inputs.getLong(MIN_SELECTABLE_DAY, 0);

        Date minSelectableDate;
        if(min == 0)
        {
            minSelectableDate = new Date(now);
        }
        else
        {
            minSelectableDate = new Date(min);
        }
        Log.i(DatePicker.class.getName(), "min date = " + minSelectableDate.toString());
        long max = inputs.getLong(MAX_SELECTABLE_DAY, 0);

        Date maxSelectableDate;
        if(max == 0)
        {
            maxSelectableDate = new Date(now.addDays(35).getTime());
        }
        else
        {
            maxSelectableDate = new Date(max);
        }


        setContentView(R.layout.date_picker);

        Button doneButton = (Button) findViewById(R.id.DatePickerDoneButton);

        if(doneButton == null)
        {
            Log.e(this.getClass().getName(), "Could not find doneButton from view id.");
            finish();
            return;
        }

        doneButton.setOnClickListener(new OnClickListener()
        {
            @Override
            public void onClick(View v) {
                Intent result = new Intent();
                result.putExtra(CHOSEN_DATE_RESULT_KEY, _selectedDate);
                setResult(RESULT_OK, result);
                finish();
            }
        });

        Button cancelButton = (Button) findViewById(R.id.DatePickerCancelButton);

        if(cancelButton == null)
        {
            Log.e(this.getClass().getName(), "Could not find cancelButton from view id.");
            finish();
            return;
        }

        cancelButton.setOnClickListener(new OnClickListener()
        {
            @Override
            public void onClick(View v) {
                setResult(RESULT_CANCELED, null);
                finish();
            }
        });

        ListView lv = (ListView)findViewById(R.id.DatePickerMonthListView);
        lv.setDividerHeight(0);


        _monthArrayAdapter = 
            new MonthArrayAdapter(this, android.R.layout.simple_list_item_1, minSelectableDate, maxSelectableDate);

        _monthArrayAdapter.addSelectedDateChangedListener(new Procedure<Date>()
                {

                    @Override
                    public void execute(Date input) {
                        _selectedDate = input.getTime();

                    }


                });

       lv.setAdapter(_monthArrayAdapter);
    }

    @Override 
    public void onRestoreInstanceState(Bundle savedInstanceState)
    {
        if(savedInstanceState.containsKey(SELECTED_DATE))
        {
            _selectedDate = savedInstanceState.getLong(SELECTED_DATE);
            _monthArrayAdapter.setSelectedDate(new Date(_selectedDate));
        }
    }

    @Override
    public void onSaveInstanceState(Bundle savedInstanceState)
    {
        savedInstanceState.putLong(SELECTED_DATE, _selectedDate);
    }

 }

4 个答案:

答案 0 :(得分:2)

遇到同样的问题,寻找答案。在我滚动列表之前,当我没有得到onClick方法时,我完全不相信它。如果我发现,我会在这里发布答案。

现在,我最好的猜测是尝试除了点击之外的其他事件(因为滚动空间正在吃掉变成点击事件的复杂触摸事件):

“downView”是一个静态变量,用于跟踪被点击的元素。

view.setOnTouchListener(new View.OnTouchListener() {
  @Override
  public boolean onTouch(View v, MotionEvent event) {
    if (event.getAction() == MotionEvent.ACTION_DOWN) {
      downView = v;
      return true;
    } else if (event.getAction() == MotionEvent.ACTION_UP) {
      if (downView == v) {
        handleClick(v);
        return true;
      }
      downView = null;
    }
    return false;
  }
});

答案 1 :(得分:2)

主要原因是ListView不喜欢具有视图数组的适配器。

所以问题是由

触发的
public View getView (int position, View convertView, ViewGroup parent)
{
    return _views[position];
}

当查看ListView代码(或者更确切地说是它的父亲AbsListView.obtainView方法)时,你会看到像

这样的代码
    if (scrapView != null) {
        ...
        child = mAdapter.getView(position, scrapView, this);
        ...
        if (child != scrapView) {
            mRecycler.addScrapView(scrapView);

使用scrapView!= _views [position]调用getView(position,...)可能会发生,因此scrapView将被回收。另一方面,很可能同样的视图也会再次添加到ListView,导致视图具有奇怪的状态(请参阅this issue

最终,这应该在ListView IMO中修复,但暂时,我建议不要使用包含视图数组的适配器。

答案 2 :(得分:0)

所以我会在触摸事件中手动编写您自己的点击事件之外添加一个完全独立的答案。

我与Android团队交换了一些电子邮件(由googly消费了一些额外费用)他们建议我手动实现ListAdapter的尝试是低效的,如果我没有正确地连接数据观察者方法适配器,它可能会导致“事件处理的有趣问题。”

所以我做了以下事情:

1)用BaseAdapter的子类替换了我的ListAdapter实现,该子类覆盖了必要的函数。

2)使用list.invalidateViews()停止并开始使用adapter.notifyDataChanged()

并且这个错误似乎消失了。

这比手动编写点击事件要多得多,但从长远来看,它也是更正确的代码。

答案 3 :(得分:0)

Aswer是:

public View getView(int position, View convertView, ViewGroup parent) {
    View v=makeMyView(position);
    v.setFocusableInTouchMode(false);
    return v;
}