需要帮助创建创建控制器以从数据库中删除项目

Question

大家晚上好！我一直在学习如何在 .NET 中编程的教程。本教程介绍了如何添加项目，但不包括删除。这是我的代码：

服务层

    public async Task Delete(int locationId)
    {
        var location = _context.Locations.Where(l => l.Id == locationId);
        _context.Remove(location);
        await _context.SaveChangesAsync();
    }

控制器

    public IActionResult Delete(int id)
    {
        _locationService.Delete(id);
        return RedirectToAction("Index", "Location");
    }

查看

<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>

我的按钮点击并重定向回我的位置索引视图。但是，它不会按预期从数据库中删除该位置。我知道问题一定出在控制器上。

如果这对将来的某人有所帮助，我将代码更改为使其正常工作： 服务层

        public async Task Delete(int locationId)
    {
        var location = _context.Locations.SingleOrDefault(l => l.Id == locationId);
        _context.Remove(location);
        await _context.SaveChangesAsync();
    }

控制器

public async Task <IActionResult> Delete(int id)
    {
        await _locationService.Delete(id);
        return RedirectToAction("Index", "Location");
    }

Answer 1

问题在于where 方法！它返回一个集合，并且 EF 的 Remove 方法不会抱怨它，因为它需要一个对象。尝试使用 FirstOrDefault 或 SingleOrDefault 或 Find 方法更改它，你会很高兴的。

public async Task Delete(int locationId)
{
    var location = _context.Locations.FirstOrDefault(l => l.Id == locationId);
    _context.Remove(location);
    await _context.SaveChangesAsync();
}

但是，如果您尝试使用Generic 版本，您肯定会立即注意到问题。更好的方法是：

//EF Core 
var item = await dbContext.FindAsync<Location>(locationId);
dbContext.Remove<Location>(item);
await dbContext.SaveChangesAsync();