[WEEX-342][android] ava.util.ConcurrentModificationException at java.…

…util.ArrayList$ArrayListIterator.next(ArrayList.java:573)at com.taobao.weex.ui.component.WXComponent.applyEvents(WXComponent.java:290)
apache · Jun 14, 2018 · c8d4d62 · c8d4d62
1 parent c60016f
commit c8d4d62
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/android/sdk/src/main/java/com/taobao/weex/ui/component/WXComponent.java b/android/sdk/src/main/java/com/taobao/weex/ui/component/WXComponent.java
@@ -290,7 +290,11 @@ private void applyEvents() {
     if (getEvents() == null || getEvents().isEmpty())
       return;
     WXEvent event = getEvents();
-    for(int i=0; i<event.size(); i++){
+    int size = event.size();
+    for (int i=0; i<size; i++) {
+      if(i >= event.size()){
+        break;
+      }
       String type = event.get(i);
       addEvent(type);
     }
@@ -1447,7 +1451,11 @@ public void removeAllEvent() {
       return;
     }
     WXEvent events = getEvents();
-    for (int i=0; i<events.size(); i++) {
+    int size = events.size();
+    for (int i=0; i<size; i++) {
+      if(i >= events.size()){
+        break;
+      }
       String event = events.get(i);
       if (event == null) {
         continue;

diff --git a/android/sdk/src/main/java/com/taobao/weex/ui/view/gesture/WXGesture.java b/android/sdk/src/main/java/com/taobao/weex/ui/view/gesture/WXGesture.java
@@ -140,7 +140,11 @@ public static boolean hasStopPropagation(WXComponent component){
     if(event == null){
       return false;
     }
-    for(int i=0; i<event.size(); i++){
+    int size = event.size();
+    for (int i=0; i<size; i++) {
+      if(i >= event.size()){
+        break;
+      }
       String type = event.get(i);
       if(isStopPropagation(type)){
         return true;