Use EventBus instead of callbacks

Author: rubengees

library/build.gradle

@@ -25,6 +25,7 @@ dependencies {
     compile fileTree(dir: 'libs', include: ['*.jar'])
     compile 'com.android.support:support-annotations:23.1.1'
     compile 'com.github.afollestad:bridge:1.6.4@aar'
+    compile 'de.greenrobot:eventbus:2.4.0'
 }
 
 // build a jar with source files

library/src/main/java/com/proxerme/library/connection/ProxerConnection.java

@@ -1,7 +1,5 @@
 package com.proxerme.library.connection;
 
-import android.os.Handler;
-import android.os.Looper;
 import android.support.annotation.CheckResult;
 import android.support.annotation.IntRange;
 import android.support.annotation.NonNull;
@@ -27,6 +25,8 @@
 import java.util.LinkedList;
 import java.util.List;
 
+import de.greenrobot.event.EventBus;
+
 import static com.proxerme.library.connection.ProxerException.ErrorCodes.PROXER;
 import static com.proxerme.library.connection.ProxerException.ErrorCodes.UNKNOWN;
 import static com.proxerme.library.connection.ProxerTag.CONFERENCES;
@@ -166,29 +166,6 @@ public static void cleanup() {
         Bridge.cleanup();
     }
 
-    /**
-     * An abstract representation of a callback, passed to the
-     * {@link ProxerRequest#execute(ResultCallback)} method.
-     *
-     * @param <T> The generic parameter.
-     */
-    public interface ResultCallback<T> {
-        /**
-         * A callback method, called if the request was successful.
-         *
-         * @param result The result of the specific request.
-         */
-        void onResult(T result);
-
-        /**
-         * A callback method, called if an error occurred during the request.
-         *
-         * @param exception The Exception that occurred.
-         * @see ProxerException
-         */
-        void onError(@NonNull ProxerException exception);
-    }
-
     /**
      * An abstract representation of a request. All requests to the API are made through this class.
      *
@@ -198,7 +175,7 @@ public static abstract class ProxerRequest<T> {
 
         /**
          * Builds the request, to be used in the
-         * {@link #execute(ResultCallback)} or
+         * {@link #execute()} or
          * {@link #executeSynchronized()} method.
          *
          * @param bridge The Bridge instance to build the request with.
@@ -216,13 +193,12 @@ public static abstract class ProxerRequest<T> {
         protected abstract int getTag();
 
         /**
-         * Asynchronously executes this request.
+         * Asynchronously executes this request. All results are passed back through the EventBus.
          *
-         * @param callback The callback for notifications about the Result.
          * @see #executeSynchronized()
          */
         @RequiresPermission(android.Manifest.permission.INTERNET)
-        public final void execute(@NonNull final ResultCallback<T> callback) {
+        public final void execute() {
             buildRequest(Bridge.client()).tag(getTag()).request(new Callback() {
                 @Override
                 public void response(Request request, final Response response, BridgeException exception) {
@@ -233,26 +209,11 @@ public void run() {
                                 try {
                                     final T result = parse(response.asJsonObject());
 
-                                    new Handler(Looper.getMainLooper()).post(new Runnable() {
-                                        @Override
-                                        public void run() {
-                                            callback.onResult(result);
-                                        }
-                                    });
+                                    EventBus.getDefault().post(result);
                                 } catch (final JSONException e) {
-                                    new Handler(Looper.getMainLooper()).post(new Runnable() {
-                                        @Override
-                                        public void run() {
-                                            callback.onError(ErrorHandler.handleException(e));
-                                        }
-                                    });
+                                    EventBus.getDefault().post(ErrorHandler.handleException(e));
                                 } catch (final BridgeException e) {
-                                    new Handler(Looper.getMainLooper()).post(new Runnable() {
-                                        @Override
-                                        public void run() {
-                                            callback.onError(ErrorHandler.handleException(e));
-                                        }
-                                    });
+                                    EventBus.getDefault().post(ErrorHandler.handleException(e));
                                 }
                             }
                         });
@@ -261,7 +222,7 @@ public void run() {
                         parseThread.start();
                     } else {
                         if (exception.reason() != BridgeException.REASON_REQUEST_CANCELLED) {
-                            callback.onError(ErrorHandler.handleException(exception));
+                            EventBus.getDefault().post(ErrorHandler.handleException(exception));
                         }
                     }
                 }
@@ -273,7 +234,7 @@ public void run() {
          *
          * @return The result, specified by this class.
          * @throws ProxerException An Exception, which might occur, while executing the request.
-         * @see #execute(ResultCallback)
+         * @see #execute()
          */
         @WorkerThread
         @RequiresPermission(android.Manifest.permission.INTERNET)