Skip to content
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

添加容器适配器管理器,并重构 DynamicContainerOperatorProxyMethodFactory 中适配器部分代码 #149

Closed
Createsequence opened this issue Sep 10, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request refactor Refactoring code to optimize or add functionality
Milestone

Comments

@Createsequence
Copy link
Collaborator

目前 DynamicContainerOperatorProxyMethodFactory 中用于将方法参数适配为容器的代码实现目前看来并不灵活。一方面,将对象适配为容器的逻辑强耦合与 DynamicContainerOperatorProxyMethodFactory,并不符合单一职责原则,也增加了测试难度,另一方面,适配器这套功能在 #135 也会用到,重复实现相同的功能并不是个好点子。

综上所述,需要将 DynamicContainerOperatorProxyMethodFactory 中的容器适配器的逻辑从该里中提前为独立组件,并额外添加一个适配器管理器/注册表,便于后面在其他的组件中复用这部分功能。

@Createsequence Createsequence added enhancement New feature or request refactor Refactoring code to optimize or add functionality labels Sep 10, 2023
@Createsequence Createsequence added this to the release 2.2.0 milestone Sep 10, 2023
@Createsequence Createsequence self-assigned this Sep 10, 2023
Createsequence added a commit that referenced this issue Sep 10, 2023
…r converting an object into a container (GitHub #149)
Createsequence added a commit that referenced this issue Sep 10, 2023
…ontainer adapters with global container adapters (GitHub #149)
Createsequence added a commit that referenced this issue Sep 10, 2023
* feat(ContainerAdapterRegister): add an adapter and adapter manager for converting an object into a container (GitHub #149)

* refactor(DynamicContainerOperatorProxyMethodFactory): replace local container adapters with global container adapters (GitHub #149)
Createsequence added a commit that referenced this issue Sep 25, 2023
* feat(ContainerAdapterRegister): add an adapter and adapter manager for converting an object into a container (GitHub #149)

* refactor(DynamicContainerOperatorProxyMethodFactory): replace local container adapters with global container adapters (GitHub #149)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor Refactoring code to optimize or add functionality
Projects
None yet
Development

No branches or pull requests

1 participant