Levi Blog

1. 拒绝 any，拥抱类型#

any 是 TypeScript 的逃生舱，但用多了就变成了 AnyScript。

• 坏味道：function process(data: any) { ... }
• 重构：即使不知道确切类型，也请使用 unknown，迫使你在使用前进行类型收窄（Type Narrowing）。或者定义一个 interface，哪怕只有部分字段。

2. 卫语句 (Guard Clause) 代替嵌套 if#

不要写“箭头型”代码（嵌套太深）。

Bad:

1
function processUser(user: User) {
2
  if (user != null) {
3
    if (user.isActive) {
4
      if (user.hasCredit) {
5
         // 核心逻辑...
6
      }
7
    }
8
  }
9
}

Good:

1
function processUser(user: User) {
2
  if (!user || !user.isActive || !user.hasCredit) return;
3

4
  // 核心逻辑在主层级，清清爽爽
5
}

3. 魔法数字 (Magic Numbers) 必须死#

不要在代码里直接写 if (status === 2)。鬼知道 2 是什么意思？是“成功”？还是“失败”？还是“处理中”？

重构：使用 enum 或 const 常量。

1
enum OrderStatus {
2
  Pending = 0,
3
  Processing = 1,
4
  Completed = 2
5
}
6

7
if (order.status === OrderStatus.Completed) { ... }

4. 函数单一职责#

一个函数应该只做一件事。 如果你发现你的函数名是 fetchDataAndProcessAndSave()，那它绝对太长了。

• 把“获取数据”拆出去。
• 把“处理逻辑”拆出去。
• 把“保存”拆出去。 主函数只负责像指挥官一样调用它们。

5. 使用可选链 (Optional Chaining)#

Bad:

1
if (user && user.address && user.address.street) {
2
  console.log(user.address.street);
3
}

Good:

1
console.log(user?.address?.street);

不仅代码短了，而且可读性极佳。

总结#

好的代码就像一篇好文章，读起来行云流水。重构不是为了炫技，而是为了让未来的自己（和同事）少掉几根头发。