第1次评审教师意见与整改清单 - xych5q/online-shop GitHub Wiki
第 1 次评审教师意见与整改清单
第 1 组 · 在线购物系统 | 评审日期 2026-10-08
来源：评审现场录音转写（说话人 1 = 潘老师，技术细节点评；说话人 6 = 主评老师，整体点评）
整理人：陈星宇（组长）
零、老师要求我们下去之后做的两件事
- 按点评更新报告，并把老师的意见整理好放上去——本次完成（已回写至《第1次阶段性评审报告》的「工作任务表」+ 本报告末尾章节）。
- 围绕最终验收要求开始工作——录屏演示、Selenium 端到端、JMeter 并发测试、Docker 部署实测，列为迭代 2 任务。
一、整体意见（主评老师，6 条）
| # |
老师意见 |
我们的现状 |
怎么改 |
责任人 |
期限 |
| A |
报告要按模板：总体计划（分几个迭代、两周一次）、当前进度百分比及算法、本次迭代起止与可度量目标，然后才是任务与成果物 |
报告的「项目总体计划」「当前迭代期定义」都已写（6 个迭代、80%、五维加权、09-17~10-19），但汇报时没有讲出来 |
PPT 把「总体计划与进度」单独讲清楚：迭代划分、迭代 1 起止与目标、进度 80% 的算法 |
陈星宇 |
迭代 2 汇报（10-22）前，PPT 补讲 |
| B |
汇报不要陷进细节：4 条矛盾逐条讲太细，要讲统计量与整体成果，举例只需 1 个 |
P5 页逐条讲 C1~C4 |
改成：几轮需求、引用别组澄清多少条、本组 10 问 8 答复、识别矛盾 4 条（列表）+ 举例 1 条（建议 C1 队首自动进入交易） |
董嘉润 |
迭代 2 汇报前 |
| C |
不要在评审现场尝试在教师机上把系统跑起来，太耗时 |
现场演示靠即时操作 |
线下录 1~2 分钟演示视频（下单到交易闭环），评审时播放 |
嵇宇锋录制、林初俊讲解 |
10-22 前，同时作为 10-15 验收「运行」材料 |
| D |
PPT 一个同学一页自己的工作，不要写「某某和某某一起做的」 |
需求文档页写的是「董嘉润、陈星宇」两人 |
拆成两页：董嘉润页 = 提问与澄清（10 问 8 答复、问题库）；陈星宇页 = 需求分析与条目化编写（FR 编号、用例、状态机、追溯矩阵） |
陈星宇 |
迭代 2 汇报前 |
| E |
前端介绍不要讲成需求，要讲页面设计、与后端如何交互、如何收集请求、拿到数据如何更新页面 |
嵇宇锋讲的是「这个页面能完成什么功能」 |
改为讲：静态页面组织、表单收集、调用 5 个买家接口、JSON 回来后如何更新 DOM、空状态与错误处理 |
嵇宇锋 |
迭代 2 汇报前 |
| F |
PPT 不要堆 AI 生成的、自己解释不清的废话，只会减分 |
部分页面表述偏套话 |
逐页复查，凡是不能用自己的话讲清楚的内容一律删除 |
陈星宇 |
迭代 2 汇报前 |
二、技术细节意见（潘老师，5 条）
| # |
老师原意 |
我们要补什么 |
落在哪 |
责任人 |
| 1 |
「自动」不是什么都不做就完成，背后要有代码或机制触发，要写清什么条件下触发 |
写明：队首买家被系统判定进入交易的那一刻，由交易服务在同一个事务里把商品状态改为冻结；触发条件 = 存在排队中意向且商品处于在售；代码位置 = 交易服务进入交易的方法 |
SRS 状态机章节、《业务流程图》、PPT 状态机页 |
王振涛 + 陈星宇 |
| 2 |
「作废/撤销」怎么算作废？要在数据中体现出来 |
写明：意向记录的终态字段取值（成功 / 撤销 / 失败），撤销后口令码立即失效、凭码查询返回 404；数据落到意向表状态字段 |
SRS 状态机与意向状态定义 |
林初俊 + 陈星宇 |
| 3 |
状态机转移箭头上要标清「打上什么标记」才转移 |
已冻结 → 已下架的触发条件写具体：卖家标记交易结果为成功（接口入参 result=SUCCESS），不是笼统的「标记成功」 |
SRS 状态机、Wiki 业务流程图、PPT P7 |
王振涛 + 陈星宇 |
| 4 |
成功和失败不能是同一个状态，应是两个状态 |
意向终态拆成「已完成」与「已失败」两个独立状态，各自写明从交易中转入的触发条件 |
SRS 状态机、PPT P7 |
王振涛、林初俊 |
| 5 |
状态不能全是单向的，要考虑回退 |
补一条：已下架商品是否可重新上架回到在售（当前实现：支持重新发布，需在状态机上画出来并说明条件） |
SRS 状态机、业务流程图 |
王振涛 + 陈星宇 |
三、整改优先级
| 优先级 |
事项 |
原因 |
| 高 |
状态机 5 条技术意见（二-1 至二-5） |
直接改需求规格说明书与图，10-15 基线验收要查需求 |
| 高 |
录屏演示（一-C） |
10-15 验收「运行」必备件，也是老师点名要的 |
| 高 |
Docker 部署实测 |
10-15 验收「部署」必备件，缺件拿不到升级需求 |
| 中 |
PPT 一人一页拆分（一-D）、前端改讲技术（一-E）、统计式表述（一-B） |
影响下次评审得分 |
| 中 |
Selenium 端到端、JMeter 并发 |
10-15 验收必备件 |
| 低 |
PPT 去废话（一-F） |
下次汇报前做即可 |
四、迭代 2（10-19 ~ 未定）任务分工
| 任务 |
责任人 |
交付物 |
对应意见 |
| 状态机修订：拆分成功/失败状态、标注转移条件、补回退路径、定义撤销的数据表现 |
王振涛（卖家端）+ 林初俊（买家端）+ 陈星宇（SRS 与图） |
SRS v1.4、业务流程图更新、PPT P7 更新 |
二-1~5 |
| Docker 部署实测 + 镜像导出 |
陈星宇 |
镜像、部署手册实测记录、验收材料 |
一-C、10-15 验收 |
| 演示录屏（1~2 分钟，下单到交易闭环） |
嵇宇锋录制、林初俊脚本 |
视频文件放仓库 test-cases/ |
一-C |
| Selenium 端到端脚本 |
待分配（建议林初俊） |
脚本 + 执行记录 |
10-15 验收 |
| JMeter 并发性能测试 |
待分配（建议王振涛） |
脚本 + 报告 |
10-15 验收 |
| PPT 分页改造与去废话 |
陈星宇 |
PPT v5 |
一-D、一-F |
| 前端页改为讲技术实现 |
嵇宇锋 |
PPT 页 + 演讲稿 |
一-E |
| 需求汇报改为统计式表述 |
董嘉润 |
PPT 页 + 演讲稿 |
一-B |
五、下一轮汇报（10-22）自检清单
- 总体计划讲了没有（几个迭代、两周一次）？
- 进度百分比讲了没有，算法讲了没有？
- 本次迭代起止日期与可度量目标讲了没有？
- 需求部分是统计式表述，不是逐条罗列？
- 每个同学一页，有没有出现「某某和某某一起」？
- 前端讲的是技术实现，不是需求？
- 演示是播放录屏，不是现场起服务？
- PPT 里有没有自己讲不清楚的套话？
- 状态机上每条转移箭头有没有写清触发条件？
- 成功和失败是不是两个独立状态？
六、写在最后
这次点评的核心一句话：老师要的是进度与完成度，不是功能细节。我们这轮把力气花在了需求澄清和代码上，方向没错，但汇报时没有把"时间、进度、边界"讲出来，也没有把人和人的边界分开。迭代 2 的两条主线：一条是补 10-15 验收的硬指标（Docker、Selenium、JMeter、录屏），一条是按意见改汇报方式。