Skip to content

TypeMap.Entryのfactoryメソッドを提供する#48

Merged
omochi merged 1 commit intomainfrom
typemap-static-func
Dec 6, 2022
Merged

TypeMap.Entryのfactoryメソッドを提供する#48
omochi merged 1 commit intomainfrom
typemap-static-func

Conversation

@omochi
Copy link
Owner

@omochi omochi commented Dec 6, 2022

ユーザーは .identity(...).coding(...) を使って構築します。
coding においては jsonType は必須に変わりました。
自動で用意される Foo_JSON 型が自明でなくてあまり嬉しくないため。

Dateのテストケースは、 jsonType として string を使うように書き換えました。

enum での実装も検討しました。
#47

associated valueを抽出するところで、_ を並べないといけなくて、
associated valueが増えたら全箇所修正が必要なところがしっくり来ないのでやめました。

computed propertyを定義しない場合は、
利用側のTypeMapConverterで直接分岐する場合、
同じ分岐を重複して出てしまうのが微妙でした。

@omochi omochi marked this pull request as ready for review December 6, 2022 09:27
@omochi omochi merged commit d0c7f2a into main Dec 6, 2022
@omochi omochi deleted the typemap-static-func branch December 6, 2022 09:27
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