上次发布代码的时候 再某个model类里面发现了一段比较奇葩的代码,代码如下:

$req_arr['PosFlag'] = 0;//上生产要改成2

顿时就被这种写法弄得雷死了。这种各种环境不同的情况不应该使用配置项来处理吗?不错,程序猿是可以懒,大家也推荐程序猿懒,

但这种做法不是懒,是没有脑子。

说起来这个还想起来上次发现的一个问题。再模板文件里面写死链接地址(链接地址也是有平台差异的)。我也是无力吐槽了。

 

一句话,请谨记:所有平台差异都不应该hardcoding在代码里面,都必须写到配置里面。

还有一句,别特么动不动就写1,2,3这样的魔术数字,你特么注释都不加,鬼知道这些带别什么意思。