treehollow / treehollow-backend

T大树洞 的Golang后端
https://thuhole.com/
GNU Affero General Public License v3.0
167 stars 47 forks source link

彻底的【代码重构】和【SQL数据库结构重构】 #20

Closed thuhole closed 3 years ago

thuhole commented 4 years ago

由于第一次发布的版本为敏捷开发方式,SQL数据表有许多地方设计欠妥,代码更是丑陋不堪,仅仅是勉强满足了用户需求。

待树洞管理完善后,计划于2020年7月底之前彻底重构代码和数据库,选择某一天的树洞低峰时段(凌晨5点)进行彻底的系统升级。

此帖用于落实重构细节,欢迎所有人来讨论!

thuhole commented 4 years ago

预计MySQL db大致分为以下几个Table:

具体的Column有待设计。

yzs981130 commented 4 years ago

golang code部分,目前的unit test的结构是有问题的,建议变成

- src
  - utils
    - utils.go
    - utils_test.go
  - config
    - config.go
    - config_test.go
...

不需要test的模块不加test即可,最终可参考类似 https://github.com/kubernetes-sigs/kube-batch/tree/master/pkg/scheduler 类似这种 https://github.com/yzs981130/thuhole-go-backend/tree/fix-email_validation/src 才可在相应目录下做go test,这里只会做utils_test.go里的test

thuhole commented 4 years ago

功能列表:

可能添加的功能:

thuhole commented 4 years ago

golang code部分,目前的unit test的结构是有问题的,建议变成

- src
  - utils
    - utils.go
    - utils_test.go
  - config
    - config.go
    - config_test.go
...

不需要test的模块不加test即可,最终可参考类似 https://github.com/kubernetes-sigs/kube-batch/tree/master/pkg/scheduler 类似这种 https://github.com/yzs981130/thuhole-go-backend/tree/fix-email_validation/src 才可在相应目录下做go test,这里只会做utils_test.go里的test

采纳。

yzs981130 commented 4 years ago

golang code部分,目前的unit test的结构是有问题的,建议变成

- src
  - utils
    - utils.go
    - utils_test.go
  - config
    - config.go
    - config_test.go
...

不需要test的模块不加test即可,最终可参考类似 https://github.com/kubernetes-sigs/kube-batch/tree/master/pkg/scheduler 类似这种 https://github.com/yzs981130/thuhole-go-backend/tree/fix-email_validation/src 才可在相应目录下做go test,这里只会做utils_test.go里的test

采纳。

我可以今晚或者明天在现在的master基础上做一下各个模块分离的代码重构,不知道attention表重构是否已经完成?可以商量一个方便的时间,我提一个模块代码结构分离+test+CI的pr,希望pr前后master尽量freeze即可

thuhole commented 4 years ago

@yzs981130 attention表已经重构完了。 非常感谢您!!!!要不邮件联系?

thuhole commented 4 years ago

@yzs981130 今晚吧?

yzs981130 commented 4 years ago

@yzs981130 attention表已经重构完了。 非常感谢您!!!!要不邮件联系?

邮件联系没问题。 单开了issue #38