1.优化代码

timcommon/src/main/java/com/tencent/qcloud/tuikit/timcommon/classicui/widget/message/MessageContentHolder.java

@@ -1,6 +1,7 @@
 package com.tencent.qcloud.tuikit.timcommon.classicui.widget.message;
 
 import android.text.TextUtils;
+import android.util.Log;
 import android.view.Gravity;
 import android.view.View;
 import android.view.ViewGroup;
@@ -9,7 +10,9 @@ import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.ProgressBar;
 import android.widget.TextView;
+
 import androidx.recyclerview.widget.RecyclerView;
+
 import com.tencent.imsdk.v2.V2TIMManager;
 import com.tencent.imsdk.v2.V2TIMMessage;
 import com.tencent.imsdk.v2.V2TIMUserFullInfo;
@@ -399,7 +402,7 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
     }
 
     private void setupAvatar(String faceUrl, boolean right) {
-        if(!TextUtils.isEmpty(faceUrl)) {
+        if (!TextUtils.isEmpty(faceUrl)) {
             List<Object> urllist = new ArrayList<>();
             urllist.add(faceUrl);
             if (right) {
@@ -407,23 +410,13 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
             } else {
                 leftUserIcon.setIconUrls(urllist);
             }
+        } else {
+            if (right) {
+                rightUserIcon.setIconUrls(null);
+            } else {
+                leftUserIcon.setIconUrls(null);
+            }
         }
-//        if (!TextUtils.isEmpty(faceUrl)) {
-//            List<Object> urllist = new ArrayList<>();
-//            urllist.add(faceUrl);
-//            if (isForwardMode || isReplyDetailMode) {
-//                leftUserIcon.setIconUrls(urllist);
-//            } else {
-//                if (right) {
-//                    rightUserIcon.setIconUrls(urllist);
-//                } else {
-//                    leftUserIcon.setIconUrls(urllist);
-//                }
-//            }
-//        } else {
-//            rightUserIcon.setIconUrls(null);
-//            leftUserIcon.setIconUrls(null);
-//        }
     }
 
     protected void setMessageAreaPadding() {
@@ -516,7 +509,7 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
                 isReadText.setText(R.string.has_all_read);
             } else if (msg.isUnread()) {
                 isReadText.setTextColor(
-                    isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
+                        isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
                 isReadText.setText(R.string.unread);
                 isReadText.setOnClickListener(new View.OnClickListener() {
                     @Override
@@ -529,7 +522,7 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
                 if (readCount > 0) {
                     isReadText.setText(isReadText.getResources().getString(R.string.someone_has_read, readCount));
                     isReadText.setTextColor(
-                        isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
+                            isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
                     isReadText.setOnClickListener(new View.OnClickListener() {
                         @Override
                         public void onClick(View v) {
@@ -545,7 +538,7 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
             } else {
                 isReadText.setText(R.string.unread);
                 isReadText.setTextColor(
-                    isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
+                        isReadText.getResources().getColor(TUIThemeManager.getAttrResId(isReadText.getContext(), R.attr.chat_read_receipt_text_color)));
                 isReadText.setOnClickListener(new View.OnClickListener() {
                     @Override
                     public void onClick(View v) {
@@ -575,22 +568,24 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
             selectableTextHelper.destroy();
         }
         selectableTextHelper = new SelectTextHelper.Builder(textView)
-                                   .setCursorHandleColor(TIMCommonService.getAppContext().getResources().getColor(R.color.font_blue))
-                                   .setCursorHandleSizeInDp(18)
-                                   .setSelectedColor(TIMCommonService.getAppContext().getResources().getColor(R.color.test_blue))
-                                   .setSelectAll(true)
-                                   .setIsEmoji(isEmoji)
-                                   .setScrollShow(false)
-                                   .setSelectedAllNoPop(true)
-                                   .setMagnifierShow(false)
-                                   .build();
+                .setCursorHandleColor(TIMCommonService.getAppContext().getResources().getColor(R.color.font_blue))
+                .setCursorHandleSizeInDp(18)
+                .setSelectedColor(TIMCommonService.getAppContext().getResources().getColor(R.color.test_blue))
+                .setSelectAll(true)
+                .setIsEmoji(isEmoji)
+                .setScrollShow(false)
+                .setSelectedAllNoPop(true)
+                .setMagnifierShow(false)
+                .build();
 
         selectableTextHelper.setSelectListener(new SelectTextHelper.OnSelectListener() {
             @Override
-            public void onClick(View v) {}
+            public void onClick(View v) {
+            }
 
             @Override
-            public void onLongClick(View v) {}
+            public void onLongClick(View v) {
+            }
 
             @Override
             public void onTextSelected(CharSequence content) {
@@ -608,10 +603,12 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
             }
 
             @Override
-            public void onClickUrl(String url) {}
+            public void onClickUrl(String url) {
+            }
 
             @Override
-            public void onSelectAllShowCustomPop() {}
+            public void onSelectAllShowCustomPop() {
+            }
 
             @Override
             public void onReset() {
@@ -620,10 +617,12 @@ public abstract class MessageContentHolder<T extends TUIMessageBean> extends Mes
             }
 
             @Override
-            public void onDismissCustomPop() {}
+            public void onDismissCustomPop() {
+            }
 
             @Override
-            public void onScrolling() {}
+            public void onScrolling() {
+            }
         });
     }