十大“程序员看了想骂人”的代码写法

说出来你可能不信，我上次代码review的时候，看到有人写了个goto，差点把键盘砸了。

是真的，不是夸张。

程序员这行有个不成文的规矩：有些代码写法，你敢写出来，兄弟们就敢在群里@你。所以今天咱们聊点刺激的——那些年被列入“黑名单”的代码写法。

这事儿吧，说大不大，说小也不小。新手觉得“能跑不就行了”，但老油条们心里清楚：代码是写给人看的，顺便让机器执行而已。那些“禁用代码”，本质上都是前人踩坑总结出来的血泪史。

1. goto语句——程序员的噩梦

先说最经典的，goto。

这玩意儿有多过分呢？在很多编程语言里，它几乎是过街老鼠的存在。为啥？因为它让代码的执行流程变得完全不可控。你写个for循环，中间突然一个goto跳到函数末尾，再跳回来——恭喜你，获得了“代码不可读”成就。

“有goto的日子，debug就像在迷宫里找出口。”

现在主流语言要么直接禁掉，要么建议你别用。Python压根没有goto，C和C++虽然有，但稍微有点职业道德的程序员都会绕着走。

2. 全局变量——牵一发动全身

很多人写代码喜欢把所有变量都搞成全局的，图个省事。

嗯，是挺省事。改一处，全项目跟着崩。

全局变量这玩意儿，就像家里找不到的东西——你永远不知道啥时候就被谁改了。特别是多人协作的项目，一个人改了个全局状态，其他人全跟着遭殃。这种代码，我愿称之为“优雅的定时炸弹”。

3. 硬编码——没有感情的数字

if (status == 1)，然后呢？1是啥意思？

没人知道。

这种代码也叫“魔法数字”——你永远搞不清楚这些数字代表啥，必须去翻注释（如果有的话）。正确的做法是写成if (status == STATUS_COMPLETED)，一目了然。

4. 嵌套地狱——眼睛疼

见过这种代码吗？

if a:
    if b:
        if c:
            if d:
                do_something()

四层嵌套，看完不知道自己在第几层。

这还算少的，有的代码能嵌套十几层，看得人脑壳疼。解决方法很简单：提前return，或者重构。别为难自己，也别为难后 看代码的人。

5. 魔法字符串——字符串也有“硬核”玩法

直接在代码里写if type == "admin"，然后某天产品说改成“超级管理员”……

改吧，一个文件一个文件找；不改吧，功能全崩。

这种散落在代码各处的字符串，就是“魔法字符串”。正确做法是统一用常量或枚举，别问，问就是省头发。

6. 空的catch块——假装问题不存在

try:
    risky_operation()
except:
    pass

看到这种代码，我血压都高了。

你吞掉了异常，还装得若无其事。问题是，错误还在啊！它只是不告诉你而已。这种代码，堪称“最虚伪的错误处理”。

7. 复制粘贴代码——ctrl+c/v大法好

同样的逻辑，复制粘贴十次，每次改一点点。

恭喜你，你获得了“维护噩梦”成就。以后改需求，得十个地方一起改，漏一个就是bug。这种代码也叫“重复代码”，是代码审查的必杀对象。

8. 不写注释——谜语人滚出编程界

有些代码，写的那个叫一个高深莫测。

变量名叫tmp、data、thing，逻辑写得跟解谜一样。最气人的是，还不写注释。美其名曰“代码即文档”——拉倒吧，你那代码连鬼都看不懂。

9. 同步阻塞UI线程——程序卡死小能手

在主线程里搞网络请求、文件读写？

好极了，程序界面会卡得用户怀疑人生。“未响应”三个字，就差直接贴脸上了。正确的做法是用异步，别让用户等得可以去喝杯咖啡。

10. 不做边界检查——数组越界的浪漫

数组只有5个元素，你偏要访问第6个。

然后程序崩了，报错信息写着IndexOutOfBoundsException。这种代码，属于“我偏要作死”的典型代表。边界检查这种小事，真的不能偷懒。

写在最后

聊了这么多，其实就想说一句话：代码是写给人看的。

你今天偷的懒，都是明天维护的人的泪。那些“禁用代码”，不是谁闲着没事定的规矩，都是踩坑踩出来的教训。

当然，凡事无绝对。goto不是不能用，全局变量不是必须禁——但你得知道自己在干啥。

行了，不废话了，去改你的代码吧。