From f8a154ff7a1716cd342cb4632ee62ffd6c960ee8 Mon Sep 17 00:00:00 2001 From: Joywayer Date: Thu, 27 Aug 2026 22:06:41 +0800 Subject: [PATCH] =?UTF-8?q?=E4=BA=8C=E4=B8=83=E7=8E=8B=EF=BC=9Aresult.chup?= =?UTF-8?q?ai=20=E7=A7=BB=E5=87=BA=20GameState=EF=BC=8C=E6=9C=AC=E5=A2=A9?= =?UTF-8?q?=E7=BB=93=E6=9E=9C=E6=94=B9=E8=B5=B0=20EQW=5FTRICK=5FEND=20?= =?UTF-8?q?=E4=BA=8B=E4=BB=B6=E8=BD=BD=E8=8D=B7?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 谁最大、这墩闲家得几分是【一次性表现】（收牌动画、得分飘字），任何 deskinfo 都不下发它， 写进 GameState 就多出一处「只活在前端、服务端不知道」的对局态——出牌中途断线重连即归零， 违反红线「丢弃 this.data、仅凭最近一次服务端快照重画，界面必须一致」。 三处一起改，缺一不可： - PlayHandler chupai3：删 result.chupai 写入，改随 EQW_TRICK_END 下发 {seat,cards,maxseat?,grade?}； aset.grade 的累加保留（它有服务端来源 PushCards.grade / chupai3.grade，是镜像量） - ResultHandler：_apply 的 map 删掉 chupai。收尾墩不发 chupai3、整包换成 jiesuan， 不改这里则删 EXCLUDE 后必转红。收尾墩改为与 chupai3 走同一对事件、同一份载荷字段 （补 emit EQW_TRICK_END 并带上 grade），否则最后一墩的得分没有出口 - GameState：删掉 result.chupai 字段本身（声明 + reset），不留永远为 null 的死字段 test_consistency 的 EXCLUDE 随之删到只剩 table.ancard3s 一条，27 checks 仍全绿； 该条改注释标明「当前夹具（65 分坐庄、单局）无法触发，此条未经验证」，不假装它在守什么。 Co-Authored-By: Claude Opus 5 (1M context) --- .../codes/net/handlers/PlayHandler.js | 29 +++++++++++++------ .../codes/net/handlers/ResultHandler.js | 12 ++++++-- client/js/01_SubGame/codes/state/Events.js | 2 +- client/js/01_SubGame/codes/state/GameState.js | 6 ++-- client/tests/test_consistency.js | 14 +++++---- client/tests/test_handlers_play.js | 13 ++++++--- client/tests/test_handlers_result.js | 14 ++++++--- 7 files changed, 61 insertions(+), 29 deletions(-) diff --git a/client/js/01_SubGame/codes/net/handlers/PlayHandler.js b/client/js/01_SubGame/codes/net/handlers/PlayHandler.js index 1b44232..41aa649 100644 --- a/client/js/01_SubGame/codes/net/handlers/PlayHandler.js +++ b/client/js/01_SubGame/codes/net/handlers/PlayHandler.js @@ -24,6 +24,8 @@ // 【这一手的信息走事件载荷，不进 GameState】：谁出了什么、几张、什么牌型是【一次性表现】 // （落牌动画），不是对局状态；deskinfo 也不下发它。放进 GameState 会立刻制造一处 // 增量/全量不一致。故随 EQW_CARD_PLAYED 事件下发给 UI。 +// 【本墩结果同理】：chupai3 的 maxseat / grade（谁最大、这墩闲家得几分）也是一次性表现， +// 随 EQW_TRICK_END 事件载荷下发，同样不进 GameState。 var EQW_PlayHandler = EQW_PlayHandler || { //order: 1/2/3，对应 chupai1/2/3 @@ -65,20 +67,19 @@ var EQW_PlayHandler = EQW_PlayHandler || { EventBus.emit(EQW_Events.EQW_CARD_PLAYED, this._playPayload(data, order)); if (data.hasOwnProperty('cardsinhand')) { EventBus.emit(EQW_Events.EQW_HAND_CHANGED); } - //—— 末家出完：本轮结束，记下谁最大、闲家得了多少分 —— + //—— 末家出完：本轮结束 —— + //【本墩结果走事件载荷，不进 GameState】谁最大、这墩闲家得了几分，是【一次性表现】 + //（收牌动画、得分飘字），与 EQW_CARD_PLAYED 同类；任何阶段的 deskinfo 都不下发它， + //写进 GameState 就多出一处「只活在前端、服务端不知道」的对局态——出牌中途断线重连 + //即归零，违反前端红线「丢弃 this.data、仅凭最近一次服务端快照重画，界面必须一致」。 + //小局累计捡分 aset.grade 不同：它有服务端来源（PushCards.grade / chupai3.grade）， + //属于镜像量，仍留在 GameState。 if (order === 3) { - if (!EQW_GameState.result.chupai) { EQW_GameState.result.chupai = {}; } - EQW_GameState._apply(EQW_GameState.result.chupai, data, { - seat: 'seat', - cards: 'cards', - maxseat: 'maxseat', - grade: 'grade' - }); //本轮闲家得分累加进小局捡分；没得分时不带 grade，累计值保持不变 if (data.hasOwnProperty('grade')) { EQW_GameState.aset.grade = EQW_GameState.aset.grade + data.grade; } - EventBus.emit(EQW_Events.EQW_TRICK_END); + EventBus.emit(EQW_Events.EQW_TRICK_END, this._trickPayload(data)); } EventBus.emit(EQW_Events.EQW_TURN_CHANGED); @@ -94,5 +95,15 @@ var EQW_PlayHandler = EQW_PlayHandler || { if (data.hasOwnProperty(keys[i])) { payload[keys[i]] = data[keys[i]]; } } return payload; + }, + + //EQW_TRICK_END 的事件载荷：这一墩的一次性结果（收牌动画、得分飘字用），不落 GameState。 + //【缺字段不兜底】grade 只在这墩被闲家收走且确实有分时才有，包里没有就不带这个键。 + //与 ResultHandler 为收尾墩发的那份载荷字段一致（收尾墩不发 chupai3，整包换成 jiesuan）。 + _trickPayload: function (data) { + var payload = { seat: data.seat, cards: data.cards }; + if (data.hasOwnProperty('maxseat')) { payload.maxseat = data.maxseat; } + if (data.hasOwnProperty('grade')) { payload.grade = data.grade; } + return payload; } }; diff --git a/client/js/01_SubGame/codes/net/handlers/ResultHandler.js b/client/js/01_SubGame/codes/net/handlers/ResultHandler.js index 1f71ad4..5a989e8 100644 --- a/client/js/01_SubGame/codes/net/handlers/ResultHandler.js +++ b/client/js/01_SubGame/codes/net/handlers/ResultHandler.js @@ -46,15 +46,20 @@ var EQW_ResultHandler = EQW_ResultHandler || { //—— 收尾轮的第三手：牌局在最后一手打完时【不发 chupai3】，整包换成 jiesuan（协议 §13/§14）—— //夹具实跑一局的包数就是证据：chupai1×28 / chupai2×28 / chupai3×27。 //只在 chupai3 归档会永远丢掉最后一轮，故在这里补齐——它必是本轮第三手，落进当前这一轮。 - //jiesuan.chupai 的字段与 chupai3 不同：只有 seat / cards / maxseat， - //没有 order / playproc / cardsinhand / grade。 + //jiesuan.chupai 的字段与 chupai3 不同：只有 seat / cards / maxseat（+ 有分时的 grade）， + //没有 order / playproc / cardsinhand。 //归档随后会被下面的结算收敛清掉（deskinfo step6 不带 PushCards），但 emit 是同步的： //落牌与收牌表现在清空之前就已取到完整的收尾轮。 + //【本墩结果只走事件载荷、不进 GameState】：谁最大、这墩得几分是一次性表现， + //deskinfo 的 Balance 不带它（它属于「出牌过程」而非结算数据），写进 GameState + //就会多出一处服务端不知道的对局态。收尾墩与 chupai3 走同一对事件、同一份载荷字段。 if (d.chupai && d.chupai.hasOwnProperty('seat') && d.chupai.hasOwnProperty('cards')) { EQW_GameState.pushPlay(d.chupai.seat, d.chupai.cards, false); var lastPlay = { seat: d.chupai.seat, cards: d.chupai.cards }; if (d.chupai.hasOwnProperty('maxseat')) { lastPlay.maxseat = d.chupai.maxseat; } + if (d.chupai.hasOwnProperty('grade')) { lastPlay.grade = d.chupai.grade; } EventBus.emit(EQW_Events.EQW_CARD_PLAYED, lastPlay); + EventBus.emit(EQW_Events.EQW_TRICK_END, lastPlay); } //结算是"清空过程量、只留结算数据"的快照点：deskinfo step6 只带 Balance（=aset）， @@ -85,8 +90,9 @@ var EQW_ResultHandler = EQW_ResultHandler || { EQW_GameState.table.liangpai = null; EQW_GameState.table.mingpai = null; + //【不写 chupai】收尾墩已在上面随事件载荷发走；deskinfo step6 的 Balance 只有 + //aset / bottom / account 三份，多存一个 chupai 就是增量路径独有的私有对局态 EQW_GameState._apply(EQW_GameState.result, d, { - chupai: 'chupai', bottom: 'bottom', aset: 'aset', account: 'account' diff --git a/client/js/01_SubGame/codes/state/Events.js b/client/js/01_SubGame/codes/state/Events.js index 8a0d672..93235d3 100644 --- a/client/js/01_SubGame/codes/state/Events.js +++ b/client/js/01_SubGame/codes/state/Events.js @@ -20,7 +20,7 @@ var EQW_Events = EQW_Events || { EQW_MAIN_SET: 'eqw.main_set', //选主 EQW_BURY_DONE: 'eqw.bury_done', //埋牌完成 EQW_CARD_PLAYED: 'eqw.card_played', //有人出牌 - EQW_TRICK_END: 'eqw.trick_end', //一轮结束（chupai3 带 maxseat） + EQW_TRICK_END: 'eqw.trick_end', //一墩结束；载荷 {seat,cards,maxseat[,grade]}，不落 GameState EQW_TURN_CHANGED: 'eqw.turn_changed', //控制权或倒计时变化 EQW_MINGPAI: 'eqw.mingpai', //收到明牌数据 EQW_TIP: 'eqw.tip', //收到对家提示 diff --git a/client/js/01_SubGame/codes/state/GameState.js b/client/js/01_SubGame/codes/state/GameState.js index 05ecb29..1523e9e 100644 --- a/client/js/01_SubGame/codes/state/GameState.js +++ b/client/js/01_SubGame/codes/state/GameState.js @@ -68,8 +68,10 @@ var EQW_GameState = EQW_GameState || { }, //—— 结算 —— + //字段集与 deskinfo step6 的 Balance 严格对齐（aset / bottom / account）。 + //【不存 chupai】收尾墩的「谁最大、得几分」是一次性表现，随 EQW_TRICK_END 事件载荷走， + //任何 deskinfo 都不下发它——存进来就是一处服务端不知道的私有对局态（见 ResultHandler / PlayHandler） result: { - chupai: null, bottom: null, aset: null, account: null @@ -87,7 +89,7 @@ var EQW_GameState = EQW_GameState || { this.my = { cards: [], mustCard: [], bottomCards: [], buryCards: [] }; this.table = { ancard3s: 0, playproc: null, pushlist: [], seatlist: [], liangpai: null, mingpai: null }; - this.result = { chupai: null, bottom: null, aset: null, account: null }; + this.result = { bottom: null, aset: null, account: null }; }, //深拷贝快照（只含数据分组，不含方法），供测试做「增量 vs 全量」比对 diff --git a/client/tests/test_consistency.js b/client/tests/test_consistency.js index e20b760..2f10529 100644 --- a/client/tests/test_consistency.js +++ b/client/tests/test_consistency.js @@ -23,13 +23,15 @@ const fx = JSON.parse(fs.readFileSync(path.join(ROOT, 'client/tests/fixtures/pac // 这些字段是「一次性事件」或「重连不重放」的，不参与一致性比对。 // 每加一条都必须写清理由——这个白名单是本测试唯一的松口处，滥用它等于废掉这条守卫。 +// +// 【清单历史】result.bottom / result.account 已随服务端 Balance 补发而删除； +// result.chupai 已随「本墩结果改走 EQW_TRICK_END 事件载荷、不进 GameState」而删除。 +// 现在只剩下面这一条。 const EXCLUDE = [ - // 70 分坐庄的 3 秒开底是上庄时的一次性事件，协议明确 deskinfo 不重放 - 'table.ancard3s', - // 收尾墩：牌局最后一手不发 chupai3，整包换成 jiesuan，其 data.chupai 由 ResultHandler - // 归档进 result.chupai；deskinfo 的 Balance 不带这一墩（它属于「出牌过程」而非结算数据）。 - // 【本条是本清单仅剩的松口处】result.bottom / result.account 已随服务端 Balance 补发而移除。 - 'result.chupai' + // 70 分坐庄的 3 秒开底是上庄时的一次性事件，协议明确 deskinfo 不重放。 + // ⚠️ 当前夹具无法触发（65 分坐庄、单局），此条未经验证——保留是因为删掉之后 + // 将来真出现不一致会静默通过；但它此刻并没有在守住任何东西，不要当成已验证的豁免。 + 'table.ancard3s' ]; function pick(snap, exclude) { diff --git a/client/tests/test_handlers_play.js b/client/tests/test_handlers_play.js index c6aae21..508240f 100644 --- a/client/tests/test_handlers_play.js +++ b/client/tests/test_handlers_play.js @@ -47,22 +47,27 @@ t.eq('无 mustcard 时清空', S.my.mustCard, []); // ================= chupai3（末家出牌，本轮结束）================= // 真实的 chupai3 也带 nextseat（下一轮首家）：服务端在 switch 前统一设置， // 只有整轮结束进入结算时才会整体换成 jiesuan rpc（不会以 chupai3 到达 handler） +// 本墩结果（谁最大 / 这墩得几分）是一次性表现，走 EQW_TRICK_END 事件载荷，【不进 GameState】—— +// 任何 deskinfo 都不下发它，存进来就是一处服务端不知道的私有对局态（出牌中途重连即归零） S.reset(); evts.length = 0; +let lastTrick = null; +EventBus.on(EQW_Events.EQW_TRICK_END, function (p) { lastTrick = p; }); EQW_PlayHandler.handle({ success: true, seat: 2, cards: [9,10], maxseat: 0, grade: 25, nextseat: 0, countdown: 15, baozhu: 1, curmultiple: -2 }, 3); -t.eq('本轮最大者', S.result.chupai.maxseat, 0); -t.eq('本轮闲家得分', S.result.chupai.grade, 25); +t.eq('本墩结果走事件载荷', lastTrick, { seat: 2, cards: [9,10], maxseat: 0, grade: 25 }); +t.eq('本墩结果不进 GameState', S.result.hasOwnProperty('chupai'), false); t.eq('累计捡分', S.aset.grade, 25); t.eq('报无主标志', S.aset.baozhu, 1); t.eq('抓分倍数（带符号）', S.aset.curmultiple, -2); t.eq('下一轮首家', S.turn.seat, 0); t.eq('本轮结束发事件', evts.indexOf('EQW_TRICK_END') >= 0, true); -// 本轮闲家没得分时不带 grade，累计分不该被抹 -S.reset(); S.aset.grade = 40; +// 本轮闲家没得分时不带 grade，累计分不该被抹；事件载荷也不得凭空补 grade（缺字段不兜底） +S.reset(); S.aset.grade = 40; lastTrick = null; EQW_PlayHandler.handle({ success: true, seat: 2, cards: [9,10], maxseat: 1, nextseat: 1, countdown: 15 }, 3); t.eq('无 grade 时累计分保持', S.aset.grade, 40); +t.eq('无 grade 时事件载荷不带该键', lastTrick.hasOwnProperty('grade'), false); t.eq('无 grade 时仍推进控制权', S.turn.seat, 1); // grade 是「本轮」得分，要跨包累加进小局捡分，不是赋值—— diff --git a/client/tests/test_handlers_result.js b/client/tests/test_handlers_result.js index 6166f43..f9d232d 100644 --- a/client/tests/test_handlers_result.js +++ b/client/tests/test_handlers_result.js @@ -10,13 +10,15 @@ load('client/js/01_SubGame/codes/net/handlers/ResultHandler.js'); const S = EQW_GameState; const evts = []; Object.keys(EQW_Events).forEach(k => EventBus.on(EQW_Events[k], () => evts.push(k))); +let lastTrick = null; +EventBus.on(EQW_Events.EQW_TRICK_END, p => { lastTrick = p; }); const ASET = { banker: 0, call: 60, multiple: 3, flower: 3, grade: 45, upgrade: 1, bangwang: 0, climb: 0, seatlist: [{ grade: 6, score: 6 }, { grade: -3, score: -3 }, { grade: -3, score: -3 }] }; // ================= 正常出牌结算 ================= -S.reset(); evts.length = 0; +S.reset(); evts.length = 0; lastTrick = null; EQW_ResultHandler.handleJiesuan({ success: true, step: 6, chupai: { seat: 2, cards: [9], maxseat: 0, grade: 10 }, bottom: { cards: [1,2,3,4,5,6,7,8], multiple: 2, grade1: 20, grade2: 40 }, @@ -24,17 +26,21 @@ EQW_ResultHandler.handleJiesuan({ success: true, step: 6, t.eq('小局结算入状态', S.result.aset, ASET); t.eq('抠底入状态', S.result.bottom.grade2, 40); -t.eq('最后一手入状态', S.result.chupai.maxseat, 0); +// 收尾墩（服务端不发 chupai3，整包换成 jiesuan）走事件载荷，与 chupai3 同一对事件、同一份字段； +// 【不进 GameState】——deskinfo step6 的 Balance 只有 aset/bottom/account，多存就是私有对局态 +t.eq('收尾墩走事件载荷', lastTrick, { seat: 2, cards: [9], maxseat: 0, grade: 10 }); +t.eq('收尾墩也发本墩结束事件', evts.indexOf('EQW_TRICK_END') >= 0, true); +t.eq('收尾墩不进 GameState', S.result.hasOwnProperty('chupai'), false); t.eq('step 读包字段（结算=6）', S.aset.step, 6); t.eq('结算发事件', evts.indexOf('EQW_ASET_RESULT') >= 0, true); t.eq('非末局不发大局结算事件', evts.indexOf('EQW_ACCOUNT_RESULT') < 0, true); // ================= 投降结算（无 chupai / 无 bottom）================= -S.reset(); evts.length = 0; +S.reset(); evts.length = 0; lastTrick = null; EQW_ResultHandler.handleJiesuan({ success: true, step: 6, aset: { banker: 0, call: 70, multiple: 1, upgrade: -99, seatlist: [] } }); t.eq('投降结算入状态', S.result.aset.upgrade, -99); -t.eq('投降无 chupai', S.result.chupai, null); +t.eq('投降无 chupai → 不发本墩结束事件', lastTrick, null); t.eq('投降无 bottom', S.result.bottom, null); t.eq('投降也发结算事件', evts.indexOf('EQW_ASET_RESULT') >= 0, true);