Jaymr/zszq-trs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 步骤3-b 多字段连续手动编辑应累计REV(不再仅最后赢)

Browse Source 
根因：applyBondManualEdit 每次调用先重置全部 bondRev 再只标当前字段，
导致连续手动编辑多字段(不回车)时只有最后编辑的字段留 REV，
中间手工编辑字段的 REV 被静默抹掉，UI 与步骤3-b需求不符。

修复：applyBondManualEdit 改为累计 REV（仅置当前字段、保留其它已手工编辑字段的 REV），
driver/AUTO 仍清。单字段编辑(步骤2/3)行为不变，无回归。

配套：
- 更新 a2d2bb31 中3个旧断言("仅最后留REV")为断言累加，避免测试守护错误行为
- 文档约定3 补充"连续编辑多字段时各手动字段累计REV"

swapCalc.js 为独立 script(?v=JsVersion)，部署 dotnet publish 即生效，无需重建 bundle。
测试：bondCalc 全套 72 例通过。
This commit is contained in:
hjhan
2026-07-30 09:55:57 +08:00
parent a2d2bb31ea
commit 0747deb909
4 changed files with 22 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
YLErpWeb/fe-tests/bondCalc.integration.test.js
+12 -13
View File
@@ -674,7 +674,7 @@ describe('约定1/2/3 状态机：端到端转换', () => {
// ---- 以下为步骤3-b 及多字段组合场景（文档明确列出但此前缺失） ----
test('Step3-b 回车CP→改DP→改YD(均不回车) → 仅最后字段(YD)留REV，前面REV被覆盖', () => {
test('Step3-b 回车CP→改DP→改YD(均不回车) → DP与YD均累计REV（修复后行为）', () => {
const item = makeBondItem();
// 步骤1：回车净价(CP)
SwapCalc.applyBondCalcSuccess(item, 'CP');
@@ -685,20 +685,19 @@ describe('约定1/2/3 状态机：端到端转换', () => {
SwapCalc.applyBondManualEdit(item, 'DP');
expect(item.bondRev).toEqual({ CP: false, DP: true, YD: false });
// ★ 步骤3-b：不回车，又改收益率(YD) → DP的REV被覆盖，仅YD留REV
// ★ 步骤3-b：不回车，又改收益率(YD) → DP的REV保留、YD也标REV（累计，不再被覆盖）
SwapCalc.applyBondManualEdit(item, 'YD');
expect(item.bondDriverType).toBeNull(); // 源已被清
expect(item.bondAuto).toEqual({ CP: false, DP: false, YD: false }); // AUTO 已清
// ⚠️ 核心断言：applyBondManualEdit 每次重置全部 REV 再只标当前字段
// 故多字段连续编辑时"最后赢"，中间字段的 REV 静默丢失
expect(item.bondRev).toEqual({ CP: false, DP: false, YD: true });
// 核心断言：applyBondManualEdit 改为累计 REV，多字段连续手动编辑时每个改过的字段都留 REV
expect(item.bondRev).toEqual({ CP: false, DP: true, YD: true });
// 数值保持不变（约定3：编辑不联动）
expect(item.PosiNetNoFeePrice).toBe(0.995); // 净价保留
expect(item.PosiGrossPrice).toBe(1.0); // 全价保留
expect(item.InitYtm).toBe(0.026); // 收益率保留
});
test('Step3-b 变体：回车CP后连续编辑三字段(CP→DP→YD) → 仅最后(YD)留REV', () => {
test('Step3-b 变体：回车CP后连续编辑三字段(CP→DP→YD) → 三个字段均累计REV', () => {
const item = makeBondItem();
SwapCalc.applyBondCalcSuccess(item, 'CP');
@@ -706,14 +705,14 @@ describe('约定1/2/3 状态机：端到端转换', () => {
SwapCalc.applyBondManualEdit(item, 'CP'); // 先改源字段本身
expect(item.bondRev).toEqual({ CP: true, DP: false, YD: false });
SwapCalc.applyBondManualEdit(item, 'DP'); // 再改第二个 → CP的REV被覆盖
expect(item.bondRev).toEqual({ CP: false, DP: true, YD: false });
SwapCalc.applyBondManualEdit(item, 'DP'); // 再改第二个 → CP的REV保留、DP也标REV
expect(item.bondRev).toEqual({ CP: true, DP: true, YD: false });
SwapCalc.applyBondManualEdit(item, 'YD'); // 最后改第三个 → DP的REV被覆盖
expect(item.bondRev).toEqual({ CP: false, DP: false, YD: true }); // 仅最后赢
SwapCalc.applyBondManualEdit(item, 'YD'); // 最后改第三个 → 三者均累计
expect(item.bondRev).toEqual({ CP: true, DP: true, YD: true }); // 全部累计，不再"最后赢"
});
test('Step3-b 三字段轮换编辑(CP→DP→YD→CP) → 仅最终(CP)留REV', () => {
test('Step3-b 三字段轮换编辑(CP→DP→YD→CP) → 三个字段均累计REV', () => {
const item = makeBondItem();
SwapCalc.applyBondCalcSuccess(item, 'CP');
@@ -723,8 +722,8 @@ describe('约定1/2/3 状态机：端到端转换', () => {
SwapCalc.applyBondManualEdit(item, 'YD');
SwapCalc.applyBondManualEdit(item, 'CP'); // 回到第一个
// 无论轮换多少次，只有最后一次 applyBondManualEdit 的字段留 REV
expect(item.bondRev).toEqual({ CP: true, DP: false, YD: false });
// 无论轮换多少次，每次编辑的字段都累计REV（不再"只有最后一次留REV"）
expect(item.bondRev).toEqual({ CP: true, DP: true, YD: true });
expect(item.bondDriverType).toBeNull();
expect(item.bondAuto).toEqual({ CP: false, DP: false, YD: false });
});
YLErpWeb/wwwroot/Scripts/app/swaptrade/swapCalc.js
+9 -3
View File
@@ -287,13 +287,19 @@
/**
* 交互约定3（纯函数，jest 可测）：编辑某字段但未回车（失焦/输入）时的标识落地。
* - 清除 源 与 AUTO 标识（不再代表任何已算结果）；
* - 仅本字段标 bondRev[type]=true（"REV"=人工输入），不联动另外两字段。
* - 本字段标 bondRev[type]=true（"REV"=人工输入），不联动另外两字段；
* - ★ 累计 REV（不清空其它已手工编辑字段的 REV）：连续手动编辑多字段(不回车)时，
* 每个被改过的字段都保留 REV，使 UI 正确显示"全部为人工输入"，且不会被后续计算器覆盖
* （见文档步骤3-b）。此前的实现每次重置全部 REV 再只标当前字段，导致"只有最后编辑的字段留 REV"，
* 中间手工编辑的字段 REV 被静默抹掉（步骤3-b 不满足需求）。
*/
function applyBondManualEdit(state, type) {
state.bondDriverType = null;
state.bondAuto = { CP: false, DP: false, YD: false };
state.bondRev = { CP: false, DP: false, YD: false };
if (type) state.bondRev[type] = true;
if (!state.bondRev || typeof state.bondRev !== 'object') {
state.bondRev = { CP: false, DP: false, YD: false };
}
if (type) state.bondRev[type] = true; // 累加：仅置当前字段，保留其它已手工编辑字段的 REV
return state;
}
YLErpWeb/wwwroot/Scripts/base/ylotc.base.js
-4
View File
@@ -1,4 +0,0 @@
(function (global) {
}(window));
项目文档/互换债券三字段互算踩坑总结与测试指南.md
+1 -1
View File
@@ -14,7 +14,7 @@
**交互约定**：
- **约定1**：在任一字段**敲回车** → 以该字段为"源"调计算器，反算另两个并覆盖。
- **约定2**：计算器调用成功→源字段标"源"、另两个标"AUTO"；失败→保留源字段值、另两个清空、三标识全清。
- **约定3**：编辑某字段（失焦/按键输入）但**未回车** → 不联动另两字段，清掉源/AUTO标识，仅本字段标"REV"（人工输入）。
- **约定3**：编辑某字段（失焦/按键输入）但**未回车** → 不联动另两字段，清掉源/AUTO标识，本字段标"REV"（人工输入）。**连续手动编辑多字段（均不回车）时各被改过的字段均累计REV**（不清空其它已手工编辑字段的REV），使 UI 正确显示"全部为人工输入"、且后续计算器回写时不会被误覆盖。
---