search

swlib / saber

⚔️ Saber, PHP异步协程HTTP客户端 | PHP Coroutine HTTP client - Swoole Humanization Library
https://packagist.org/packages/swlib/saber
Apache License 2.0
980 stars 124 forks source link

文档里面对php版本要求是7.1实际上7.0也可以吧 #2

Closed gouchaoer closed 5 years ago

gouchaoer commented 6 years ago

我看composer里是70，源码里也没啥不兼容70的地方，这个库简直php爬虫工程师的核弹啊

twose commented 6 years ago

@gouchaoer 可能因为一些问题, swoole2.x一度移除了PHP7.0支持, 现在swoole4.0是绝对稳定, 可以在任意场景使用协程(之前不能使用魔术方法等), 所以最近会抽时间更新这个lib, 发正式版.

shuyabin commented 6 years ago
twose commented 6 years ago

@shuyabin 看了一下, 还真是, PHP71才支持null的类型声明, 这个库全都是用最新写法的, 既然用swoole了, 最好还是持续拥抱最新版本吧

PHP 7.1 allows for void and null return types by preceding the type declaration with a ?

那么最低版本还是得PHP71了, 如果说5 => 7有历史困难, 70 => 71 其实并不存在这种问题.

gouchaoer commented 6 years ago

依赖swoole的话，和swoole/swoft之类的保持一致从70开始是不是更好一点，因为是基础组件，那几个返回值地方感觉不用影响不大

twose commented 6 years ago

最近发现swoole在PHP7.0下有部分测试跑不通, 70底层还是有点问题的, 说不定会踩雷... 结合swoole的一些小bug(最近才修的), 甚至可能需要搭配swoole4.0.3才能发挥100%威力(比如连接池等) 讲道理感觉现在71占有比会高一些? 推荐还是71及以上比较省心, 兼容的话有时间清理一下返回值(不支持?的话返回值类型提示也不能带, 不然返回null会报错)

xuhuaiqu commented 5 years ago

公司版本升级到7.1 也需要不少人配合才行 虽然眼馋saber目前看来无法支持7.0啊

twose commented 5 years ago

7.0已被PHP官方废弃, Swoole也即将移除PHP7.0的支持