Merge branch 'main' into main

Author: HanNayeoniee

Cargo.lock

@@ -13,10 +13,10 @@ readme = "../../README.md"
 homepage = "https://braillify.kr"
 
 [dependencies]
-phf = { version = "0.11", features = ["macros"] }
+phf = { version = "0.12", features = ["macros"] }
 
 
 [dev-dependencies]
-csv = "1.3.0"
+csv = "1.3.1"
 serde_json = "^1"
 proptest = "1.7"

libs/braillify/Cargo.toml

@@ -14,7 +14,7 @@ pub struct KoreanChar {
 impl KoreanChar {
     pub fn new(c: char) -> Result<Self, String> {
         let code = c as u32;
-        if !(0xAC00 <= code && code <= 0xD7A3) {
+        if !(0xAC00..=0xD7A3).contains(&code) {
             return Err("Invalid Korean character".to_string());
         }
 
@@ -76,13 +76,13 @@ impl CharType {
             return Ok(Self::MathSymbol(c));
         }
         let code = c as u32;
-        if 0x3131 <= code && code <= 0x3163 {
+        if (0x3131..=0x3163).contains(&code) {
             return Ok(Self::KoreanPart(c));
         }
         // if !(0xAC00 <= code && code <= 0xD7A3) {
         //     return Ok(Self::Char(c));
         // }
-        if 0xAC00 <= code && code <= 0xD7A3 {
+        if (0xAC00..=0xD7A3).contains(&code) {
             return Ok(Self::Korean(KoreanChar::new(c)?));
         }
         if c.is_whitespace() {

libs/braillify/src/char_struct.rs

@@ -52,16 +52,14 @@ pub fn encode_korean_char(korean: &KoreanChar) -> Result<Vec<u8>, String> {
             result.extend(encode_jungsong(korean.jung)?);
             result.extend(encode_jongseong(jong)?);
         }
+    } else if let Ok(code) = char_shortcut::encode_char_shortcut(build_char(cho0, korean.jung, None)) {
+        result.extend(code);
     } else {
-        if let Ok(code) = char_shortcut::encode_char_shortcut(build_char(cho0, korean.jung, None)) {
-            result.extend(code);
-        } else {
-            // shortcut 이 없으므로 초성 중성, 모두 결합
-            if cho0 != 'ㅇ' {
-                result.push(encode_choseong(cho0)?);
-            }
-            result.extend(encode_jungsong(korean.jung)?);
+        // shortcut 이 없으므로 초성 중성, 모두 결합
+        if cho0 != 'ㅇ' {
+            result.push(encode_choseong(cho0)?);
         }
+        result.extend(encode_jungsong(korean.jung)?);
     }
 
     Ok(result)

libs/braillify/src/korean_char.rs

@@ -210,17 +210,15 @@ impl Encoder {
                                     // 8항 - 단독으로 쓰인 자모
                                     result.push(63);
                                     result.extend(korean_part::encode_korean_part(c)?);
+                                } else if has_korean_char {
+                                    // 10항 - 단독으로 쓰인 자음자가 단어에 붙어 나올 때
+                                    result.push(56);
+                                    result.extend(korean_part::encode_korean_part(c)?);
                                 } else {
-                                    if has_korean_char {
-                                        // 10항 - 단독으로 쓰인 자음자가 단어에 붙어 나올 때
-                                        result.push(56);
-                                        result.extend(korean_part::encode_korean_part(c)?);
-                                    } else {
-                                        // 10항 - 단독으로 쓰인 자음자가 단어에 붙어 나올 때
-                                        // 8항 - 단독으로 쓰인 자모
-                                        result.push(63);
-                                        result.extend(korean_part::encode_korean_part(c)?);
-                                    }
+                                    // 10항 - 단독으로 쓰인 자음자가 단어에 붙어 나올 때
+                                    // 8항 - 단독으로 쓰인 자모
+                                    result.push(63);
+                                    result.extend(korean_part::encode_korean_part(c)?);
                                 }
                             }
                         }
@@ -337,8 +335,8 @@ impl Encoder {
             }
         }
 
-        if self.triple_big_english {
-            if !(remaining_words
+        if self.triple_big_english
+            && !(remaining_words
                 .first()
                 .is_some_and(|w| w.chars().all(|c| c.is_ascii_alphabetic())))
             {
@@ -349,7 +347,6 @@ impl Encoder {
                 result.push(4);
                 self.triple_big_english = false; // Reset after adding terminator
             }
-        }
         if !remaining_words.is_empty() {
             if self.english_indicator
                 && !remaining_words[0]
@@ -590,7 +587,8 @@ mod test {
             for (line_num, result) in reader.into_records().enumerate() {
                 total += 1;
                 file_total += 1;
-                let record = result.unwrap();
+                let error = format!("CSV 레코드를 읽는 중 오류 발생: {:?} at {}", result, line_num);
+                let record = result.expect(&error);
                 let input = &record[0];
                 let expected = record[2].replace(" ", "⠀");
                 match encode(input) {