ASP.NETコアアプリケーションでDependency Injectionを実装中にInvalidOperationExceptionが発生しました

Question

私は依存性注入について学んでいるので、私も理解しました（あなたがそうする気持ちがあればどこでも修正してください）。次のサンプルのコンセプトは、依存関係注入を使用する利点を確認することです。具体的な定義（クラス）の傾向がある場合に、将来変わる。

IEmailService - インタフェース：

public interface IEmailService 
{ 
    void SendMail(); 
} 

EmailService - クラス

public class EmailService : IEmailService 
{ 
    public EmailService(string emailFrom, string emailTo) 
    { 

    } 

    public void SendMail() 
    { 
     // Code here 
    } 
} 

インターフェース上に継承にHomeController

public class HomeController : Controller 
{ 
    private IEmailService _service; 

    public HomeController(IEmailService service) 
    { 
     _service = service; 
    } 

    public IActionResult Index() 
    { 
     _service.SendMail(); 
     return View(); 
    } 
} 

Startup.cs

public void ConfigureServices(IServiceCollection services) 
{ 
    // Add framework services. 
    ... 
    services.AddTransient<IEmailService, EmailService>(); 
    ... 
} 

実践的な仮定

は、私が以前EmailServiceクラスには、パラメータ化コンストラクタがなかったと仮定し、私は追加する必要があるように、将来的に、私は感じて（HomeControllerのような）抽象化（インタフェース）を使用してthにアクセスするコントローラに影響を与えるべきではありません間接的に。

残念ながら、上記のコードを実行しているときに、パラメータ化されたコンストラクタをEmailServiceクラスから削除すると消えるような次の例外が発生しています。

InvalidOperationException： 'DependencyInjectionDemo.Services.EmailService'をアクティブ化しようとしているときに、 'System.String'タイプのサービスを解決できません。

Answer 1

あなたは、ラムダを使用してEmailService登録することができます。

services.AddTransient<IEmailService>(_ => new EmailService("from@", "to@")); 

emailFromとemailToしかしコントローラがIEmailServiceにこの情報を提供する責任があるかもしれないことを意味し、ランタイムデータに見えます。 EmailServiceはコントローラーからのデカップリングされているため、コントローラーがその作成を担当していないことを意味します。一般的に

、あなたはhereが説明したように、実行時のデータを使用してコンポーネント（あなたのケースでEmailService）を初期化する必要が防ぐ必要があり、アドバイスは次のとおりです。

時にアプリケーションコンポーネントにランタイムデータを注入しないでください。建設;あいまいさを引き起こし、余分な責任で構成ルートを複雑にし、DI構成の正確性を確認することが非常に困難になります。私のアドバイスは、実行時データが、構築されたオブジェクトグラフのメソッド呼び出しを流れるようにすることです。あなたのケースでは

これは、基本的には次のIEmailService抽象化を変更することを意味：

public interface IEmailService 
{ 
    void SendMail(string emailFrom, string emailTo); 
}