fix: address PR review comments

- Refactor FontSizeUtil to reduce code duplication by extracting common helper method
- Change layout margin from 8dp to 2dp as requested
- Adjust line spacing multiplier from 1.3 to 1.1 for better readability

Addresses feedback from Copilot and @graycreate in PR #131

Author: graycreate

app/src/main/java/me/ghui/v2er/util/FontSizeUtil.java

@@ -6,64 +6,60 @@
 
 public class FontSizeUtil {
 
-    public static float getContentSize() {
+    /**
+     * Common helper method to get font dimension based on size preference
+     * @param smallDimen dimension resource for "小" size
+     * @param mediumDimen dimension resource for "中" size (default)
+     * @param largeDimen dimension resource for "大" size
+     * @param extraLargeDimen dimension resource for "特大" size
+     * @return font size in pixels
+     */
+    private static float getFontSizeForPreference(int smallDimen, int mediumDimen,
+                                                   int largeDimen, int extraLargeDimen) {
         String size = Pref.read(R.string.pref_key_fontsize);
         int id;
         switch (size) {
             case "小":
-                id = R.dimen.smallTextSize;
+                id = smallDimen;
                 break;
             case "大":
-                id = R.dimen.largeTextSize;
+                id = largeDimen;
                 break;
             case "特大":
-                id = R.dimen.extralargeTextSize;
+                id = extraLargeDimen;
                 break;
             case "中":
             default:
-                id = R.dimen.mediumTextSize;
+                id = mediumDimen;
         }
         return App.get().getResources().getDimension(id);
     }
 
+    public static float getContentSize() {
+        return getFontSizeForPreference(
+            R.dimen.smallTextSize,
+            R.dimen.mediumTextSize,
+            R.dimen.largeTextSize,
+            R.dimen.extralargeTextSize
+        );
+    }
+
     public static float getTitleSize() {
-        String size = Pref.read(R.string.pref_key_fontsize);
-        int id;
-        switch (size) {
-            case "小":
-                id = R.dimen.smallTextSize;
-                break;
-            case "大":
-                id = R.dimen.largeTextSize;
-                break;
-            case "特大":
-                id = R.dimen.extralargeTextSize;
-                break;
-            case "中":
-            default:
-                id = R.dimen.mediumTextSize;
-        }
-        return App.get().getResources().getDimension(id);
+        return getFontSizeForPreference(
+            R.dimen.smallTextSize,
+            R.dimen.mediumTextSize,
+            R.dimen.largeTextSize,
+            R.dimen.extralargeTextSize
+        );
     }
 
     public static float getSubTextSize() {
-        String size = Pref.read(R.string.pref_key_fontsize);
-        int id;
-        switch (size) {
-            case "小":
-                id = R.dimen.microTextSize;
-                break;
-            case "大":
-                id = R.dimen.mediumTextSize;
-                break;
-            case "特大":
-                id = R.dimen.largeTextSize;
-                break;
-            case "中":
-            default:
-                id = R.dimen.smallTextSize;
-        }
-        return App.get().getResources().getDimension(id);
+        return getFontSizeForPreference(
+            R.dimen.microTextSize,
+            R.dimen.smallTextSize,
+            R.dimen.mediumTextSize,
+            R.dimen.largeTextSize
+        );
     }
 
     public static float getHtmlFontSize() {

app/src/main/res/layout/common_list_item.xml

@@ -13,7 +13,7 @@
         android:id="@+id/title_tv"
         style="@style/TopicTitle"
         android:layout_below="@id/time_tv"
-        android:layout_marginTop="8dp"
+        android:layout_marginTop="2dp"
         tools:text="[macOS]有人用 magnet 分屏的吗？好像有会导致系统卡顿的奇怪是的呵呵，里" />
 
     <de.hdodenhof.circleimageview.CircleImageView

app/src/main/res/values/styles.xml

@@ -134,7 +134,7 @@
         <item name="android:textSize">@dimen/mediumTextSize</item>
         <item name="android:textStyle">normal</item>
         <item name="android:paddingTop">6dp</item>
-        <item name="android:lineSpacingMultiplier">1.3</item>
+        <item name="android:lineSpacingMultiplier">1.1</item>
         <!--<item name="android:transitionName">@string/share_element_topic_title</item>-->
     </style>