Awesome
代码整洁的 TypeScript ![Tweet](https://img.shields.io/twitter/url/http/shields.io.svg?style=social)
将代码整洁的理念适配至 TypeScript , 灵感来自于代码整洁的 JavaScript 。
目录
简介
将源自 Robert C. Martin 的 Clean Code 的软件工程原则适配到 TypeScript 。 这不是一个代码风格指南, 它是一个使用 TypeScript 来生产 可读的, 可重用的, 以及可重构的软件的指南。
这里的每一项原则都不是必须遵守的, 甚至只有更少的能够被广泛认可。 这些仅仅是指南而已, 但是却是 Clean Code 作者多年经验的结晶。
我们的软件工程行业只有短短的 50 年, 依然有很多要我们去学习。 当软件架构与建筑架构一样古老时, 也许我们将会有硬性的规则去遵守。 而现在, 让这些指南做为你和你的团队生产的 TypeScript 代码的 质量的标准。
还有一件事: 知道这些指南并不能马上让你成为一个更加出色的软件开发者, 并且使用它们工作多年也并 不意味着你不再会犯错误。 每一段代码最开始都是草稿, 像湿粘土一样被打造成最终的形态。 最后当我们 和搭档们一起审查代码时清除那些不完善之处, 不要因为最初需要改善的草稿代码而自责, 而是对那些代 码下手。
变量
使用有意义的变量名称
使用可以区分的名称, 让读者知道他们的区别是什么。
不好的:
function between<T>(a1: T, a2: T, a3: T): boolean {
return a2 <= a1 && a1 <= a3;
}
好的:
function between<T>(value: T, left: T, right: T): boolean {
return left <= value && value <= right;
}
使用可拼读的变量名称
如果你不能把它读出来, 那你就不能和同事讨论它, 这看起来像块纱布。
不好的:
type DtaRcrd102 = {
genymdhms: Date;
modymdhms: Date;
pszqint: number;
}
好的:
type Customer = {
generationTimestamp: Date;
modificationTimestamp: Date;
recordId: number;
}
为相同类型的变量使用相同的词汇
不好的:
function getUserInfo(): User;
function getUserDetails(): User;
function getUserData(): User;
好的:
function getUser(): User;
使用可搜索的名称
我们要阅读的代码比要写的代码多得多, 所以我们写出的代码的可读性和可搜索性是很重要的。 使用没有 意义的变量名将会导致我们的程序难于理解, 将会伤害我们的读者, 所以请使用可搜索的变量名。 类似 TSLint 的工具可以帮助我们找到未命名的常量。
不好的:
// What the heck is 86400000 for?
// 艹, 86400000 是什么鬼?
setTimeout(restart, 86400000);
好的:
// Declare them as capitalized named constants.
// 将他们声明为大写的变量
const MILLISECONDS_IN_A_DAY = 24 * 60 * 60 * 1000;
setTimeout(restart, MILLISECONDS_IN_A_DAY);
使用解释性的变量
不好的:
declare const users: Map<string, User>;
for (const keyValue of users) {
// iterate through users map
}
好的:
declare const users: Map<string, User>;
for (const [id, user] of users) {
// iterate through users map
}
避免心理映射
显示比隐式更好。 清晰为王!
不好的:
const u = getUser();
const s = getSubscription();
const t = charge(u, s);
好的:
const user = getUser();
const subscription = getSubscription();
const transaction = charge(user, subscription);
不添加不必要的上下文
如果你的类/类型/对象名有意义, 不必在变量名上再重复。
不好的:
type Car = {
carMake: string;
carModel: string;
carColor: string;
}
function print(car: Car): void {
console.log(`${car.carMake} ${car.carModel} (${car.carColor})`);
}
好的:
type Car = {
make: string;
model: string;
color: string;
}
function print(car: Car): void {
console.log(`${car.make} ${car.model} (${car.color})`);
}
使用默认变量替代短路运算或条件
默认参数通常比短路运算更清晰。
不好的:
function loadPages(count?: number) {
const loadCount = count !== undefined ? count : 10;
// ...
}
好的:
function loadPages(count: number = 10) {
// ...
}
函数
函数参数 (两个以下最理想)
限制函数参数的个数是非常重要的, 因为这样将使你的函数容易进行测试。 一旦超过三个参数将会导致组 合爆炸, 因为你不得不编写大量针对每个参数的测试用例。
一个或者两个参数是理想状况, 如果可能的话, 三个参数的情况应该避免, 超过三个应该被重构。 通常, 如果你有一个超过两个函数的参数, 那就意味着你的函数尝试做太多的事情。 如果不是, 多数情况下一个 更高级对象可能会满足需求。
当你发现你自己需要大量的参数时, 考虑使用一个对象。
为了让函数需要的属性更明显, 可以使用解构语法。 它有三个优点:
-
当有人查看函数签名时, 会立刻清楚用到了哪些属性。
-
解构还克隆传递给函数的参数对象的指定原始值。 这有助于预防副作用。 注意:不会克隆参数对象中解构的对象和数组。
-
TypeScript 会警告您未使用的属性,如果没有解构,这将是不可能的。
不好的:
function createMenu(title: string, body: string, buttonText: string, cancellable: boolean) {
// ...
}
createMenu('Foo', 'Bar', 'Baz', true);
好的:
function createMenu(options: { title: string, body: string, buttonText: string, cancellable: boolean }) {
// ...
}
createMenu({
title: 'Foo',
body: 'Bar',
buttonText: 'Baz',
cancellable: true
});
你可以通过类型别名来显著提高可读性:
type MenuOptions = { title: string, body: string, buttonText: string, cancellable: boolean };
function createMenu(options: MenuOptions) {
// ...
}
createMenu({
title: 'Foo',
body: 'Bar',
buttonText: 'Baz',
cancellable: true
});
函数应当只做一件事情
这是软件工程中最重要的一条规则, 当函数需要做更多的事情时, 它们将会更难进行编写、 测试和推理。 当你能将一个函数隔离到只有一个动作, 他们将能够被容易的进行重构并且你的代码将会更容易阅读。 如 果你严格遵守本指南中的这一条, 你将会领先于许多开发者。
不好的:
function emailClients(clients: Client) {
clients.forEach((client) => {
const clientRecord = database.lookup(client);
if (clientRecord.isActive()) {
email(client);
}
});
}
好的:
function emailClients(clients: Client) {
clients.filter(isActiveClient).forEach(email);
}
function isActiveClient(client: Client) {
const clientRecord = database.lookup(client);
return clientRecord.isActive();
}
函数名称应该说明它要做什么
不好的:
function addToDate(date: Date, month: number): Date {
// ...
}
const date = new Date();
// It's hard to tell from the function name what is added
// 很难从函数名看出加了什么
addToDate(date, 1);
好的:
function addMonthToDate(date: Date, month: number): Date {
// ...
}
const date = new Date();
addMonthToDate(date, 1);
函数应该只有一个抽象级别
当在你的函数中有多于一个抽象级别时, 你的函数通常做了太多事情。 拆分函数将会提升重用性和测试性。
不好的:
function parseCode(code: string) {
const REGEXES = [ /* ... */ ];
const statements = code.split(' ');
const tokens = [];
REGEXES.forEach((regex) => {
statements.forEach((statement) => {
// ...
});
});
const ast = [];
tokens.forEach((token) => {
// lex...
});
ast.forEach((node) => {
// parse...
});
}
好的:
const REGEXES = [ /* ... */ ];
function parseCode(code: string) {
const tokens = tokenize(code);
const syntaxTree = parse(tokens);
syntaxTree.forEach((node) => {
// parse...
});
}
function tokenize(code: string): Token[] {
const statements = code.split(' ');
const tokens: Token[] = [];
REGEXES.forEach((regex) => {
statements.forEach((statement) => {
tokens.push( /* ... */ );
});
});
return tokens;
}
function parse(tokens: Token[]): SyntaxTree {
const syntaxTree: SyntaxTree[] = [];
tokens.forEach((token) => {
syntaxTree.push( /* ... */ );
});
return syntaxTree;
}
移除冗余代码
竭尽你的全力去避免冗余代码。 冗余代码是不好的, 因为它意味着当你需要修改一些逻辑时会有多个地方 需要修改。
想象一下你在经营一家餐馆, 你需要记录所有的库存西红柿, 洋葱, 大蒜, 各种香料等等。 如果你有多 个记录列表, 当你用西红柿做一道菜时你得更新多个列表。 如果你只有一个列表, 就只有一个地方需要更 新!
你有冗余代码通常是因为你有两个或多个稍微不同的东西, 它们共享大部分, 但是它们的不同之处迫使你使 用两个或更多独立的函数来处理大部分相同的东西。 移除冗余代码意味着创建一个可以处理这些不同之处的 抽象的函数/模块/类。
让这个抽象正确是关键的, 这是为什么要你遵循 Classes 那一章的 SOLID 的原因。 不好的抽象比冗 余代码更差, 所以要谨慎行事。 既然已经这么说了, 如果你能够做出一个好的抽象, 才去做。 不要重复 你自己, 否则你会发现当你要修改一个东西时时刻需要修改多个地方。
不好的:
function showDeveloperList(developers: Developer[]) {
developers.forEach((developer) => {
const expectedSalary = developer.calculateExpectedSalary();
const experience = developer.getExperience();
const githubLink = developer.getGithubLink();
const data = {
expectedSalary,
experience,
githubLink
};
render(data);
});
}
function showManagerList(managers: Manager[]) {
managers.forEach((manager) => {
const expectedSalary = manager.calculateExpectedSalary();
const experience = manager.getExperience();
const portfolio = manager.getMBAProjects();
const data = {
expectedSalary,
experience,
portfolio
};
render(data);
});
}
好的:
class Developer {
// ...
getExtraDetails() {
return {
githubLink: this.githubLink,
}
}
}
class Manager {
// ...
getExtraDetails() {
return {
portfolio: this.portfolio,
}
}
}
function showEmployeeList(employee: Developer | Manager) {
employee.forEach((employee) => {
const expectedSalary = employee.calculateExpectedSalary();
const experience = employee.getExperience();
const extra = employee.getExtraDetails();
const data = {
expectedSalary,
experience,
extra,
};
render(data);
});
}
您应该对代码冗余持批判的态度。 有时需要在冗余代码和通过因不必要的抽象而增加的复杂性之间做权衡。 当来自两个不同模块的两个实现看起来相似但存在于不同的域中时,冗余(可能)是可接受的并且优于提取公共代码。 在这种情况下,提取的公共代码引入了两个模块之间的间接依赖关系。
使用 Object.assign 设置默认对象或者解构
不好的:
type MenuConfig = { title?: string, body?: string, buttonText?: string, cancellable?: boolean };
function createMenu(config: MenuConfig) {
config.title = config.title || 'Foo';
config.body = config.body || 'Bar';
config.buttonText = config.buttonText || 'Baz';
config.cancellable = config.cancellable !== undefined ? config.cancellable : true;
// ...
}
createMenu({ body: 'Bar' });
好的:
type MenuConfig = { title?: string, body?: string, buttonText?: string, cancellable?: boolean };
function createMenu(config: MenuConfig) {
const menuConfig = Object.assign({
title: 'Foo',
body: 'Bar',
buttonText: 'Baz',
cancellable: true
}, config);
// ...
}
createMenu({ body: 'Bar' });
另外, 也可以使用解构来处理默认值:
type MenuConfig = { title?: string, body?: string, buttonText?: string, cancellable?: boolean };
function createMenu({ title = 'Foo', body = 'Bar', buttonText = 'Baz', cancellable = true }: MenuConfig) {
// ...
}
createMenu({ body: 'Bar' });
要避免显示传递 undefined
或者 null
值产生的负面影响或异常行为, 可以设置 TypeScript 编译器来禁止。 请查看 TypeScript 的 --strictNullChecks
选项。
不要使用标记位做为函数参数
标记位是告诉你的用户这个函数做了不只一件事情。 函数应该只做一件事情。 如果你的函数因为一个布尔值 出现不同的代码路径, 请拆分它们。
不好的:
function createFile(name: string, temp: boolean) {
if (temp) {
fs.create(`./temp/${name}`);
} else {
fs.create(name);
}
}
好的:
function createTempFile(name: string) {
createFile(`./temp/${name}`);
}
function createFile(name: string) {
fs.create(name);
}
避免副作用(第 1 部分)
如果一个函数做了除接受一个值然后返回一个值或多个值之外的任何事情, 它将会产生副作用, 它可能是 写入一个文件, 修改一个全局变量, 或者意外的把你所有的钱连接到一个陌生人那里。
现在在你的程序中确实偶尔需要副作用, 就像上面的代码, 你也许需要写入到一个文件, 你需要做的是集 中化你要做的事情, 不要让多个函数或者类写入一个特定的文件, 用一个服务来实现它, 一个并且只有一 个。
重点是避免这些常见的易犯的错误, 比如在对象之间共享状态而不使用任何结构, 使用任何地方都可以写入 的可变的数据类型, 没有集中化导致副作用。 如果你能做到这些, 那么你将会比其它的码农大军更加幸福。
不好的:
// Global variable referenced by following function.
// 全局变量被下面的函数引用
let name = 'Robert C. Martin';
function toBase64() {
name = btoa(name);
}
toBase64();
// If we had another function that used this name, now it'd be a Base64 value
// 如果我们有另一个函数使用这个 name , 现在它应该是一个 Base64 字符串值。
// expected to print 'Robert C. Martin' but instead 'Um9iZXJ0IEMuIE1hcnRpbg=='
// 期望打印出 'Robert C. Martin' 但是却是 'Um9iZXJ0IEMuIE1hcnRpbg=='
console.log(name);
好的:
const name = 'Robert C. Martin';
function toBase64(text: string): string {
return btoa(text);
}
const encodedName = toBase64(name);
console.log(name);
避免副作用 (第 2 部分)
在 JavaScript 中, 基本类型通过值进行传递而对象/类通过引用传递。 以对象和数组为例, 如果你的函数对一个购物车数组做出了更改, 比如添加了一个要购买的东西, 那么其它使用这个购物车数组的任何函数都会受到影响。 这样貌似挺不错的, 不过也可能很糟糕。 让我们来想象一个糟糕的情况:
用户点击“购买”按钮, 调用一个 purchase
函数, 发出一个网络请求, 经购物车数组发送到服务器。 由于网络情况比较差, purchase
函数只能尝试重新发送请求。 现在, 用户在网络请求开始之前, 突然点击了“添加到购物车”按钮, 添加了一项并不是真心想买的东西, 那么 purchase
函数将会发送这个突然被添加的项目, 因为它们引用了同一个购物车数组对象, 而这个对象 addItemToCart
函数修改了, 添加了一个不想要的项目。
一个好的方案应该是让 addItemToCart
始终克隆一个购物车副本, 编辑并返回副本。 这样能够保证它不会被其它任何函数引用, 也就不能进行修改。
这种方案下需要注意以下 2 个问题:
-
可能在有些情况下确实需要修改输入对象, 但是当你采用这种编程实践之后, 你会发现这种情况是寥寥无几的。 很多东西可以被重构来消除负面影响。 (参考纯函数/Pure function)
-
克隆大对象可能比较消耗性能。 幸运的是, 在实际操作上, 并不是一个多大的问题, 因为有优秀的类库可以让这一操作变得非常快, 同时也比手工克隆这些对象和数组节省很多内存。
不好的:
function addItemToCart(cart: CartItem[], item: Item): void {
cart.push({ item, date: Date.now() });
};
好的:
function addItemToCart(cart: CartItem[], item: Item): CartItem[] {
return [...cart, { item, date: Date.now() }];
};
不要写入全局函数
污染全局在 JavaScript 中是一个不好的做法, 因为你可能会和另外一个类库冲突, 你的 API 的用户
可能不够聪明, 直到他们得到在生产环境得到一个异常。 让我们来考虑这样一个例子: 假设你要扩展
JavaScript 的 原生 Array
, 添加一个可以显示两个数组的不同之处的 diff
方法, 你可以在
Array.prototype
中写一个新的方法, 但是它可能会和尝试做相同事情的其它类库发生冲突。 如果有
另外一个类库仅仅使用 diff
方法来查找数组的第一个元素和最后一个元素之间的不同之处呢? 这就是
为什么使用 ES2015/ES6 的类是一个更好的做法的原因, 只要简单的扩展全局的 Array
即可。
不好的:
declare global {
interface Array<T> {
diff(other: T[]): Array<T>;
}
}
if (!Array.prototype.diff) {
Array.prototype.diff = function <T>(other: T[]): T[] {
const hash = new Set(other);
return this.filter(elem => !hash.has(elem));
};
}
好的:
class MyArray<T> extends Array<T> {
diff(other: T[]): T[] {
const hash = new Set(other);
return this.filter(elem => !hash.has(elem));
};
}
函数式编程优于指令式编程
当你可以使用函数式编程风格时请尽情使用。
不好的:
const contributions = [
{
name: 'Uncle Bobby',
linesOfCode: 500
}, {
name: 'Suzie Q',
linesOfCode: 1500
}, {
name: 'Jimmy Gosling',
linesOfCode: 150
}, {
name: 'Gracie Hopper',
linesOfCode: 1000
}
];
let totalOutput = 0;
for (let i = 0; i < contributions.length; i++) {
totalOutput += contributions[i].linesOfCode;
}
好的:
const contributions = [
{
name: 'Uncle Bobby',
linesOfCode: 500
}, {
name: 'Suzie Q',
linesOfCode: 1500
}, {
name: 'Jimmy Gosling',
linesOfCode: 150
}, {
name: 'Gracie Hopper',
linesOfCode: 1000
}
];
const totalOutput = contributions
.reduce((totalLines, output) => totalLines + output.linesOfCode, 0);
封装条件语句
不好的:
if (subscription.isTrial || account.balance > 0) {
// ...
}
好的:
function canActivateService(subscription: Subscription, account: Account) {
return subscription.isTrial || account.balance > 0
}
if (canActivateService(subscription, account)) {
// ...
}
避免负面条件
不好的:
function isEmailNotUsed(email: string): boolean {
// ...
}
if (isEmailNotUsed(email)) {
// ...
}
好的:
function isEmailUsed(email): boolean {
// ...
}
if (!isEmailUsed(node)) {
// ...
}
避免条件语句
这看起来似乎是一个不可能的任务。 第一次听到这个时, 多数人会说: “没有 if
语句还能期望我干
啥呢”, 答案是多数情况下你可以使用多态来完成同样的任务。 第二个问题通常是 “好了, 那么做很棒,
但是我为什么想要那样做呢”, 答案是我们学到的上一条代码整洁之道的理念: 一个函数应当只做一件事情。
当你有使用 if
语句的类/函数是, 你在告诉你的用户你的函数做了不止一件事情。 记住: 只做一件
事情。
不好的:
class Airplane {
private type: string;
// ...
getCruisingAltitude() {
switch (this.type) {
case '777':
return this.getMaxAltitude() - this.getPassengerCount();
case 'Air Force One':
return this.getMaxAltitude();
case 'Cessna':
return this.getMaxAltitude() - this.getFuelExpenditure();
default:
throw new Error('Unknown airplane type.');
}
}
private getMaxAltitude(): number {
// ...
}
}
好的:
abstract class Airplane {
protected getMaxAltitude(): number {
// shared logic with subclasses ...
}
// ...
}
class Boeing777 extends Airplane {
// ...
getCruisingAltitude() {
return this.getMaxAltitude() - this.getPassengerCount();
}
}
class AirForceOne extends Airplane {
// ...
getCruisingAltitude() {
return this.getMaxAltitude();
}
}
class Cessna extends Airplane {
// ...
getCruisingAltitude() {
return this.getMaxAltitude() - this.getFuelExpenditure();
}
}
避免类型检查
TypeScript 是 JavaScript 的一个严格语法的超急, 为这门语言增加了可选的静态类型检查。 始终倾向于给变量、 参数以及返回值定义类型一体现 TypeScript 的完整特征。 这将使重构变得更加容易。
不好的
function travelToTexas(vehicle: Bicycle | Car) {
if (vehicle instanceof Bicycle) {
vehicle.pedal(currentLocation, new Location('texas'));
} else if (vehicle instanceof Car) {
vehicle.drive(currentLocation, new Location('texas'));
}
}
好的
type Vehicle = Bicycle | Car;
function travelToTexas(vehicle: Vehicle) {
vehicle.move(currentLocation, new Location('texas'));
}
不要过度优化
现代化浏览器运行时在幕后做大量的优化, 在大多数的时间, 做优化就是在浪费你的时间。 这些是好的 资源, 用来 查看那些地方需要优化。 为这些而优化, 直到他们被修正。
不好的:
// On old browsers, each iteration with uncached `list.length` would be costly
// because of `list.length` recomputation. In modern browsers, this is optimized.
// 在旧的浏览器上, 每次循环 `list.length` 都没有被缓存, 会导致不必要的开销, 因为要重新计
// 算 `list.length` 。 在现代化浏览器上, 这个已经被优化了。
for (let i = 0, len = list.length; i < len; i++) {
// ...
}
好的:
for (let i = 0; i < list.length; i++) {
// ...
}
移除僵尸代码
僵死代码和冗余代码同样糟糕。 没有理由在代码库中保存它。 如果它不会被调用, 就删掉它。 当你需要 它时, 它依然保存在版本历史记录中。
不好的:
function oldRequestModule(url: string) {
// ...
}
function requestModule(url: string) {
// ...
}
const req = requestModule;
inventoryTracker('apples', req, 'www.inventory-awesome.io');
好的:
function requestModule(url: string) {
// ...
}
const req = requestModule;
inventoryTracker('apples', req, 'www.inventory-awesome.io');
使用枚举器和生成器
当像流一样处理数据集时, 使用生成器和枚举器。 这样做的好处是:
- 被调用者与生成器解耦, 这样被调用者可以决定处理多少项;
- 延迟执行, 元素按流式按需处理;
- 允许为枚举模式实现进行优化;
不好的:
function fibonacci(n: number): number[] {
if (n === 1) return [0];
if (n === 2) return [0, 1];
const items: number[] = [0, 1];
while (items.length < n) {
items.push(items[items.length - 2] + items[items.length - 1]);
}
return items;
}
function print(n: number) {
fibonacci(n).forEach(fib => console.log(fib));
}
// Print first 10 Fibonacci numbers.
print(10);
好的:
// 生成一个无限长的斐波那契数字流, 生成器并没有保存数字数组。
function* fibonacci(): IterableIterator<number> {
let [a, b] = [0, 1];
while (true) {
yield a;
[a, b] = [b, a + b];
}
}
function print(n: number) {
let i = 0;
for (const fib of fibonacci()) {
if (i++ === n) break;
console.log(fib);
}
}
// Print first 10 Fibonacci numbers.
print(10);
一些类库可以与原生数组类似的方式使用枚举, 将 map
, slice
, forEach
等方法串联到一起。 请参考 itiriri 的高级枚举处理示例(或者 itiriri-async 的异步枚举处理)。
import itiriri from 'itiriri';
function* fibonacci(): IterableIterator<number> {
let [a, b] = [0, 1];
while (true) {
yield a;
[a, b] = [b, a + b];
}
}
itiriri(fibonacci())
.take(10)
.forEach(fib => console.log(fib));
对象和数据结构
使用 getter 和 setter
TypeScript 支持 getter/setter 语法。 正因为如此, 使用 getters 和 setters 来访问对象上的数据比简单的在一个对象上查找属性 要好得多。 “为什么?” 你可能会问, 好吧, 原因请看下面的列表:
- 当你想在获取一个对象属性的背后做更多的事情时, 你不需要在代码库中查找和修改每一处访问;
- 使用
set
可以让添加验证变得容易; - 封装内部实现;
- 使用 getting 和 setting 时, 容易添加日志和错误处理;
- 你可以延迟加载对象的属性, 比如说从服务器获取。
不好的:
type BankAccount = {
balance: number;
// ...
}
const value = 100;
const account: BankAccount = {
balance: 0,
// ...
};
if (value < 0) {
throw new Error('Cannot set negative balance.');
}
account.balance = value;
好的:
class BankAccount {
private accountBalance: number = 0;
get balance(): number {
return this.accountBalance;
}
set balance(value: number) {
if (value < 0) {
throw new Error('Cannot set negative balance.');
}
this.accountBalance = value;
}
// ...
}
// 现在 `BankAccount` 封装了验证逻辑, 如果某一天需求变化了, 需要添加额外的验证规则, 我们只需要修改 `setter` 即可, 其它依赖性代码则无需修改。
const account = new BankAccount();
account.balance = 100;
让对象拥有私有的/受保护的成员
TypeScript 支持在类成员上添加 public
(默认) , protected
和 private
修饰符。
不好的:
class Circle {
radius: number;
constructor(radius: number) {
this.radius = radius;
}
perimeter() {
return 2 * Math.PI * this.radius;
}
surface() {
return Math.PI * this.radius * this.radius;
}
}
好的:
class Circle {
constructor(private readonly radius: number) {
}
perimeter() {
return 2 * Math.PI * this.radius;
}
surface() {
return Math.PI * this.radius * this.radius;
}
}
倾向于不可变性
TypeScript 的类型系统允许将类/接口的某些属性标记为 只读 。 这允许你在一个很舒适的方式下工作(不需要考虑意外的变化)。 针对更加高级的场景, 有一个内置的 Readonly
类型, 它接受一个类型 T
, 实用类型映射, 将类型 T
的全部属性统统标记为只读(参考 mapped types)。
不好的:
interface Config {
host: string;
port: string;
db: string;
}
好的:
interface Config {
readonly host: string;
readonly port: string;
readonly db: string;
}
类型 vs. 接口
当需要并集或者交集时, 实用类型。 当需要扩展或实现时, 实用接口。 然而并没有严格的规则, 哪个适合就用哪个。 若需要一个更加详细的解释, 请参考关于 TypeScript 的类型和接口之间的不同的这个答案。
不好的:
interface EmailConfig {
// ...
}
interface DbConfig {
// ...
}
interface Config {
// ...
}
//...
type Shape = {
// ...
}
好的:
type EmailConfig = {
// ...
}
type DbConfig = {
// ...
}
type Config = EmailConfig | DbConfig;
// ...
interface Shape {
// ...
}
class Circle implements Shape {
// ...
}
class Square implements Shape {
// ...
}
类
类应当小
类的大小由它的职责衡量。 根据 单一职责原则 一个类应该小。
不好的:
class Dashboard {
getLanguage(): string { /* ... */ }
setLanguage(language: string): void { /* ... */ }
showProgress(): void { /* ... */ }
hideProgress(): void { /* ... */ }
isDirty(): boolean { /* ... */ }
disable(): void { /* ... */ }
enable(): void { /* ... */ }
addSubscription(subscription: Subscription): void { /* ... */ }
removeSubscription(subscription: Subscription): void { /* ... */ }
addUser(user: User): void { /* ... */ }
removeUser(user: User): void { /* ... */ }
goToHomePage(): void { /* ... */ }
updateProfile(details: UserDetails): void { /* ... */ }
getVersion(): string { /* ... */ }
// ...
}
好的:
class Dashboard {
disable(): void { /* ... */ }
enable(): void { /* ... */ }
getVersion(): string { /* ... */ }
}
// 将其它方法移到其它类以拆分职责
// ...
高内聚和低耦合
内聚定义了类成员彼此相关的程度。 理想情况下,每个方法都应使用类中的所有字段。 然后我们说这个类是最高内聚。 实际上, 这并非总是可行, 甚至不可取。 但是你应该更喜欢高内聚。
耦合指的是两个类相互依赖或相互依赖的程度。 如果其中一个类中的更改不影响其它的, 则称这些类是低耦合的。
好的软件设计拥有 高内聚 和 低耦合 。
不好的:
class UserManager {
// 不好的: 每一个私有变量仅仅被一组或另一组方法使用, 这很明显说明这个类在兼负多个职责。 当我仅仅需要一个服务来获取用户事务时, 还得传递另外一个 `emailSender` 的实例。
constructor(
private readonly db: Database,
private readonly emailSender: EmailSender) {
}
async getUser(id: number): Promise<User> {
return await db.users.findOne({ id });
}
async getTransactions(userId: number): Promise<Transaction[]> {
return await db.transactions.find({ userId });
}
async sendGreeting(): Promise<void> {
await emailSender.send('Welcome!');
}
async sendNotification(text: string): Promise<void> {
await emailSender.send(text);
}
async sendNewsletter(): Promise<void> {
// ...
}
}
好的:
class UserService {
constructor(private readonly db: Database) {
}
async getUser(id: number): Promise<User> {
return await this.db.users.findOne({ id });
}
async getTransactions(userId: number): Promise<Transaction[]> {
return await this.db.transactions.find({ userId });
}
}
class UserNotifier {
constructor(private readonly emailSender: EmailSender) {
}
async sendGreeting(): Promise<void> {
await this.emailSender.send('Welcome!');
}
async sendNotification(text: string): Promise<void> {
await this.emailSender.send(text);
}
async sendNewsletter(): Promise<void> {
// ...
}
}
组合优先于继承
正如设计模式四人帮所述, 如果可能, 你应该优先使用组合而不是继承。 有许多好的理由去使用继承, 也有许多好的理由去使用组合。这个格言 的重点是, 如果你本能的观点是继承, 那么请想一下组合能否更好的为你的问题建模。 很多情况下它真的 可以。
那么你也许会这样想, “我什么时候改使用继承?” 这取决于你手上的问题, 不过这儿有一个像样的列表说 明什么时候继承比组合更好用:
- 你的继承表示"是一个"的关系而不是"有一个"的关系(人类->动物 vs 用户->用户详情);
- 你可以重用来自基类的代码(人可以像所有动物一样行动);
- 你想通过基类对子类进行全局的修改(改变所有动物行动时的热量消耗);
不好的:
class Employee {
constructor(
private readonly name: string,
private readonly email: string) {
}
// ...
}
// Bad because Employees "have" tax data. EmployeeTaxData is not a type of Employee
// 不好是因为雇员“有”税率数据, EmployeeTaxData 不是一个 Employee 类型。
class EmployeeTaxData extends Employee {
constructor(
name: string,
email: string,
private readonly ssn: string,
private readonly salary: number) {
super(name, email);
}
// ...
}
不好的:
class Employee {
private taxData: EmployeeTaxData;
constructor(
private readonly name: string,
private readonly email: string) {
}
setTaxData(ssn: string, salary: number): Employee {
this.taxData = new EmployeeTaxData(ssn, salary);
return this;
}
// ...
}
class EmployeeTaxData {
constructor(
public readonly ssn: string,
public readonly salary: number) {
}
// ...
}
使用方法链
这个模式在 JavaScript 中是非常有用的, 并且在许多类库使用。 它使你的代码变得富有表现力, 并减少啰嗦。 因为这个原因, 我说, 使用方法链然后再看看你的代码会变得多么简洁。
不好的:
class QueryBuilder {
private collection: string;
private pageNumber: number = 1;
private itemsPerPage: number = 100;
private orderByFields: string[] = [];
from(collection: string): void {
this.collection = collection;
}
page(number: number, itemsPerPage: number = 100): void {
this.pageNumber = number;
this.itemsPerPage = itemsPerPage;
}
orderBy(...fields: string[]): void {
this.orderByFields = fields;
}
build(): Query {
// ...
}
}
// ...
const queryBuilder = new QueryBuilder();
queryBuilder.from('users');
queryBuilder.page(1, 100);
queryBuilder.orderBy('firstName', 'lastName');
const query = queryBuilder.build();
好的:
class QueryBuilder {
private collection: string;
private pageNumber: number = 1;
private itemsPerPage: number = 100;
private orderByFields: string[] = [];
from(collection: string): this {
this.collection = collection;
return this;
}
page(number: number, itemsPerPage: number = 100): this {
this.pageNumber = number;
this.itemsPerPage = itemsPerPage;
return this;
}
orderBy(...fields: string[]): this {
this.orderByFields = fields;
return this;
}
build(): Query {
// ...
}
}
// ...
const query = new QueryBuilder()
.from('users')
.page(1, 100)
.orderBy('firstName', 'lastName')
.build();
SOLID
单一职责原则 (SRP)
正如代码整洁之道所述, “永远不要有超过一个理由来修改一个类”。 给一个类塞满许多功能, 就像你在航 班上只能带一个行李箱一样, 这样做的问题你的类不会有理想的内聚性, 将会有太多的理由来对它进行修改。 最小化需要修改一个类的次数时很重要的, 因为如果一个类拥有太多的功能, 一旦你修改它的一小部分, 将会很难弄清楚会对代码库中的其它模块造成什么影响。
不好的:
class UserSettings {
constructor(private readonly user: User) {
}
changeSettings(settings: UserSettings) {
if (this.verifyCredentials()) {
// ...
}
}
verifyCredentials() {
// ...
}
}
好的:
class UserAuth {
constructor(private readonly user: User) {
}
verifyCredentials() {
// ...
}
}
class UserSettings {
private readonly auth: UserAuth;
constructor(private readonly user: User) {
this.auth = new UserAuth(user);
}
changeSettings(settings: UserSettings) {
if (this.auth.verifyCredentials()) {
// ...
}
}
}
开闭原则 (OCP)
Bertrand Meyer 说过, “软件实体 (类, 模块, 函数等) 应该为扩展开放, 但是为修改关闭。” 这 是什么意思呢? 这个原则基本上说明了你应该允许用户添加功能而不必修改现有的代码。
不好的:
class AjaxAdapter extends Adapter {
constructor() {
super();
}
// ...
}
class NodeAdapter extends Adapter {
constructor() {
super();
}
// ...
}
class HttpRequester {
constructor(private readonly adapter: Adapter) {
}
async fetch<T>(url: string): Promise<T> {
if (this.adapter instanceof AjaxAdapter) {
const response = await makeAjaxCall<T>(url);
// transform response and return
} else if (this.adapter instanceof NodeAdapter) {
const response = await makeHttpCall<T>(url);
// transform response and return
}
}
}
function makeAjaxCall<T>(url: string): Promise<T> {
// request and return promise
}
function makeHttpCall<T>(url: string): Promise<T> {
// request and return promise
}
好的:
abstract class Adapter {
abstract async request<T>(url: string): Promise<T>;
// code shared to subclasses ...
}
class AjaxAdapter extends Adapter {
constructor() {
super();
}
async request<T>(url: string): Promise<T>{
// request and return promise
}
// ...
}
class NodeAdapter extends Adapter {
constructor() {
super();
}
async request<T>(url: string): Promise<T>{
// request and return promise
}
// ...
}
class HttpRequester {
constructor(private readonly adapter: Adapter) {
}
async fetch<T>(url: string): Promise<T> {
const response = await this.adapter.request<T>(url);
// transform response and return
}
}
里氏代换原则 (LSP)
这是针对一个非常简单的里面的一个恐怖意图, 它的正式定义是: “如果 S 是 T 的一个子类型, 那么类 型为 T 的对象可以被类型为 S 的对象替换(例如, 类型为 S 的对象可作为类型为 T 的替代品)儿不需 要修改目标程序的期望性质 (正确性、 任务执行性等)。” 这甚至是个恐怖的定义。
最好的解释是, 如果你又一个基类和一个子类, 那个基类和字类可以互换而不会产生不正确的结果。 这可 能还有有些疑惑, 让我们来看一下这个经典的正方形与矩形的例子。 从数学上说, 一个正方形是一个矩形, 但是你用 "is-a" 的关系用继承来实现, 你将很快遇到麻烦。
不好的:
class Rectangle {
constructor(
protected width: number = 0,
protected height: number = 0) {
}
setColor(color: string): this {
// ...
}
render(area: number) {
// ...
}
setWidth(width: number): this {
this.width = width;
return this;
}
setHeight(height: number): this {
this.height = height;
return this;
}
getArea(): number {
return this.width * this.height;
}
}
class Square extends Rectangle {
setWidth(width: number): this {
this.width = width;
this.height = width;
return this;
}
setHeight(height: number): this {
this.width = height;
this.height = height;
return this;
}
}
function renderLargeRectangles(rectangles: Rectangle[]) {
rectangles.forEach((rectangle) => {
const area = rectangle
.setWidth(4)
.setHeight(5)
.getArea(); // BAD: Returns 25 for Square. Should be 20.
rectangle.render(area);
});
}
const rectangles = [new Rectangle(), new Rectangle(), new Square()];
renderLargeRectangles(rectangles);
好的:
abstract class Shape {
setColor(color: string): this {
// ...
}
render(area: number) {
// ...
}
abstract getArea(): number;
}
class Rectangle extends Shape {
constructor(
private readonly width = 0,
private readonly height = 0) {
super();
}
getArea(): number {
return this.width * this.height;
}
}
class Square extends Shape {
constructor(private readonly length: number) {
super();
}
getArea(): number {
return this.length * this.length;
}
}
function renderLargeShapes(shapes: Shape[]) {
shapes.forEach((shape) => {
const area = shape.getArea();
shape.render(area);
});
}
const shapes = [new Rectangle(4, 5), new Rectangle(4, 5), new Square(5)];
renderLargeShapes(shapes);
接口隔离原则 (ISP)
接口隔离原则说的是 “客户端不应该强制依赖他们不需要的接口。” 这个原则与单一职责原则紧密相关。
它的真正含义是你应该总是设计一种抽象的方式,使用部分方法的客户端不会得到全部实现。 这还包括禁止客户端实现他们不需要的方法。
不好的:
interface SmartPrinter {
print();
fax();
scan();
}
class AllInOnePrinter implements SmartPrinter {
print() {
// ...
}
fax() {
// ...
}
scan() {
// ...
}
}
class EconomicPrinter implements SmartPrinter {
print() {
// ...
}
fax() {
throw new Error('Fax not supported.');
}
scan() {
throw new Error('Scan not supported.');
}
}
好的:
interface Printer {
print();
}
interface Fax {
fax();
}
interface Scanner {
scan();
}
class AllInOnePrinter implements Printer, Fax, Scanner {
print() {
// ...
}
fax() {
// ...
}
scan() {
// ...
}
}
class EconomicPrinter implements Printer {
print() {
// ...
}
}
依赖反转原则 (DIP)
这个原则阐述了两个重要的事情:
- 高级模块不应该依赖于低级模块, 两者都应该依赖与抽象;
- 抽象不应当依赖于具体实现, 具体实现应当依赖于抽象。
这个一开始会很难理解, 但是如果你使用过 Angular.js , 你应该已经看到过通过依赖注入来实现的这个原则, 虽然他们不是相同的概念, 依赖反转原则让高级模块远离低级模块的细节和创建, 可以通过 DI 来实现。 这样做的巨大益处是降低模块间的耦合。 耦合是一个非常糟糕的开发模式, 因为会导致代码难于重构。
DIP 通常通过使用控制反转 (IoC) 容器来达到。 InversifyJs 是 TypeScript 下的一个强大的 IoC 容器示例。
不好的:
import { readFile as readFileCb } from 'fs';
import { promisify } from 'util';
const readFile = promisify(readFileCb);
type ReportData = {
// ..
}
class XmlFormatter {
parse<T>(content: string): T {
// Converts an XML string to an object T
}
}
class ReportReader {
// BAD: We have created a dependency on a specific request implementation.
// We should just have ReportReader depend on a parse method: `parse`
private readonly formatter = new XmlFormatter();
async read(path: string): Promise<ReportData> {
const text = await readFile(path, 'UTF8');
return this.formatter.parse<ReportData>(text);
}
}
// ...
const reader = new ReportReader();
await report = await reader.read('report.xml');
好的:
import { readFile as readFileCb } from 'fs';
import { promisify } from 'util';
const readFile = promisify(readFileCb);
type ReportData = {
// ..
}
interface Formatter {
parse<T>(content: string): T;
}
class XmlFormatter implements Formatter {
parse<T>(content: string): T {
// Converts an XML string to an object T
}
}
class JsonFormatter implements Formatter {
parse<T>(content: string): T {
// Converts a JSON string to an object T
}
}
class ReportReader {
constructor(private readonly formatter: Formatter) {
}
async read(path: string): Promise<ReportData> {
const text = await readFile(path, 'UTF8');
return this.formatter.parse<ReportData>(text);
}
}
// ...
const reader = new ReportReader(new XmlFormatter());
await report = await reader.read('report.xml');
// or if we had to read a json report
const reader = new ReportReader(new JsonFormatter());
await report = await reader.read('report.json');
测试
测试比发布更加重要。 如果你没有测试或者测试不够充分, 每次发布时你就不能确认没有破坏任何事情。 测试的量由你的团队决定, 但是拥有 100% 的覆盖率(包括所有的语句和分支)是你为什么能达到高度自信 和内心的平静。 这意味着需要一个额外的伟大的测试框架, 也需要一个好的覆盖率工具。
没有理由不写测试。 这里有大量的优秀的 JS 测试框架, 选一个适合你的团队的即可。 当为团队选择了测试框架之后, 接下来的目标是为生产的每一个新的功能/模 块编写测试。 如果你倾向于测试驱动开发(TDD), 那就太棒了, 但是要点是确认你在上线任何功能或者重 构一个现有功能之前, 达到了需要的目标覆盖率。
TDD 的三大纪律
- 不得写任何生产代码, 除非是修复了一个失败的单元测试;
- 出现任何失败, 不得继续编写任何单元测试; 编译失败也是失败;
- 只要有失败的单元测试, 不得继续编写任何生产代码;
F.I.R.S.T. 规则
简洁测试应当遵循的规则:
-
快速 (Fast) 测试应当很快, 因为我们希望经常运行他们。
-
独立 (Independent) 测试不应当相互依赖。 不管是单独运行还是一起以任意顺序运行, 应当有相同的输出。
-
重复 (Repeatable) 测试应当是在任何环境下可重复的, 测试失败不应有任何理由。
-
自验证 (Self-Validating) 一个测试的结果应该是 通过 或者 失败 。 不需要去比较日志文件来判断测试是否通过。
-
及时 (Timely) 单元测试应该在产品代码之前编写。 如果你在产品代码之后编写测试, 就会发现写测试太难了。
每个测试单个概念
单元测试也应当遵守 单一职责原则 。 让每个单元测试只包含一个断言。
不好的:
import { assert } from 'chai';
describe('AwesomeDate', () => {
it('handles date boundaries', () => {
let date: AwesomeDate;
date = new AwesomeDate('1/1/2015');
assert.equal('1/31/2015', date.addDays(30));
date = new AwesomeDate('2/1/2016');
assert.equal('2/29/2016', date.addDays(28));
date = new AwesomeDate('2/1/2015');
assert.equal('3/1/2015', date.addDays(28));
});
});
好的:
import { assert } from 'chai';
describe('AwesomeDate', () => {
it('handles 30-day months', () => {
const date = new AwesomeDate('1/1/2015');
assert.equal('1/31/2015', date.addDays(30));
});
it('handles leap year', () => {
const date = new AwesomeDate('2/1/2016');
assert.equal('2/29/2016', date.addDays(28));
});
it('handles non-leap year', () => {
const date = new AwesomeDate('2/1/2015');
assert.equal('3/1/2015', date.addDays(28));
});
});
测试的名称应当揭示它的动机
当测试失败时, 它的名称就是错误的第一指示。
不好的:
describe('Calendar', () => {
it('2/29/2020', () => {
// ...
});
it('throws', () => {
// ...
});
});
好的:
describe('Calendar', () => {
it('should handle leap year', () => {
// ...
});
it('should throw when format is invalid', () => {
// ...
});
});
并发
倾向于 Promise 而不是回调
回调不够简洁, 因为他们会产生过多的嵌套 (回调地狱) 。 这些工具可以将使用回调函数转换成返回 Promise 的函数 (对于 Node.js , 参考 util.promisify
, 而对于更加通用的场景, 参考 pify, es6-promisify )。
不好的:
import { get } from 'request';
import { writeFile } from 'fs';
function downloadPage(url: string, saveTo: string, callback: (error: Error, content?: string) => void) {
get(url, (error, response) => {
if (error) {
callback(error);
} else {
writeFile(saveTo, response.body, (error) => {
if (error) {
callback(error);
} else {
callback(null, response.body);
}
});
}
});
}
downloadPage('https://en.wikipedia.org/wiki/Robert_Cecil_Martin', 'article.html', (error, content) => {
if (error) {
console.error(error);
} else {
console.log(content);
}
});
好的:
import { get } from 'request';
import { writeFile } from 'fs';
import { promisify } from 'util';
const write = promisify(writeFile);
function downloadPage(url: string, saveTo: string): Promise<string> {
return get(url)
.then(response => write(saveTo, response));
}
downloadPage('https://en.wikipedia.org/wiki/Robert_Cecil_Martin', 'article.html')
.then(content => console.log(content))
.catch(error => console.error(error));
Promise 提供了一些帮助方法来让代码变得更加简洁:
Pattern | Description |
---|---|
Promise.resolve(value) | 将一个值转换为一个已解决的 Promise 。 |
Promise.reject(error) | 将一个错误转换为一个已拒绝的 Promise 。 |
Promise.all(promises) | 从一组 Promise 返回一个新的 Promise , 如果这组 Promise 全部解决, 则解决新生成的 Promise , 否则拒绝新生成的 Promise 。 |
Promise.race(promises) | 从多个 Promise 生成一个新的 Promise , 返回值由第一个解决或者拒绝的 Promise 决定。 |
Promise.all
在需要并行运行任务时非常有用, 而 Promise.race
则可以比较容易的实现类似超时的 Promise 。
Async/Await 比 Promise 更加简洁
使用async
/await
语法,您可以编写更清晰,更易理解的链接承诺的代码。 通过在方法前面标记 async
关键字, 可以让 JavaScript 运行时在遇到 async
关键字时暂停(当使用 Promise 时才能这样做)。
不好的:
import { get } from 'request';
import { writeFile } from 'fs';
import { promisify } from 'util';
const write = util.promisify(writeFile);
function downloadPage(url: string, saveTo: string): Promise<string> {
return get(url).then(response => write(saveTo, response));
}
downloadPage('https://en.wikipedia.org/wiki/Robert_Cecil_Martin', 'article.html')
.then(content => console.log(content))
.catch(error => console.error(error));
Good:
import { get } from 'request';
import { writeFile } from 'fs';
import { promisify } from 'util';
const write = promisify(writeFile);
async function downloadPage(url: string, saveTo: string): Promise<string> {
const response = await get(url);
await write(saveTo, response);
return response;
}
// somewhere in an async function
try {
const content = await downloadPage('https://en.wikipedia.org/wiki/Robert_Cecil_Martin', 'article.html');
console.log(content);
} catch (error) {
console.error(error);
}
错误处理
抛出错误是件好事! 他们意味着当程序出错时, 成功的通知运行时, 并通过停止执行当前堆栈上的函数, 终止进程(在 Node 中), 并且在控制台打印错误堆栈信息以通知你。
始终使用为抛出或拒绝使用错误对象 (Error)
JavaScript 以及 TypeScript 允许你 抛出
任意对象, 一个 Promise 也能够用任意对象进行拒绝。 使用 抛出 (throw)
语法和 错误 (Error)
类型是非常明智的, 这是因为错误消息可能会被更高级的语句用 cache
捕获到。 而捕获一个字符串可能会非常令人疑惑的, 同时也会让调试更加痛苦。 同样的理由, 拒绝 Promise 时, 也应该使用 Error
类型。
不好的:
function calculateTotal(items: Item[]): number {
throw 'Not implemented.';
}
function get(): Promise<Item[]> {
return Promise.reject('Not implemented.');
}
好的:
function calculateTotal(items: Item[]): number {
throw new Error('Not implemented.');
}
function get(): Promise<Item[]> {
return Promise.reject(new Error('Not implemented.'));
}
// or equivalent to:
async function get(): Promise<Item[]> {
throw new Error('Not implemented.');
}
使用 Error
类型的好处是它被 try/catch/finally
支持, 并且所有的 Error 对象都有一个隐式属性 stack
, 在调试时很有用。 还有一个选择, 那就是不使用 throw
语法, 始终返回自定义的错误对象。 TypeScript 下更加容易, 参看下面的例子:
type Result<R> = { isError: false, value: R };
type Failure<E> = { isError: true, error: E };
type Failable<R, E> = Result<R> | Failure<E>;
function calculateTotal(items: Item[]): Failable<number, 'empty'> {
if (items.length === 0) {
return { isError: true, error: 'empty' };
}
// ...
return { isError: false, value: 42 };
}
要查看这个主意的更详细说明, 请参考原帖。
不要忽略捕获到的错误
捕获到错误后,什么都不做, 既不能让你修复错误, 也不能让你响应错误。 使用 console.log
将错误输出到控制台并不是十分高明, 因为经常会有大量的内容被打印输出到控制台, 很难再被找到。 一旦你在 try/catch
中包括了任何一点儿代码, 这就意味着你认为这里可能会有错误发生, 你应当针对它有一个计划, 或者一段代码来进行处理。
不好的:
try {
functionThatMightThrow();
} catch (error) {
console.log(error);
}
// or even worse
// 更糟糕的是
try {
functionThatMightThrow();
} catch (error) {
// ignore error
// 完全忽略错误
}
好的:
import { logger } from './logging'
try {
functionThatMightThrow();
} catch (error) {
logger.log(error);
}
不要忽略被拒绝的 Promise
由于同样的原因, 你不应该忽略由 try/catch
捕获到的错误。
不好的:
getUser()
.then((user: User) => {
return sendEmail(user.email, 'Welcome!');
})
.catch((error) => {
console.log(error);
});
好的:
import { logger } from './logging'
getUser()
.then((user: User) => {
return sendEmail(user.email, 'Welcome!');
})
.catch((error) => {
logger.log(error);
});
// 或者使用 async/await 语法:
try {
const user = await getUser();
await sendEmail(user.email, 'Welcome!');
} catch (error) {
logger.log(error);
}
格式化
格式化是主观的。 就像其它规则一样, 没有必须让你遵守的硬性规则。 重点是不要因为格式去争论, 有大量的工具来自动格式化, 使用其中的一个即可! 因为做为工程师去争论格式化就是在浪费时间和金钱。 要遵守的通用规则是 保持一致的格式化规则 。
对 TypeScript 来说, 有一个强大的工具叫做 TSLint 。 它是一个可以显著提高代码的可读性和可维护性的静态分析工具。 也已经有一些可用的 TSLint 配置供你在项目中参考:
-
标准的 TSLint 配置 - 标准风格规则
-
Airbnb 的 TSLint 配置 - Airbnb 风格指南
-
简洁代码的 TSLint 配置 - 受 Clean Code: A Handbook of Agile Software Craftsmanship 影响的 TSLint 规则
-
React 的 TSLint 配置 - React & JSX 相关的 TSLint 配置
-
TSLint + Prettier - Prettier 的代码检查规则
-
TypeScript 的 ESLint 规则 - TypeScript 的 ESLint 规则
-
Immutable - TypeScript 中禁止突变的规则
也请参考这个伟大的 TypeScript 风格指南和编码约定 。
使用一致的大小写
大小写可以告诉你很多关于你的变量, 函数等等。 这些规则是主观的, 所以你的团队可以选择他们想要的任何东西。 关键是, 无论你选择什么, 只要 保持一致 即可。
不好的:
const DAYS_IN_WEEK = 7;
const daysInMonth = 30;
const songs = ['Back In Black', 'Stairway to Heaven', 'Hey Jude'];
const Artists = ['ACDC', 'Led Zeppelin', 'The Beatles'];
function eraseDatabase() {}
function restore_database() {}
type animal = { /* ... */ }
type Container = { /* ... */ }
好的:
const DAYS_IN_WEEK = 7;
const DAYS_IN_MONTH = 30;
const SONGS = ['Back In Black', 'Stairway to Heaven', 'Hey Jude'];
const ARTISTS = ['ACDC', 'Led Zeppelin', 'The Beatles'];
function eraseDatabase() {}
function restoreDatabase() {}
type Animal = { /* ... */ }
type Container = { /* ... */ }
建议类、 接口、 类型和命名空间使用 PascalCase
风格, 变量、 函数和类成员使用 camelCase
风格。
函数的调用方与被调用方应该靠近
如果一个函数调用另一个, 则在代码中这两个函数的竖直位置应该靠近。 理想情况下,保持被调用函数在被调用函数的正上方。 我们倾向于从上到下阅读代码, 就像读一章报纸。 由于这个原因, 保持你的代码可 以按照这种方式阅读。
不好的:
class PerformanceReview {
constructor(private readonly employee: Employee) {
}
private lookupPeers() {
return db.lookup(this.employee.id, 'peers');
}
private lookupManager() {
return db.lookup(this.employee, 'manager');
}
private getPeerReviews() {
const peers = this.lookupPeers();
// ...
}
review() {
this.getPeerReviews();
this.getManagerReview();
this.getSelfReview();
// ...
}
private getManagerReview() {
const manager = this.lookupManager();
}
private getSelfReview() {
// ...
}
}
const review = new PerformanceReview(employee);
review.review();
好的:
class PerformanceReview {
constructor(private readonly employee: Employee) {
}
review() {
this.getPeerReviews();
this.getManagerReview();
this.getSelfReview();
// ...
}
private getPeerReviews() {
const peers = this.lookupPeers();
// ...
}
private lookupPeers() {
return db.lookup(this.employee.id, 'peers');
}
private getManagerReview() {
const manager = this.lookupManager();
}
private lookupManager() {
return db.lookup(this.employee, 'manager');
}
private getSelfReview() {
// ...
}
}
const review = new PerformanceReview(employee);
review.review();
组织导入
使用简洁易读的 import 语句, 您可以快速查看当前代码的依赖关系。 确保对 import
语句应用以下良好实践:
- 导入应当排序并分组;
- 未使用的导入应当删除;
- 命名导入应当排序 (比如:
import {A, B, C} from 'foo';
) - 导入源应当在分组内排序, 比如:
import * as foo from 'a'; import * as bar from 'b';
- 分组导入之间保留一个空行;
- 分组之间应当遵守下面的顺序:
- 兼容性填充 (比如:
import 'reflect-metadata';
) - Node 内置模块 (比如:
import fs from 'fs';
) - 外部模块 (比如:
import { query } from 'itiriri';
) - 内部模块 (i.e
import { UserService } from 'src/services/userService';
) - 来自父目录的模块 (比如:
import foo from '../foo'; import qux from '../../foo/qux';
) - 来自相同目录或同级目录的模块 (比如:
import bar from './bar'; import baz from './bar/baz';
)
- 兼容性填充 (比如:
不好的:
import { TypeDefinition } from '../types/typeDefinition';
import { AttributeTypes } from '../model/attribute';
import { ApiCredentials, Adapters } from './common/api/authorization';
import fs from 'fs';
import { ConfigPlugin } from './plugins/config/configPlugin';
import { BindingScopeEnum, Container } from 'inversify';
import 'reflect-metadata';
好的:
import 'reflect-metadata';
import fs from 'fs';
import { BindingScopeEnum, Container } from 'inversify';
import { AttributeTypes } from '../model/attribute';
import { TypeDefinition } from '../types/typeDefinition';
import { ApiCredentials, Adapters } from './common/api/authorization';
import { ConfigPlugin } from './plugins/config/configPlugin';
使用 TypeScript 别名
通过在 tsconfig.json
文件中的 compilerOptions 对象内定义路径和基础路径, 可以创建更漂亮的导入。
这将避免导入时出现太长的相对路径。
不好的:
import { UserService } from '../../../services/UserService';
好的:
import { UserService } from '@services/UserService';
// tsconfig.json
...
"compilerOptions": {
...
"baseUrl": "src",
"paths": {
"@services": ["services/*"]
}
...
}
...
注释
使用注释就意味着代码的表达失败。 代码应该是唯一的事实来源。
不是为烂代码添加注释, 而是重写它们。
- Brian W. Kernighan 和 P. J. Plaugher
倾向于自描述的代码而不是注释
评论是代码的辩解, 不是要求。 多数情况下, 好的代码就是文档。
不好的:
// Check if subscription is active.
if (subscription.endDate > Date.now) { }
好的:
const isSubscriptionActive = subscription.endDate > Date.now;
if (isSubscriptionActive) { /* ... */ }
不要在代码库中保存注释掉的代码
因为有版本控制, 把旧的代码留在历史记录即可
不好的:
type User = {
name: string;
email: string;
// age: number;
// jobPosition: string;
}
好的:
type User = {
name: string;
email: string;
}
不要有日志式的注释
记住, 使用版本控制! 不需要僵尸代码, 注释掉的代码, 尤其是日志式的评论。 使用 git log
来
获取历史记录。
不好的:
/**
* 2016-12-20: Removed monads, didn't understand them (RM)
* 2016-10-01: Improved using special monads (JP)
* 2016-02-03: Added type-checking (LI)
* 2015-03-14: Implemented combine (JR)
*/
function combine(a: number, b: number): number {
return a + b;
}
好的:
function combine(a: number, b: number): number {
return a + b;
}
避免占位符
它们仅仅添加了干扰。 让函数和变量名称与合适的缩进和格式化为你的代码提供视觉结构。
绝大多数 IDE 支持代码折叠, 允许你展开/关闭代码段 (查看 Visual Studio Code folding regions ) 。
不好的:
////////////////////////////////////////////////////////////////////////////////
// Client class
////////////////////////////////////////////////////////////////////////////////
class Client {
id: number;
name: string;
address: Address;
contact: Contact;
////////////////////////////////////////////////////////////////////////////////
// public methods
////////////////////////////////////////////////////////////////////////////////
public describe(): string {
// ...
}
////////////////////////////////////////////////////////////////////////////////
// private methods
////////////////////////////////////////////////////////////////////////////////
private describeAddress(): string {
// ...
}
private describeContact(): string {
// ...
}
};
好的:
class Client {
id: number;
name: string;
address: Address;
contact: Contact;
public describe(): string {
// ...
}
private describeAddress(): string {
// ...
}
private describeContact(): string {
// ...
}
};
TODO 注释
当你发现你需要在代码中做一些后期改进是, 请使用 // TODO
注释。 大多数 IDE 对这种类型的注释有着特殊的支持, 你可以快速的发现全部的 TODO 列表。
记住, TODO 注视并不是烂代码的借口。
不好的:
function getActiveSubscriptions(): Promise<Subscription[]> {
// ensure `dueDate` is indexed.
return db.subscriptions.find({ dueDate: { $lte: new Date() } });
}
好的:
function getActiveSubscriptions(): Promise<Subscription[]> {
// TODO: ensure `dueDate` is indexed.
return db.subscriptions.find({ dueDate: { $lte: new Date() } });
}
翻译
本文也有其它语言版本:
巴西葡萄牙语: vitorfreitas/clean-code-typescript
简体中文:
日文: MSakamaki/clean-code-typescript
还有一些语言的版本也正在翻译中:
韩语
一旦翻译完成, 就会在这里添加链接。 关注这个讨论以获得更多的详情以及进度。 您可以通过将其翻译成您的语言,为* Clean Code *社区做出不可或缺的贡献。