OnTimeSet также вызывается при закрытии TimePickerDialog.

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

Метод 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();

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

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

Повторюсь: это подтвержденная ошибка в 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
    }
}

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

Использование подсчета, чтобы избежать этого. когда 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);

Спасибо Тони за публикацию обходного пути. Это работает в большинстве случаев, но не всегда. Мы выпустили наше приложение с этим обходным решением (вместе с проверкой версии); однако это решение не помогло на 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
                    }

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