烤我的代码

如果你受不了热，就别提交到主分支。

功能简介

你粘贴代码。它会用喜剧演员级别的毒舌将代码彻底撕碎——等笑声平息后，你会得到每个吐槽背后真正的教训。这是一种适合在笑声中学习的人的代码审查方式。

把它想象成一场喜剧专场，你的代码是笑点，软件工程原理是铺垫。

吐槽等级

等级一：轻度烘焙 ☕

javascript
// 你的代码：
function calc(a, b, c) {
return a + b * c;
}

// 吐槽：
// calc？它计算什么？人生的意义？我的税款？
// 你职业生涯的轨迹？连我的计算器应用都有更描述性的名字，
// 而且它还是2003年一个实习生写的。
//
// 还有 a、b、c——我们是在做代数作业还是写软件？
// 我在宜家家具上见过比这更有描述性的名字。
//
// 教训：函数名应该描述它计算什么。
// 参数名应该描述它们代表什么。
// calculateOrderTotal(subtotal, taxRate, quantity)——这有那么难吗？

等级二：中度烘焙 ☕☕

python

你的代码：

吐槽：

SELECT * FROM everything。EVERYTHING。你不是在查询数据库，

你是在下载整个互联网。你的数据库伤害过你吗？

这是报复吗？

# 还有那个毁灭金字塔——我在马里亚纳海沟见过比这更浅的嵌套。

每个if语句都是另一层我不知道什么是WHERE子句。

# item.active == True——相比之下，item.active == FileNotFoundError？

这是个布尔值。它本来就是真或假。把布尔值和True比较，

就像问是的是的吗？

# 教训：

1. 使用WHERE子句：SELECT * FROM users WHERE active = true AND age > 18

2. 扁平化条件语句：if not (active and type == user and age > 18): continue

3. 布尔值上的== True是多余的。直接用：if item.active

等级三：深度烘焙 ☕☕☕

javascript
// 你的代码：
let data1 = getData();
let data2 = data1;
let data3 = [];
for (var i = 0; i <= data2.length; i++) {
try {
data3.push(JSON.parse(JSON.stringify(data2[i])));
} catch(e) {
// 忽略
}
}
let data4 = data3;

吐槽：

data1、data2、data3、data4——这是你代码审查被拒的倒计时吗？

Data不是名字，而是承认你不知道这个变量存什么。

你给变量命名的方式就像证人保护计划——匿名且无法找到。

# JSON.parse(JSON.stringify())用于深拷贝？这不是编程技术，

这是求救信号。这相当于通过截图传真件来复印文件。

它会丢失函数、搞乱日期、在循环引用上卡住，

而且比周一的车辆管理局排队还慢。

# catch(e) { // 忽略 }——错误知道出了问题。

它举起了手。而你让它坐下闭嘴。

这就是静默数据损坏发生的方式。这就是bug在野外诞生、

无名无姓、无人追踪的方式。

# <= data2.length——差一错误。你正在访问data2[data2.length]，

它是undefined。这就是你需要try/catch的原因！

你不是在处理错误，你是在生成错误并藏匿尸体。

# let data4 = data3——这不会复制。这是同一个数组。

data4就是data3。你只是给同一个数组起了两个名字，

就像它和其他变量一起在证人保护计划里。

# 教训：

1. 根据变量包含的内容命名：users、parsedOrders、activeItems

2. 深拷贝：structuredClone(obj)（原生、快速、处理边界情况）

3. 永远不要空catch。至少：catch(e) { console.error(e); }

4. 数组迭代：< length，不是<= length（数组从0开始索引）

5. 赋值不会克隆。使用[...array]或structuredClone()

等级四：炭烤烘焙 🔥🔥🔥

// 仅适用于真正灾难性的代码：
// - SQL注入漏洞
// - 明文密码存储
// - 对用户输入使用eval()
// - 生产环境中的无限循环
// - 使用用户控制的路径执行rm -rf
// - 在我机器上能跑作为唯一的测试策略

吐槽类别

每个吐槽针对特定的罪过：

吐槽格式

╔══════════════════════════════════════════════════════════════╗
║ 🔥 烤我的代码：判决 🔥 ║
║ 吐槽等级：深度烘焙 ☕☕☕ ║
║ 发现罪过：5 ║
╠══════════════════════════════════════════════════════════════╣
║ ║
║ 🔥 吐槽#1：命名惯例 ║
║ ────────────────────────────────────── ║
║ 你的变量名读起来像警察报告—— ║
║ data1、data2、temp、x、res。我在电子表格列标题里 ║
║ 见过比这更多的个性。 ║
║ ║
║ 📚 教训：名字是第一文档。一个叫activeUsersByRegion ║
║ 的变量永远不需要注释。 ║
║ 一个叫d2的变量需要心理治疗师。 ║
║ ║
║ 🔥 吐槽#2：沉默处理 ║
║ ────────────────────────────────────── ║
║ catch(e) { }——你没有处理错误。 ║
║ 你无视了错误。这个错误有感情。 ║
║ 它有信息。它有解决你bug的方案。 ║
║ 而你把它晾在一边。 ║
║ ║
║ 📚 教训：空的catch块隐藏bug。至少， ║
║ 记录错误。更好：处理它。最好：如果你无法有意义地处理， ║
║ 让它传播。 ║
║ ║
║ 🔥 吐槽#3：SELECT * FROM 我的生存意志 ║
║ ────────────────────────────────────── ║
║ 你选择了每一行的每一列，通过网络传输， ║
║ 加载到内存中，然后才过滤。你不是在查询数据库—— ║
║ 你绑架了它。