ui operation shouldn't in stn thread, like req2Buf buf2Resp function call

shockcao · shockcao · commit b66da6ec70e2 · 2017-01-05T15:33:22.000+08:00
diff --git a/samples/android/marsSampleChat/app/src/main/java/com/tencent/mars/sample/ConversationActivity.java b/samples/android/marsSampleChat/app/src/main/java/com/tencent/mars/sample/ConversationActivity.java
@@ -216,21 +216,7 @@ public void onPreEncode(Main.ConversationListRequest req) {
 
             @Override
             public void onPostDecode(Main.ConversationListResponse response) {
-                // update data list only
-                if (response.list == null) {
-                    Log.i(TAG, "getconvlist: empty response list");
-                    progressBar.setVisibility(View.VISIBLE);
-                    return;
-                }
-                else if (response.list.length == 0) {
-                    Log.i(TAG, "getconvlist: empty response list");
-                    progressBar.setVisibility(View.VISIBLE);
-                    return;
-                }
 
-                for (Main.Conversation conv : response.list) {
-                    dataList.add(new Conversation(conv.name, conv.topic, conv.notice));
-                }
             }
 
             @Override
@@ -239,6 +225,12 @@ public void onTaskEnd() {
 
                     @Override
                     public void run() {
+                        if (response != null) {
+                            for (Main.Conversation conv : response.list) {
+                                dataList.add(new Conversation(conv.name, conv.topic, conv.notice));
+                            }
+                        }
+
                         if (!dataList.isEmpty()) {
                             progressBar.setVisibility(View.INVISIBLE);
                             conversationListAdapter.list.clear();
