refactor(): Remove unnecessary null check from subscriptionId (#3739)

codewithtamim · AnGgIt886 · commit b80db25e4f77 · 2024-10-22T20:02:20.000+07:00
- Replaced `subscriptionId.isNullOrEmpty()` with `subscriptionId.isEmpty()` since `subscriptionId` is not nullable.
- This refactor simplifies the logic and improves code readability by eliminating the redundant null check.
diff --git a/app/src/main/kotlin/com/neko/v2ray/util/V2rayConfigUtil.kt b/app/src/main/kotlin/com/neko/v2ray/util/V2rayConfigUtil.kt
@@ -566,7 +566,7 @@ object V2rayConfigUtil {
             return returnPair
         }
 
-        if (subscriptionId.isNullOrEmpty()) {
+        if (subscriptionId.isEmpty()) {
             return returnPair
         }
         try {

Original file line number	Diff line number	Diff line change
`@@ -566,7 +566,7 @@ object V2rayConfigUtil {`
`566`	`566`	`return returnPair`
`567`	`567`	`}`
`568`	`568`
`569`		`- if (subscriptionId.isNullOrEmpty()) {`
	`569`	`+ if (subscriptionId.isEmpty()) {`
`570`	`570`	`return returnPair`
`571`	`571`	`}`
`572`	`572`	`try {`