-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
建议能不能让控制器、模块这些都能支持完整的命名空间 #539
Comments
有同样的需求,现在带action的方案看起来不太好 |
有同样的需求,最好是不需要 modules 文件夹,直接 app/admin/controllers 或者 app/controllers 就是单模块模式,取消 action 后缀 |
多模块还是有需求场景的,比如一个大项目中不同的模块有不同的二级域名绑定之类的。 |
我的意思并不是取消多模块,而是程序第一次运行时只能识别时候是多模块模式,而不需要额外添加一个 modules 文件夹来支持多模块。如项目文件夹是 app/controllers 这种结构,那就是单模块,若是 app/admin/controllers,app/mail/controllers 这种结构,则是多模块 |
附议 |
根据 约定大于配置 的原则,这些都不是问题 |
现在基本开源类库都遵循psr规范,希望yaf支持下标准规范,不然引入类库后有文件是命名空间格式,有的不是,感觉很乱 |
Springboot其实也不支持有同名的controller类,在不同的包里面都不行。 |
多模块我认为不是非用不可
这是一个单应用结构,其中application是应用目录,可以看到conf在application外面,而library是在application中的。
|
其实一点都不乱,yaf 的controllers models plugins actions 目录下的类其实都是在根命名空间下的,也就是 |
举例
Admin中有个User控制器,目前文件是
modules/Admin/controllers/User.php,代码是:
期望代码是:
如果能配置把Action后缀给去掉就更好了
目前的问题是:
modules/Admin/controllers/User.php
modules/Api/controllers/User.php
两个文件的全部没有命名空间,IDE会报相同的方法已在项目中多次定义,强迫症看着很难受,而且也不符合完整的命名空间规范。建议可以配置,支持完整的命名空间
The text was updated successfully, but these errors were encountered: