search

misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.95k stars 1.35k forks source link

!(DI in frontend) #14529

Open KisaragiEffective opened 1 month ago

KisaragiEffective commented 1 month ago

https://github.com/misskey-dev/misskey/pull/13929#issuecomment-2322826495 でDIをやりたいと聞いたが、DIをフロントエンドに導入するありがたみが見えない。 差し替える機会が乏しい割にメンテナンスコストが高くつきそう。

KisaragiEffective commented 1 month ago

補足: タイトルは仮

syuilo commented 1 month ago

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

Sayamame-beans commented 1 month ago

(多分13929)

syuilo commented 1 month ago

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

KisaragiEffective commented 1 month ago

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

samunohito commented 1 month ago

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

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

KisaragiEffective commented 1 month ago

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

kakkokari-gtyih commented 1 month ago

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

KisaragiEffective commented 1 month ago

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

🙏

kakkokari-gtyih commented 4 weeks ago

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

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

これはおもっている

syuilo commented 4 weeks ago

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

syuilo commented 4 weeks ago

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

kakkokari-gtyih commented 4 weeks ago

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

これは？

syuilo commented 4 weeks ago

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

samunohito commented 4 weeks ago

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

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

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

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

samunohito commented 4 weeks ago

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