137-rick / Dora-RPC

DoraRPC is an RPC For the PHP MicroService by The Swoole
468 stars 101 forks source link

提几点建议 #6

Open nosun opened 9 years ago

nosun commented 9 years ago

代码写的很工整,注释也比较清楚,赞一个!这里提几点建议,仅供参考~ 1、虽然很容易看懂,也有例子,但最好再详细介绍一下,比如协议设定,使用方法。 2、服务端需要在初始化的时候载入类库,例子中没有说明。 3、最好采用配置文件的方式,而不是每次去修改类。 4、服务端可以引入控制层,可以启动服务,也可以关闭服务,便于管理。 5、从抽象的角度,应该可以适配更多的协议。 6、可以参考psr-4规范,引入命名空间,自动载入。

nosun commented 9 years ago

'package_length_offset' => 0, 'package_body_offset' => 4,

最新版的swoole文档中没有看到这两个配置

137-rick commented 9 years ago

:)很高兴你的参与~ 以上问题已经记录,稍晚我将继续优化以上所提出的问题~ 另外后面两个配置没有显示,是因为swoole官方的wiki是多人维护的这个选项可能忘记弄了~ 如果对swoole有兴趣建议别光看wiki了,加入群可以和作者互动:)

maievshadow commented 8 years ago

小龙要支持代码写的规范些,也支持别人写接口参与~哈哈哈哈