From 17b81245f31d032d5eed69c190e42edb1949f46b Mon Sep 17 00:00:00 2001
From: hxb <hxb@hdlchina.com.cn>
Date: 星期二, 12 四月 2022 16:37:39 +0800
Subject: [PATCH] 修复接听后闪退的问题

---
 HDLSDK/hdl-common/src/main/java/com/hdl/sdk/common/event/EventDispatcher.java |  213 ++++++++++++++++++++++++++++++++---------------------
 1 files changed, 128 insertions(+), 85 deletions(-)

diff --git a/HDLSDK/hdl-common/src/main/java/com/hdl/sdk/common/event/EventDispatcher.java b/HDLSDK/hdl-common/src/main/java/com/hdl/sdk/common/event/EventDispatcher.java
index 172ed9b..bfb38b3 100644
--- a/HDLSDK/hdl-common/src/main/java/com/hdl/sdk/common/event/EventDispatcher.java
+++ b/HDLSDK/hdl-common/src/main/java/com/hdl/sdk/common/event/EventDispatcher.java
@@ -30,116 +30,151 @@
 
     private static final ExecutorService ioThread = ThreadToolUtils.getInstance().newFixedThreadPool(3);
 
+
     private EventDispatcher() {
     }
 
-//    private static class SingletonInstance {
-        private static final EventDispatcher instance = new EventDispatcher();
+    //    private static class SingletonInstance {
+    private static final EventDispatcher instance = new EventDispatcher();
 //    }
 
     public static EventDispatcher getInstance() {
         return instance;
     }
 
-    public synchronized void register(Object tag, EventListener listener) {
-        if (!EVENT.containsKey(tag)) {
-            EVENT.put(tag, new ArrayList<>());
+    public void register(Object tag, EventListener listener) {
+        synchronized (this) {
+            try {
+                LogUtils.i("娉ㄥ唽涓婚锛�" + tag);
+                if (tag == null) return;
+                if (!EVENT.containsKey(tag)) {
+                    EVENT.put(tag, new ArrayList<>());
+                }
+                if (listener == null) {
+                    LogUtils.i("鐩戝惉浜嬩欢涓虹┖");
+                    return;
+                }
+                Objects.requireNonNull(EVENT.get(tag)).add(listener);
+                TYPE.put(listener, MAIN_TYPE);
+            } catch (Exception e) {
+                LogUtils.e("register锛�" + e.getMessage());
+            }
         }
-        List<EventListener> events = EVENT.get(tag);
-        if (events != null && !events.contains(listener)) {
-            events.add(listener);
-        }
-        TYPE.put(listener, MAIN_TYPE);
     }
 
-    public synchronized void registerIo(Object tag, EventListener listener) {
-        if (!EVENT.containsKey(tag)) {
-            EVENT.put(tag, new ArrayList<>());
+    public  void registerIo(Object tag, EventListener listener) {
+        synchronized (this) {
+            LogUtils.i("娉ㄥ唽涓婚锛�" + tag);
+            if (tag == null) return;
+            if (!EVENT.containsKey(tag)) {
+                EVENT.put(tag, new ArrayList<>());
+            }
+            try {
+                if (listener == null) {
+                    LogUtils.i("鐩戝惉浜嬩欢涓虹┖");
+                    return;
+                }
+                Objects.requireNonNull(EVENT.get(tag)).add(listener);
+                TYPE.put(listener, IO_TYPE);
+            } catch (Exception e) {
+                LogUtils.e("registerIo锛�" + e.getMessage());
+            }
         }
-        List<EventListener> events = EVENT.get(tag);
-        if (events != null && !events.contains(listener)) {
-            events.add(listener);
-        }
-        TYPE.put(listener, IO_TYPE);
     }
 
-    public synchronized void remove(Object tag) {
-        try {
-
-            ioThread.execute(new Runnable() {
-                @Override
-                public void run() {
-                    try {
-                        if (EVENT.containsKey(tag)) {
-                            List<EventListener> list = EVENT.get(tag);
-                            for (EventListener eventListener : list) {
-                                TYPE.remove(eventListener);
-                            }
-                            EVENT.remove(tag);
+    public  void remove(Object tag) {
+        synchronized (this) {
+            if (tag == null) {
+                return;
+            }
+            try {
+                if (EVENT.containsKey(tag)) {
+                    LogUtils.i("绉婚櫎key锛�" + tag);
+                    List<EventListener> list = EVENT.get(tag);
+                    for (EventListener eventListener : list) {
+                        if (eventListener == null) {
+                            continue;
                         }
-                    } catch (Exception ignored) {
-                        LogUtils.e("绉婚櫎event寮傚父1锛�"+ignored.getMessage());
+                        TYPE.remove(eventListener);
+                    }
+                    EVENT.remove(tag);
+                }
+            } catch (Exception ignored) {
+                LogUtils.e("绉婚櫎event寮傚父1锛�" + ignored.getMessage());
+            }
+        }
+    }
+
+    public  void remove(Object tag, EventListener listener) {
+        synchronized (this) {
+            try {
+                if (tag == null || listener == null) {
+                    return;
+                }
+                if (EVENT.containsKey(tag)) {
+                    LogUtils.i("绉婚櫎key锛�" + tag);
+                    List<EventListener> ev = EVENT.get(tag);
+                    if (ev != null && !ev.isEmpty()) {
+                        TYPE.remove(listener);
+                        ev.remove(listener);
                     }
                 }
-            });
-        }catch (Exception e){
-            LogUtils.e("绉婚櫎event寮傚父2锛�"+e.getMessage());
+            } catch (Exception ignored) {
+                LogUtils.e("绉婚櫎event寮傚父1锛�" + ignored.getMessage());
+            }
         }
     }
-
-    public synchronized void remove(Object tag, EventListener listener) {
-        try {
-            ioThread.execute(new Runnable() {
-                @Override
-                public void run() {
-                    try {
-                        if (EVENT.containsKey(tag)) {
-                            List<EventListener> ev = EVENT.get(tag);
-                            if (ev != null && !ev.isEmpty()) {
-                                TYPE.remove(listener);
-                                ev.remove(listener);
-                            }
-                        }
-                    } catch (Exception ignored) {
-                        LogUtils.e("绉婚櫎event寮傚父1锛�"+ignored.getMessage());
-                    }
-
-                }
-            });
-        }catch (Exception e){
-            LogUtils.e("绉婚櫎event寮傚父2锛�"+e.getMessage());
-        }
-    }
-
 
     public synchronized void post(Object tag, @NonNull Object o) {
-        if (EVENT.containsKey(tag)) {
-            List<EventListener> list = EVENT.get(tag);
-            if (list != null && !list.isEmpty()) {
-                for (EventListener listener : list) {
+        if (tag == null) {
+            LogUtils.i("post tag涓虹┖");
+            return;
+        }
+        try {
+            if (EVENT.containsKey(tag)) {
+                LogUtils.i("post锛�" + tag);
+                List<EventListener> list = EVENT.get(tag);
+                if (list != null && !list.isEmpty()) {
+                    for (EventListener listener : list) {
+                        if (listener == null) {
+                            continue;
+                        }
+                        ThreadToolUtils.getInstance().runOnUiThread(new Runnable() {
+                            @Override
+                            public void run() {
+                                try {
+                                    if (listener != null) {
+                                        listener.onMessage(o);
+                                    }
+                                } catch (Exception e) {
+                                    LogUtils.e("post寮傚父1锛�" + e.getMessage());
+                                }
+                            }
+                        });
+                    }
+                }
+            }
+            //鎵�鏈変富棰樼殑Listener閫氱煡
+            if (ALL_TOPICS_EVENT != null && !ALL_TOPICS_EVENT.isEmpty()) {
+                for (EventListener listener : ALL_TOPICS_EVENT) {
+                    if (listener == null) {
+                        continue;
+                    }
                     ThreadToolUtils.getInstance().runOnUiThread(new Runnable() {
                         @Override
                         public void run() {
-                            if (listener != null) {
-                                listener.onMessage(o);
+                            try {
+                                if (listener != null) {
+                                    listener.onMessage(o);
+                                }
+                            } catch (Exception e) {
                             }
                         }
                     });
                 }
             }
-        }
-        //鎵�鏈変富棰樼殑Listener閫氱煡
-        if (ALL_TOPICS_EVENT != null && !ALL_TOPICS_EVENT.isEmpty()) {
-            for (EventListener listener : ALL_TOPICS_EVENT) {
-                ThreadToolUtils.getInstance().runOnUiThread(new Runnable() {
-                    @Override
-                    public void run() {
-                        if (listener != null) {
-                            listener.onMessage(o);
-                        }
-                    }
-                });
-            }
+        }catch (Exception e){
+            LogUtils.e("post寮傚父2锛�" + e.getMessage());
         }
     }
 
@@ -148,10 +183,17 @@
      * @param listener
      */
     public synchronized void registerAllTopicsListener(EventListener listener) {
-        if (ALL_TOPICS_EVENT != null && !ALL_TOPICS_EVENT.contains(listener)) {
-            ALL_TOPICS_EVENT.add(listener);
+        if(listener==null){
+            return;
         }
-        TYPE.put(listener, MAIN_TYPE);
+        try {
+            if (ALL_TOPICS_EVENT != null && !ALL_TOPICS_EVENT.contains(listener)) {
+                ALL_TOPICS_EVENT.add(listener);
+            }
+            TYPE.put(listener, MAIN_TYPE);
+        }catch (Exception e){
+            LogUtils.e("registerAllTopicsListener锛�" + e.getMessage());
+        }
     }
 
     /**
@@ -164,6 +206,9 @@
                 @Override
                 public void run() {
                     try {
+                        if(listener==null){
+                            return;
+                        }
                         if (ALL_TOPICS_EVENT != null && !ALL_TOPICS_EVENT.isEmpty()) {
                             TYPE.remove(listener);
                             ALL_TOPICS_EVENT.remove(listener);
@@ -189,6 +234,4 @@
         clear();
         ioThread.shutdownNow();
     }
-
-
 }

--
Gitblit v1.8.0