How to prevent rapid double click on a button

Question

I have looked at the answers here - Android Preventing Double Click On A Button and implemented qezt's solution like and I've tried setEnabled(false) like so -

doneButton.setOnClickListener(new View.OnClickListener() {
    @Override
        public void onClick(View v) {

            // mis-clicking prevention, using threshold of 1 second
            if (SystemClock.elapsedRealtime() - doneButtonClickTime < 1000){
                return;
            }

            //store time of button click
            doneButtonClickTime = SystemClock.elapsedRealtime();

            doneButton.setEnabled(false);

            //do actual work 

        }
    });

Neither of these work against super fast double clicks.

Note - I'm not setting doneButton.setEnabled(true) after my processing is done. I simply finish() the activity so there is no issue of the button getting enabled too soon.

What are you clicking? its a button r8? what is the name of button? — Vinay Jayaram, Sep 28 '15 at 09:22
not sure but maybe it works if you put everything that is in onClick in a `synchronized` block? — Ivo, Sep 28 '15 at 09:33
@Blackbelt [this comment](http://stackoverflow.com/questions/5608720/android-preventing-double-click-on-a-button/16514644#comment13900011_5608757) seems to answer why. — Sufian, Sep 28 '15 at 09:39
check `if (!v.isEnabled()) { return; }` as first thing in your onClick — Blackbelt, Sep 28 '15 at 09:42
@Blackbelt it won't work. Please see the comment I linked to in my first comment. — Sufian, Sep 28 '15 at 11:14
for kotlin handle click look at this link https://stackoverflow.com/a/56880661/7176189 — Jai Khambhayta, Jul 04 '19 at 04:31
Possible duplicate of [Avoid button multiple rapid clicks](https://stackoverflow.com/questions/16534369/avoid-button-multiple-rapid-clicks) — Malwinder Singh, Jul 12 '19 at 17:19

score 19 · Accepted Answer · answered Sep 28 '15 at 09:26

I am doing like this it works very well.

public abstract class OnOneOffClickListener implements View.OnClickListener {

   private static final long MIN_CLICK_INTERVAL=600;

   private long mLastClickTime;

   public static boolean isViewClicked = false;


   public abstract void onSingleClick(View v);

   @Override
   public final void onClick(View v) {
       long currentClickTime=SystemClock.uptimeMillis();
       long elapsedTime=currentClickTime-mLastClickTime;

       mLastClickTime=currentClickTime;

       if(elapsedTime<=MIN_CLICK_INTERVAL)
           return;
       if(!isViewClicked){
           isViewClicked = true;
           startTimer();
       } else {
           return;
       }
       onSingleClick(v);        
   }
    /**
     * This method delays simultaneous touch events of multiple views.
     */
    private void startTimer() {
        Handler handler = new Handler();

        handler.postDelayed(new Runnable() {

            @Override
            public void run() {
                    isViewClicked = false;
            }
        }, 600);

    }

}

score 8 · Answer 2 · edited Jun 18 '20 at 13:59

8

I use a function like this in the listener of a button:

public static long lastClickTime = 0;
public static final long DOUBLE_CLICK_TIME_DELTA = 500;

public static boolean isDoubleClick(){
    long clickTime = System.currentTimeMillis();
    if(clickTime - lastClickTime < DOUBLE_CLICK_TIME_DELTA){
        lastClickTime = clickTime;
        return true;
    }
    lastClickTime = clickTime;
    return false;
}

edited Jun 18 '20 at 13:59

Cxom

3
1

answered Sep 28 '15 at 09:32

Jean-baptiste Valero

81
8

This worked for me after reversing the return booleans. I have edited the answer. – Steve Kamau Feb 10 '20 at 05:35

score 3 · Answer 3 · answered Aug 31 '16 at 18:09

Probably not most efficient, but minimal inversive:

...
onClick(View v) {
    MultiClickPreventer.preventMultiClick(v);
    //your op here
}
...
public class MultiClickPreventer {
    private static final long DELAY_IN_MS = 500;

    public static void preventMultiClick(final View view) {
        if (!view.isClickable()) {
            return;
        }
        view.setClickable(false);
        view.postDelayed(new Runnable() {
            @Override
            public void run() {
                view.setClickable(true);
            }
        }, DELAY_IN_MS);
    }
}

score 3 · Answer 4 · answered Jan 16 '18 at 07:25

You can use this method. By using post delay you can take care for double click events.

void debounceEffectForClick(View view) {

view.setClickable(false);

view.postDelayed(new Runnable() {
    @Override
    public void run() {
        view.setClickable(true);

    }
}, 500);

}

Mallika Khullar · Answer 5 · 2015-09-28T11:23:00.307

1

Suru's answer worked well for me, thanks!

I'd like to add the following answer for anybody who's using https://github.com/balysv/material-ripple/blob/master/library/src/main/java/com/balysv/materialripple/MaterialRippleLayout.java and is facing this problem -

app:mrl_rippleDelayClick is true by default. This means, onClick will be executed only after it's finished showing the ripple. Hence setEnabled(false)inside onClick will be executed after a delay because the ripple isn't done executing and in that period you may double click the view.

Set app:mrl_rippleDelayClick="false" to fix this. This means the call to onClick will happen as soon as the view is clicked, instead of waiting for the ripple to finish showing.

edited Sep 28 '15 at 11:23

answered Sep 28 '15 at 10:45

Mallika Khullar

1,725
3
22
37

1

It's more of a comment than an answer – Sufian Sep 28 '15 at 11:13
1

It was better decision to post your answer than a comment. Because i had the same problem with MaterialRippleLayout and i found separate solution for that. Thanks @Mallika. – Jigar Dec 17 '16 at 05:14

score 1 · Answer 6 · answered Sep 30 '19 at 08:30

1

Set the launchMode of the activity in manifest as singleTop

 <activity
        android:name="com.example.MainActivity"
        android:screenOrientation="portrait"
        android:launchMode="singleTop"/>

answered Sep 30 '19 at 08:30

Zayd Khan

124
2
11

score 1 · Answer 7 · edited Apr 28 '20 at 18:41

private static final long MIN_CLICK_INTERVAL = 1000;

private boolean isViewClicked = false;

private View.OnClickListener onClickListener = new View.OnClickListener() {
@Override
final public void onClick(View view) {
    if (isViewClicked) {
      return:
    }
    // place your onClick logic here
    isViewClicked = true;
    view.postDelayed(new Runnable() {
        @Override
        public void run() {
            isViewClicked = false;
        }
    }, MIN_CLICK_INTERVAL);
}

}

score 0 · Answer 8 · answered Sep 28 '15 at 09:29

0

try to set yourbutton.setClickable(false) like this :

onclick(){
yourbutton.setClickable(false) ;
ButtonLogic();
}

ButtonLogic(){
//your button logic
yourbutton.setClickable(true);
}

answered Sep 28 '15 at 09:29

Sarah Maher

790
1
10
21

Hey, I've tried this but the button is still clickable if I press it in quick succession. I guess disabling the button or setting unclickable is not enough if you are doing computationally intensive work in onClick() since click events can get queued up before the button can be disabled. Any ideas? – Mallika Khullar Sep 28 '15 at 09:32
1

I usually use a progress dialog to prevent the user from interacting with UI if necessary . you may want to try it if the process you are doing takes longer than you wish . – Sarah Maher Sep 28 '15 at 09:39

score 0 · Answer 9 · edited Aug 07 '18 at 16:58

0

Qezt solution is already fine. But if you want to prevent super fast double-click, then you simply reduce the threshhold

    // half a second
    if (SystemClock.elapsedRealtime() - doneButtonClickTime < 500) {
        return;
    }

    // or 100ms (1/10 of second)
    if (SystemClock.elapsedRealtime() - doneButtonClickTime < 100) {
        return;
    }

edited Aug 07 '18 at 16:58

Jean-baptiste Valero

81
8

answered Sep 28 '15 at 09:42

Fadils

1,508
16
21

score 0 · Answer 10 · answered Aug 01 '17 at 12:24

public static void disablefor1sec(final View v) {
        try {
            v.setEnabled(false);
            v.setAlpha((float) 0.5);
            v.postDelayed(new Runnable() {
                @Override
                public void run() {
                    try {
                        v.setEnabled(true);
                        v.setAlpha((float) 1.0);
                    } catch (Exception e) {
                        Log.d("disablefor1sec", " Exception while un hiding the view : " + e.getMessage());
                    }
                }
            }, 1000);
        } catch (Exception e) {
            Log.d("disablefor1sec", " Exception while hiding the view : " + e.getMessage());
        }
    }

I kept above method in a static file and i will call this method for all the buttonclick like this

    button_or_textview.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            StaticFile.disablefor1sec(button_or_textview);
            // Do Somthing.
        }
    });

score 0 · Answer 11 · edited Jul 15 '21 at 07:52

0

Kotlin short version:

private var lastClickTime: Long = 0
    
//in click listener
if (SystemClock.elapsedRealtime() - lastClickTime < 1000) {
    return@setOnClickListener
}
lastClickTime = SystemClock.elapsedRealtime()

edited Jul 15 '21 at 07:52

Waldmann

1,563
12
25

answered Jan 30 '19 at 12:23

Bogdan Stolyarov

86
2
5

How to prevent rapid double click on a button

11 Answers11

Linked