Fixing double click event handling: making isBusy variable outer of mHandler.postDelayed. Now we can truly receive double click events.; Making DOUBLE_CLICK_INTERVAL as a static field and adjusting his value to best fit double click sense. Some Google checkstyle was applied.

Author: samirtf

app/src/main/java/com/pedromassango/doubleclick/DoubleClick.java

@@ -4,45 +4,76 @@
 import android.view.View;
 
 /**
- * Created by pedromassango on 12/20/17.
+ * @author pedromassango on 12/20/17.
+ *
+ * @contributing to development samirtf
+ * Interface to be notified from a new click event.
+ *
  */
-
 public class DoubleClick implements View.OnClickListener {
 
-    private int clicks;
-    private boolean isBussy = false;
-    private final DoubleClickListener doubleClickListener;
-    Handler mHandler = new Handler();
-    public DoubleClick( DoubleClickListener doubleClickListener) {
-        this.doubleClickListener = doubleClickListener;
-    }
+  /*
+   * Duration of click interval.
+   * 200 milliseconds is a best fit to double click interval.
+   */
+  private static final long DOUBLE_CLICK_INTERVAL = 200L;  // Time to wait the second click.
+
+  /*
+   * Handler to process click event.
+   */
+  final Handler mHandler = new Handler();
+
+  /*
+   * Click callback.
+   */
+  private final DoubleClickListener doubleClickListener;
+
+  /*
+   * Number of clicks in @DOUBLE_CLICK_INTERVAL interval.
+   */
+  private int clicks;
 
-    @Override
-    public void onClick( final View v) {
+  /*
+   * Flag to check if click handler is busy.
+   */
+  private boolean isBusy = false;
 
-        if(!isBussy) {
-            //  Prevent multiple click in this short time
-            isBussy = true;
+  /**
+   * Builds a DoubleClick.
+   *
+   * @param doubleClickListener
+   */
+  public DoubleClick(final DoubleClickListener doubleClickListener) {
+    this.doubleClickListener = doubleClickListener;
+  }
 
-            // Increase clicks count
-            this.clicks++;
-            long DOUBLE_CLICK_INTERVAL = 250L;  // Time to wait the second click.
-            mHandler.postDelayed(new Runnable() {
-                public final void run() {
+  @Override
+  public void onClick(final View view) {
 
-                    if (clicks >= 2) {  // Double tap.
-                        doubleClickListener.onDoubleClick(v);
-                    }
+    if (!isBusy) {
+      //  Prevent multiple click in this short time
+      isBusy = true;
 
-                    if (clicks == 1) {  // Single tap
-                        doubleClickListener.onSingleClick(v);
-                    }
+      // Increase clicks count
+      clicks++;
 
-                    // we need to  restore clicks count
-                    clicks = 0;
-                    isBussy = false;
-                }
-            }, DOUBLE_CLICK_INTERVAL);
+      mHandler.postDelayed(new Runnable() {
+        public final void run() {
+
+          if (clicks >= 2) {  // Double tap.
+            doubleClickListener.onDoubleClick(view);
+          }
+
+          if (clicks == 1) {  // Single tap
+            doubleClickListener.onSingleClick(view);
+          }
+
+          // we need to  restore clicks count
+          clicks = 0;
         }
+      }, DOUBLE_CLICK_INTERVAL);
+      isBusy = false;
     }
+
+  }
 }

app/src/main/java/com/pedromassango/doubleclick/DoubleClickListener.java

@@ -4,20 +4,20 @@
 
 
 /**
- * Created by pedromassango on 12/20/17.
- */
-/*
-    Interface to be notified from a new click event.
+ * @author pedromassango on 12/20/17.
+ * @contributing to development samirtf
+ *
+ * Interface to be notified from a new click event.
  */
 public interface DoubleClickListener {
 
-    /*
-        Called when the user make a single click.
-    */
-    void onSingleClick( View view);
+  /**
+   * Called when the user make a single click.
+   */
+  void onSingleClick(final View view);
 
-    /*
-        Called when the user make a double click.
-    */
-    void onDoubleClick( View view);
+  /**
+   * Called when the user make a double click.
+   */
+  void onDoubleClick(final View view);
 }