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

[go522000]

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

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

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

[Orenoid]

说烂不至于，说它很清晰可就算了吧。
这么一大堆业务逻辑，不做分段注释，也不做进一步的函数封装。代码里也有明显的分段，每一段稍微注释一下不难吧？

[CookCoder]

@go522000 根据变量名字，地址信息，然后后面的省市 ID ，差不多根据 id 找到更具体的省市信息，这一段也需要注释么？

[CookCoder]

不过，注释写的肯定很差，输入输出起码写，每一个关键变量和业务逻辑起码加一个，619 行这样的不需要注释。

[meeop]

代码挺整洁的,个人觉得并不需要太多注释

以及程序员最讨厌别人不写注释,同时最讨厌自己写注释

注释和文档由于会因为代码变更缺乏维护而辞不达意,更好的做法还是把代码结构写清晰,变量名说清楚

[CookCoder]

说还行的原因是，起码根据变量名字，方法名字，几乎已经看明白这些代码准备干什么，难道说个还行都过分么？

[really28]

又不是不能用，建议各位熟读 https://coderlmn.github.io/frontEndCourse/unmaintainable.html

提升自己核心竞争力。

🐶

[GeorgeGalway]

作为多年的 Phper ，这代码写的，确实还不错

[caixiangyu17]

这命名不太行呀，各种 foo ，v ，r ，k ，data ，temp
不过主要问题是方法太长了，我们 kotlin 项目 linting 要求 function 少于 40 行，一行少于 120char ，多了过不了 pipeline

[QlanQ]

作为多年的 Phper ，这代码写的，确实还不错

[forbreak]

并不是注释越多越好啊。。。。

[mingl0280]

这代码还要注释？
@go522000 619 应该是返回一个集合，包含省市区的地址（字符串）。这个有啥看不懂的？

[ytmsdy]

小伙子，你太年轻了。
至少变量命名清晰，结构也不乱。
你是没有见过糟糕的代码！

[alanyuan]

就这么把公司代码发出来，建议删帖&拉黑

[bugFactory]

@really28 大佬，看了下，真是牛逼坏了呀

[mrgeneral]

看着还行。

你把屏幕整宽点，去掉软换行，就会发现还挺整洁的，基本都是参数转化和提取而已。

[pengtdyd]

这就看懵了？？？小伙子，你太年轻了。1000 行没有注释的 SQL 就问你怕不怕！！！话说 SQL 没注释也正常。

[fyooo]

后续还是不要贴代码吧，严重违背职业道德

[ZoR]

代码看完后 感觉还行能猜到百分之 8 90%了，还是太年轻 有的代码看完以后 不知所云 让人欲仙欲死

[lap510200]

这已经算好的了 。。

[lap510200]

@ZoR 觉得还行的才是久经沙场，比起代码缺注释，数据库字段缺注释的老项目都有很多