TypeScript中的代码审查：检查什么以及如何提供反馈

轻松学习Python，无需理论负担。在浏览器中直接解决带自动检查的实践任务，无需安装任何东西。

引言：为什么需要对 TypeScript 代码进行评审

代码评审不仅仅是合并前的一道形式化检查。它是提升项目质量、促进团队知识共享以及在早期阶段预防缺陷的强大工具。对于 TypeScript 而言，代码评审具有特殊意义：静态类型、强大的泛型系统以及严格的编译器设置，使得许多错误甚至在运行应用程序前就能被捕获。然而，即使在 TypeScript 中，也有一些容易忽略的陷阱。

在本文中，我们将详细阐述评审 TypeScript 代码时需要注意哪些方面，以及如何提供既有用又不伤人的反馈。

1. 类型与泛型：别让 `any` 毁了一切

TypeScript 的主要目标之一是让类型明确。如果代码中出现了 any，这几乎总是一个危险信号。在代码评审时，请检查：

避免使用 any。 将其替换为 unknown 并随后进行类型检查，或者替换为具体的类型。
泛型的正确性。 确保类型参数既不会丢失，也不会变得过于宽泛。例如：

// 不好：丢失了类型function getFirst(arr: any[]): any {    return arr[0];}

// 好：通过泛型保留了类型function getFirst<T>(arr: T[]): T | undefined {    return arr[0];}

API 响应的类型。 如果 JSON 数据来自服务器，应该用接口来描述，而不是 any。
工具类型。 使用 Partial、Pick、Omit 来灵活处理对象——这可以减少代码重复。

提示：在 tsconfig.json 中启用 strict: true 和 noImplicitAny: true 标志——这将在编译层面消除大部分问题。

2. 逻辑与错误处理：别对问题保持沉默

TypeScript 并不能防止逻辑错误，因此代码评审应检查业务逻辑和异常处理。

需要关注的点：

处理边界情况。 如果数组为空、对象为 null 或字符串为空，会发生什么？示例：

// 不好：我们未检查元素是否存在function getUserName(users: User[], id: string): string {    return users.find(u => u.id === id)!.name; // 可能抛出错误}

// 好：我们进行了检查function getUserName(users: User[], id: string): string | null {    const user = users.find(u => u.id === id);    return user ? user.name : null;}