diff --git a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSonPath.kt b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSonPath.kt
index 298c1d2fa..60a3510c2 100644
--- a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSonPath.kt
+++ b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSonPath.kt
@@ -62,17 +62,17 @@ class AnalyzeByJSonPath {
                 return result
             }
         } else {
-            val sb = StringBuilder()
+            val textList = arrayListOf<String>()
             for (rl in rules) {
                 val temp = getString(rl)
-                if (!TextUtils.isEmpty(temp)) {
-                    sb.append(temp)
+                if (!temp.isNullOrEmpty()) {
+                    textList.add(temp)
                     if (elementsType == "|") {
                         break
                     }
                 }
             }
-            return sb.toString()
+            return TextUtils.join(",", textList)
         }
     }
 
diff --git a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSoup.kt b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSoup.kt
index 879d18b05..7be879bbe 100644
--- a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSoup.kt
+++ b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByJSoup.kt
@@ -75,7 +75,7 @@ class AnalyzeByJSoup {
         //拆分规则
         val sourceRule = SourceRule(ruleStr)
         if (isEmpty(sourceRule.elementsRule)) {
-            textS.add(element!!.data())
+            textS.add(element?.data() ?: "")
         } else {
             val elementsType: String
             val ruleStrS: Array<String>
@@ -105,7 +105,7 @@ class AnalyzeByJSoup {
                 } else {
                     getResultList(ruleStrX)
                 }
-                if (temp != null && temp.isNotEmpty()) {
+                if (!temp.isNullOrEmpty()) {
                     results.add(temp)
                     if (results.isNotEmpty() && elementsType == "|") {
                         break
diff --git a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByXPath.kt b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByXPath.kt
index 23030393c..0989f88d5 100644
--- a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByXPath.kt
+++ b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeByXPath.kt
@@ -154,17 +154,17 @@ class AnalyzeByXPath {
             }
             return null
         } else {
-            val sb = StringBuilder()
+            val textList = arrayListOf<String>()
             for (rl in rules) {
                 val temp = getString(rl)
-                if (!TextUtils.isEmpty(temp)) {
-                    sb.append(temp)
+                if (!temp.isNullOrEmpty()) {
+                    textList.add(temp)
                     if (elementsType == "|") {
                         break
                     }
                 }
             }
-            return sb.toString()
+            return TextUtils.join(",", textList)
         }
     }
 }