使用HTTP Patch更新车辆状态却意外删除数据的问题求助

核心原因分析 #

1. 参数传递方式不匹配
   前端通过请求Body传递statusId，但后端UpdateStatus接口是从**URL查询参数（Query）**获取状态值，导致后端无法正确接收你要设置的Canceled状态。如果CarStatusResourceParameters类中Status属性的默认值被设为CarStatus.Delete，就会触发删除车辆的逻辑。

2. 状态值格式不兼容
   前端传递的是数字989，但后端期望的是CarStatus枚举类型（如字符串"Canceled"），两者格式不匹配，进一步导致后端无法识别正确的状态指令。

3. 潜在路由匹配风险
   前端请求路径末尾带斜杠/car/${id}/status/，虽然ASP.NET Core默认忽略尾斜杠，但极端情况下可能引发路由匹配异常，导致请求误匹配到其他接口（不过当前代码中UpdateStatus路由更具体，此概率较低）。

解决办法 #

方案一：调整前端请求，适配后端查询参数要求 #

修改前端请求，将状态值以查询参数形式传递，无需发送Body：

const handleCancelTask = async () => {
    try {
        setSubmitting(true);
        const as = await asStorage.getItem();
        const app = await client.tokenClient();

        // 直接通过查询参数传递Canceled状态
        const response = await app!.patch(
            `/car/${id}/status?Status=Canceled`,
        );

        if (response.status === 200) {
            console.log('Updated car status');
        } else {
            console.log('Failed to update car status');
        }
    } catch (error) {
        console.log({ error });
    } finally {
        setSubmitting(false); // 确保操作结束后重置按钮状态
    }
};

方案二：修改后端接口，适配前端Body传递方式 #

如果希望保持前端Body传参的方式，修改后端接口从Body接收状态ID，并添加映射逻辑：

// 新增接收前端参数的DTO
public class UpdateStatusDto
{
    public int StatusId { get; set; }
}

// 修改UpdateStatus接口
[ProducesResponseType(StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status400BadRequest)]
[HttpPatch, Route("{id}/status")]
public async Task<IActionResult> UpdateStatus([FromRoute] string id, [FromBody] UpdateStatusDto dto)
{
    await _authMethods.VerifyHasPermissionForCar(id, Create);
    
    // 根据前端传递的statusId映射到对应的枚举
    CarStatus targetStatus = dto.StatusId switch
    {
        989 => CarStatus.Canceled,
        // 补充其他状态ID与枚举的映射
        _ => CarStatus.Invalid
    };

    if (targetStatus == CarStatus.Invalid)
    {
        return BadRequest("Invalid statusId");
    }

    switch (targetStatus)
    {
        case CarStatus.Delete:
            await _carMethods.UpdateStatus(id, TaskStatusAsNumeric.DELETED);
            break;
        case CarStatus.Completed:
            await _carMethods.UpdateStatus(id, TaskStatusAsNumeric.COMPLETED);
            break;
        case CarStatus.Canceled:
            await _carMethods.UpdateStatus(id, TaskStatusAsNumeric.CANCELLED);
            break;
        default:
            return BadRequest("Invalid status");
    }

    return Ok();
}

额外优化建议 #

• 检查CarStatusResourceParameters类中Status属性的默认值，确保不会默认设为CarStatus.Delete，避免参数传递异常时误删车辆。
• 前端请求路径去掉末尾斜杠，改为/car/${id}/status，与后端路由定义保持一致，减少路由匹配风险。
• 在后端接口中增加参数校验逻辑，确保未传递有效状态时直接返回错误，而非执行默认操作。

内容的提问来源于stack exchange，提问作者User2023-UsEr