ASP.NET MVC：控制器能否调用其他控制器实现关联数据删除？

首先明确：不要直接调用其他控制器的Action，这种做法不符合MVC职责划分，还会带来请求重复、原子性无法保证等问题。正确的思路是把关联数据的删除逻辑集中处理，以下是几种可行方案：

方案1：直接在DeleteConfirmed中批量删除关联数据 #

在删除主数据集的同一个方法里，直接操作其他表的DbSet，一次性删除所有关联数据后统一提交，保证操作原子性（要么全删，要么全不删）。

修改DeleteConfirmed方法：

// POST: Datasets/Delete/5
[HttpPost, ActionName("Delete")]
[ValidateAntiForgeryToken]
public async Task<IActionResult> DeleteConfirmed(string id)
{
    // 删除主数据集
    var datasets = await _context.Datasets.FindAsync(id);
    if (datasets != null)
    {
        _context.Datasets.Remove(datasets);
    }

    // 删除关联的ATS表数据（替换为你的实际表名）
    var atsRecords = await _context.ATSRecords.Where(r => r.dataset == id).ToListAsync();
    _context.ATSRecords.RemoveRange(atsRecords);

    // 依次处理剩下4个表，示例：
    // var xxxRecords = await _context.XxxRecords.Where(r => r.dataset == id).ToListAsync();
    // _context.XxxRecords.RemoveRange(xxxRecords);
    // ...其他表同理

    // 统一提交所有删除操作
    await _context.SaveChangesAsync();

    return RedirectToAction(nameof(Index));
}

方案2：抽离删除逻辑到服务层（推荐解耦） #

如果后续业务逻辑可能变化，或想让控制器更简洁，可把删除主数据集+关联数据的逻辑抽成独立服务：

1. 创建服务接口与实现类：

public interface IDatasetService
{
    Task DeleteDatasetAndRelatedData(string datasetId);
}

public class DatasetService : IDatasetService
{
    private readonly YourDbContext _dbContext;

    public DatasetService(YourDbContext dbContext)
    {
        _dbContext = dbContext;
    }

    public async Task DeleteDatasetAndRelatedData(string datasetId)
    {
        // 删除主数据集
        var dataset = await _dbContext.Datasets.FindAsync(datasetId);
        if (dataset != null)
        {
            _dbContext.Datasets.Remove(dataset);
        }

        // 批量删除所有关联表数据
        _dbContext.ATSRecords.RemoveRange(_dbContext.ATSRecords.Where(r => r.dataset == datasetId));
        _dbContext.XxxRecords.RemoveRange(_dbContext.XxxRecords.Where(r => r.dataset == datasetId));
        // ...其他3个表同理

        await _dbContext.SaveChangesAsync();
    }
}

2. 在Startup/Program.cs中注册服务：

// .NET 6+
builder.Services.AddScoped<IDatasetService, DatasetService>();

3. 在DatasetsController中注入并调用：

private readonly IDatasetService _datasetService;

public DatasetsController(IDatasetService datasetService)
{
    _datasetService = datasetService;
}

[HttpPost, ActionName("Delete")]
[ValidateAntiForgeryToken]
public async Task<IActionResult> DeleteConfirmed(string id)
{
    await _datasetService.DeleteDatasetAndRelatedData(id);
    return RedirectToAction(nameof(Index));
}

方案3：数据库级联删除（最省心，需修改数据库结构） #

如果业务允许且可修改数据库模型，给其他表的dataset字段配置外键+级联删除，删除主数据集时数据库会自动清理关联数据：

1. 修改关联表模型（以ATSRecord为例）：

public class ATSRecord
{
    // 其他字段...
    public string dataset { get; set; }

    // 添加导航属性
    [ForeignKey("dataset")]
    public Datasets Dataset { get; set; }
}

2. 在DbContext的OnModelCreating中配置级联删除：

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    modelBuilder.Entity<ATSRecord>()
        .HasOne(r => r.Dataset)
        .WithMany() // 若Datasets无对应导航属性，使用WithMany()
        .HasForeignKey(r => r.dataset)
        .OnDelete(DeleteBehavior.Cascade);

    // 其他4个表同理配置级联删除
}

配置完成后，原DeleteConfirmed方法无需修改，删除主数据集时数据库会自动处理关联数据。

为什么你之前的RedirectToAction没用？ #

RedirectToAction的参数顺序错误，正确写法是RedirectToAction("ActionName", "ControllerNameWithoutSuffix", 参数)，比如RedirectToAction("Delete", "ATS", new { id = id })，但这种方式是发起新的GET请求，会导致多次HTTP请求，且无法保证所有删除操作的原子性（比如某请求失败，其他已执行），完全不适合批量关联删除场景。

内容的提问来源于stack exchange，提问作者tj26