From 6209fde93e93f93a8e88b30c781d3bbbc008b746 Mon Sep 17 00:00:00 2001
From: kunfei <gekunfei@live.com>
Date: Thu, 4 Jul 2019 14:58:54 +0800
Subject: [PATCH 1/6] up

---
 app/build.gradle                                           | 4 ++--
 app/src/main/java/io/legado/app/data/dao/BookChapterDao.kt | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index f36dfe7f5..c15bcc12f 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -80,8 +80,8 @@ dependencies {
 
     //androidX
     implementation 'androidx.core:core-ktx:1.2.0-alpha02'
-    implementation 'androidx.appcompat:appcompat:1.1.0-beta01'
-    implementation 'androidx.preference:preference:1.1.0-beta01'
+    implementation 'androidx.appcompat:appcompat:1.1.0-rc01'
+    implementation 'androidx.preference:preference:1.1.0-rc01'
     implementation 'androidx.constraintlayout:constraintlayout:2.0.0-beta1'
     implementation 'com.google.android.material:material:1.1.0-alpha07'
     implementation 'com.google.android:flexbox:1.1.0'
diff --git a/app/src/main/java/io/legado/app/data/dao/BookChapterDao.kt b/app/src/main/java/io/legado/app/data/dao/BookChapterDao.kt
index ad45e6290..9cfe22e8d 100644
--- a/app/src/main/java/io/legado/app/data/dao/BookChapterDao.kt
+++ b/app/src/main/java/io/legado/app/data/dao/BookChapterDao.kt
@@ -1,9 +1,11 @@
 package io.legado.app.data.dao
 
+import androidx.room.Dao
 import androidx.room.Insert
 import androidx.room.OnConflictStrategy
 import io.legado.app.data.entities.BookChapter
 
+@Dao
 interface BookChapterDao {
 
 

From 3939004ed77436ba2677930b4d385f04a0d87fd8 Mon Sep 17 00:00:00 2001
From: kunfei <gekunfei@live.com>
Date: Thu, 4 Jul 2019 15:05:54 +0800
Subject: [PATCH 2/6] up

---
 .../java/io/legado/app/model/analyzeRule/AnalyzeByJSoup.kt     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

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 8b4bcc811..cb93e6758 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
@@ -2,7 +2,6 @@ package io.legado.app.model.analyzeRule
 
 import android.text.TextUtils.isEmpty
 import android.text.TextUtils.join
-import io.legado.app.utils.startWithIgnoreCase
 import org.jsoup.Jsoup
 import org.jsoup.nodes.Element
 import org.jsoup.select.Collector
@@ -414,7 +413,7 @@ class AnalyzeByJSoup {
         var replacement = ""
 
         init {
-            if (ruleStr.startWithIgnoreCase("@CSS:")) {
+            if (ruleStr.startsWith("@CSS:", true)) {
                 isCss = true
                 elementsRule = ruleStr.substring(5).trim { it <= ' ' }
             } else {

From 654d8d1d3cd1f67f33e7ff1319654aeaef48d413 Mon Sep 17 00:00:00 2001
From: kunfei <gekunfei@live.com>
Date: Fri, 5 Jul 2019 15:25:23 +0800
Subject: [PATCH 3/6] up

---
 .../app/model/analyzeRule/AnalyzeUrl.kt       | 42 +++++++++++--------
 1 file changed, 25 insertions(+), 17 deletions(-)

diff --git a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeUrl.kt b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeUrl.kt
index 273b76fad..28067b32a 100644
--- a/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeUrl.kt
+++ b/app/src/main/java/io/legado/app/model/analyzeRule/AnalyzeUrl.kt
@@ -6,10 +6,9 @@ import androidx.annotation.Keep
 import io.legado.app.constant.AppConst.SCRIPT_ENGINE
 import io.legado.app.constant.Pattern.EXP_PATTERN
 import io.legado.app.data.entities.Book
-import io.legado.app.utils.Encoder
-import io.legado.app.utils.GSON
-import io.legado.app.utils.NetworkUtils
-import io.legado.app.utils.fromJsonObject
+import io.legado.app.utils.*
+import okhttp3.FormBody
+import okhttp3.MediaType
 import okhttp3.RequestBody
 import java.net.URLEncoder
 import java.util.*
@@ -33,6 +32,7 @@ class AnalyzeUrl(
 ) {
     companion object {
         private val pagePattern = Pattern.compile("<(.*?)>")
+        private val jsonType = MediaType.parse("application/json; charset=utf-8")
     }
 
     lateinit var url: String
@@ -43,7 +43,7 @@ class AnalyzeUrl(
         private set
     var queryStr: String? = null
         private set
-    private val queryMap = LinkedHashMap<String, String>()
+    private val fieldMap = LinkedHashMap<String, String>()
     private val headerMap = HashMap<String, String>()
     private var charset: String? = null
     private var bodyTxt: String? = null
@@ -55,9 +55,9 @@ class AnalyzeUrl(
     val postData: ByteArray
         get() {
             val builder = StringBuilder()
-            val keys = queryMap.keys
+            val keys = fieldMap.keys
             for (key in keys) {
-                builder.append(String.format("%s=%s&", key, queryMap[key]))
+                builder.append(String.format("%s=%s&", key, fieldMap[key]))
             }
             builder.deleteCharAt(builder.lastIndexOf("&"))
             return builder.toString().toByteArray()
@@ -130,9 +130,7 @@ class AnalyzeUrl(
             options?.let {
                 options["method"]?.let { if (it.equals("POST", true)) method = Method.POST }
                 options["headers"]?.let { headers ->
-                    GSON.fromJsonObject<Map<String, String>>(
-                        headers
-                    )?.let { headerMap.putAll(it) }
+                    GSON.fromJsonObject<Map<String, String>>(headers)?.let { headerMap.putAll(it) }
                 }
                 options["body"]?.let { bodyTxt = it }
                 options["charset"]?.let { charset = it }
@@ -143,11 +141,21 @@ class AnalyzeUrl(
                 urlArray = url.split("\\?".toRegex())
                 url = urlArray[0]
                 if (urlArray.size > 1) {
-                    analyzePutFields(urlArray[1])
+                    analyzeFields(urlArray[1])
                 }
             }
             Method.POST -> {
-                bodyTxt
+                bodyTxt?.let {
+                    if (it.isJson()) {
+                        body = RequestBody.create(jsonType, it)
+                    } else {
+                        analyzeFields(it)
+                        val builder = FormBody.Builder()
+                        for (item in fieldMap)
+                            builder.add(item.key, item.value)
+                        body = builder.build()
+                    }
+                }
             }
         }
     }
@@ -157,7 +165,7 @@ class AnalyzeUrl(
      * 解析QueryMap
      */
     @Throws(Exception::class)
-    private fun analyzePutFields(fieldsTxt: String) {
+    private fun analyzeFields(fieldsTxt: String) {
         queryStr = fieldsTxt
         val queryS = fieldsTxt.split("&".toRegex()).dropLastWhile { it.isEmpty() }.toTypedArray()
         for (query in queryS) {
@@ -165,14 +173,14 @@ class AnalyzeUrl(
             val value = if (queryM.size > 1) queryM[1] else ""
             if (TextUtils.isEmpty(charset)) {
                 if (NetworkUtils.hasUrlEncoded(value)) {
-                    queryMap[queryM[0]] = value
+                    fieldMap[queryM[0]] = value
                 } else {
-                    queryMap[queryM[0]] = URLEncoder.encode(value, "UTF-8")
+                    fieldMap[queryM[0]] = URLEncoder.encode(value, "UTF-8")
                 }
             } else if (charset == "escape") {
-                queryMap[queryM[0]] = Encoder.escape(value)
+                fieldMap[queryM[0]] = Encoder.escape(value)
             } else {
-                queryMap[queryM[0]] = URLEncoder.encode(value, charset)
+                fieldMap[queryM[0]] = URLEncoder.encode(value, charset)
             }
         }
     }

From 6cb217ed6481a1f1e15b21d5822f465b1a94f477 Mon Sep 17 00:00:00 2001
From: kunfei <gekunfei@live.com>
Date: Fri, 5 Jul 2019 15:38:03 +0800
Subject: [PATCH 4/6] up

---
 app/src/main/java/io/legado/app/model/book/All.kt             | 2 ++
 app/src/main/java/io/legado/app/model/book/BookChapterList.kt | 2 ++
 app/src/main/java/io/legado/app/model/book/BookContent.kt     | 2 ++
 app/src/main/java/io/legado/app/model/book/BookInfo.kt        | 2 ++
 app/src/main/java/io/legado/app/model/book/BookList.kt        | 2 ++
 5 files changed, 10 insertions(+)
 create mode 100644 app/src/main/java/io/legado/app/model/book/All.kt
 create mode 100644 app/src/main/java/io/legado/app/model/book/BookChapterList.kt
 create mode 100644 app/src/main/java/io/legado/app/model/book/BookContent.kt
 create mode 100644 app/src/main/java/io/legado/app/model/book/BookInfo.kt
 create mode 100644 app/src/main/java/io/legado/app/model/book/BookList.kt

diff --git a/app/src/main/java/io/legado/app/model/book/All.kt b/app/src/main/java/io/legado/app/model/book/All.kt
new file mode 100644
index 000000000..a85cebe46
--- /dev/null
+++ b/app/src/main/java/io/legado/app/model/book/All.kt
@@ -0,0 +1,2 @@
+package io.legado.app.model.book
+
diff --git a/app/src/main/java/io/legado/app/model/book/BookChapterList.kt b/app/src/main/java/io/legado/app/model/book/BookChapterList.kt
new file mode 100644
index 000000000..a85cebe46
--- /dev/null
+++ b/app/src/main/java/io/legado/app/model/book/BookChapterList.kt
@@ -0,0 +1,2 @@
+package io.legado.app.model.book
+
diff --git a/app/src/main/java/io/legado/app/model/book/BookContent.kt b/app/src/main/java/io/legado/app/model/book/BookContent.kt
new file mode 100644
index 000000000..a85cebe46
--- /dev/null
+++ b/app/src/main/java/io/legado/app/model/book/BookContent.kt
@@ -0,0 +1,2 @@
+package io.legado.app.model.book
+
diff --git a/app/src/main/java/io/legado/app/model/book/BookInfo.kt b/app/src/main/java/io/legado/app/model/book/BookInfo.kt
new file mode 100644
index 000000000..a85cebe46
--- /dev/null
+++ b/app/src/main/java/io/legado/app/model/book/BookInfo.kt
@@ -0,0 +1,2 @@
+package io.legado.app.model.book
+
diff --git a/app/src/main/java/io/legado/app/model/book/BookList.kt b/app/src/main/java/io/legado/app/model/book/BookList.kt
new file mode 100644
index 000000000..a85cebe46
--- /dev/null
+++ b/app/src/main/java/io/legado/app/model/book/BookList.kt
@@ -0,0 +1,2 @@
+package io.legado.app.model.book
+

From 9a735562171dbc0a39e072065436b9a7bc7d67d7 Mon Sep 17 00:00:00 2001
From: kunfei <gekunfei@live.com>
Date: Fri, 5 Jul 2019 15:40:55 +0800
Subject: [PATCH 5/6] up

---
 app/src/main/java/io/legado/app/model/book/All.kt             | 4 ++++
 app/src/main/java/io/legado/app/model/book/BookChapterList.kt | 3 +++
 app/src/main/java/io/legado/app/model/book/BookContent.kt     | 3 +++
 app/src/main/java/io/legado/app/model/book/BookInfo.kt        | 3 +++
 app/src/main/java/io/legado/app/model/book/BookList.kt        | 3 +++
 5 files changed, 16 insertions(+)

diff --git a/app/src/main/java/io/legado/app/model/book/All.kt b/app/src/main/java/io/legado/app/model/book/All.kt
index a85cebe46..e75b4ac5a 100644
--- a/app/src/main/java/io/legado/app/model/book/All.kt
+++ b/app/src/main/java/io/legado/app/model/book/All.kt
@@ -1,2 +1,6 @@
 package io.legado.app.model.book
 
+class All {
+
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/model/book/BookChapterList.kt b/app/src/main/java/io/legado/app/model/book/BookChapterList.kt
index a85cebe46..38e297239 100644
--- a/app/src/main/java/io/legado/app/model/book/BookChapterList.kt
+++ b/app/src/main/java/io/legado/app/model/book/BookChapterList.kt
@@ -1,2 +1,5 @@
 package io.legado.app.model.book
 
+class BookChapterList {
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/model/book/BookContent.kt b/app/src/main/java/io/legado/app/model/book/BookContent.kt
index a85cebe46..8d6341b32 100644
--- a/app/src/main/java/io/legado/app/model/book/BookContent.kt
+++ b/app/src/main/java/io/legado/app/model/book/BookContent.kt
@@ -1,2 +1,5 @@
 package io.legado.app.model.book
 
+class BookContent {
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/model/book/BookInfo.kt b/app/src/main/java/io/legado/app/model/book/BookInfo.kt
index a85cebe46..6dd7e1e07 100644
--- a/app/src/main/java/io/legado/app/model/book/BookInfo.kt
+++ b/app/src/main/java/io/legado/app/model/book/BookInfo.kt
@@ -1,2 +1,5 @@
 package io.legado.app.model.book
 
+class BookInfo {
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/model/book/BookList.kt b/app/src/main/java/io/legado/app/model/book/BookList.kt
index a85cebe46..489d694c7 100644
--- a/app/src/main/java/io/legado/app/model/book/BookList.kt
+++ b/app/src/main/java/io/legado/app/model/book/BookList.kt
@@ -1,2 +1,5 @@
 package io.legado.app.model.book
 
+class BookList {
+
+}
\ No newline at end of file

From 3eb22e5ce082c546a5e1d801b2e1773ee4b2c56d Mon Sep 17 00:00:00 2001
From: atbest <atbest09@gmail.com>
Date: Sat, 6 Jul 2019 00:26:33 -0400
Subject: [PATCH 6/6] Optimize utils

---
 .../main/java/io/legado/app/utils/Encoder.kt  | 28 ++++++---------
 .../java/io/legado/app/utils/FileUtils.kt     | 22 ++----------
 .../java/io/legado/app/utils/NetworkUtils.kt  | 34 ++++---------------
 .../main/java/io/legado/app/utils/README.md   |  1 -
 .../io/legado/app/utils/StringExtensions.kt   |  2 +-
 5 files changed, 22 insertions(+), 65 deletions(-)
 delete mode 100644 app/src/main/java/io/legado/app/utils/README.md

diff --git a/app/src/main/java/io/legado/app/utils/Encoder.kt b/app/src/main/java/io/legado/app/utils/Encoder.kt
index b74266fa3..8f97c4c83 100644
--- a/app/src/main/java/io/legado/app/utils/Encoder.kt
+++ b/app/src/main/java/io/legado/app/utils/Encoder.kt
@@ -3,26 +3,20 @@ package io.legado.app.utils
 object Encoder {
 
     fun escape(src: String): String {
-        var i = 0
-        var char: Char
         val tmp = StringBuilder()
-        tmp.ensureCapacity(src.length * 6)
-        while (i < src.length) {
-            char = src[i]
-            if (Character.isDigit(char) || Character.isLowerCase(char)
-                || Character.isUpperCase(char)
-            )
+        for (char in src) {
+            val charCode = char.toInt()
+            if (charCode in 48..57 || charCode in 65..90 || charCode in 97..122) {
                 tmp.append(char)
-            else if (char.toInt() < 256) {
-                tmp.append("%")
-                if (char.toInt() < 16)
-                    tmp.append("0")
-                tmp.append(char.toInt().toString(16))
-            } else {
-                tmp.append("%u")
-                tmp.append(char.toInt().toString(16))
+                continue
             }
-            i++
+
+            val prefix = when {
+                charCode < 16 -> "%0"
+                charCode < 256 -> "%"
+                else -> "%u"
+            }
+            tmp.append(prefix).append(charCode.toString(16))
         }
         return tmp.toString()
     }
diff --git a/app/src/main/java/io/legado/app/utils/FileUtils.kt b/app/src/main/java/io/legado/app/utils/FileUtils.kt
index 8c5ba3823..e64869d11 100644
--- a/app/src/main/java/io/legado/app/utils/FileUtils.kt
+++ b/app/src/main/java/io/legado/app/utils/FileUtils.kt
@@ -21,20 +21,7 @@ object FileUtils {
     fun getSdPath() = Environment.getExternalStorageDirectory().absolutePath
 
     fun getFileByPath(filePath: String): File? {
-        return if (isSpace(filePath)) null else File(filePath)
-    }
-
-    fun isSpace(s: String?): Boolean {
-        if (s == null) return true
-        var i = 0
-        val len = s.length
-        while (i < len) {
-            if (!Character.isWhitespace(s[i])) {
-                return false
-            }
-            ++i
-        }
-        return true
+        return if (filePath.isBlank()) null else File(filePath)
     }
 
     fun getSdCardPath(): String {
@@ -101,12 +88,9 @@ object FileUtils {
         return paths
     }
 
-    @TargetApi(Build.VERSION_CODES.KITKAT)
     fun getPath(context: Context, uri: Uri): String? {
-        val isKitKat = Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT
-
         // DocumentProvider
-        if (isKitKat && DocumentsContract.isDocumentUri(context, uri)) {
+        if (DocumentsContract.isDocumentUri(context, uri)) {
             // ExternalStorageProvider
             if (isExternalStorageDocument(uri)) {
                 val docId = DocumentsContract.getDocumentId(uri)
@@ -119,7 +103,7 @@ object FileUtils {
 
             } else if (isDownloadsDocument(uri)) {
                 val id = DocumentsContract.getDocumentId(uri)
-                val split = id.split(":".toRegex()).dropLastWhile { it.isEmpty() }.toTypedArray()
+                val split = id.split(":").dropLastWhile { it.isEmpty() }.toTypedArray()
                 val type = split[0]
                 if ("raw".equals(
                         type,
diff --git a/app/src/main/java/io/legado/app/utils/NetworkUtils.kt b/app/src/main/java/io/legado/app/utils/NetworkUtils.kt
index 79ce84ef9..ae2d42d47 100644
--- a/app/src/main/java/io/legado/app/utils/NetworkUtils.kt
+++ b/app/src/main/java/io/legado/app/utils/NetworkUtils.kt
@@ -8,37 +8,18 @@ object NetworkUtils {
 
     private val notNeedEncoding: BitSet by lazy {
         val bitSet = BitSet(256)
-        var i: Int = 'a'.toInt()
-        while (i <= 'z'.toInt()) {
+        for (i in 'a'.toInt()..'z'.toInt()) {
             bitSet.set(i)
-            i++
         }
-        i = 'A'.toInt()
-        while (i <= 'Z'.toInt()) {
+        for (i in 'A'.toInt()..'Z'.toInt()) {
             bitSet.set(i)
-            i++
         }
-        i = '0'.toInt()
-        while (i <= '9'.toInt()) {
+        for (i in '0'.toInt()..'9'.toInt()) {
             bitSet.set(i)
-            i++
         }
-        bitSet.set('+'.toInt())
-        bitSet.set('-'.toInt())
-        bitSet.set('_'.toInt())
-        bitSet.set('.'.toInt())
-        bitSet.set('$'.toInt())
-        bitSet.set(':'.toInt())
-        bitSet.set('('.toInt())
-        bitSet.set(')'.toInt())
-        bitSet.set('!'.toInt())
-        bitSet.set('*'.toInt())
-        bitSet.set('@'.toInt())
-        bitSet.set('&'.toInt())
-        bitSet.set('#'.toInt())
-        bitSet.set(','.toInt())
-        bitSet.set('['.toInt())
-        bitSet.set(']'.toInt())
+        for (char in "+-_.$:()!*@&#,[]") {
+            bitSet.set(char.toInt())
+        }
         return@lazy bitSet
     }
 
@@ -69,7 +50,6 @@ object NetworkUtils {
             // 其他字符，肯定需要urlEncode
             needEncode = true
             break
-            i++
         }
 
         return !needEncode
@@ -79,7 +59,7 @@ object NetworkUtils {
      * 判断c是否是16进制的字符
      */
     private fun isDigit16Char(c: Char): Boolean {
-        return c >= '0' && c <= '9' || c >= 'A' && c <= 'F'
+        return c in '0'..'9' || c in 'A'..'F' || c in 'a'..'f'
     }
 
     /**
diff --git a/app/src/main/java/io/legado/app/utils/README.md b/app/src/main/java/io/legado/app/utils/README.md
deleted file mode 100644
index a31b943b5..000000000
--- a/app/src/main/java/io/legado/app/utils/README.md
+++ /dev/null
@@ -1 +0,0 @@
-## 放置一些工具类
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/utils/StringExtensions.kt b/app/src/main/java/io/legado/app/utils/StringExtensions.kt
index 0ec34e16e..de179f1da 100644
--- a/app/src/main/java/io/legado/app/utils/StringExtensions.kt
+++ b/app/src/main/java/io/legado/app/utils/StringExtensions.kt
@@ -17,7 +17,7 @@ fun String?.isJson(): Boolean = this?.run {
 } ?: false
 
 fun String?.htmlFormat(): String = if (this.isNullOrBlank()) "" else
-    this.replace("(?i)<(br[\\s/]*|/*p.*?|/*div.*?)>".toRegex(), "\n")// 替换特定标签为换行符
+    this.replace("(?i)<(br[\\s/]*|/*p\\b.*?|/*div\\b.*?)>".toRegex(), "\n")// 替换特定标签为换行符
         .replace("<[script>]*.*?>|&nbsp;".toRegex(), "")// 删除script标签对和空格转义符
         .replace("\\s*\\n+\\s*".toRegex(), "\n　　")// 移除空行,并增加段前缩进2个汉字
         .replace("^[\\n\\s]+".toRegex(), "　　")//移除开头空行,并增加段前缩进2个汉字