准备修改离职同事的一个项目，所有代码只有开头一行注释，看着头晕。

[go522000]

截一张图给大家感受一下。 https://static01.imgkr.com/temp/372735ce126444a283e8856dd5a4c56e.png

没办法修改，实在想不通这个同事是什么意思。 是对自己代码的自信？ 还是故意留坑？

唉，我自己 15 号也要走了，只能和大老板说自己能力有限，无能为力。

[XiLingHost]

这个命名看着挺舒服的，似乎也不是很需要注释吧

[monkeylmj]

个人觉得注释并不是越多越好，最舒服的代码是通过合理命名就能看懂意思，根本不需要注释

[mayday526]

这不是看着还行吗，还需要啥注释？

[rabbbit]

你们公司允许就这么把代码贴出来?

[CookCoder]

这代码很差？

[BiChengfei]

除了方法上注释不要使用单行注释，其他没什么问题。这个业务逻辑感觉挺简单，写的也很清晰

[cpstar]

这就当一个黑盒，不也没啥问题啊

[codespots]

连个业务逻辑相关的说明都没有，那么多人觉得这叫还行？可能我水平不够，所以只能在写代码时多加注释了

[ration]

这业务代码确实需要多注释文档才容易理解。不然要花很多时间。可能改成面向对象的写法好点。

[haah]

你有时间发帖，却没时间怼他？

[461da73c]

这 PHP 写的挺干净的。

[go522000]

向支持不用注释的各位请教一下，第 619 行，JoinService::JoinField(...) 这个函数里面大约还有 40 多行代码（整个文件都没注释），大家在不把代码过一遍的情况下，能不能猜出这一行是什么意思呢？最后会返回什么内容呢？

[go522000]

@haah 哈，不怼他，我还有 9 天也离职了，直接告诉老板我能力有限最方便了。

[anerevol]

@go522000 就是 useraddress 存的是 province_id city_id ， 要 join 表查一下对应的 province_name city_name 这些吧
其实这种通用的业务规则还好 那种专有的业务规则不写注释才是会死人的

[fgwmlhdkkkw]

我觉得写的挺好的……

[rabbbit]

@go522000
都要走了还改啥啊.把图删了吧,别让你们老板抓住把柄.

[zhzy0077]

说实话这代码比绝大多数屎山强太多 最要命的其实不是这个 JoinService 顶天了点进去看一遍 反而是 $flag, getCode, $data 这些名字 比如$flag == 0 的判断虽然可以从 if 里看出应该是判断发货还是收货的 这样的变量名其实毫无意义 还不如 abcd 至少写的人会有愧疚感

[cloudsigma]

删图吧

[Building]

我觉得写的挺好的……

[zengguibo]

这种代码一看就好理解，那种大神写的奇巧淫技才看不懂呢