How to prevent double code running by clicking twice fast to a button in Android

Question

If i click fast to my button in my Android app, it seems that code behind it runs twice. If i click my menu button twice the activity that has to be launch onclick just starts twice and i have to quit from it twice.

This is really annoying because if i click too fast for menu buttons i can load up a whole bunch of activities in the background and i must quit them one by one, so this is clearly a buggy state of my app i want to fix this.

What can i do with this issue?

I use simple onClickListeners and Buttons

EDIT:

Regarding to answers and comments my menu buttons look like this:

top20Button.setOnClickListener(new OnClickListener() { public void onClick(View v) { favButton.setClickable(false); nearButton.setClickable(false); highlightedButton.setClickable(false); top20Button.setClickable(false); Intent i = new Intent(); i.putExtra("showDialog", false); i.addFlags(Intent.FLAG_ACTIVITY_SINGLE_TOP); i.setClass(Search.this, Top20.class); startActivity(i); finish(); } });

After all this correction its still the same :S When i click like a mad person multiple activites are on the history stack and i must quit multiple times.

Any suggestions ? What m i doing wrong?

Try disabling button on click. So when ever it will get clicked first, it will become disabled and will not capture any future clicks, I guess. — Harry Joy
– Harry Joy, Commented Jul 2, 2012 at 9:03
Still having multiple activites on stack, even with SINGLE_TOP? Check my updated answer... — Bojan Radivojevic
– Bojan Radivojevic, Commented Jul 3, 2012 at 11:00

Shrikant Ballal · Accepted Answer · 2012-07-02 09:03:24Z

8

You can use following code: btn.setEnabled(false);

btn.setOnclickListener(new View.onClickListener(){ public void onClick(View v) { btn.setEnabled(false); } });

answered Jul 2, 2012 at 9:03

Shrikant Ballal

7,0877 gold badges43 silver badges61 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Miguel Q Over a year ago

but it will then disable further clicking of other, and then how do you enable again?

Shrikant Ballal Over a year ago

@Miguel: In above example, the code written in OnClickListener, starts an activity, so when user comes back to previous activity, it will automatically gets enabled. But in other cases, like if any async task is started in onClickListener, then onPostExecute() would be the best placewhere you will enable the button again.

Simon K. Gerges · Accepted Answer · 2018-10-03 12:07:41Z

You can Disable the view temporarily like this

public static void disableTemporarly(final View view) { view.setEnabled(false); view.post(new Runnable() { @Override public void run() { view.setEnabled(true); } }); }

Edit:

The above solution will work fine. But it will become even better when using the power of Kotlin

1) Create the SafeClikc Listener

class SafeClickListener( private var defaultInterval: Int = 1000, private val onSafeCLick: (View) -> Unit ) : View.OnClickListener { private var lastTimeClicked: Long = 0 override fun onClick(v: View) { if (SystemClock.elapsedRealtime() - lastTimeClicked < defaultInterval) { return } lastTimeClicked = SystemClock.elapsedRealtime() onSafeCLick(v) } }

2) Add extension function to make it works with any view, this will create a new SafeClickListener and delegate the work to it

fun View.setSafeOnClickListener(onSafeClick: (View) -> Unit) { val safeClickListener = SafeClickListener { onSafeClick(it) } setOnClickListener(safeClickListener) }

3) Now it is very easy to use it

settingsButton.setSafeOnClickListener { showSettingsScreen() }

Happy Kotlin ;)

I used a variation of the above to solve a similar issue. Efficient and effective answer without the complexity of other approaches.

anand krish · Accepted Answer · 2017-08-28 11:10:19Z

 Button.setOnClickListener(new View.OnClickListener { @Override public void onClick(final View v) { v.setEnabled(false); v.postDelayed(new Runnable() { @Override public void run() { v.setEnabled(true); } },150); //150 is in milliseconds } });

Bojan Radivojevic · Accepted Answer · 2012-07-03 10:59:35Z

2

Well, that's the expected behaviour...

Launch your new acvitity with SINGLE_TOP flag

Or try setting android:launchMode="singleInstance" for Top20 activity in your AndroidManifest.xml

edited Jul 3, 2012 at 10:59

answered Jul 2, 2012 at 9:03

Bojan Radivojevic

7373 gold badges12 silver badges24 bronze badges

2 Comments

Adam Varhegyi Over a year ago

If i add this statement a strange animation is appearing on activity changing: The layout moves to left a blank screen appears then my new activity is being shown. For some reason this setting skips all my activity animations which is currently: overridePendingTransition(0, 0); If you can tell me how to disable this blank screen animation i can select your answer as an answer to my problem because now i dont have duplicated activities. =)

Bojan Radivojevic Over a year ago

try adding NO_ANIMATION flag when launching activity :)

Miguel Q · Accepted Answer · 2014-05-18 10:53:14Z

This solves the issue of multiple Instances of activity, and still play the default animation

intent.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);

Community · Accepted Answer · 2017-05-23 12:06:47Z

This solution fits better to my requirements. Instead using enable/disable view feature. I use the time, first, create a variable to store the last time user pressed the view* then when the user presses the view, capture the time in a variable in order to compare the current moment and the last time user pressed the view, if the differential time between times is too small, which means fast,or extremely fast just ignore it and that's it. But if its a reasonable time to you, set the last time variable with the current time and do whatever else you want to do.

@Override public void onListItemClick(ListView list, View item, int position, long id) { Long now = System.currentTimeMillis(); if((now - lastTimeSelected) > 500){ lastTimeSelected = now; // your code goes here } else { Log.i(TAG, "Too fast!"); } }

I saw a similar answer on other post, but nobody seems to like this approach.

nzala · Accepted Answer · 2016-10-12 07:05:01Z

Add the following code snipet to your Activity definition in Androidmanifest.xml

android:launchMode = "singleTask"

Vyacheslav · Accepted Answer · 2016-10-12 07:14:34Z

Use frozen variable inside Application class.

That is

public class App extends Application { public static boolean frozen = false; }

somewhere while click:

//... onClick() { if (frozen) { return; } frozen = true } //...

Somewhere to release, for instance, when new Activity has been launched

onCreate (...) { super.onCreate(...); frozen = false; }

in AndroidManifest.xml:

<application android:allowBackup="true" android:largeHeap="true" android:name="com.example.App" android:icon="@drawable/icon" android:label="@string/app_alias" android:theme="@style/AppTheme" >

Collectives™ on Stack Overflow

How to prevent double code running by clicking twice fast to a button in Android

8 Answers 8

2 Comments

1 Comment

Comments

2 Comments

Comments

Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

8 Answers 8

2 Comments

1 Comment

Comments

2 Comments

Comments

Comments

Comments

Comments

Linked

Related