Bug487-WuYuhan-Refactor #57

mrlan · 2022-11-03T22:16:42+08:00

mrlan commented

2022-11-03 22:16:42 +08:00

重构了吴宇涵(@wuyuhan)在分支 Bug487-WuYuhan 中提交的检查username代码。

新增类 WarningMessage 对 username 的检查（见Login.py），将检查代码封装在 WarningMessage 类中。

好处：account_service.py中，只需要3行代码即可完成对username的检查，见函数"def signup()"。如果还要添加其他的检查，只需要更新 WarningMessage 类即可，改善代码易读性。

-Hui

重构了吴宇涵(@wuyuhan)在分支 Bug487-WuYuhan 中提交的检查username代码。新增类 WarningMessage 对 username 的检查（见Login.py），将检查代码封装在 WarningMessage 类中。好处：account_service.py中，只需要3行代码即可完成对username的检查，见函数"def signup()"。如果还要添加其他的检查，只需要更新 WarningMessage 类即可，改善代码易读性。 -Hui

mrlan added 4 commits 2022-11-03 22:16:43 +08:00

286e884dd8 Refactor Wu Yuhan's code

3cfec31c3f Login.py: add missing colon

f0b5adc5e4 Login.py: fix function name

702205940c Login.py: must convert warn to string before comparing to OK

mrlan added 1 commit 2022-11-03 22:21:45 +08:00

d3a796428d account_service.py: module re is no longer necessary.

mrlan added 1 commit 2022-11-03 22:28:35 +08:00

29ffada7eb Login.py: improve comments.

mrlan merged commit f909201615 into master

2022-11-08 19:53:10 +08:00

mrlan referenced this issue from a commit