ASP.NET Core Web API能否添加非CRUD操作？以删除文件为例

可以在ASP.NET Core Web API中添加删除文件这类非CRUD操作，但从RESTful设计规范和代码职责划分的角度来看，你当前的实现方式存在明显不合理之处，具体分析和优化建议如下：

现有实现的问题 #

• 违反RESTful语义：HttpGet请求的设计初衷是幂等、只读操作，仅用于获取资源，不应该修改服务器状态（比如删除文件）。重复调用该接口会重复执行删除逻辑，引发非预期的副作用。
• 职责边界混乱：GetById接口的核心职责是查询Issue数据，删除文件的逻辑属于修改操作，两者耦合在一起会导致代码维护难度提升，后续扩展时容易引发问题。
• 代码语法错误：你的DeleteFile方法缺少返回值类型，正确定义应为public void DeleteFile(string myPath)；路径字符串中的反斜杠未转义，需改为@"C:\New\amit.txt"或"C:\\New\\amit.txt"，否则会被识别为转义字符导致路径无效。

优化方案 #

方案1：拆分操作，单独提供文件删除接口（推荐） #

遵循RESTful规范，将文件删除逻辑拆分到独立的HttpDelete接口中，明确职责边界：

[HttpDelete("files/{fileName}")]
[ProducesResponseType(StatusCodes.Status204NoContent)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
public IActionResult DeleteFile(string fileName)
{
    var filePath = Path.Combine(@"C:\New", fileName);
    if (File.Exists(filePath))
    {
        File.Delete(filePath);
        return NoContent();
    }
    return NotFound();
}

方案2：若业务需绑定查询与删除（不推荐），确保操作幂等性 #

如果业务逻辑确实要求查询Issue后触发文件删除，需通过数据库标记避免重复执行删除操作，保证接口幂等：

[HttpGet("{id}")]
[ProducesResponseType(typeof(Issue), StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
public async Task<IActionResult> GetById(int id)
{
    var issue = await _context.Issues.FindAsync(id);
    if (issue == null)
    {
        return NotFound();
    }

    // 检查是否已执行过文件删除，避免重复操作
    if (!issue.HasFileDeleted)
    {
        var filePath = @"C:\New\amit.txt";
        if (File.Exists(filePath))
        {
            File.Delete(filePath);
            // 更新数据库标记
            issue.HasFileDeleted = true;
            await _context.SaveChangesAsync();
        }
    }

    return Ok(issue);
}

同时修正DeleteFile方法的语法错误：

public void DeleteFile(string myPath) 
{
    if (File.Exists(myPath))
    {
        File.Delete(myPath);
    }
}

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