Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: GemsFarming hard mode #4432

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

sui-feng-cb
Copy link
Contributor

#4022 #4381
3油c图紧急委托的实现,通过选择舰队界面进入换装备和换船,先长按进入换装,换装完成后多一步撤下该船,然后再根据原来的逻辑换船换装。
通过在GemsFarming类中覆写父类中的fleet_enter, fleet_enter_ship, fleet_back函数实现换装
当没有可用的白皮航母时,会选择船坞内的高等级航母

Comment on lines 73 to 94
HARDMODEMAPS = [
'c1', 'c2', 'c3',
'd1', 'd2', 'd3',
'ht1', 'ht2', 'ht3', 'ht4', 'ht5', 'ht6',
]
if self.config.Campaign_Name.lower() in HARDMODEMAPS:
logger.info('Is in hard mode, switch ship changing method.')
self.hard_mode = True
self.page_fleet_check_button = FLEET_PREPARATION
if self.config.Fleet_FleetOrder == 'fleet1_standby_fleet2_all':
self.fleet_detail_enter_flagship = self.fleet_enter_flagship = FLEET_ENTER_FLAGSHIP_HARD_2
self.fleet_detail_enter = self.fleet_enter_ = FLEET_ENTER_HARD_2
else:
self.fleet_detail_enter_flagship = self.fleet_enter_flagship = FLEET_ENTER_FLAGSHIP_HARD_1
self.fleet_detail_enter = self.fleet_enter_ = FLEET_ENTER_HARD_1
else:
self.hard_mode = False
self.page_fleet_check_button = page_fleet.check_button
self.fleet_detail_enter_flagship = FLEET_DETAIL_ENTER_FLAGSHIP
self.fleet_detail_enter = FLEET_DETAIL_ENTER
self.fleet_enter_flagship = FLEET_ENTER_FLAGSHIP
self.fleet_enter_ = FLEET_ENTER
Copy link
Owner

Choose a reason for hiding this comment

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

emm,这样的代码质量可不过关哦

Copy link
Contributor Author

Choose a reason for hiding this comment

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

请问是要更换判断困难图的方式吗,还有允许用这样的变量来表示普通图和困难图的不同图片素材吗

Copy link
Owner

Choose a reason for hiding this comment

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

肯定不能这样写啊,一堆游离的类变量 还被patch,谁维护谁要骂人,你要分别实现普通和困难的换船和换装备

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改,把换船相关的函数放到了一个新文件里,定义了两个类分别实现普通和困难的换船换装

@LmeSzinc LmeSzinc added the 需要修改 / Request changes Changes required in PR label Dec 20, 2024
@sui-feng-cb sui-feng-cb force-pushed the gemsfarming_hardmode branch 2 times, most recently from 6148a11 to 8a10f28 Compare December 21, 2024 16:22
@sui-feng-cb
Copy link
Contributor Author

@LmeSzinc 已修改

@sui-feng-cb sui-feng-cb force-pushed the gemsfarming_hardmode branch 3 times, most recently from 2aef7ae to 481ba69 Compare January 7, 2025 08:14
@sui-feng-cb sui-feng-cb force-pushed the gemsfarming_hardmode branch from 481ba69 to 2a01e44 Compare January 9, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
需要修改 / Request changes Changes required in PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants