苍穹外卖day11

*解答**：这是一个典型的**场景化数据展示**问题。**解答**：这是一个**空对象更新**的风险点！// 没看到需要订单菜品信息啊。**教训**：前端传参可能只包含必要字段（这里只有订单ID），后端应根据**业务语义**确定状态值，而不是盲目依赖前端传入的状态。**分析**：待付款订单的支付状态为`UN_PAID`，不满足`PAID`条件，因此无法进入取消逻辑。**问题**：为什么`order

otto

248人浏览 · 2026-01-21 10:29:03

otto · 2026-01-21 10:29:03 发布

昨天的笔记,忘记发了.

# 订单管理功能开发实战笔记：7个典型问题与解决方案

> 记录订单模块开发中的常见“坑”与填坑思路，适合后端开发者参考

## 前言

在最近的一次餐饮系统订单模块开发中，我遇到了不少典型业务场景下的编码与设计问题。本文将梳理其中7个关键问题点，分享解决方案与思考过程，希望能为正在开发类似功能的同学提供参考。

---

## 一、订单菜品信息的条件性展示

在实现订单分页查询时，我注意到以下代码片段：

```java
public PageResult conditionSearch(OrdersPageQueryDTO ordersPageQueryDTO) {
PageHelper.startPage(ordersPageQueryDTO.getPage(), ordersPageQueryDTO.getPageSize());
Page<Orders> page = orderMapper.pageQuery(ordersPageQueryDTO);

// 部分订单状态，需要额外返回订单菜品信息，将Orders转化为OrderVO
List<OrderVO> orderVOList = getOrderVOList(page); // 没看到需要订单菜品信息啊

PageResult pageResult = new PageResult();
pageResult.setTotal(page.getTotal());
pageResult.setRecords(orderVOList);
return pageResult;
}
```

**问题**：为什么代码中注释说要返回订单菜品信息，但实际需求中似乎并不需要？

**解答**：这是一个典型的**场景化数据展示**问题。在“全部订单”列表中，确实不需要展示菜品详情，但在“待接单”页面中，商家需要看到用户点了哪些菜，以便备餐。因此，`getOrderVOList()`方法内部实际上根据订单状态判断是否需要查询并封装菜品信息。

**启示**：同一实体在不同业务场景下可能需要不同的VO（视图对象），不要假设所有列表接口都返回相同字段。

---

## 二、订单统计VO的设计疑惑

订单状态有6种，但统计VO只定义了3个字段：

```java
@Data
public class OrderStatisticsVO implements Serializable {
//待接单数量
private Integer toBeConfirmed;
//待派送数量
private Integer confirmed;
//派送中数量
private Integer deliveryInProgress;
}
```

**问题**：为什么只有3种状态统计，而不是6种？

**解答**：这个VO是用于**商家工作台顶部状态统计**的，只关注当前需要处理的关键状态：
- 待接单（2）—— 需要立即处理
- 待派送（3）—— 已接单待出餐
- 派送中（4）—— 正在配送中

已完成、已取消、退款等状态不属于当前待办事项，因此不需要统计展示。

**启示**：DTO/VO的设计应紧密贴合具体业务场景，避免过度设计。

---

## 三、确认订单接口的“走神时刻”

在确认订单接口实现中，我差点犯了一个错误：

```java
public void confirm(OrdersConfirmDTO ordersConfirmDTO) {
Orders orders = Orders.builder()
.id(ordersConfirmDTO.getId())
// .status(ordersConfirmDTO.getStatus()) // 错误！接收的DTO只有id属性
.status(Orders.CONFIRMED) // 正确：接单就是将状态改为“已接单”
.build();
orderMapper.update(orders);
}
```

**教训**：前端传参可能只包含必要字段（这里只有订单ID），后端应根据**业务语义**确定状态值，而不是盲目依赖前端传入的状态。

---

## 四、为什么把更新语句放在if语句中？

在拒单逻辑中，更新操作被放在条件判断内：

```java
public void rejection(OrdersRejectionDTO ordersRejectionDTO) throws Exception {
// ... 验证逻辑

Orders orders = new Orders();
if (payStatus == Orders.PAID) {
// 用户已支付，需要退款
// 这里省略退款调用代码...

// 只有已支付的订单才需要设置这些字段
orders.setId(ordersDB.getId());
orders.setStatus(Orders.CANCELLED);
orders.setRejectionReason(ordersRejectionDTO.getRejectionReason());
orders.setCancelTime(LocalDateTime.now());
}
orderMapper.update(orders); // 为什么放在这里？
}
```