Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ユーザー辞書を読み込みエラー時はnilでもつようにする #88

Merged
merged 5 commits into from
Jan 13, 2024

Conversation

mtgto
Copy link
Owner

@mtgto mtgto commented Jan 9, 2024

ユーザー辞書は手で編集することが想定されるため正しくないエントリが生じることが考えられます。
そのようなエントリが正しく読めなかったことをユーザーに伝え、辞書の読み込みエラー状態を持てるようにするためにユーザー辞書の型をFileDictからFileDict?に変更します。

それに加えて設定画面の辞書の画面でどの辞書で何行読めなかった行があるかを表示するようにします(下のスクショ)。
また何行目で読み込みエラーになったかをログに出すようにします。

image

ほんとうはこのPull Requestで読み込みエラーになったときは正常な辞書として扱わない、としようと思ったんですが、https://skk-dev.github.io/dict/ で配布しているSKK-JISYO.Lはスラッシュで終わってないエントリがあるためこのPull Requestではロギングだけに留めて見送ります。

skk-dev/dictのmasterブランチでは直っているようですが、gh-pagesブランチに取り込まれてないのが壊れたSKK-JISYO.Lが配布されている原因みたいです。
skk-dev/dict#43

また別の辞書でBOMありのテキストファイルのBOMを無視できてなさそうなことにも気付いたので別途対応します。

@mtgto
Copy link
Owner Author

mtgto commented Jan 10, 2024

ユーザー辞書で読み込みエラーがある場合は通知センターに通知して読み込みしてない状態にしようかな。
そうしないと読み込めた行だけがシリアライズされて上書きされてしまう。

@mtgto mtgto merged commit b347b1b into main Jan 13, 2024
2 checks passed
@mtgto mtgto deleted the read-error-user-dict branch January 13, 2024 03:05
@mtgto
Copy link
Owner Author

mtgto commented Jan 13, 2024

ユーザー辞書で読み込みエラーがある場合は通知センターに通知して読み込みしてない状態にしようかな。
そうしないと読み込めた行だけがシリアライズされて上書きされてしまう。

このPull Requestとは別でやります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant