ray-di / ray-di.github.io

Ray.Di user guide
https://ray-di.github.io/
6 stars 14 forks source link

improve description of Setter Injection in Injections.md #27

Closed 77web closed 2 years ago

77web commented 2 years ago

To clarify the meaning and improve DeepL translation :grin:

before

Ray.Diは `#[Inject]` 属性を持つメソッドをインジェクトすることができます。依存関係はパラメータの形式をとり、インジェクタはメソッドを呼び出す前にそれを解決します。注入されるメソッドは任意の数のパラメータを持つことができ、メソッド名は注入に影響を与えません。

after

Ray.Diは `#[Inject]` 属性を持つメソッドを通して依存関係を注入することができます。依存関係はパラメータの形式をとり、インジェクタはメソッドを呼び出す前にそれを解決します。注入されるメソッドは任意の数のパラメータを持つことができ、メソッド名は注入に影響を与えません。
77web commented 2 years ago

@koriym please review :pray:

koriym commented 2 years ago

良さそうですが、オリジナルを校正でいいのかという疑問も https://github.com/google/guice/wiki/Injections#method-injection

いやしかし、こっちが正しいですよね。

cc/ @NaokiTsuchiya

koriym commented 2 years ago

"Guice can be injected "in" methods" ?

NaokiTsuchiya commented 2 years ago

期待する文章としては、 Ray.Di は `#[Inject]` 属性を持つメソッドに依存を注入します のような感じですよね。

オリジナル側はたしかに変な感じで、 Guice can inject dependencies into methods のような感じになるんでしょうか

koriym commented 2 years ago

Ray.Di can inject by methods with the #[Inject] attribute.

じゃないでしょうか?