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

Packedなobjectのtype定義を用意して使用する #4018

Closed
mei23 opened this issue Jan 27, 2019 · 4 comments · Fixed by #4572
Closed

Packedなobjectのtype定義を用意して使用する #4018

mei23 opened this issue Jan 27, 2019 · 4 comments · Fixed by #4572
Labels
💚Refactor Rewriting code without changing behavior

Comments

@mei23
Copy link
Contributor

mei23 commented Jan 27, 2019

Summary

現状、Pack (API向けにNote等のEntitiyを返している部分) はanyで回しているが
これをIPackedNoteのような専用のtypeにする。
→ そのtypeはAPIドキュメントに記述するEntityと一緒になるので、APIドキュメント保守しやすくなることを期待。

現状、PackではDBのModelから不要なものを削除しているが、
これを必要なプロパティのみIPackedNoteのようなものにコピーするようにする。
→ 不要なものを削除するという方法だと、消し忘れた時に漏洩するリスクがある。
→ 不要なものを削除→条件に一致しない場合は削除→NOTやORだらけになって感覚的にわかりづらくなる。

Environment


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@mei23 mei23 added the 💚Refactor Rewriting code without changing behavior label Jan 27, 2019
@acid-chicken
Copy link
Member

Related to #3622

@acid-chicken
Copy link
Member

現状DBのインターフェースに刺さってる子達の名前をIHogeからIHogeModelに改名して、IHogeをPackedなインターフェースに置き換えるのはどうでしょうか。

@syuilo
Copy link
Member

syuilo commented Feb 1, 2019

一度Packしたものはすぐレスポンスとして返す場合がほとんどで、サーバー内でPackしたものの型情報を参照することはDBから取得した生の状態のものと比べると少ないので、型を定義するとしたらめいめいが言及したように新たに IPackedNote のようにするのがいいと思ってます

@acid-chicken
Copy link
Member

ではその方向で。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💚Refactor Rewriting code without changing behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants