A debounced onClickListener for Android

ThrottledOnClickListener.kt

package com.westwingnow.android.utils

import android.os.SystemClock
import android.view.View
import java.util.*

/**
 * A Throttled OnClickListener
 * Rejects clicks that are too close together in time.
 * This class is safe to use as an OnClickListener for multiple views, and will throttle each one separately.
 *
 *  * @param minimumIntervalMsec The minimum allowed time between clicks - any click sooner than this after a previous click will be rejected
 */
private const val minimumInterval = 1000L

class ThrottledOnClickListener(private val onClick: (view: View) -> Unit) : View.OnClickListener {
    private val lastClickMap: MutableMap<View, Long> = WeakHashMap()

    override fun onClick(clickedView: View) {
        val previousClickTimestamp = lastClickMap[clickedView]
        val currentTimestamp = SystemClock.uptimeMillis()

        lastClickMap[clickedView] = currentTimestamp
        if (previousClickTimestamp == null || currentTimestamp - previousClickTimestamp.toLong() > minimumInterval) {
            onClick.invoke(clickedView)
        }
    }
}

Already solved, thanks for the great class

I think, androidrx/rxjava library is too heavy for just debounce or throttle

There is an error on 25 line, it's a recursive call. There needs to be onClick.invoke(clickedView) to use the lambda passed in the constructor.

Thumbs up for this.

@artnest @tieorange
Is there a particular reason to create a separate class for it. Can't we do it as an extension function on View class? Something like this:

fun View.setDebouncedClickListener(
  debounceIntervalMs: Int = 700,
  listener: (view: View?) -> Unit
) {
  var lastTapTimestamp: Long = 0
  val customListener = View.OnClickListener {
    val currentTime = System.currentTimeMillis()
    if (currentTime - lastTapTimestamp > debounceIntervalMs) {
      lastTapTimestamp = currentTime
      listener(it)
    }
  }
  this.setOnClickListener(customListener)
}

There is an error on 25 line, it's a recursive call. There needs to be onClick.invoke(clickedView) to use the lambda passed in the constructor.

Works like a charm!

Maybe edit the gist to correct the error.

There is an error on 25 line, it's a recursive call. There needs to be onClick.invoke(clickedView) to use the lambda passed in the constructor.

Works like a charm!

Maybe edit the gist to correct the error.

Fixed. Thanks!