Conversation
| export const engineIdSchema = z.string().uuid().brand<"EngineId">(); | ||
| export type EngineId = z.infer<typeof engineIdSchema>; | ||
| export const EngineId = (id: string): EngineId => engineIdSchema.parse(id); |
There was a problem hiding this comment.
ここがEngineIdを型定義している場所です。
|
が入ったあとに修正してマージが良さそう？ |
|
#1092 が入ったmainをマージしました･･･！ |
| @@ -88,16 +89,19 @@ const engineInfos = computed( | |||
| () => | |||
| new Map( | |||
| Object.entries(store.state.characterInfos).map( | |||
There was a problem hiding this comment.
型見る感じ単純なstringとして扱ってそうに見えたけれども、ここの変更は意図通りですか？
あと実際に動かしていないのですが、symbolを付けたstring（すなわちEngineIdですが）を再度EngineIdコンストラクタみたいなのを通すとどうなるんですか？（って質問しようとしてコード見たらzodでstringとして扱ってみたいな感じだったから、単純なtoString()が呼ばれて再度symbol付け直す感じなのかな…
There was a problem hiding this comment.
単純なstringとして扱ってそう
これってどこのことでしょうか 👀
engineIdStrであればEngineIdでラップしてるはず･･･？
再度EngineIdコンストラクタみたいなのを通すと
試してみた感じ同じ型になってそうでした！！
There was a problem hiding this comment.
characterInfosのkeyがstringとして扱われてるのですよね
EngineIdがkeyなのであれば、これは型として表現した方がいいなと思ったのでした（ということで、types.tsの方の変更requestです）
コメントを付けた箇所は、よくわからないstringをEngineIdとして扱っているわけではなさそうなので、ここに関しては変更不要です（Object.entriesの弊害がまた出てしまったみたいな顔をしながら
There was a problem hiding this comment.
あ、なるほどです！ほんとですね！！
stringに代入できてしまうんですね･･･。
EngineIdになってなかったRecordがいくつかあったので一緒に直してみました！！
|
レビューありがとうございます！！！ |
内容
EngineIdをただのstringからユニークな型（brand型）に変更します。
これによりRecordのキーなどの変数が厳密になり、コーディングミスを失くしやすくなるはず･･･！
関連 Issue
その他
方針としては、string型になっていたところを全てEngineIdにしています。
またzod schemaが必要なとろろはengineIdSchemaを指定しています。
一部undefinableにする必要があり、throw Errorするロジック周りがたされていたりします。