Trying to remove item based on ID in EF Core

My goal is to remove an item based on the ID. Here is the code I currently have:

public async Task<IActionResult> RemoveTodoItem(int id)
{
    // TODO
    // Use EFCore to remove the item based on id

    var todoItem = await _context.TodoItems.FindAsync(id);

    if (todoItem == null)
    {
        return NotFound();
    }

    _context.TodoItems.Remove(todoItem);
    await _context.SaveChangesAsync();

    return NoContent();
}

My issue is that TodoItems keeps returning the error code CS1061, saying TodoContext does not contain a definition of TodoItems.

Here is the code I have in TodoContext:

public class TodoContext: DbContext
{
    public DbSet<Todo> Todos { get; set; }

    public TodoContext(DbContextOptions<TodoContext> options) : base(options) {   }
}

Todo class:

public class Todo
{
    public int Id { get; set; }
    public string? Name { get; set; }
    public bool IsComplete { get; set; }
}

How could I fix this error?

I'm not sure if this has to do with dependency injection? I believe the database has already been injected to the controller.

> My issue is that TodoItems keeps returning the error code CS1061.
> Saying TodoContext does not contain a definition of TodoItems

Actually, you might get the error for numerous reasons. For instance, if you don't initialize the TodoContext while your controller being called. Even, if you write the name with typos.

In additional, if you don't set primary key on your table or POCO class or if correct dbContext has not been passed while initialize the builder.Services.AddDbContext within your program.cs file these also can cause the issue.

> I'm not sure if this has to do with dependency injection? I believe
> the database has already been injected to the controller.

Although, you haven't shared your full controller class along with your constructor. Thus, I have tried following way along with your code snippet and its working as expected. Please have a look if you have implemented the same way.

Model:

public class Todo
{
    [Key]
    public int Id { get; set; }

    public string? Name { get; set; }

    public bool IsComplete { get; set; }
}

DbContext:

public class TodoContext: DbContext
{
    public TodoContext(DbContextOptions<TodoContext> options) : base(options)
    {
    }
       
    public DbSet<Todo> TodoItems { get; set; }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Todo>().ToTable("TodoItems");
    }
}

Note: I have set the table name as TodoItems in database.

Program.cs file:

var connectionString = builder.Configuration.GetConnectionString("DefaultConnection");
builder.Services.AddDbContext<TodoContext>(x => x.UseSqlServer(connectionString));

Note: Be sure, you have configure your database and server name in your appsettings.json file.

Controller:

public class TodoController : Controller
{
    private readonly TodoContext _context;

    public TodoController(TodoContext context)
    {
        _context = context;
    }
       
    [HttpDelete]
    public async Task<IActionResult> RemoveTodoItem(int id)
    {
        var todoItem = await _context.TodoItems.FindAsync(id);
        _context.TodoItems.Remove(todoItem);
        await _context.SaveChangesAsync();
        return NoContent();
    }
}

Note: Make sure you have initialize the TodoContext within your controller constructor. Apparently, the error pointing this or please check if you have type the name correctly.

Output:

Note: If you would like to know more details on Delete or Removing items you could check our official document here