xuyue/OJ2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity
220 Commits 1 Branch 0 Tags
2bf0e7bfc1a35ce998ccf536def2a0dd16450145
Commit Graph

7 Commits

Author SHA1 Message Date
yuetsh
f66bafafaf feat(教师统计): 提交统计面板返修 —— 统计口径、未完成分两栏、多题、自动刷新 
排查这个面板时发现的一串问题和缺口，一起修掉。改动集中在
`GET /submissions/statistics` 和 `StatisticsPanel.vue`，两边互相咬着，
拆不成独立的 commit。

**「已解决」数的不是题数**（真在显示错数字）。`count(*) filter (accepted)`
按用户名分组、没有 distinct problem_id，同一道题重复 AC 会重复计数。查单道题时
看不出来，老师查「这节课全班」时「已解决 5」可能是同一道题交了 5 次。新增
`solvedCount = count(distinct problem_id) filter (accepted)`，表格那一列换成它；
`acceptedCount` 保留，正确率的分子仍然是提交条数。

**正确率把判题中的算进了分母**。PENDING / JUDGING 进分母不进分子，全班同时交卷的
那几秒正确率凭空掉一截 —— 而老师盯着看的就是这个数。分母改成判完的条数，
`UNJUDGED_RESULTS` 提到 judge/status.ts 共用。总提交仍是全部条数（交了就算交过，
否则人数口径会跟着变，正在判的学生会掉进「没交」名单），另外下发 `judgingCount`，
非零时面板多显示一块「判题中」，三个数字才对得上。

**「未完成」实际是「一次没交」**。做了但一次没对的学生既不在「完成人数」也不在
未完成名单，等于从屏幕上消失 —— 而那恰恰是最该去看一眼的人。新增
`dataAttempted`，未完成栏拆成「还没交」/「交了没对」两组，tab 计数是两者之和。
请假隐藏对两组同时生效：他们同样占着班级人数这个分母，只藏一半会把完成度算错。

**点名字能看到错在哪**。`dataAttempted` 带上最近一条提交的题号、状态和
`statistic_info.err_info`（截断 400 字），点名字弹出来，还能一步跳到代码。
老师不用再切到提交列表、翻到这个人、点开代码。

**支持一次查几道题**。题号框接受 `1001,1005,1010`（中英文逗号、分号、空格都当
分隔符，投影前手敲不该因为打了全角逗号就查不出来），有一个题号不存在就整体 404。
**完成 = 这几道全解决**；只填一道时和原来完全等价，不填题号时退回「至少做出一道」。
差一道的人落在「交了没全对」里，名字后面缀 `2/3题`。

**零提交时整块面板消失**。判空条件是 `count.total > 0`，可一节课刚开始一条提交都
没有、后端已经把整份花名册当作「未完成」返回了 —— 最该看名单的时刻反而只显示
「暂无数据」。流程图那边更彻底：`/flowcharts/statistics` 的零提交分支把
`dataUnaccepted` 写死成空数组，名单压根没下发。

**面板不会自己刷新**。打开是空的、要点一次按钮，拿到的是那一刻的快照。改成打开即查
+ 每 15 秒滚动重查，页面切到后台就停，关掉面板随组件卸载停掉。上一次没回来就跳过
这一次。班级从手打 `ks251` 换成下拉（选项来自网站配置的 class_list，和登录框同一份），
保留自由输入，查过的班级记进 localStorage 下次带上 —— 机房电脑一台对一个班。

**明细查询没有 LIMIT**。展开行用的 submissionItems 原来把窗口内**全部**提交捞进内存
再原样序列化，「全部时段 + 不填条件」就是十几万条。改成只取有 AC 的人、每人最近 50 条，
截断用窗口函数发生在数据库侧；表格「提交数」仍是真实总数。

**顺带**：`personRate` 前端从来没读过（完成度是前端按「减掉请假人数的分母」自己算的），
从契约里删掉；「语法未过」的题数单列出来（AST_CHECK_FAILED 全站仍然算通过，口径没动，
只是让老师看得见谁是绕过语法要求做出来的，那批人教学上没达标）。

实跑验证（dev 全栈 + 浏览器）：

- 已解决：student 两条 AC 都在题号 5 上 —— 改前显示 2，改后显示 1
- 正确率：临时把两条改成 PENDING/JUDGING，12 条提交 2 条通过 —— 16.67% → 20%，
  面板多出「判题中 2」，表格显示「12（2 条判题中）」
- 未完成两栏：student2 没交、student 交了 12 次没对，请假隐藏 student 后
  班级人数 2→1、tab 2→1、出现「恢复 1 位」
- 错因弹层：点 student 弹出「最近一次：1020 · 编译失败 / Test case not found / 看代码」
- 多题：`1004` 完成 1 人；`1004,1005` 完成 0 人、交了没全对 student 1/2题 8次；
  全角逗号同上；`1004,9999` 报 `Problem 9999 does not exist`
- 自动刷新：打开即发请求，之后 04:44:22 → 04:44:36 → 04:44:51 → 04:45:06 每 15 秒
  一次且窗口跟着滚；关掉面板后 20 秒请求数不再增长
- 班级下拉：选「25计算机1班」→ ks251，重开面板自动带上并立刻查

tsc / vue-tsc / check:routes 全过。验证用的 dev 库改动已还原。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KqjE6qPo67fqVDKn6Bx7yd
 2026-09-08 05:20:51 -06:00
yuetsh
2c4d56b29a refactor(后端): 清掉 Django 遗留的死列与手工级联，角色字符串收成一份 
三条迁移，一次部署（0008/0009 含 DROP COLUMN，需要 OJ2_ALLOW_DESTRUCTIVE=1）：

- 0008 删 IP 相关：比赛 IP 白名单（前端本来就没有输入框，detail.vue 无条件置空）、
  submission.ip（前端从未显示过）、以及一次都没被调用过的 IP 限流桶。
  judge_server.ip 是运维数据，保留。
- 0009 删九个只有 Django 时代写过、OJ2 一次都没读过的列：user 的 auth_token /
  open_api / open_api_appkey / session_keys，user_profile 的 blog / github /
  school / major / language。open_api 后台连开关都没有，那段「已经开着就不重置
  appkey」的逻辑从上线起没进过 if。判据是「全仓零读取」而不是「看着没用」——
  raw_password 同样刺眼却是在用的，别一起清掉。
- 0010 给 17 条外键补上删除动作，不再是 Django 留下的一律 NO ACTION。父行消失后
  必然无意义、且不构成学生留痕的走 CASCADE（中间表、题单/教程/成就的组成部分、
  user_profile 与 user_stat）；需要人看见的继续拦着——submission.problem_id、
  以及 user 的绝大多数外键，删用户撞外键会被 handler 翻译成「请改为禁用账号」，
  这是有意的：全 CASCADE 会静默抹掉成就与进度，而 submission.user_id 压根没有
  外键，结果是一半删一半留。六处手工级联随之删掉。

角色字符串收进 packages/contract/src/roles.ts：原先 ADMIN_ROLES / TEACHER_ROLES
在两个文件各抄一份、学生口径在四个文件各写一遍、前端 USER_TYPE 是第三份副本。
AuthUser.adminType 与 drizzle 的列都收窄成联合类型，二十多处 `=== "Super Admin"`
从此受编译器管着（$type 是纯 TS 层的，generate 确认不产生任何 SQL 变更）。

顺带删掉 db/relations.ts —— drizzle-kit pull 的产物，全仓零引用。

一处行为变化：后台用户列表传非法的 ?type= 回 400，不再静默返回空列表；界面上的
下拉只有合法值，打不到这条。

验证：tsc / vue-tsc / vite build / check:routes 全过；三条迁移在 dev 库执行，
并逐条建 fixture 走 HTTP 接口验过删除连坐与拦截（题单五张子表连坐、user_badge
二级连坐、删有提交的题目仍 409、删有表情的用户仍 409）。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AeJoYc2t2d7cThVqMBYrBF
 2026-09-02 22:57:45 -06:00
yuetsh
047e6dd7d3 feat(登录): 学生登录改成班级+姓名两个下拉，一个字都不用打
Some checks failed
Deploy / deploy (push) Has been cancelled
Details 
登录框拆成「学生登录 / 管理员登录」两个页签。学生页签选完班级选姓名，
ks 前缀提交时自动拼，选完姓名焦点直接跳到密码框，回车登录；上次选的班级
记在 localStorage 下次自动填好（姓名不记 —— 同一台机器换个人坐就是别人）。
管理员页签只有用户名+密码，不拼前缀。

「没有我所在的班级」保留在班级列表末尾，往届的班级从后台 class_list 里删掉后
学生仍能从这里写完整用户名登录。为了把「还没选」和「没有我所在的班级」分开，
loginForm.class 的初值从空串改成 null —— 原来两者都是空串，一打开就默认落在
「没有我所在的班级」那一档、显示文本框，正好和「方便学生」相反。username 同样
改成 nullable：空串在 n-select 里是「选了一个空值」，占位符不显示。

clearProfile 的 storage.clear() 给 LOGIN_CLASS 开了个口子：机房一台机器对一个班，
下课登出、下节课再来还是同一个班，清掉的话每个人都得重新选一遍。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YQCLjAmz6pPXMNGoZmvAmC
 2026-09-01 06:57:24 -06:00
yuetsh
8861393529 fix(流程图): 评分明细的顺序是乱的，40 分的那项排在最后
Some checks failed
Deploy / deploy (push) Has been cancelled
Details 
AI 是按分值从高到低返回的：`逻辑正确性(40) → 完整性(30) → 规范性(20) → 清晰度(10)`。
但 `ai_criteria_details` 存在 jsonb 列里，而 Postgres 的 jsonb **不保留键序** ——
它按「键长度 + 字节序」重排，读出来变成 `完整性 → 清晰度 → 规范性 → 逻辑正确性`，
权重最高的那项被排到了最后。

评分弹框和教师端的评分详情都是直接 `v-for` 遍历这个对象，所以两处都乱。
统计面板因为用的是写死的 `CRITERIA_ORDER`，一直是对的 —— 于是同一份数据在两个
地方的顺序还不一致。

把顺序抽到 `utils/constants` 共享，三处统一走 `sortFlowchartCriteria()`；
表里没有的键排到后面，AI 万一返回别的评分项也不会丢。

顺带把限流的提示改得能看懂：撞上 429 时原来只显示「流程图提交失败」，学生会以为
是自己的图有问题然后反复点，越点等得越久。现在按错误码分支，提示「提交太频繁了，
缓一会儿再交」。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
 2026-08-27 10:53:29 -06:00
yuetsh
36a4663193 chore(前端): 清掉四处零引用的死代码
Some checks failed
Deploy / deploy (push) Has been cancelled
Details 
按名字扫了一遍 api 层导出和 utils 常量，四处全仓零引用：

**`getContestProblem`**（admin/api.ts）—— 和紧挨在它上面的 `getProblem` 逐字
相同，同一个 URL、同一个类型。旧后端 admin 侧比赛题和公开题是两个端点，合并
之后这个壳留下来了。

**`createMessage`**（oj/api.ts）—— 两代前端都只有定义没有调用。参数名还是旧
后端那套（recipient / submission），函数体里再映射成契约的
recipientId / submissionId，等于给一个不存在的调用方写了个兼容层。
d3b05b8 删掉了它的类型 CreateMessage，函数漏了。

  后端 `POST /messages` 是完整实现的，只是后台那个页面
  （admin/communication/messages.vue）两代都是一句「未完待续」的占位。
  在端点上加了注释说明这件事，免得下次扫「无人调用的端点」时被当成可删。

**`CONTEST_TYPE`**（constants.ts）—— 和上面 30 行处的 `ContestType` 枚举
一模一样。枚举那份有 8 处在用，这个对象零引用。

**`LANGUAGE_ID`**（constants.ts）—— Judge0 的语言 id。真正在用的那份在
utils/judge.ts 的 JUDGE0_LANGUAGE_ID，而且**只有那份是对的**：死掉的这份把
Golang / JavaScript / Python2 全写成了 0，谁要是拿它去调 Judge0，这三种语言
的在线试运行会静默发出 `language_id: 0`。

顺带确认过没有 `constants["LANGUAGE_ID"]` 这类动态取值。

验证：vue-tsc 0 error、vite build 通过、tsc(apps/api) 0 error、
check:routes 168 条无遮蔽。四处都是零文本引用的纯删除。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
 2026-08-25 23:32:50 -06:00
yuetsh
c2a81209d7 refactor(前端): 拆掉 camelCase→snake_case 转换层，契约成为唯一真相 
utils/legacy.ts 是迁移期的临时层：新后端一律 camelCase，而组件读的还是
旧 Django 的 snake_case，于是在 api 层做一次递归键名重写。它自己的注释就
写了「迁移完成后这一层应当整体拆掉」。现在拆了。

代价不只是那 96 处包装：每个响应都要递归遍历整个对象重写一遍键名，而且
utils/types.ts 和 packages/contract 是两份真相 —— 手抄的那份还抄歪了好几处。

做法是按域推进，每域都用 vue-tsc 相对基线做差，确认零新增错误后再往下走。
前端的类型现在一律以契约为准，只在必要处窄化（比如 languages/template 的键
窄化成 LANGUAGE），删掉的重复定义包括 WebsiteConfig、LoginSummary、
AchievementSummary、ProblemSet、Contest、User、Profile、AdminTag、
StuckProblem 等等，其中 ClassComparison 有两个组件各手抄了一份。

## 顺带修掉的真 bug

- 管理端公告列表的「可见」开关每次都 400：列表响应被契约 omit 掉了 content，
  而更新接口要求 content 必填，toggleVisible 把列表行原样回传。而且是乐观
  翻转、不 await 不 catch，管理员看到开关动了、实际没存也没有提示。
  改成先 GET 整条再 PUT，加失败提示。

- 删有提交的题时只显示笼统的「删除失败」：前端还在 match 旧 Django 的英文
  文案，而后端返回的是 problem-has-submissions + 中文。连同另外 8 处同类
  匹配一起改成判错误码 —— 文案是后端随时能改的，match 文案改一个字就静默失效。

- SubmissionStatus.time_limit_exceeded 写成 `1 | 2`，TS 按位或算成 3，和
  memory_limit_exceeded 撞了同一个值。后端 judge/status.ts 里这是分开的
  两个码，按后端拆成 cpu_/real_ 两项。当前没有代码读这两个成员，但
  CLAUDE.md 明确要求判题状态码三处同步。

- 流程图历史翻到没有提交的那一页会直接抛：契约里 submission 是 nullable，
  被 any 掩盖成看起来非空。补了 null 分支。

## 契约里被逼出来的三处不诚实

- grade 写成 z.string()，但 averageGrade() 在没有可用数据时返回空串，
  前端三张图表拿它查 Record<Grade,...> 会查出 undefined。按实际收紧成
  z.enum([...,""])，四个查表点都补了「无评级」分支。

- difficulty 写成 z.string()。核对过生产库 dump：956 道题只有
  Low/Mid/High 三个值（761/149/46）。收紧成枚举。

- topReaction 写成 z.string()，既对不上前端渲染的 {type,count}，也对不上
  旧后端 get_top_reactions 下发的形状。改成正确形状并注明当前恒传 null。

## 明确保留 snake_case 的 54 处

判题沙箱原始输出（cpu_time/exit_code/output_md5/compile_output）、
statistic_info 内容（err_info/time_cost/ast_results）、submission_info
JSONB（is_ac/ac_time/error_number，回滚时旧后端还要读）、SQL 判题引擎的
total_rows/order_sensitive/changed_tables、WebSocket 的 submission_id、
以及数据库选项键 enable_maxkb。每一处都在类型定义旁写了为什么不能改。

language 没有跟着收紧契约 —— 它是配置项、随时可能加语言，收紧会让新语言
在后端 parse 时直接抛。改在 api 边界一处窄化。

## 另外

- utils/http.ts 整个模块已是死代码（四处引用全是 import type），删除。
- profile 的 blog/github/school/major/language 五个字段全链路空转，没有
  任何组件读，从契约到类型一并摘除（数据库列不动）。
- admin/account.ts 往 user_profile 塞的 totalScore 是 OI 模式遗留，表里
  没这一列。Drizzle 按表定义拼列名会把它静默丢弃，所以没出过错，是死代码。

验证：vue-tsc 143 → 54 条且无新增，apps/api tsc、check:routes、web build
全通过；各域响应形状逐条打接口核对过。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
 2026-08-25 13:44:24 -06:00
yuetsh
ae1fb329b5 feat(阶段1): 搬入 ojnext 为 apps/web，未改业务代码 2026-08-06 21:18:16 -06:00