OnTimeSet вызывается также при отклонении TimePickerDialog

Question

Сегодня я пытался использовать TimePickerDialog но я заметил пару недостатков.

1. OnTimeSet также вызывается, когда диалоговое окно закрывается (например, нажав снаружи)
2. OnTimeSet вызывается дважды, когда пользователь нажимает кнопку "Готово"

API я использую 18.

Кто-нибудь еще испытывал эти проблемы? Как вы их решили?

Answer 1

Столкнулся с точно такой же проблемой сегодня. Не мог понять, почему это происходит, но нашел простое решение:

Метод onTimeSet() вызывается один раз, когда диалог закрывается, и дважды, когда нажимается кнопка "Готово". В любом случае, есть один нежелательный вызов onTimeSet(). Поэтому я решил всегда игнорировать первый звонок.

Вот код:

Calendar mcurrentTime = Calendar.getInstance();
int hour = mcurrentTime.get(Calendar.HOUR_OF_DAY);
int minute = mcurrentTime.get(Calendar.MINUTE);

TimePickerDialog mTimePicker;
mTimePicker = new TimePickerDialog(MainActivity.this, new TimePickerDialog.OnTimeSetListener() 
    {
        int callCount = 0;   //To track number of calls to onTimeSet()

        @Override
        public void onTimeSet(TimePicker timePicker, int selectedHour, int selectedMinute) 
        {
             if(callCount == 1)    // On second call
             {
                 timeString = selectedHour + ":" + selectedMinute + ":00";
                 Log.d("TEST", "Chosen time : "+ timeString);           
             }

             callCount++;    // Incrementing call count.

        }
    }, hour, minute, true);

    mTimePicker.setTitle("Pick Time");
    mTimePicker.show();

Answer 2

Вы должны использовать уже заданный метод класса View:

new TimePickerDialog.OnTimeSetListener() {
    @Override
    public void onTimeSet(TimePicker view, int hour, int minute) {  
        if (view.isShown()) {
            // This method will return true only once...
        }
    }
};

Answer 3

Повторим: это подтвержденная ошибка в Android для нескольких типов диалогов. Уже было предложено два обходных пути: сохранение состояния в переменной (экземпляра) или запрос диалога, если оно isShown(), Но isShown() кажется ненадежным в Android 4.0.4, и сохранение состояния становится беспорядочным, если вы хотите снова показать диалоговое окно.
Лучшее решение - сохранить состояние внутри самого диалога, потому что это тот же экземпляр, который вызывает метод:

public void onDateSet(DatePicker picker, int year, int monthOfYear, int dayOfMonth) {
    if (picker.getTag() == null) {
        picker.setTag("TAGGED");
        // Only gets called once per Dialog
    }
}

Это чисто и эффективно.

Answer 4

Используя счет, чтобы избежать этого. когда TimePickDialog был выбран более двух раз, он также должен работать хорошо.

            TimePickerDialog tpd = new TimePickerDialog(this, new TimePickerDialog.OnTimeSetListener() {
            int count = 0;
            @Override
            public void onTimeSet(TimePicker view, int setHour, int setMinute) {
                if(count % 2 == 0) {
                   //set time here
                }
                count++;

            } }, hour, minute, true);

Answer 5

Спасибо Тони за публикацию обходного пути. Это работает большую часть времени, но не всегда. Мы выпустили наше приложение с этим обходным путем (вместе с проверкой версий); Однако это решение не удалось на Samsung Galaxy Note GT-8000 (Android 4.4.2). У устройств 4.4.2 по умолчанию есть эта ошибка, и решение работает, однако Samsung, кажется, исправил эту проблему в выпуске 4.4.2, поэтому onTimeSet() вызывается только один раз, когда мы игнорируем его, и второго вызова не происходит.

Мы публикуем решение, которое мы применили сегодня. Хотя я не доволен решением, так как это еще один взлом / обходной путь, но он может помочь в тех случаях, когда проверка версий не помогает, а OEM-производители объединяют выборочные исправления.

Наша ранняя реализация была

                        if((android.os.Build.VERSION.SDK_INT >=
                            Build.VERSION_CODES.ICE_CREAM_SANDWICH) &&
                            (android.os.Build.VERSION.SDK_INT <
                                    Build.VERSION_CODES.LOLLIPOP)){
                        if(ccount == 1){
                            // Do Your Processing
                            count = 0;
                        }else{
                            // Ignore event. Bug in Android API
                            count++;
                        }
                    }else{
                        // Do Your Processing
                    }

Наша новая реализация

                        if((android.os.Build.VERSION.SDK_INT >=
                            Build.VERSION_CODES.ICE_CREAM_SANDWICH) &&
                            (android.os.Build.VERSION.SDK_INT <
                                    Build.VERSION_CODES.LOLLIPOP)){
                        StackTraceElement[] stacktrace = Thread.currentThread().getStackTrace();
                        StackTraceElement e = stacktrace[4];
                        String methodName = e.getMethodName();
                        if(methodName.equals("onClick")){
                            // Do Your Processing
                        }else{
                            // Ignore event. Bug in Android API
                        }
                    }else{
                        // Do Your Processing
                    }

Надеюсь, что это может помочь другим.