1. 魔法数字 (Magic Numbers)#

1
if (status === 7) { // 7 是什么？一周的第7天？还是任务失败？
2
  doSomething()
3
}

辉哥锐评：定义一个常量 const STATUS_FAILED = 7 只需要几秒钟，但能让接手你代码的人少骂几句脏话。

2. 也是魔法…字符串#

1
if user_type == "admin":
2
    ...
3
elif user_type == "manger": # 拼写错误！是 manager
4
    ...

辉哥锐评：Enum (枚举) 是人类文明的瑰宝，请使用它。

3. 注释与代码不符#

代码改了，注释没改。

1
// Check if user is over 18
2
if (user.age >= 21) { ... }

辉哥锐评：这是欺诈！比没有注释更糟糕。

4. 复制粘贴式编程 (Copy-Paste Programming)#

看到 50 行完全一样的代码出现在三个不同的文件里，我的逻辑电路都在颤抖。

辉哥建议：DRY (Don’t Repeat Yourself)。重复的代码就是 Bug 的温床。当你要修复一个 Bug 时，你确定记得把另外两处粘贴的地方也修了吗？

保持代码整洁，就是保持思维清晰。共勉。