ASP.NET WebAPI控制器继承引发AmbiguousMatchException问题求助

问题根源 #

你遇到的AmbiguousMatchException是因为ASP.NET Core路由系统会把基类控制器的公共Action方法也视为派生控制器的可用Action。当用new关键字定义派生类的Get方法时，派生控制器实际存在两个GET Action：一个是继承自基类的IEnumerable<Document> Get()，另一个是派生类自身的IEnumerable<Invoice> Get()，两者的HTTP方法和路由模板完全一致，导致路由匹配冲突。

可行解决方案 #

方案1：提取公共逻辑到服务类，取消控制器继承（推荐）

控制器继承容易引发路由与职责混淆，更优雅的做法是将文档/发票的查询逻辑抽离到独立服务中，让两个控制器各自独立：

// 公共服务类，封装业务逻辑
public class DocumentService
{
    public IEnumerable<Document> GetAllDocuments()
    {
        // 实现获取所有文档的逻辑
    }

    public IEnumerable<Invoice> GetAllInvoices()
    {
        // 实现获取所有发票的逻辑
    }
}

// 文档控制器
[ApiController]
[Route("Documents")]
public class DocumentController : ControllerBase
{
    private readonly DocumentService _documentService;

    public DocumentController(DocumentService documentService)
    {
        _documentService = documentService;
    }

    [HttpGet]
    public IEnumerable<Document> Get()
    {
        return _documentService.GetAllDocuments();
    }
}

// 发票控制器
[ApiController]
[Route("Documents/Invoices")]
public class InvoiceController : ControllerBase
{
    private readonly DocumentService _documentService;

    public InvoiceController(DocumentService documentService)
    {
        _documentService = documentService;
    }

    [HttpGet]
    public IEnumerable<Invoice> Get()
    {
        return _documentService.GetAllInvoices();
    }
}

这种方式符合单一职责原则，彻底规避路由冲突，也更利于后续维护。

方案2：在派生控制器中标记基类Action为非路由可用

如果必须保留控制器继承，可以在派生控制器中重写基类的Get方法，并添加[NonAction]属性，让路由系统忽略这个继承来的Action：

[ApiController]
[Route("Documents/Invoices")]
public class InvoiceController : DocumentController
{
    // 标记基类继承的Get方法为非Action，路由系统会忽略它
    [NonAction]
    public override IEnumerable<Document> Get()
    {
        // 此处可返回base.Get()或直接抛出异常，不会被路由调用
        throw new NotImplementedException();
    }

    [HttpGet]
    public new IEnumerable<Invoice> Get()
    {
        // 实现获取发票列表的逻辑
    }
}

这样派生控制器只有自身的Get方法会被路由系统识别，访问Documents/Invoices时就不会出现冲突。

方案3：给Action指定明确的路由模板

通过给基类和派生类的Get方法分别指定唯一的路由模板，让路由系统能够区分：

// 基类控制器
[ApiController]
[Route("Documents")]
public class DocumentController : ControllerBase
{
    // 明确指定路由为当前控制器的根路径
    [HttpGet("")]
    public virtual IEnumerable<Document> Get()
    {
        // 实现逻辑
    }
}

// 派生类控制器
[ApiController]
[Route("Documents/Invoices")]
public class InvoiceController : DocumentController
{
    // 指定当前控制器路由的根路径，用new隐藏基类方法
    [HttpGet("")]
    public new IEnumerable<Invoice> Get()
    {
        // 实现逻辑
    }
}

这种方式确保两个Action的路由模板在各自控制器前缀下唯一，也能解决冲突问题。

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