.gitea/workflows/pull-request.yaml

@@ -18,9 +18,9 @@ jobs:
           java-version: 21
       - name: Setup Gradle
         uses: gradle/actions/setup-gradle@v4
-      - name: Lint
-        run: ./gradlew detekt
       - name: Build
-        run: ./gradlew build testClasses -x check
+        run: ./gradlew build testClasses -x test -x detekt
+      - name: Detekt
+        run: ./gradlew detekt
       - name: Test
         run: ./gradlew test

build.gradle.kts

@@ -9,16 +9,11 @@ version = "1.0-SNAPSHOT"
 
 repositories {
     mavenCentral()
-    maven {
-        url = uri("https://gitea.marvinelsen.com/api/packages/marvinelsen/maven")
-    }
 }
 
 dependencies {
     detektPlugins(libs.detekt.formatting)
 
-    implementation("com.marvinelsen:chinese-transliteration:1.1-SNAPSHOT") { isChanging = true }
-
     testImplementation(libs.kotest.core)
     testImplementation(libs.kotest.assertions)
 }

src/main/kotlin/com/marvinelsen/cedict/api/CedictEntry.kt

@@ -1,10 +1,8 @@
 package com.marvinelsen.cedict.api
 
-import com.marvinelsen.chinese.transliteration.Syllable
-
 data class CedictEntry(
     val traditional: String,
     val simplified: String,
-    val pinyinSyllables: List<Syllable>,
+    val pinyinSyllables: List<PinyinSyllable>,
     val definitions: List<CedictDefinition>,
 )

src/main/kotlin/com/marvinelsen/cedict/api/PinyinSyllable.kt

@@ -0,0 +1,27 @@
+package com.marvinelsen.cedict.api
+
+data class PinyinSyllable(
+    val syllable: String,
+    val tone: Tone,
+) {
+    companion object {
+        fun fromString(pinyinWithNumbers: String): PinyinSyllable {
+            require(pinyinWithNumbers.isNotBlank()) { "Argument pinyinWithNumbers must not be blank." }
+
+            val lastCharacter = pinyinWithNumbers.last()
+            return if (lastCharacter.isDigit()) {
+                PinyinSyllable(
+                    syllable = pinyinWithNumbers.substring(0, pinyinWithNumbers.lastIndex),
+                    tone = Tone.fromDigit(lastCharacter)
+                )
+            } else {
+                PinyinSyllable(
+                    syllable = pinyinWithNumbers,
+                    tone = Tone.NONE
+                )
+            }
+        }
+    }
+
+    override fun toString() = syllable + tone.toString()
+}

src/main/kotlin/com/marvinelsen/cedict/api/Tone.kt

@@ -0,0 +1,25 @@
+package com.marvinelsen.cedict.api
+
+enum class Tone {
+    NONE, FIRST, SECOND, THIRD, FORTH, FIFTH;
+
+    companion object {
+        fun fromDigit(digit: Char) = when (digit) {
+            '1' -> FIRST
+            '2' -> SECOND
+            '3' -> THIRD
+            '4' -> FORTH
+            '5' -> FIFTH
+            else -> error("Digit $digit is not a valid tone")
+        }
+    }
+
+    override fun toString() = when (this) {
+        NONE -> ""
+        FIRST -> "1"
+        SECOND -> "2"
+        THIRD -> "3"
+        FORTH -> "4"
+        FIFTH -> "5"
+    }
+}

src/main/kotlin/com/marvinelsen/cedict/internal/CedictParserImpl.kt

@@ -3,7 +3,7 @@ package com.marvinelsen.cedict.internal
 import com.marvinelsen.cedict.api.CedictDefinition
 import com.marvinelsen.cedict.api.CedictEntry
 import com.marvinelsen.cedict.api.CedictParser
-import com.marvinelsen.chinese.transliteration.Syllable
+import com.marvinelsen.cedict.api.PinyinSyllable
 import java.io.InputStream
 
 internal class CedictParserImpl : CedictParser {
@@ -42,9 +42,10 @@ internal class CedictParserImpl : CedictParser {
     }
 
     private fun toPinyinSyllables(pinyinWithNumbers: String) = pinyinWithNumbers
+        .lowercase()
+        .replace("u:", "ü")
         .split(" ")
-        .filter { Syllable.isValidPinyinWithToneNumberSyllable(it) }
+        .map { PinyinSyllable.fromString(it) }
-        .map { Syllable.fromPinyinWithToneNumber(it) }
 
     private fun toCedictDefinitions(definitions: String) = definitions
         .split(DEFINITION_SEPARATOR)

src/test/kotlin/com/marvinelsen/cedict/internal/CedictParserImplTest.kt

@@ -1,8 +1,8 @@
 package com.marvinelsen.cedict.internal
 
 import com.marvinelsen.cedict.api.CedictDefinition
-import com.marvinelsen.chinese.transliteration.Syllable
+import com.marvinelsen.cedict.api.PinyinSyllable
-import com.marvinelsen.chinese.transliteration.Tone
+import com.marvinelsen.cedict.api.Tone
 import io.kotest.core.spec.style.ShouldSpec
 import io.kotest.matchers.shouldBe
 import java.util.zip.GZIPInputStream
@@ -16,8 +16,8 @@ class CedictParserImplTest : ShouldSpec({
         cedictEntry.traditional shouldBe "皮實"
         cedictEntry.simplified shouldBe "皮实"
         cedictEntry.pinyinSyllables shouldBe listOf(
-            Syllable("pi", Tone.SECOND),
+            PinyinSyllable("pi", Tone.SECOND),
-            Syllable("shi", Tone.FIFTH)
+            PinyinSyllable("shi", Tone.FIFTH)
         )
         cedictEntry.definitions shouldBe listOf(
             CedictDefinition(listOf("(of things) durable")),