读代码的时候，什么时刻最破防？

[gpsbird]

[Justin13]

破防到没有，最多偶尔气笑了

[iyiluo]

给一份屎山代码，需要加一个新功能，明天上线

[falsemask]

@Dream95 这个是不是因为项目配了 check style 不允许出现魔法数字

[godloveplay]

公司有个项目，注释都是看代码能看出来的：“遍历 list 去出 字段 xxx ”，“去 xxx_xxx 表中查询字段 xxx”
反倒是一句业务含义的注释都没有，十分无语。

[AchieveHF]

屎上雕花

[xiangran0028]

进这个公司的时候，上一任已经跑了，没有交接，留下了 4 个机型，然后我发现每个机型居然都用的不同的框架（因为前任有好几个），最离谱的是一个 function 居然有 15K 行，里面各种全局变量各种赋值

[Aruok]

变量随意命名, 拼音变量就不说了, a1, a2 这种的, 方法名 handleData, deleteItem 的, 不注释的话根据不知道干嘛的. 一个方法还写的老长了...

[GeruzoniAnsasu]

[28Sv0ngQfIE7Yloe]

昨天发现同事写了一段

delete （ redisKey ）;
set(redisKey,value);

不知道要干啥，直接 set 不行吗？

[assiadamo]

for (int i = 0; i < 3; i++) {
if (i == 0) {
// ...
} else if (i == 1) {
// ...
} else if (i == 2) {
// ...
}
}

[chawuchiren]

@Dream95 这个有什么问题嘛，不是业务变量，我们常量池就这么定义的

[lucifer9]

刚工作时候接过个紧急需求，改一位前 acm 金牌同事的代码。里面变量名真的是 a ，b ，c…这种

[hhhh115]

@hahastudio #5 回旋镖

[cndenis]

@MillaMaxwell 你还别说, 这是 Python 的 EAFP 风格, 请求宽恕比许可更容易(Easier to Ask for Forgiveness than Permission), 用 try...except 代替前置的 if 判断
https://docs.python.org/3/glossary.html#term-EAFP

对于并发的场合, 这种做法是必要的, 比如你从一个并发的任务列队中取值, 即使先用 if 判断队列非空, 取回来的值也有可能是空的, 正确的做法是 try 取值干活 except Empty ...

[fredweili]

用 copilot refactor

[Sawyerhou]

debug 一个老同事的跑不起来的脚本，一打开

一行注释都没有，一个空行都没有
空格一会没有，一会一个，一会几个，完全随机

自认为脾气挺好，当时差点直接骂人

[MillaMaxwell]

@cndenis #34 但是这种的前提是代码是可控的，捕获到的异常是已知且清晰的，我遇到的情况就是单纯的懒得做判断，直接 try except 一把梭

[fo0o7hU2tr6v6TCe]

全局变量在项目里乱飞的时候

[ExiFeng]

@Morii 更新缓存吧， 先删了重新添加

[28Sv0ngQfIE7Yloe]

@ExiFeng

直接 set 效果一样额