Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

提几点建议 #6

Open
nosun opened this issue Jul 30, 2015 · 3 comments
Open

提几点建议 #6

nosun opened this issue Jul 30, 2015 · 3 comments

Comments

@nosun
Copy link

nosun commented Jul 30, 2015

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

@nosun
Copy link
Author

nosun commented Jul 30, 2015

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

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

@137-rick
Copy link
Owner

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

@maievshadow
Copy link

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants