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.

Sign up for GitHub

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

Jump to bottom

replace project struct to depend inject(di) #1

Open

DGuang21 wants to merge 1 commit into gogf:main from DGuang21:inject

Member

DGuang21 commented Mar 11, 2023

No description provided.


          replace project struct to depend inject(di)

937114a

gqcn added the discus label

Member

gqcn commented Mar 13, 2023

@DGuang21
在wire引入的前提下，如何解决以下两点：

接口化维护wire依赖，而不是直接依赖logic实现
去掉service后如何解决循环import问题
如果这两点能解决得好，引入wire确实能提高目前的工程化管理。

Member

gqcn commented Mar 13, 2023

目前gf的工程管理存在以下两点问题：

不太优雅的初始化管理，严重依赖代码生成，手工维护较繁琐。这点也是 @深圳-大广-打杂目前提的pr期待解决得。
不支持di特性，使得业务项目对单测编写不是很友好。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels