Skip to content

Conversation

@xizifu
Copy link
Collaborator

@xizifu xizifu commented Oct 13, 2025

PR受影响的平台

诱因和背景

1.目前本体仅支持仁库这种游戏外的公共区域,#Player.lose、#Player.gain、#Player.addToExpansion、Game.cardsGoToSpecial等均仅限制toRenku/fromRenku。考虑到存在其他可能的的公共区域,比如额外牌堆、欢杀谋黄月英的牛马区,现在支持类似仁库的处理方式的游戏外的公共区域的添加,可以通过在lib.arenaReady中/技能中对lib.commonArea进行操作设置新区域,并添加支持关闭的显示公共区域的设置
2.lib.element.dilaog支持使用handle方法直接操作对话框,详见赵襄【扶汉】、刘渊【兼习】等技能;
3.bug反馈 #3046

PR描述

1.将仁库设置到lib.commonArea中进行统一管理;
2.修改本体与操作仁库相关的技能;
3.修改lib.element.player.lose、lib.element.player.gain、lib.element.player.addToExpansion、Game.cardsGoToSpecial、lib.element.content.lose、lib.element.content.addToExpansion、lib.element.content.cardsGoToSpecial、lib.element.content.cardsGoToPile、lib.element.player.content.useCard、lib.element.player.content.equip支持类似仁库的游戏外公共区域的添加/移出/刷新操作;
4.添加lib.config.show_commonCardpile(显示游戏外公共区域(仁库等)按钮),开启后可与查看牌堆一样查看各个游戏外的公共区域的牌;
5.lib.element.dilaog支持使用handle方法直接操作对话框,并调整手杀曹髦【清正】、手杀陆郁生【润微】、孙韶【筑墼】、标郑玄【整经】、刘渊【兼习】、十周年赵襄【扶汉】等技能,用法也请参考以上技能;
6.修复 #3046 反馈的bug;
7.lib.element.content.guanxing改为async content;
8.将player.getStat().card[name]相关的技能进行数值判断再执行减值操作;
9.武将称号拆分到各武将包文件中;

PR测试

已使用张机进行联机/单机测试
ib.element.dilaog支持使用handle方法已使用调整武将技能进行联机/单机/托管测试

扩展适配

修改了上述提到的方法/技能的扩展需要跟进

检查清单

  • 我没有把该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遵循项目中.editorconfigeslint.config.mjsprettier.config.mjs所规定的代码样式,并且已经通过prettier格式化过代码

@xizifu xizifu changed the title 添加游戏外公共区域功能 添加查看/设置游戏外公共区域功能 Oct 13, 2025
@mengxinzxz
Copy link
Collaborator

范进不看便罢,看了一遍,又念一遍,自己把两手拍了一下,笑了一声,道:“噫!好了!我中了!”说着,往后一跤跌倒,牙关咬紧,不省人事。老太太慌了,慌将几口开水灌了过来。他爬将起来,又拍着手大笑道:“噫!好!我中了!”笑着,不由分说,就往门外飞跑,把报录人和邻居都吓了一跳。走出大门不多路,一脚踹在塘里,挣起来,头发都跌散了,两手黄泥,淋淋漓漓一身的水。众人拉他不住,拍着笑着,一直走到集上去了。众人大眼望小眼,一齐道:“原来新贵人欢喜疯了。”
9CP)48 YT1ZIDNASY9`8MVT

* 处理区域中移出的卡牌,目前仅作为刷新区域的简单使用
* @param {Card[]} cards 区域中需要移出的卡牌
*/
removeHandeler(cards) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里removeHandler能不能也变成content呢

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

对,我也感觉是不是也做成事件好一点,一半是一半不是有点小怪()

@Spmario233
Copy link
Collaborator

与其对牌移动事件狠狠加屎山
要不要考虑直接重构牌移动事件呢

@Spmario233
Copy link
Collaborator

如果重构的话,不仅仁区的溢出,装备区等区域的溢出和阻断或许也可以采取同样的类似方式来管理

@rintim rintim requested review from a team October 14, 2025 11:20
@mengxinzxz
Copy link
Collaborator

approve目前的修改只代表代码可以正常跑,但特定区域进出卡牌建议统一为事件,方便进行事件管理(移动事件重构也可以用这个思路)

@xizifu xizifu changed the title 添加查看/设置游戏外公共区域功能 添加查看/设置游戏外公共区域功能;bugfix Oct 16, 2025
@xizifu xizifu merged commit b556f7a into libnoname:PR-Branch Oct 16, 2025
1 check passed
yx-lingmeng pushed a commit to yx-lingmeng/noname that referenced this pull request Oct 16, 2025
添加查看/设置游戏外公共区域功能;bugfix
yx-lingmeng pushed a commit to yx-lingmeng/noname that referenced this pull request Oct 16, 2025
添加查看/设置游戏外公共区域功能;bugfix
yx-lingmeng added a commit to yx-lingmeng/noname that referenced this pull request Oct 18, 2025
Merge pull request libnoname#3036 from xizifu/PR-Branch

添加查看/设置游戏外公共区域功能;bugfix

武将称号拆分到各武将包文件

将player.getStat().card[name]相关的技能进行数值判断再执行减值操作

removeHandeler事件化;lib.element.content.guanxing改为async content

Merge pull request libnoname#3036 from xizifu/PR-Branch

添加查看/设置游戏外公共区域功能;bugfix

Merge branch 'libnoname:PR-Branch' into PR-Branch
yx-lingmeng pushed a commit to yx-lingmeng/noname that referenced this pull request Oct 18, 2025
添加查看/设置游戏外公共区域功能;bugfix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants