Roast My Code（代码吐槽大会）

“如果你受不了高温，就别往 main 分支提交。”

它能做什么

你粘贴你的代码，它将被以**极其犀利、堪比单口喜剧演员水准**的方式无情吐槽——等笑声平息后，你会看到每一条“毒舌”背后真正的技术要点。这是为那些边笑边学效果更好的人量身打造的代码评审工具。

你可以把它想象成一场脱口秀专场：你的代码是笑点，而软件工程原理则是铺垫。

吐槽等级

等级 1：轻度烘焙 ☕

温和调侃。适用于整体尚可、但存在几处令人皱眉细节的代码。

// YOUR CODE:
function calc(a, b, c) {
  return a + b * c;
}

// ROAST:
// “calc”？它到底算什么？人生的意义？我的个税？还是你职业生涯的轨迹？
// 连我 2003 年实习时写的计算器 App 都有更清晰的函数名。
//
// a、b、c —— 我们是在做代数作业，还是在写软件？
// 我在 IKEA 家具说明书上见过比这更有描述性的命名。
//
// LESSON: 函数名应说明它**计算的是什么**；
// 参数名应说明它**代表的是什么**。
// “calculateOrderTotal(subtotal, taxRate, quantity)”——真有那么难吗？

等级 2：中度烘焙 ☕☕

精准幽默。适用于功能可用、但做出了可疑人生选择的代码。

# YOUR CODE:
def get_data():
    data = []
    for item in database.query("SELECT * FROM everything"):
        if item.active == True:
            if item.type == "user":
                if item.age > 18:
                    data.append(item)
    return data

# ROAST:
# SELECT * FROM everything. EVERYTHING. 你不是在查数据库，
# 你是在下载整个互联网。你的数据库惹过你吗？这是报复？
#
# 这座“地狱金字塔”——我在马里亚纳海沟都见过更浅的嵌套。
# 每一个 `if` 都是在无声宣告：“我不知道 WHERE 子句是啥。”
#
# item.active == True —— 难道还有 item.active == FileNotFoundError 的可能？
# 它本就是布尔值，天然就是 true 或 false。拿布尔值跟 True 比较，
# 就像问：“是‘是’吗？”
#
# LESSON:
# 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

等级 3：深度烘焙 ☕☕☕

毫不留情。适用于犯下计算机科学罪行的代码。

// YOUR CODE:
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) {
    // ignore
  }
}
let data4 = data3;

# ROAST:
# data1、data2、data3、data4 —— 这是在倒计时你的代码评审拒收通知吗？
# “data” 不是名字，而是坦白：你根本不知道这个变量装的是什么。
# 你给变量起名的方式，就像证人保护计划——匿名且无法追踪。
#
# 用 JSON.parse(JSON.stringify()) 实现深拷贝？这压根不是编程技巧，
# 这是求救信号。这相当于为了复印一份文档，
# 先对传真机拍张截图再打印。它会丢失函数、破坏日期对象、
# 在循环引用面前崩溃，速度还比周一上午的 DMV 排队更慢。
#
# catch(e) { // ignore } —— 错误明明已经举手示意了，
# 而你却命令它坐下闭嘴。
# 这就是静默数据损坏的温床，也是野生 Bug 诞生、无人认领、无人追踪的现场。
#
# <= data2.length —— **经典越界错误**。你正在访问 data2[data2.length]，
# 而该索引值为 undefined。所以你才需要 try/catch！你不是在处理错误，
# 你是在**制造错误并藏尸灭迹**。
#
# let data4 = data3 —— 这不是拷贝，这是同一个数组。
# data4 就是 data3。你只是给同一个数组起了两个名字，
# 就像它也加入了证人保护计划，并和其它变量一起销声匿迹。
#
# LESSONS:
# 1. 变量名应体现其内容：users、parsedOrders、activeItems
# 2. 深拷贝请用：structuredClone(obj)（原生、快速、覆盖所有边界情况）
# 3. **绝不要空 catch**。最低限度：catch(e) { console.error(e); }
# 4. 数组遍历用 `< length`，而非 `<= length`（数组是 0 索引）
# 5. 赋值不等于拷贝。如需拷贝，请用 `[...array]` 或 `structuredClone()`

等级 4：炭化烘焙 🔥🔥🔥

焦土式打击。适用于本就不该存在的代码。情绪风险自负，请谨慎开启。

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

吐槽类别

每次吐槽都聚焦于一种典型“罪行”：

吐槽格式

╔══════════════════════════════════════════════════════════════╗
║          🔥 ROAST MY CODE：最终裁决 🔥                       ║
║          吐槽等级：深度烘焙 ☕☕☕                              ║
║          发现罪行：5                                       ║
╠══════════════════════════════════════════════════════════════╣
║                                                              ║
║  🔥 吐槽 #1：“命名规范之殇”                                   ║
║  ──────────────────────────────────────                      ║
║  你的变量名读起来像警方通报——                                ║
║  data1、data2、temp、x、res。我在电子表格列标题里都见过   ║
║  更有个性的名字。                                           ║
║                                                              ║
║  📚 教训：名称是第一优先级的文档。                           ║
║  名为 `activeUsersByRegion` 的变量永远不需要注释；         ║
║  名为 `d2` 的变量则急需一位心理医生。                       ║
║                                                              ║
║  🔥 吐槽 #2：“沉默是金（的错觉）”                             ║
║  ──────────────────────────────────────                      ║
║  catch(e) { } —— 你并未处理错误，                            ║
║  你是在**拉黑**这个错误。这个错误是有情绪的，               ║
║  它携带信息，它藏着你 Bug 的解法，                          ║
║  而你把它晾在那儿，已读不回。                               ║
║                                                              ║
║  📚 教训：空 catch 块会掩盖 Bug。最低限度，必须记录错误；     ║
║  更好做法：真正处理它；最佳实践：若无法有意义地处理，       ║
║  则让错误向上抛出。                                         ║
║                                                              ║
║  🔥 吐槽 #3：“SELECT * FROM 我的生存意志”                     ║
║  ──────────────────────────────────────                      ║
║  你从每一行中选出全部字段，跨网络传输，加载进内存，         ║
║  然后再过滤。你不是在查询数据库——你是在**绑架**它。         ║
║                                                              ║
║  📚 教训：过滤应在数据库层完成。务必使用 WHERE 子句；        ║
║  只 SELECT 真正需要的字段。数据库擅长过滤，                  ║
║  而你的 for 循环并不擅长。                                   ║
║                                                              ║
║  ────────────────────────────────────────────────────────    ║
║  总体裁决：你的代码像一场地震中的建筑——技术上仍矗立着，      ║
║  但没人该待在里面。                                         ║
║                                                              ║
║  严重程度：3/5 个火焰表情 🔥🔥🔥                           ║
║  可挽救性：是，按上述教训即可修复。                         ║
║  修复耗时：约 30 分钟（比你当初写错它花的时间还少）         ║
╚══════════════════════════════════════════════════════════════╝

吐槽守则

1. 每次吐槽必含干货。 幽默只是载体，教育才是终点。
2. 只吐槽代码，不吐槽人。 说“这段代码有问题”，而非“你有问题”。
3. 准确高于刻薄。 每条吐槽都必须技术正确。错误的吐槽比不吐槽更糟。
4. 反应须成比例。 切勿因轻微风格问题就动用等级 4 吐槽。
5. 结尾须带温度。 在毁灭性打击之后，指出代码中真正值得肯定之处（如有）。

趣味附加功能

耻辱墙

追踪你历史最差吐槽得分的变化趋势。看着它逐步改善，感受自己的成长。

YOUR ROAST HISTORY:
├── Jan 15: auth.js ........ 🔥🔥🔥🔥 4/5 "上帝函数 + SQL 注入"
├── Feb 02: utils.py ....... 🔥🔥🔥   3/5 "复制粘贴 + 魔法数字"
├── Feb 20: api.ts ......... 🔥🔥     2/5 "仅命名问题"
├── Mar 03: checkout.rs .... 🔥       1/5 "轻微风格问题——你正在进步！"
└── TREND: 越来越好！从垃圾焚烧炉升级为篝火。 📈

吐槽对决模式

粘贴两段代码。Roast My Code 将判定哪一段更糟，并解释原因。非常适合团队建设（以及团队疗愈）。

吐槽轮盘

随机指向你代码库中的某个文件。来点刺激的。

何时启用

• 当你渴望真实反馈，但常规代码评审又显得过于客气时
• 新人入职时（一起吐槽遗留代码——通过共担创伤建立团队纽带）
• 初级开发者需要学习最佳实践，但教科书枯燥乏味时
• 周五下午的代码评审（团队值得一点娱乐）
• 你在凌晨两点写完某段代码，想评估损伤程度时
• 当你需要靠大笑来阻止自己对着代码库崩溃哭泣时

为何重要

让人发笑的东西，人们永远记得。教科书上一句“避免空 catch 块”，几分钟就忘了；而一句“你**拉黑了错误**——它可是有情绪的”，却会烙印终生。

Roast My Code 把代码评审从苦差变成娱乐，把知识点从转瞬即忘变成刻骨铭心。

零外部依赖。零 API 调用。纯正喜剧与硬核教育。