Skip to content

Conversation

@zhonghui1966
Copy link
Contributor

@zhonghui1966 zhonghui1966 commented Oct 4, 2025

PR受影响的平台

诱因和背景

chooseToCompare内的filterCard接口不完善,ai检测不到并且如果所有手牌都不通过检测会导致卡住

PR描述

完善chooseToCompare事件:
1.增加lib.filter.canBeComapred,传参有card, player, event,包含checkMod,传参同上,event传进mod里是技能名,想整花活自己getHistory去
2.修改player.js中的canCompare, chooseToCompare函数,适配修改
3.修改content.js中的三种chooseToCompare事件,均适配event.position和新的filterCard
4.修改手杀势杨弘,梦司马昭,手杀笮融,界太史慈的技能以适配新的修改
prettier格式化

PR测试

测过了

扩展适配

修改filterCard的扩展要么重新apply一遍lib.filter.canBeComapred(可能导致相互覆盖),要么也改成mod的形式

检查清单

  • 我没有把该PR提交到master分支
  • commit中没有无用信息,和没有具体内容的“bugfix”
  • 我已经进行了充足的测试,且现有的测试都已通过
  • 若我拥有PR标签权限,则已确保为该PR打上标签;若我未拥有PR标签权限且该PR仍需继续提交内容,则已确保为该PR名称打上WIP直到本PR内容全部提交
  • 如果此次PR中添加了新的武将,则我已在character/rank.js中添加对应的武将强度评级,并对双人武将/复姓武将添加name:xxx的参数
  • 如果此次PR中添加了新的语音文件,则我已在lib.translate中加入语音文件的文字台词
  • 如果此次PR涉及到新功能的添加,我已在PR描述中写入详细文档
  • 如果此次PR需要扩展跟进,我已在扩展适配中写入详细文档
  • 如果这个PR解决了一个issue,我在诱因和背景中明确链接到该issue
  • 我保证该PR中没有随意修改换行符等内容,没有制造出大量的Diff
  • 我保证该PR遵循项目中.editorconfig、eslint.config.mjs和prettier.config.mjs所规定的代码样式,并且已经通过prettier格式化过代码

修复chooseToCompare和chooseToCompareMultiple事件的filterCard接口如果均无法用于拼点卡死的问题
@Cicardo-Thicavasco
1.补齐了剑阁武将的介绍
2.给明刑柱加了个换形态原画的功能
3.修改了颛顼的翻译
This reverts commit f124bfc.
参数描述修改
适配幻曹冲的canCompare及其技能修改
为什么漏了个括号?
修复势杨弘,手杀笮融mod会导致其他s区牌也被用于拼点的问题
修复梦司马昭插入结算后拼点效果无法维持的问题
知道错了,别拷打我了~(∠・ω< )⌒
以及配置好的格式化重新回调
@zhonghui1966
Copy link
Contributor Author

现在没有大量diff了 😄

@mengxinzxz
Copy link
Collaborator

拼点存在规则漏洞,已经进入拼点流程之后的无法拼点判定只有无手牌,直接对初始手牌进行filter可能会让秦宓天辩类技能无法使用,建议完善“可拼点”概念

@zhonghui1966
Copy link
Contributor Author

拼点存在规则漏洞,已经进入拼点流程之后的无法拼点判定只有无手牌,直接对初始手牌进行filter可能会让秦宓天辩类技能无法使用,建议完善“可拼点”概念

就是兼容onCompare?让秦宓这些有手牌就能天辩?

把所有的判断都挪到canCompare那里
以及checkMod有个写错了
@zhonghui1966
Copy link
Contributor Author

拼点存在规则漏洞,已经进入拼点流程之后的无法拼点判定只有无手牌,直接对初始手牌进行filter可能会让秦宓天辩类技能无法使用,建议完善“可拼点”概念

我按我的理解改完了,但是还有一个问题,秦宓这一类的武将如果手牌全部无法用于拼点,进入选牌的时候技能按钮会出不来,我看tianbian的filter是过了的,不知道为什么

@rintim rintim requested review from a team October 14, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants