Snailclimb / guide-rpc-framework

A custom RPC framework implemented by Netty+Kyro+Zookeeper.(一款基于 Netty+Kyro+Zookeeper 实现的自定义 RPC 框架-附详细实现过程和相关教程。)
https://gitee.com/SnailClimb/guide-rpc-framework
Other
3.97k stars 2.06k forks source link

为SPI机制增加策略枚举 #75

Closed xiaoguyueyue closed 1 year ago

xiaoguyueyue commented 1 year ago

guide哥你好,之前看了你开源rpc-guide项目收益匪浅,也在你星球生活了一段时间了,最近冲浪的时候看到策略者模式的一篇文章,后面想到SPI机制也是策略者模式,同时发现项目中用于SPI机制的文件有些没有对应的枚举类,对于SPI机制的每一个文件,我们都需要维护一个策略枚举,让别人知道你当前具有哪些策略,方便其他开发者在使用时进行查看,当然他也可以去配置文件中查看,但是你编程过程经常去翻配置文件,然后填写对应的字符串key,总让人觉得很奇怪,而且如果配置文件里面的值需要进行修改,还得手动全局替换用到该修改值的地方,所以我为SPI文件加了几个枚举类,也修改了代码中直接使用字符串的地方,运行测试都没问题,提交了一个PR,如有错误或者不规范的地方请guide哥指点,再次对duige哥开源的项目表示感谢

Snailclimb commented 1 year ago

guide哥你好,之前看了你开源rpc-guide项目收益匪浅,也在你星球生活了一段时间了,最近冲浪的时候看到策略者模式的一篇文章,后面想到SPI机制也是策略者模式,同时发现项目中用于SPI机制的文件有些没有对应的枚举类,对于SPI机制的每一个文件,我们都需要维护一个策略枚举,让别人知道你当前具有哪些策略,方便其他开发者在使用时进行查看,当然他也可以去配置文件中查看,但是你编程过程经常去翻配置文件,然后填写对应的字符串key,总让人觉得很奇怪,而且如果配置文件里面的值需要进行修改,还得手动全局替换用到该修改值的地方,所以我为SPI文件加了几个枚举类,也修改了代码中直接使用字符串的地方,运行测试都没问题,提交了一个PR,如有错误或者不规范的地方请guide哥指点,再次对duige哥开源的项目表示感谢

不错不错,学习了,这优化对于代码质量清晰度的提升还是很有用的。