ローマ字→ひらがな変換ロジックをリファクタリング

Converter.romajiToHiragana の変換ロジックを RomajiToHiragana クラスに
抽出し、可読性を改善:
- String[] の代わりに Entry record を使用
- マジックナンバー(4)を変換テーブルから自動計算
- 巻き戻り値を事前にintへパース（実行時パース廃止）
- Minecraft/Forge に依存しない純粋なJavaクラスとして分離
- 227件のユニットテストを追加（JUnit 5）

https://claude.ai/code/session_01D2dE5uXJ2exvdTGNvXjKJA

Author: claude

build.gradle

@@ -171,6 +171,9 @@ dependencies {
     runtimeOnly fg.deobf("curse.maven:modern-ui-352491:6199942")
 
     annotationProcessor 'org.spongepowered:mixin:0.8.5:processor'
+
+    testImplementation 'org.junit.jupiter:junit-jupiter:5.10.2'
+    testRuntimeOnly 'org.junit.platform:junit-platform-launcher'
 }
 
 mixin {
@@ -240,6 +243,10 @@ tasks.named('jar', Jar).configure {
 //    }
 //}
 
+test {
+    useJUnitPlatform()
+}
+
 tasks.withType(JavaCompile).configureEach {
     options.encoding = 'UTF-8' // Use the UTF-8 charset for Java compilation
 }

src/main/java/io/github/meatwo310/tsukichat/util/Converter.java

@@ -13,97 +13,23 @@
 import java.net.URLEncoder;
 import java.nio.charset.StandardCharsets;
 import java.util.Arrays;
-import java.util.LinkedHashMap;
-import java.util.Objects;
 import java.util.regex.Pattern;
 import java.util.stream.Stream;
 
 public class Converter {
-    /**
-     * ローマ字からひらがなへの変換テーブル。
-     * キーにローマ字、値にひらがなと巻き戻り数を持つ
-     */
-    private static final LinkedHashMap<String, String[]> hiraganaMap = new LinkedHashMap<>();
     private static final Logger LOGGER = TsukiChat.LOGGER;
-
-    /**
-     * ひらがな変換テーブルを初期化する
-     * @param resourceName リソース名
-     */
-    private static void initMap(String resourceName) {
-        URL pathToTable = Converter.class.getResource(resourceName);
-
-        if (Objects.isNull(pathToTable)) {
-            System.out.println("Could not find resource: " + resourceName);
-            return;
-        }
-
-        // テーブルを読み込む
-        try (java.io.InputStream stream = pathToTable.openStream()) {
-            BufferedReader reader = new BufferedReader(new InputStreamReader(stream, StandardCharsets.UTF_8));
-            String line;
-            while ((line = reader.readLine()) != null) {
-                if (line.isBlank()) continue;
-                String[] parts = line.split(":"); // 0: ローマ字, 1: ひらがな, 2: 巻き戻り数
-                String romaji = parts[0];
-                String hiragana = parts[1];
-                String back;
-
-                if (parts.length == 2) { // ローマ字+ひらがな
-                    back = "0";
-                } else if (parts.length == 3) { // ローマ字+ひらがな+巻き戻り数
-                    // Int型に変換可能か確認
-                    try {
-                        Integer.parseInt(parts[2]);
-                    } catch (Exception e) {
-                        LOGGER.warn("Could not parse int in {}; This line will be ignored: {}", resourceName, line);
-                        continue;
-                    }
-                    back = parts[2];
-                } else {
-                    LOGGER.warn("Invalid line in {}; This line will be ignored: {}", resourceName, line);
-                    continue;
-                }
-
-                // ローマ字をキーにしてひらがなと巻き戻り数を保存
-                hiraganaMap.put(romaji, new String[]{hiragana, back});
-            }
-        } catch (java.io.IOException e) {
-            throw new RuntimeException(e);
-        }
-    }
-
-    static {
-        initMap("/assets/japaneseromajiconverter/romaji_to_hiragana.txt");
-    }
+    private static final RomajiToHiragana ROMAJI_TO_HIRAGANA = RomajiToHiragana.fromResource(
+            "/assets/japaneseromajiconverter/romaji_to_hiragana.txt",
+            LOGGER::warn
+    );
 
     /**
      * ローマ字をひらがなに変換する
      * @param romaji ローマ字
      * @return ひらがな
      */
     public static String romajiToHiragana(String romaji) {
-        StringBuilder hiragana = new StringBuilder();
-        int i = 0;
-        while (i < romaji.length()) {
-            boolean found = false;
-            //           ↓ MAGIC NUMBER !!!!
-            for (int j = 4; j >= 1; j--) {
-                if (!(i + j <= romaji.length())) continue;
-                String substring = romaji.substring(i, i + j);
-
-                if (!hiraganaMap.containsKey(substring)) continue;
-                hiragana.append(hiraganaMap.get(substring)[0]);
-                i += j + Integer.parseInt(hiraganaMap.get(substring)[1]);
-                found = true;
-                break;
-            }
-            if (!found) {
-                hiragana.append(romaji.charAt(i));
-                i++;
-            }
-        }
-        return hiragana.toString();
+        return ROMAJI_TO_HIRAGANA.convert(romaji);
     }
 
     /**

src/main/java/io/github/meatwo310/tsukichat/util/RomajiToHiragana.java

@@ -0,0 +1,143 @@
+package io.github.meatwo310.tsukichat.util;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
+import java.nio.charset.StandardCharsets;
+import java.util.LinkedHashMap;
+import java.util.Map;
+import java.util.Objects;
+
+/**
+ * ローマ字からひらがなへの変換を行うクラス。
+ * Minecraft / Forge に依存しない純粋なJavaクラス。
+ */
+public class RomajiToHiragana {
+
+    /**
+     * 変換テーブルの1エントリ。
+     * @param hiragana 変換先のひらがな（またはそのまま出力する文字列）
+     * @param backtrack 巻き戻り文字数（通常0、促音などで負の値）
+     */
+    record Entry(String hiragana, int backtrack) {}
+
+    private final Map<String, Entry> conversionTable;
+    private final int maxKeyLength;
+
+    RomajiToHiragana(Map<String, Entry> conversionTable) {
+        this.conversionTable = Map.copyOf(conversionTable);
+        this.maxKeyLength = conversionTable.keySet().stream()
+                .mapToInt(String::length)
+                .max()
+                .orElse(0);
+    }
+
+    /**
+     * リソースファイルから変換テーブルを読み込んでインスタンスを生成する。
+     *
+     * <p>ファイル形式（1行ごと）:</p>
+     * <ul>
+     *   <li>{@code ローマ字:ひらがな} — 巻き戻りなし</li>
+     *   <li>{@code ローマ字:ひらがな:巻き戻り数} — 促音など巻き戻りあり</li>
+     * </ul>
+     *
+     * @param resourcePath クラスパス上のリソースパス
+     * @param logger 警告ログの出力先（nullの場合はログを出力しない）
+     * @return 生成されたインスタンス
+     * @throws IllegalArgumentException リソースが見つからない場合
+     * @throws RuntimeException 読み込み中にIOエラーが発生した場合
+     */
+    public static RomajiToHiragana fromResource(String resourcePath, Logger logger) {
+        InputStream stream = RomajiToHiragana.class.getResourceAsStream(resourcePath);
+        if (stream == null) {
+            throw new IllegalArgumentException("Resource not found: " + resourcePath);
+        }
+
+        LinkedHashMap<String, Entry> table = new LinkedHashMap<>();
+        try (BufferedReader reader = new BufferedReader(new InputStreamReader(stream, StandardCharsets.UTF_8))) {
+            String line;
+            while ((line = reader.readLine()) != null) {
+                if (line.isBlank()) continue;
+                parseLine(line, resourcePath, table, logger);
+            }
+        } catch (IOException e) {
+            throw new RuntimeException("Failed to read resource: " + resourcePath, e);
+        }
+
+        return new RomajiToHiragana(table);
+    }
+
+    /**
+     * ローマ字をひらがなに変換する。
+     * 変換テーブルに一致しない文字はそのまま出力される。
+     *
+     * @param romaji 変換元のローマ字文字列
+     * @return 変換後のひらがな文字列
+     */
+    public String convert(String romaji) {
+        StringBuilder result = new StringBuilder();
+        int pos = 0;
+
+        while (pos < romaji.length()) {
+            Entry matched = null;
+            int matchedLength = 0;
+
+            // 最長一致: 長い候補から順に試す
+            int maxLen = Math.min(maxKeyLength, romaji.length() - pos);
+            for (int len = maxLen; len >= 1; len--) {
+                String candidate = romaji.substring(pos, pos + len);
+                Entry entry = conversionTable.get(candidate);
+                if (entry != null) {
+                    matched = entry;
+                    matchedLength = len;
+                    break;
+                }
+            }
+
+            if (matched != null) {
+                result.append(matched.hiragana());
+                pos += matchedLength + matched.backtrack();
+            } else {
+                // 一致なし: 1文字そのまま出力
+                result.append(romaji.charAt(pos));
+                pos++;
+            }
+        }
+
+        return result.toString();
+    }
+
+    /**
+     * 変換テーブルの1行をパースしてテーブルに追加する。
+     */
+    private static void parseLine(String line, String resourceName,
+                                  Map<String, Entry> table, Logger logger) {
+        String[] parts = line.split(":");
+        if (parts.length == 2) {
+            table.put(parts[0], new Entry(parts[1], 0));
+        } else if (parts.length == 3) {
+            try {
+                int backtrack = Integer.parseInt(parts[2]);
+                table.put(parts[0], new Entry(parts[1], backtrack));
+            } catch (NumberFormatException e) {
+                if (logger != null) {
+                    logger.warn("Could not parse backtrack value in {}; line ignored: {}", resourceName, line);
+                }
+            }
+        } else {
+            if (logger != null) {
+                logger.warn("Invalid line in {}; line ignored: {}", resourceName, line);
+            }
+        }
+    }
+
+    /**
+     * ログ出力用のインターフェース。
+     * slf4j等の具体的なロガーに依存しないようにするための抽象化。
+     */
+    @FunctionalInterface
+    public interface Logger {
+        void warn(String format, Object... args);
+    }
+}