search

sanmusen214 / BAAH

A script that can automatically finish daily tasks in Blue Archive (global/Japan/CN/CN bilibili server). 碧蓝档案国际服/日服/蔚蓝档案国服/B服每日任务脚本
MIT License
438 stars 17 forks source link

Refactor gui #113

Closed ly5519 closed 2 weeks ago

ly5519 commented 2 months ago

when I want to test the first module, gui, is diffcult, so I want to refactor it first, do you like it? first submit a check for you, give me some advise if you dont like this behavior

103

sanmusen214 commented 2 months ago

目前这个GUI的文件结构更改看起来不错。感觉GUI架构的主要缺点是没有对象化各个配置项，纯列表操作导致扩展性和理解性较差，后面可以看看修改下对象化各个配置项以支持解析与导出（可以先改一个配置项然后review一下效果

ly5519 commented 1 month ago

中场休息，森哥检视一下, 模块化正在弄，先拆每个json文件的，最后拆每个json文件里面每一个小项的，必须要先提交一波了，要不然搞不清楚自己到底在写啥，需要commit摆正自己正在干的事

sanmusen214 commented 1 month ago

必须要先提交一波了，要不然搞不清楚自己到底在写啥，需要commit摆正自己正在干的事

目前为止well done，不过你好像没有保留启动时给定参数的功能， 定位到 #96 和commit 04bfec2

ly5519 commented 1 month ago

必须要先提交一波了，要不然搞不清楚自己到底在写啥，需要commit摆正自己正在干的事

目前为止well done，不过你好像没有保留启动时给定参数的功能， 定位到 #96 和commit 04bfec2

因为本次重构还是为了unittest服务，所以搞不懂的代码 & 删了好像还能跑的代码一律删除，有点破坏性修改了，看了PR和issue大概知道是干啥的了，之后重构这部分重新添加上去

ly5519 commented 1 month ago

因为有部分功能排在了此PR之后不能，再拖了😌菜的安详，麻烦森哥看一眼吧帮忙排一下版之类的，nicegui太难写了

sanmusen214 commented 1 month ago

最近有点忙，等几天哈

sanmusen214 commented 2 weeks ago

已合并，我测几天