再也没有了当年的激情,去写出优雅和兼容并存的代码

查看 1024|回复 81
brader
OP
  
@idealhs 抛开业务场景的话,肯定很难想象啦,这种业务代码要结合业务看才好理解
brader
OP
  
@xinyewdz 我已经摆烂好多年了
eilisonLiang   
@brader 估计要看结果导向吧,有的领导要求能跑就行,有的是要求质量,不同情况写不同的代码,优雅的代码,bug 会少一些,就是要折腾一下
oneisall8955   
能看懂就是好代码
nikola11   
时间有限,就顾不上优雅
zjj19950716   
一头扎进了大染缸
iSNN   
@eilisonLiang 有的领导...有的领导...其实只有一种领导。开发快、质量高、可维护、BUG 少、稳定性好
Rache1   
如果要说你列出的代码是优雅的正例的话,还真有些问题。
1 、魔法字符串,这里的 1000000...、还有 bcdiv 的那个 18 、0x
2 、dexToHex 和 hexToDec 这俩方法 php 都自带了的,我不太确定你这个实现跟自带的有什么区别。
3 、你的 remove0x 和 add0x ,在这里就是个摆设吗,71 行、48 行、这些明显就应该用你这个函数了吧。
4 、再说回你这个 add0x 和 remove0x ,如果是我的话,我会改成 withPrefix 和 trimPrefix
5 、你这里面判断以 0x 开头的,也至少存在 2 处调用了,那就应该也单独提成函数。
6 、你这 48 行的 decToHex 前面拼接的 0x ,不是可以直接传给 dexToHex 的 mark 么
QlanQ   
我还没有,最近接手的代码,让我 无从下手
比起代码,数据库设计的烂才是最难受的
去年的项目用的 yii 1.1 + layerui ,还说是自研
借楼问下,
@coderxy
@NessajCN
@eilisonLiang
@brader
@pkoukk
数据库关联表,冗余数据才是正常的吗?
最近看到好几个项目这样做,我都怀疑自己是不是错了?
azui999   
能跑就行,很多东西,我也把握不了
您需要登录后才可以回帖 登录 | 立即注册

返回顶部