WEBAPI、マイクロソフトの記事を読んだが、エラー

Question

が含まれています、私はこの記事を読んでいます：

https://docs.asp.net/en/latest/tutorials/first-web-api.htmlを。

と言われています。

using System.Collections.Generic; 
using Microsoft.AspNetCore.Mvc; 
using TodoApi.Models; 

    namespace TodoApi.Controllers 
    { 
     [Route("api/[controller]")] 
     public class TodoController : Controller 
     { 
      public TodoController(ITodoRepository todoItems) 
      { 
       TodoItems = todoItems; 
      } 
     } 
    } 

しかしTodoItemsが宣言されて：あなたはこれを置く必要がありますか？

何か問題がありますか？それとも、記事で間違っていますか？

私はファイルでこれを置く場合ので：

public ITodoRepository TodoItems { get; private set; }   

エラーが表示されなくなります。しかしこれは正しいのですか？

Answer 1

おそらく間違いでしょう。

public class TodoController : Controller 
    { 
     private readonly ITodoRepository TodoItems; 

     public TodoController(ITodoRepository todoItems) 
     { 
      TodoItems = todoItems; 
     } 
    } 

ここでは、コンストラクタによってdependency injectionを使用しています。
TodoItemsをpublicというプロパティとして宣言するのは良い考えではないと思います。 private readonlyフィールドはコンストラクタでのみ初期化されます。それ以外の場合は、継承を使用する場合はprotectedと宣言される可能性があります。