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

!(DI in frontend) #14529

Open
KisaragiEffective opened this issue Sep 9, 2024 · 16 comments
Open

!(DI in frontend) #14529

KisaragiEffective opened this issue Sep 9, 2024 · 16 comments
Labels
💬Discussion Being discussed or needs discussion packages/frontend Client side specific issue/PR

Comments

@KisaragiEffective
Copy link
Collaborator

KisaragiEffective commented Sep 9, 2024

#13929 (comment) でDIをやりたいと聞いたが、DIをフロントエンドに導入するありがたみが見えない。
差し替える機会が乏しい割にメンテナンスコストが高くつきそう。

@KisaragiEffective KisaragiEffective added 💬Discussion Being discussed or needs discussion packages/frontend Client side specific issue/PR labels Sep 9, 2024
@KisaragiEffective
Copy link
Collaborator Author

補足: タイトルは仮

@syuilo
Copy link
Member

syuilo commented Sep 9, 2024

テストがより簡単に行えるようになるし、設計もきれいになるからメンテナンスコストはむしろ下がる

@Sayamame-beans
Copy link
Member

(多分13929)

@syuilo
Copy link
Member

syuilo commented Sep 9, 2024

現在のfrontendはいろいろなところが密結合になっておりメンテナンス性が著しく低い

@KisaragiEffective
Copy link
Collaborator Author

モジュールの切り方が悪いだけであって別にDIなくても解決できる可能性はある

@samunohito
Copy link
Member

off-topic:
仮にもしやるのであれば…埋め込みの正式リリース後にして頂けると嬉しいかもです。

というのも、埋め込みは埋め込みで巨大な対応となっており、不具合の切り分けなどのコストが上がるため。
まずは、埋め込みが安定した状態で正式リリースを打って、ユーザ各位にお届けしたい気持ちがあります。

@KisaragiEffective
Copy link
Collaborator Author

$i程度ならまぁという気もする
ただそれ以外は疑わしいと思う

@kakkokari-gtyih
Copy link
Contributor

(かんけいない)一番上のコメントの出処編集して直しておきました

@KisaragiEffective
Copy link
Collaborator Author

(かんけいない)一番上のコメントの出処編集して直しておきました

🙏

@kakkokari-gtyih
Copy link
Contributor

off-topic: 仮にもしやるのであれば…埋め込みの正式リリース後にして頂けると嬉しいかもです。

というのも、埋め込みは埋め込みで巨大な対応となっており、不具合の切り分けなどのコストが上がるため。 まずは、埋め込みが安定した状態で正式リリースを打って、ユーザ各位にお届けしたい気持ちがあります。

これはおもっている

@syuilo
Copy link
Member

syuilo commented Sep 14, 2024

やるとしたら3ヶ月くらいはかかりそう

@syuilo
Copy link
Member

syuilo commented Sep 14, 2024

DIすること自体はリファクタリングとして当然やった方がよくて、議論の余地があるのはDIコンテナを入れるべきか・入れるとしたら何か になりそう

@kakkokari-gtyih
Copy link
Contributor

モジュールの切り方が悪いだけであって別にDIなくても解決できる可能性はある

これは?

@syuilo
Copy link
Member

syuilo commented Sep 14, 2024

その可能性はなさそう
仮にあったとしても普通にDIした方が良さそう

@samunohito
Copy link
Member

別にDIなくても解決できる可能性はある

なモジュールも出てくるし、

仮にあったとしても普通にDIした方が良さそう

なモジュールも出てくるというだけだと思います。
DIは設計指針の一つでしかないので、関心ごとを分離したほうがそうするし、必要が無ければしないだけかと…

@samunohito
Copy link
Member

(ほんとうにやる余地はあるのかという点は調べないとわからないけれども)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💬Discussion Being discussed or needs discussion packages/frontend Client side specific issue/PR
Projects
None yet
Development

No branches or pull requests

5 participants