团队中成员编程风格差异好大

[rming]

个人不敢说哪个好，哪个坏，但是这种编程风格的差异，直接导致一些代码不能够被有效的复用，来张图大家吐槽

[teriyaki]

1.模块解耦好 这个问题不大
2.不要总想着复用写类 有时copypaste更好

[Septembers]

“软件，就像所有技术一样，具有天然的政治性。代码必然会反映作者的选择、偏见和期望。”
see http://kb.cnblogs.com/page/79308/

[Matrixbirds]

所以要统一一下coding style

[MrEggNoodle]

大家坐下来讨论吧，选择一种规范。

[haiyang416]

这东西不是规范要求怎么这就怎么写么？如果没有规范，那就没有槽点。

[Septembers]

@haiyang416 像golang那样强行格式化统一代码风格不是每个团队/人都能接受的

[Decadent]

为什么谷歌要执行严格的代码编写规范 | 外刊IT评论
http://www.vaikan.com/google-coding-standards/

[alphonsez]

没必要统一也统一不了。就像最终的代码质量也是参差不齐。
有用的代码一定会被一遍一遍重写到足够好为止。没用的代码who cares.

[crystaldust]

没怎么写过PHP，没看出来啥问题啊？

[sobigfish]

php-cs-fixer
在git push 前hook

[rming]

@Septembers

[rming]

@Septembers
“软件，就像所有技术一样，具有天然的政治性。代码必然会反映作者的选择、偏见和期望。”
学习了。
如果是我，我觉得可能会这样写


[Septembers]

@rming $conditon 这里应该用复数形式吧？

[picasso250]

纠结大括号的程序员将来都不是成功的CTO

[raincious]

想知道能写这样代码的程序员值多少钱。

一个叫tableCondition的方法硬依赖了$_GET（直接$_GET然后intval就不多说了）。

正确的API应该是：
public function tableCondition($pid)

（其实不应该是buildTableCond么？）

只有控制器能够接受用户输入，其他模块接收来自内部的传递的参数（比如来自控制器处理后的）。然而看截图的样子应该是Model吧？

[rming]

@Septembers 😅
@raincious 看看就好，不要激动

[hellokittyer]

就看leader了

[Keita1314]

@raincious 不太清楚硬依赖$_GET是什么意思，想请教一下

[rming]

@Keita1314 quick and dirty 仅供吐槽

[sogood]

用 phpfmt