Need help creating Creating Controller to Delete item from database - entity-framework

Question

Good evening everyone! I have been following along with a tutorial to learn how to program in .NET. The tutorial goes over how to add an item but it doesn't cover deletion. Here is my code:
Service Layer
public async Task Delete(int locationId)
{
var location = _context.Locations.Where(l => l.Id == locationId);
_context.Remove(location);
await _context.SaveChangesAsync();
}
Controller
public IActionResult Delete(int id)
{
_locationService.Delete(id);
return RedirectToAction("Index", "Location");
}
View
<table class="table table-hover" id="locationIndexTable">
<thead>
<tr>
<th>Name</th>
<th>Address</th>
<th>Phone Number</th>
<th>Delete Location</th>
</tr>
</thead>
<tbody>
#foreach (var location in Model.LocationList)
{
<tr>
<td>
<a asp-controller="Location" asp-action="Detail" asp-route-id="#location.Id">
#location.Name
</a>
</td>
<td>
#location.Address
</td>
<td>
#location.PhoneNumber
</td>
<td>
<a asp-controller="Location" asp-action="Delete" asp-route-id="#location.Id" class="btn btn-sm btn-primary">TODO Delete</a>
</td>
</tr>
}
</tbody>
My button clicks through and redirects back to my location index view. However it does not delete the location from the database as intended. I know the issue must lie with the controller.
In the off chance that this will help someone in the future here is what I changed my code to to get it to work:
Service Layer
public async Task Delete(int locationId)
{
var location = _context.Locations.SingleOrDefault(l => l.Id == locationId);
_context.Remove(location);
await _context.SaveChangesAsync();
}
Controller
public async Task <IActionResult> Delete(int id)
{
await _locationService.Delete(id);
return RedirectToAction("Index", "Location");
}

Answer 1

The problem is because of the where method! It returns a collection and EF's Remove method does not complain about it as it takes an object. Try changing that with FirstOrDefault or SingleOrDefault or the Find method and you will be good to go.
public async Task Delete(int locationId)
{
var location = _context.Locations.FirstOrDefault(l => l.Id == locationId);
_context.Remove(location);
await _context.SaveChangesAsync();
}
However if you tried using the Generic version, you would surely notice the problem immediately. A better way would be to to:
//EF Core
var item = await dbContext.FindAsync<Location>(locationId);
dbContext.Remove<Location>(item);
await dbContext.SaveChangesAsync();