leader 一定要我用 array()不让我用[]怎么办?

2016-10-25 10:06:31 +08:00
 checgg

我真是、、、、

5739 次点击
所在节点    PHP
46 条回复
overlords
2016-10-25 10:08:21 +08:00
你的 leader 考虑兼容性吧
congjf
2016-10-25 10:10:43 +08:00
那你为什么一定要用 [] ?
FrankFang128
2016-10-25 10:12:50 +08:00
怼领导有好处?
qhxin
2016-10-25 10:12:58 +08:00
兼容性,没毛病
penjianfeng
2016-10-25 10:17:34 +08:00
新特性自己项目用用就行,公司项目如果不知道线上版本,老老实实写旧语法,不然等你遇到坑了你就会知道你 leader 当初让你写旧语法的原因了
xujif
2016-10-25 10:21:29 +08:00
[] 是 5.4 开始的吧,那就是 trait ,命名空间都不能用喽。。那就一点都不好玩了啊。
xujif
2016-10-25 10:21:57 +08:00
[]对于 php 来说不能说是新特性了吧。。。
@penjianfeng
checgg
2016-10-25 10:22:08 +08:00
公司 PHP 版本本来是 5.4 以上的版本。
我喜欢用[]啊。这样写起来舒服,好看。
array 好丑啊。
而且现在 return 我也尽量用 True 和 False 。
这样就不用考虑我是在写 python 还是 php 了。。。
zsdsz
2016-10-25 10:27:46 +08:00
我一直用 array()
array_column 这个函数我之前也一直避免使用
看到别人的代码没人考虑向下兼容后才开始用的
shuimugan
2016-10-25 10:28:41 +08:00
如果你项目依赖的库都用了[]和 Traits 等 php5.4 起的特性,那么你项目为什么还不用[]?根本就换不回 5.3 了啊
Sunyanzi
2016-10-25 10:31:52 +08:00
各个公司情况不同啊 ... 线上在边边角角还有不少 5.3 的机器的话不这么写还能怎么写 ..?

说来我倒是强制要求我的团队必须写 [] 不能写 array() ... 但我们的开发环境是 php 7.1 RC ...

我还要求每个人必须显式声明参数类型和函数返回呢 ...
checgg
2016-10-25 10:40:50 +08:00
@Sunyanzi
我自己本地跑的也是 7.0 的版本。强制要求写[]真是极好的。
不过参数类型和函数返回值我觉得没必要。
还有我个人认为良好的项目代码,函数前面基本不需要写注释,函数名和参数名就已经说明了一切了。
在每个函数前面加上这么一小坨真的好丑。
bombless
2016-10-25 11:03:50 +08:00
可怕,不知道啥时候能一步到位上 7
poke707
2016-10-25 11:27:00 +08:00
怕增加其他队员学习成本?
checgg
2016-10-25 11:46:24 +08:00
@bombless 你是说他可怕还是我可怕。
cuebyte
2016-10-25 11:47:35 +08:00
你写你的,他还能把你开了不成?我特么最受不了的就是 array()了
elvba
2016-10-25 11:48:22 +08:00
虽然不是你的问题,但是你怼不过你的 leader 啊
话说我这边的环境还是 5.2 , 12 年的项目,命名空间啥的都没,不过在自己手撸了一套中间件、简单的 ORM 、表单验证之后,用起来也没那么差
akira
2016-10-25 11:52:25 +08:00
公司一定要有一个统一的代码规范要求。

要么你说服 leader 改规范,
要么你就按照 leader 要求走
Lonely
2016-10-25 11:56:35 +08:00
这么点事也要上来说,能接受就用,不接受就走人。
zhanglintc
2016-10-25 12:08:30 +08:00
同意 19 楼啊, 在团队中就按照团队要求来.

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/315222

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX