eslint-config-tencent v1.1.2
JavaScript 编码规范
前言
为前端开发提供良好的基础编码风格修行指南。
基于 Airbnb 编码规范。所有规范分为三个等级:必须、推荐、可选。
必须(Mandatory) 级别要求员工必须严格按照规范的编码格式编写,否则会在代码扫描和自动化构建中报错。对应 RFC 2119 中的 MUST, MUST NOT 和 REQUIRED 级别。
推荐(Preferable) 级别希望员工尽量按照规范编写,但如有特殊情况,可以不采用。对应 RFC 2119 中的 SHOULD, SHOULD NOT 级别。
可选(Optional) 级别并不对员工编码提出要求,一般是JavaScript常识以及ES6、ES7的新语法,但仍希望员工按参考规范编写。委员会将定期review编码规范,不断提高规范等级要求。对应 RFC 2119 中的 MAY 级别。
本文档中的代码为示例代码,出于演示目的,不一定完全符合本文档所规定的所有规则要求。
本文档中的示例代码中会有 Good 或 Best 的提示,表示这是遵守代码规范的一种写法。 需要说明的是,虽然 Good 写法符合这条规范,但不代表这是满足规范的唯一方式。
未尽事宜,应当首先以同一文件中的其他类似风格为准,若该文件中没有类似结构,则以项目风格为准。一致性是最重要的。
License
基于 MIT 协议开源。
查看 Notice 文件了解注明和鸣谢信息。
0. 相关工具
eslint-config-tencent 规则包
安装:
javascript
tnpm install eslint @tencent/eslint-config-tencent --save-devESLint 传统配置 Legacy Config
.eslintrc.js:
module.exports = {
  extends: ['@tencent/eslint-config-tencent'],
}typescript 还需要安装依赖
tnpm install @typescript-eslint/eslint-plugin @typescript-eslint/parser --save-dev.eslintrc.js:
module.exports = {
  extends: ['@tencent/eslint-config-tencent', '@tencent/eslint-config-tencent/ts'],
}使用 prettier 的用户需要安装对应的依赖
tnpm install prettier eslint-plugin-prettier --save-dev.eslintrc.js:
module.exports = {
  extends: ['@tencent/eslint-config-tencent', '@tencent/eslint-config-tencent/ts', '@tencent/eslint-config-tencent/prettier'],
}注意:使用 prettier 规则之后,会自动禁用掉与之相冲突的格式相关的规则。由于prettier与ESLint存在冲突,开源扫描仍以ESLint扫描结果为准。
ESLint 新版扁平配置 Flat Config
推荐使用 cjs 作为配置文件后缀,可兼容 type: module 和 type: commonjs 的项目。
eslint.config.cjs:
const tencentEslintConfig = require('@tencent/eslint-config-tencent/flat');
module.exports = tencentEslintConfig({});使用 TypeScript 的项目需要传入 tsconfig.json 所在的目录位置,
eslint.config.cjs:
const tencentEslintConfig = require('@tencent/eslint-config-tencent/flat');
module.exports = tencentEslintConfig({
  tsconfigRootDir: __dirname,
});Orange-ci 插件
增量检查
master:
  merge_request:
    - stages:
      - name: make changelist
        type: git:changeList
        options:
          changed: changed.txt
      - name: git diff eslint-config-tencent
        image: csighub.tencentyun.com/standards/eslint-config-tencent:latest
        settings:
          change_file: changed.txt全量检查
master:
  merge_request:
    - stages:
      - name: git eslint-config-tencent
        image: csighub.tencentyun.com/standards/eslint-config-tencent:latest工蜂CI 插件
全量检查
mr:
  branches:
    include:
      - master
stages:
  - stage:
    - job:
        name: eslint
        steps:
        - taskType: dockerRun@latest
          displayName: dockerRun
          inputs:
            image: csighub.tencentyun.com/standards/eslint-config-tencent:latest
            cmd: ""1. 类型
- 1.1 【可选】 基本类型: 当你访问一个基本类型时,直接操作它的值。 - string
- number
- boolean
- null
- undefined
- symbol
 - const foo = 1; let bar = foo; bar = 9; console.log(foo, bar); // => 1, 9- 符号(Symbols)不能完全的被 polyfill,因此在不能原生支持symbol类型的浏览器或环境中,不应该使用symbol类型。
 
- 1.2 【可选】 复杂类型: 当你访问一个复杂类型时,直接操作其值的引用。 - object
- array
- function
 - const foo = [1, 2]; const bar = foo; bar[0] = 9; console.log(foo[0], bar[0]); // => 9, 9
2. 引用
- 2.1 【必须】 使用 - const定义你的所有引用;避免使用- var。 eslint:- prefer-const,- no-const-assign- 原因? 这样能够确保你不能重新赋值你的引用,否则可能导致错误或者产生难以理解的代码。 - // bad var a = 1; var b = 2; // good const a = 1; const b = 2;
- 2.2 【必须】 如果你必须重新赋值你的引用, 使用 - let代替- var。 eslint:- no-var- 原因? - let是块级作用域,而不像- var是函数作用域。- // bad var count = 1; if (true) { count += 1; } // good, use the let. let count = 1; if (true) { count += 1; }
- 2.3 【可选】 注意,let 和 const 都是块级作用域。 - // const 和 let 只存在于他们定义的块级作用域中。 { let a = 1; const b = 1; } console.log(a); // ReferenceError console.log(b); // ReferenceError
3. 对象
- 3.1 【必须】 使用字面量语法创建对象。 eslint: - no-new-object- // bad const item = new Object(); // good const item = {};
- 3.2 【推荐】 在创建具有动态属性名称的对象时使用计算属性名。 - 原因? 它允许你在一个地方定义对象的所有属性。 - function getKey(k) { return `a key named ${k}`; } // bad const obj = { id: 5, name: 'San Francisco', }; obj[getKey('enabled')] = true; // good const obj = { id: 5, name: 'San Francisco', [getKey('enabled')]: true, };
- 3.3 【推荐】 用对象方法简写。 eslint: - object-shorthand- // bad const value = 1; const atom = { value: value, addValue: function (newValue) { return atom.value + newValue; }, }; // good const value = 1; const atom = { value, addValue(newValue) { return atom.value + newValue; }, };
- 3.4 【推荐】 用属性值简写。 eslint: - object-shorthand- 原因? 它更加简洁并更具描述性。 - const lukeSkywalker = 'Luke Skywalker'; // bad const obj = { lukeSkywalker: lukeSkywalker, }; // good const obj = { lukeSkywalker, };
- 3.5 【推荐】 声明对象时,将简写的属性放在前面。 - 原因? 这样更容易的判断哪些属性使用的简写。 - const anakinSkywalker = 'Anakin Skywalker'; const lukeSkywalker = 'Luke Skywalker'; // bad const obj = { episodeOne: 1, twoJediWalkIntoACantina: 2, lukeSkywalker, episodeThree: 3, mayTheFourth: 4, anakinSkywalker, }; // good const obj = { lukeSkywalker, anakinSkywalker, episodeOne: 1, twoJediWalkIntoACantina: 2, episodeThree: 3, mayTheFourth: 4, };
- 3.6 【必须】 只使用引号标注无效标识符的属性。 eslint: - quote-props- 原因? 一般来说,我们认为这样更容易阅读。 它能更好地适配语法高亮显示功能,并且更容易通过许多 JS 引擎进行优化。 - // bad const bad = { 'foo': 3, 'bar': 4, 'data-blah': 5, }; // good const good = { foo: 3, bar: 4, 'data-blah': 5, };
- 3.7 【推荐】 不能直接调用 - Object.prototype的方法,如:- hasOwnProperty、- propertyIsEnumerable和- isPrototypeOf。 eslint:- no-prototype-builtins- 原因? 这些方法可能被有问题的对象上的属性覆盖 - 如 - { hasOwnProperty: false }- 或者,对象是一个空对象 (- Object.create(null))。- // bad console.log(object.hasOwnProperty(key)); // good console.log(Object.prototype.hasOwnProperty.call(object, key)); // best const has = Object.prototype.hasOwnProperty; // 在模块范围内的缓存中查找一次 console.log(has.call(object, key)); /* or */ import has from 'has'; // https://www.npmjs.com/package/has console.log(has(object, key));
- 3.8 【推荐】 使用对象扩展操作符(spread operator)浅拷贝对象,而不是用 - Object.assign方法。 使用对象的剩余操作符(rest operator)来获得一个新对象,该对象省略了某些属性。 eslint:- prefer-object-spread- // very bad const original = { a: 1, b: 2 }; const copy = Object.assign(original, { c: 3 }); // 变异的 `original` ಠ_ಠ delete copy.a; // 这.... // bad const original = { a: 1, b: 2 }; const copy = Object.assign({}, original, { c: 3 }); // copy => { a: 1, b: 2, c: 3 } // good const original = { a: 1, b: 2 }; const copy = { ...original, c: 3 }; // copy => { a: 1, b: 2, c: 3 } const { a, ...noA } = copy; // noA => { b: 2, c: 3 }
4. 数组
- 4.1 【必须】 使用字面量语法创建数组。 eslint: - no-array-constructor- // bad const items = new Array(); // good const items = [];
- 4.2 【必须】 使用 Array#push 代替直接赋值来给数组添加项。 - const someStack = []; // bad someStack[someStack.length] = 'abracadabra'; // good someStack.push('abracadabra');
- 4.3 【必须】 使用数组展开符 - ...来拷贝数组。- // bad const len = items.length; const itemsCopy = []; let i; for (i = 0; i < len; i += 1) { itemsCopy[i] = items[i]; } // good const itemsCopy = [...items];
- 4.4 【推荐】 使用展开符 - ...代替- Array.from,将一个可迭代对象转换成一个数组。- const foo = document.querySelectorAll('.foo'); // good const nodes = Array.from(foo); // best const nodes = [...foo];
- 4.5 【必须】 使用 Array.from 将一个类数组(array-like)对象转换成一个数组。 - const arrLike = { 0: 'foo', 1: 'bar', 2: 'baz', length: 3 }; // bad const arr = Array.prototype.slice.call(arrLike); // good const arr = Array.from(arrLike);
- 4.6 【必须】 使用 Array.from 代替展开符 - ...映射迭代器,因为它避免了创建一个中间数组。- // bad const baz = [...foo].map(bar); // good const baz = Array.from(foo, bar);
- 4.7 【推荐】 在数组回调函数中使用 return 语句。 如果函数体由单个语句的返回表达式组成,并且无副作用,那么可以省略返回值, 具体查看 8.2。 eslint: - array-callback-return- // good [1, 2, 3].map((x) => { const y = x + 1; return x * y; }); // good [1, 2, 3].map(x => x + 1); // bad - 没有返回值,意味着在第一次迭代后 `acc` 没有被定义 [[0, 1], [2, 3], [4, 5]].reduce((acc, item, index) => { const flatten = acc.concat(item); acc[index] = flatten; }); // good [[0, 1], [2, 3], [4, 5]].reduce((acc, item, index) => { const flatten = acc.concat(item); acc[index] = flatten; return flatten; }, []); // bad inbox.filter((msg) => { const { subject, author } = msg; if (subject === 'Mockingbird') { return author === 'Harper Lee'; } else { return false; } }); // good inbox.filter((msg) => { const { subject, author } = msg; if (subject === 'Mockingbird') { return author === 'Harper Lee'; } return false; });
- 4.8 【推荐】 如果数组有多行,则在数组开始括号 - [的时候换行,然后在数组结束括号- ]的时候换行。 eslint:- array-bracket-newline- // bad const arr = [ [0, 1], [2, 3], [4, 5], ]; const objectInArray = [{ id: 1, }, { id: 2, }]; const numberInArray = [ 1, 2, ]; // good const arr = [[0, 1], [2, 3], [4, 5]]; const objectInArray = [ { id: 1, }, { id: 2, }, ]; const numberInArray = [ 1, 2, ];
5. 解构
- 5.1 【推荐】 在访问和使用对象的多个属性时使用对象解构。 eslint: - prefer-destructuring- 原因? 解构可以避免为这些属性创建临时引用。 - // bad function getFullName(user) { const firstName = user.firstName; const lastName = user.lastName; return `${firstName} ${lastName}`; } // good function getFullName(user) { const { firstName, lastName } = user; return `${firstName} ${lastName}`; } // best function getFullName({ firstName, lastName }) { return `${firstName} ${lastName}`; }
- 5.2 【推荐】 使用数组解构。 eslint: - prefer-destructuring- const arr = [1, 2, 3, 4]; // bad const first = arr[0]; const second = arr[1]; // good const [first, second] = arr;
- 5.3 【必须】 在有多个返回值时, 使用对象解构,而不是数组解构。 - 原因? 你可以随时添加新的属性或者改变属性的顺序,而不用修改调用方。 - // bad function processInput(input) { // 处理代码... return [left, right, top, bottom]; } // 调用者需要考虑返回数据的顺序。 const [left, __, top] = processInput(input); // good function processInput(input) { // 处理代码... return { left, right, top, bottom }; } // 调用者只选择他们需要的数据。 const { left, top } = processInput(input);
6. 字符
- 6.1 【推荐】 使用单引号 - ''定义字符串。 eslint:- quotes- // bad const name = "Capt. Janeway"; // bad - 模板文字应该包含插值或换行。 const name = `Capt. Janeway`; // good const name = 'Capt. Janeway';
- 6.2 【必须】 不应该用字符串跨行连接符的格式来跨行编写,这样会使当前行长度超过100个字符。 - 原因? 断开的字符串维护起来很痛苦,并且会提高索引难度。 - // bad const errorMessage = 'This is a super long error that was thrown because \ of Batman. When you stop to think about how Batman had anything to do \ with this, you would get nowhere \ fast.'; // bad const errorMessage = 'This is a super long error that was thrown because ' + 'of Batman. When you stop to think about how Batman had anything to do ' + 'with this, you would get nowhere fast.'; // good const errorMessage = 'This is a super long error that was thrown because of Batman. When you stop to think about how Batman had anything to do with this, you would get nowhere fast.';
- 6.3 【必须】 构建字符串时,使用字符串模板代替字符串拼接。 eslint: - prefer-template- template-curly-spacing- 原因? 字符串模板为您提供了一种可读的、简洁的语法,具有正确的换行和字符串插值特性。 - // bad function sayHi(name) { return 'How are you, ' + name + '?'; } // bad function sayHi(name) { return ['How are you, ', name, '?'].join(); } // bad function sayHi(name) { return `How are you, ${ name }?`; } // good function sayHi(name) { return `How are you, ${name}?`; }
- 6.4 【必须】 永远不要使用 - eval()执行放在字符串中的代码,它导致了太多的漏洞。 eslint:- no-eval
- 6.5 【必须】 不要在字符串中转义不必要的字符。 eslint: - no-useless-escape- 原因? 反斜杠损害了可读性,因此只有在必要的时候才可以出现。 - // bad const foo = '\'this\' \i\s \"quoted\"'; // good const foo = '\'this\' is "quoted"'; const foo = `my name is '${name}'`;
7. 函数
- 7.1 【可选】 使用命名的函数表达式代替函数声明。 eslint: - func-style- 原因? 函数声明时作用域被提前了,这意味着在一个文件里函数很容易(太容易了)在其定义之前被引用。这样伤害了代码可读性和可维护性。如果你发现一个函数又大又复杂,并且它干扰了对这个文件其他部分的理解,那么是时候把这个函数单独抽成一个模块了!别忘了给表达式显式的命名,不用管这个名字是不是由一个确定的变量推断出来的(这在现代浏览器和类似babel编译器中很常见)。这消除了由匿名函数在错误调用栈产生的所有假设。 * (Discussion) - // bad function foo() { // ... } // also good * const foo = function () { // ... }; // good const short = function longUniqueMoreDescriptiveLexicalFoo() { // ... };
- 7.2 【必须】 把立即执行函数包裹在圆括号里。 eslint: - wrap-iife- 原因? 立即调用的函数表达式是个独立的单元 - 将它和它的调用括号还有入参包装在一起可以非常清晰的表明这一点。请注意,在一个到处都是模块的世界中,您几乎用不到 IIFE。 - // immediately-invoked function expression (IIFE) 立即调用的函数表达式 (function () { console.log('Welcome to the Internet. Please follow me.'); }());
- 7.3 【必须】 切记不要在非功能块中声明函数 ( - if,- while, 等)。 请将函数赋值给变量。 浏览器允许你这样做, 但是不同浏览器会有不同的行为, 这并不是什么好事。 eslint:- no-loop-func
- 7.4 【必须】 ECMA-262 将 - block定义为语句列表。 而函数声明并不是语句。- // bad if (currentUser) { function test() { console.log('Nope.'); } } // good let test; if (currentUser) { test = () => { console.log('Yup.'); }; }
- 7.5 【必须】 永远不要给一个参数命名为 - arguments。 这将会覆盖函数默认的- arguments对象。 eslint:- no-shadow-restricted-names- // bad function foo(name, options, arguments) { // ... } // good function foo(name, options, args) { // ... }
- 7.6 【推荐】 使用 rest 语法 - ...代替- arguments。 eslint:- prefer-rest-params- 原因? - ...明确了你想要拉取什么参数。 而且, rest 参数是一个真正的数组,而不仅仅是类数组的- arguments。- // bad function concatenateAll() { const args = Array.prototype.slice.call(arguments); return args.join(''); } // good function concatenateAll(...args) { return args.join(''); }
- 7.7 【推荐】 使用默认的参数语法,而不是改变函数参数。 - // really bad function handleThings(opts) { // 不!我们不应该修改参数。 // 更加错误的是: 如果 opts 是一个 "非正值"(falsy)它将被设置成一个对象 // 这或许正是你想要的,但它可能会导致一些难以察觉的错误。 opts = opts || {}; // ... } // still bad function handleThings(opts) { if (opts === void 0) { opts = {}; } // ... } // good function handleThings(opts = {}) { // ... }
- 7.8 【必须】 使用默认参数时避免副作用。 - 原因? 他们很容易混淆。 - var b = 1; // bad function count(a = b++) { console.log(a); } count(); // 1 count(); // 2 count(3); // 3 count(); // 3
- 7.9 【推荐】 总是把默认参数放在最后。 eslint: - default-param-last- // bad function handleThings(opts = {}, name) { // ... } // good function handleThings(name, opts = {}) { // ... }
- 7.10 【推荐】 永远不要使用函数构造器来创建一个新函数。 eslint: - no-new-func- 原因? 以这种方式创建一个函数跟 - eval()差不多,将会导致漏洞。- // bad var add = new Function('a', 'b', 'return a + b'); // still bad var subtract = Function('a', 'b', 'return a - b');
- 7.11 【必须】 函数声明语句中需要空格。 eslint: - space-before-function-paren- space-before-blocks- 原因? 一致性很好,在删除或添加名称时不需要添加或删除空格。 - // bad const f = function(){}; const g = function (){}; const h = function() {}; // good const x = function () {}; const y = function a() {};
- 7.12 【推荐】 不要改变入参。 eslint: - no-param-reassign- 原因? 操作入参对象会导致原始调用位置出现意想不到的副作用。 - // bad function f1(obj) { obj.key = 1; } // good function f2(obj) { const key = Object.prototype.hasOwnProperty.call(obj, 'key') ? obj.key : 1; }
- 7.13 【推荐】 不要对入参重新赋值,也不要给入参的属性赋值。部分要求修改入参的常用库(如 Koa、Vuex)可以豁免。 eslint: - no-param-reassign- 原因? 重新赋值参数会导致意外的行为,尤其是在访问 - arguments对象的时候。 它还可能导致性能优化问题,尤其是在 V8 中。- // bad function f1(a) { a = 1; // ... } function f2(a) { if (!a) { a = 1; } // ... } // good function f3(a) { const b = a || 1; // ... } function f4(a = 1) { // ... }
- 7.14 【推荐】 优先使用扩展运算符 - ...来调用可变参数函数。 eslint:- prefer-spread- 原因? 它更加清晰,你不需要提供上下文,并且能比用 - apply来执行可变参数的- new操作更容易些。- // bad const x = [1, 2, 3, 4, 5]; console.log.apply(console, x); // good const x = [1, 2, 3, 4, 5]; console.log(...x); // bad new (Function.prototype.bind.apply(Date, [null, 2016, 8, 5])); // good new Date(...[2016, 8, 5]);
- 7.15 【推荐】 调用或者书写一个包含多个参数的函数应该像这个指南里的其他多行代码写法一样: 每行值包含一个参数,并且最后一行也要以逗号结尾。eslint: - function-paren-newline- // bad function foo(bar, baz, quux) { // ... } // good function foo( bar, baz, quux, ) { // ... } // bad console.log(foo, bar, baz); // good console.log( foo, bar, baz, );
8. 箭头函数
- 8.1 【推荐】 当你必须使用匿名函数时 (当传递内联函数时), 使用箭头函数。 eslint: - prefer-arrow-callback,- arrow-spacing- 原因? 它创建了一个在 - this上下文中执行的函数版本,它通常是你想要的,并且是一个更简洁的语法。- 什么时候不适用? 如果你有一个相当复杂的函数,你可能会把这些逻辑转移到它自己的命名函数表达式里。 - // bad [1, 2, 3].map(function (x) { const y = x + 1; return x * y; }); // good [1, 2, 3].map((x) => { const y = x + 1; return x * y; });
- 8.2 【推荐】 如果函数体由一个没有副作用的 表达式 语句组成,删除大括号和 - return。否则,保留括号并继续使用- return语句。 eslint:- arrow-parens,- arrow-body-style- 原因? 语法糖。 多个函数被链接在一起时,提高可读性。 - // bad [1, 2, 3].map(number => { const nextNumber = number + 1; `A string containing the ${nextNumber}.`; }); // good [1, 2, 3].map(number => `A string containing the ${number + 1}.`); // good [1, 2, 3].map((number) => { const nextNumber = number + 1; return `A string containing the ${nextNumber}.`; }); // good [1, 2, 3].map((number, index) => ({ [index]: number, })); // 没有副作用的隐式返回 function foo(callback) { const val = callback(); if (val === true) { // 如果回调返回 true 执行 } } let bool = false; // bad foo(() => bool = true); // good foo(() => { bool = true; });
- 8.3 【推荐】 如果表达式跨越多个行,用括号将其括起来,以获得更好的可读性。 - 原因? 它清楚地表明了函数的起点和终点。 
  // bad
  ['get', 'post', 'put'].map(httpMethod => Object.prototype.hasOwnProperty.call(
      httpMagicObjectWithAVeryLongName,
      httpMethod,
    )
  );
  // good
  ['get', 'post', 'put'].map(httpMethod => (
    Object.prototype.hasOwnProperty.call(
      httpMagicObjectWithAVeryLongName,
      httpMethod,
    )
  ));- 8.4 【推荐】 如果你的函数只有一个参数并且函数体没有大括号,就删除圆括号。 否则,为了保证清晰和一致性,请给参数加上括号。 注意:总是使用括号是可以接受的,在这种情况下,我们使用 “always” option 来配置 eslint. eslint: - arrow-parens- 原因? 让代码看上去不那么乱。 - // bad [1, 2, 3].map((x) => x * x); // good [1, 2, 3].map(x => x * x); // good [1, 2, 3].map(number => ( `A long string with the ${number}. It’s so long that we don’t want it to take up space on the .map line!` )); // bad [1, 2, 3].map(x => { const y = x + 1; return x * y; }); // good [1, 2, 3].map((x) => { const y = x + 1; return x * y; });
- 8.5 【推荐】 避免搞混箭头函数符号 ( - =>) 和比较运算符 (- <=,- >=)。 eslint:- no-confusing-arrow- // bad const itemHeight = item => item.height > 256 ? item.largeSize : item.smallSize; // bad const itemHeight = (item) => item.height > 256 ? item.largeSize : item.smallSize; // good const itemHeight = item => (item.height > 256 ? item.largeSize : item.smallSize); // good const itemHeight = (item) => { const { height, largeSize, smallSize } = item; return height > 256 ? largeSize : smallSize; };
- 8.6 【推荐】 在箭头函数用隐式 return 时强制将函数体的位置约束在箭头后。 eslint: - implicit-arrow-linebreak- // bad (foo) => bar; (foo) => (bar); // good (foo) => bar; (foo) => (bar); (foo) => ( bar );
9. 类和构造器
- 9.1 【推荐】 尽量使用 - class. 避免直接操作- prototype.- 原因? - class语法更简洁,更容易看懂。- // bad function Queue(contents = []) { this.queue = [...contents]; } Queue.prototype.pop = function () { const value = this.queue[0]; this.queue.splice(0, 1); return value; }; // good class Queue { constructor(contents = []) { this.queue = [...contents]; } pop() { const value = this.queue[0]; this.queue.splice(0, 1); return value; } }
- 9.2 【推荐】 使用 - extends来实现继承。- 原因? 它是一个内置的方法,可以在不破坏 - instanceof的情况下继承原型功能。- // bad const inherits = require('inherits'); function PeekableQueue(contents) { Queue.apply(this, contents); } inherits(PeekableQueue, Queue); PeekableQueue.prototype.peek = function () { return this.queue[0]; }; // good class PeekableQueue extends Queue { peek() { return this.queue[0]; } }
- 9.3 【可选】 类的成员方法,可以返回 - this, 来实现链式调用。- // bad Jedi.prototype.jump = function () { this.jumping = true; return true; }; Jedi.prototype.setHeight = function (height) { this.height = height; }; const luke = new Jedi(); luke.jump(); // => true luke.setHeight(20); // => undefined // good class Jedi { jump() { this.jumping = true; return this; } setHeight(height) { this.height = height; return this; } } const luke = new Jedi(); luke.jump() .setHeight(20);
- 9.4 【可选】 只要在确保能正常工作并且不产生任何副作用的情况下,编写一个自定义的 - toString()方法也是可以的。- class Jedi { constructor(options = {}) { this.name = options.name || 'no name'; } getName() { return this.name; } toString() { return `Jedi - ${this.getName()}`; } }
- 9.5 【推荐】 如果没有具体说明,类有默认的构造方法。一个空的构造函数或只是代表父类的构造函数是不需要写的。 eslint: - no-useless-constructor- // bad class Jedi { constructor() {} getName() { return this.name; } } // bad class Rey extends Jedi { // 这种构造函数是不需要写的 constructor(...args) { super(...args); } } // good class Rey extends Jedi { constructor(...args) { super(...args); this.name = 'Rey'; } }
- 9.6 【必须】 避免定义重复的类成员。 eslint: - no-dupe-class-members- 原因? 重复的类成员声明将会默认使用最后一个 - 具有重复的类成员可以说是一个bug。 - // bad class Foo { bar() { return 1; } bar() { return 2; } } // good class Foo { bar() { return 1; } }
- 9.7 【推荐】 类成员要么引用 - this,要么声明为静态方法,除非一个外部库或框架需要使用某些非静态方法。当一个方法为非静态方法时,一般表明它在不同的实例上会表现得不同。
10. 模块
- 10.1 【可选】 使用ES6的模块 ( - import/- export) 语法来定义模块。- // bad const AirbnbStyleGuide = require('./AirbnbStyleGuide'); module.exports = AirbnbStyleGuide.es6; // ok import AirbnbStyleGuide from './AirbnbStyleGuide'; export default AirbnbStyleGuide.es6; // best import { es6 } from './AirbnbStyleGuide'; export default es6;
- 10.2 【推荐】 不要使用import * 通配符 - 原因? 这确保你有单个默认的导出。 - // bad import * as AirbnbStyleGuide from './AirbnbStyleGuide'; // good import AirbnbStyleGuide from './AirbnbStyleGuide';
- 10.3 【推荐】 不要在import语句中直接export。 - 原因? 虽然写在一行很简洁,但是有一个明确的导入和一个明确的导出能够保证一致性。 - // bad // filename es6.js export { es6 as default } from './AirbnbStyleGuide'; // good // filename es6.js import { es6 } from './AirbnbStyleGuide'; export default es6;
- 10.4 【必须】 对于同一个路径,只在一个地方引入所有需要的东西。 eslint: - no-duplicates- 原因? 对于同一个路径,如果存在多行引入,会使代码更难以维护。 - // bad import foo from 'foo'; // … 其他导入 … // import { named1, named2 } from 'foo'; // good import foo, { named1, named2 } from 'foo'; // good import foo, { named1, named2, } from 'foo';
- 10.5 【推荐】 不要导出可变的引用。 eslint: - import/no-mutable-exports- 原因? 在一般情况下,应该避免导出可变引用。虽然在某些特殊情况下,可能需要这样,但是一般情况下只需要导出常量引用。 - // bad let foo = 3; export { foo }; // good const foo = 3; export { foo };
- 10.6 【可选】 在只有单一导出的模块里,用 export default 更好。 eslint: - import/prefer-default-export- 原因? 鼓励更多的模块只做单一导出,会增强代码的可读性和可维护性。 - // bad export function foo() {} // good export default function foo() {}
- 10.7 【必须】 将所有的 - imports 语句放在其他语句之前。 eslint:- import/first- 原因? 将所有的 - imports 提到顶部,可以防止某些诡异行为的发生。- // bad import foo from 'foo'; foo.init(); import bar from 'bar'; // good import foo from 'foo'; import bar from 'bar'; foo.init();
- 10.8 【可选】 多行引入应该像多行数组和对象字面量一样缩进。 - 原因? 这里的花括号和其他地方的花括号是一样的,遵循相同的缩进规则。末尾的逗号也是一样的。 - // bad import {longNameA, longNameB, longNameC, longNameD, longNameE} from 'path'; // good import { longNameA, longNameB, longNameC, longNameD, longNameE, } from 'path';
- 10.9 【推荐】 在模块导入语句中禁止使用 Webpack 加载器语法。 eslint: - import/no-webpack-loader-syntax- 原因? 因为在导入语句中使用 webpack 语法,会将代码和打包工具耦合在一起。应该在 - webpack.config.js中使用加载器语法。- // bad import fooSass from 'css!sass!foo.scss'; import barCss from 'style!css!bar.css'; // good import fooSass from 'foo.scss'; import barCss from 'bar.css';
11. 迭代器和发生器
- 11.1 【推荐】 不要使用迭代器。 推荐使用 JavaScript 的高阶函数代替 - for-in。 eslint:- no-iterator- no-restricted-syntax- 原因? 这有助于不可变性原则。 使用带有返回值的纯函数比使用那些带有副作用的方法,更具有可读性。 - 使用 - map()/- every()/- filter()/- find()/- findIndex()/- reduce()/- some()/ ... 遍历数组, 和使用- Object.keys()/- Object.values()/- Object.entries()迭代你的对象生成数组。- const numbers = [1, 2, 3, 4, 5]; // bad let sum = 0; for (let num of numbers) { sum += num; } sum === 15; // good let sum = 0; numbers.forEach((num) => { sum += num; }); sum === 15; // best (use the functional force) const sum = numbers.reduce((total, num) => total + num, 0); sum === 15; // bad const increasedByOne = []; for (let i = 0; i < numbers.length; i++) { increasedByOne.push(numbers[i] + 1); } // good const increasedByOne = []; numbers.forEach((num) => { increasedByOne.push(num + 1); }); // best (keeping it functional) const increasedByOne = numbers.map(num => num + 1);
- 11.2 【可选】 现在不要使用generator。 - 原因? 它们不能很好的转译为 ES5。但可以在Nodejs中使用。* 
- 11.3 【推荐】 如果你必须要使用generator,请确保正确使用空格。 eslint: - generator-star-spacing- 原因? - function和- *是同一个概念关键字的一部分 -- *不是- function的修饰符,- function*是一个不同于- function的构造器。- // bad function * foo() { // ... } // bad const bar = function * () { // ... }; // bad const baz = function *() { // ... }; // bad const quux = function*() { // ... }; // bad function*foo() { // ... } // bad function *foo() { // ... } // very bad function * foo() { // ... } // very bad const wat = function * () { // ... }; // good function* foo() { // ... } // good const foo = function* () { // ... };
12. 属性
- 12.1 【推荐】 访问属性时使用点符号。 eslint: - dot-notation- const luke = { jedi: true, age: 28, }; // bad const isJedi = luke['jedi']; // good const isJedi = luke.jedi;
- 12.2 【可选】 使用变量访问属性时,用 - []表示法。- const luke = { jedi: true, age: 28, }; function getProp(prop) { return luke[prop]; } const isJedi = getProp('jedi');
- 12.3 【推荐】 计算指数时,可以使用 - **运算符。 eslint:- no-restricted-properties.- // bad const binary = Math.pow(2, 10); // good const binary = 2 ** 10;
13. 变量
- 13.1 【必须】 变量应先声明再使用,禁止引用任何未声明的变量,除非你明确知道引用的变量存在于当前作用域链上。禁止不带任何关键词定义变量,这样做将会创建一个全局变量,污染全局命名空间,造成程序意料之外的错误。 eslint: - no-undef- prefer-const- // bad, 这会创建一个全局变量 superPower = new SuperPower(); // good const superPower = new SuperPower(); // bad, 容易污染外部变量 let superPower = 'a'; (function() { superPower = 'b'; })(); console.log(superPower); // good let superPower = 'a'; (function() { let superPower = 'b'; })(); console.log(superPower); // bad, 更常见的情况是这样的,在 for 循环里的 i 将会污染外部的变量 i let i = 1; (function() { for (i = 0; i < 10; i++) { console.log('inside', i); } console.log('outside', i) })(); console.log('global', i); // good let i = 1; (function() { // i 的作用域在 for 循环内 for (let i = 0; i < 10; i++) { console.log('inside i', i); } // 如果真的需要在 for 循环外使用循环变量,应该先定义在外部 let j; for (j = 0; j < 10; j++) { console.log('inside j:', j); } console.log('outside j', j); })(); console.log('global', i);
- 13.2 【推荐】 声明多个变量应该分开声明,避免使用 - ,一次声明多个变量。 eslint:- one-var- 原因? 这样更容易添加新的变量声明,不必担心是使用 - ;还是使用- ,所带来的代码差异。使用版本管理工具如 git ,最后那行的- ;就不会被标记为修改成- ,。 并且可以通过 debugger 逐步查看每个声明,而不是立即跳过所有声明。- // bad const items = getItems(), goSportsTeam = true, dragonball = 'z'; // bad const items = getItems(), goSportsTeam = true; // 定义成了全局变量 dragonball = 'z'; // good const items = getItems(); const goSportsTeam = true; const dragonball = 'z';
- 13.3 【推荐】 把 - const声明语句放在一起,把- let声明语句放在一起。- 原因? 这在后边如果需要根据前边的赋值变量指定一个变量时很有用,且更容易知道哪些变量是不希望被修改的。 - // bad let i; const items = getItems(); let dragonball; const goSportsTeam = true; let len; // good const goSportsTeam = true; const items = getItems(); let dragonball; let i; let length;
- 13.4 【推荐】 在你真正需要使用到变量的代码块内定义变量。 - 原因? - let和- const是块级作用域而不是函数作用域,不存在变量提升的情况。- // bad, 不必要的函数调用 function checkName(hasName) { const name = getName(); if (hasName === 'test') { return false; } if (name === 'test') { this.setName(''); return false; } return name; } // good function checkName(hasName) { if (hasName === 'test') { return false; } const name = getName(); if (name === 'test') { this.setName(''); return false; } return name; }
- 13.5 【必须】 不要链式变量赋值。 eslint: - no-multi-assign- 原因? 链式变量赋值会创建隐式全局变量。 - // bad (function example() { /* * JavaScript 把它解释为 * let a = ( b = ( c = 1 ) ); * let 关键词只适用于变量 a,变量 b 和变量 c 则变成了全局变量。 */ let a = b = c = 1; }()); // throws ReferenceError console.log(a); // 1 console.log(b); // 1 console.log(c); // good (function example() { let a = 1; let b = a; let c = a; }()); // throws ReferenceError console.log(a); // throws ReferenceError console.log(b); // throws ReferenceError console.log(c); // 对于 `const` 也一样
- 13.6 【必须】 避免使用不必要的递增和递减操作符 ( - ++,- --)。 eslint- no-plusplus- 原因? 在eslint文档中,一元操作符 - ++和- --会自动添加分号,不同的空白可能会改变源代码的语义。建议使用- num += 1这样的语句来做递增和递减,而不是使用- num++或- num ++。同时- ++num和- num++的差异也使代码的可读性变差。不必要的增量和减量语句会导致无法预先明确递增/预递减值,这可能会导致程序中的意外行为。- 但目前依然允许在 for loop 中使用 - ++、- --的语法,但依然建议尽快迁移到- += 1、- -= 1的语法。 #22- // bad, i = 11, j = 20 let i = 10; let j = 20; i ++ j // bad, i = 10, j = 21 let i = 10; let j = 20; i ++ j // bad const array = [1, 2, 3]; let num = 1; num++; --num; // not good, just acceptable for upforward compatible. let sum = 0; let truthyCount = 0; for (let i = 0; i < array.length; i++) { let value = array[i]; sum += value; if (value) { truthyCount++; } } // good const array = [1, 2, 3]; let num = 1; num += 1; num -= 1; // good const sum = array.reduce((a, b) => a + b, 0); const truthyCount = array.filter(Boolean).length;
- 13.7 【必须】 避免在赋值语句 - =前后换行。如果你的代码单行长度超过了- max-len定义的长度而不得不换行,那么使用括号包裹。 eslint- operator-linebreak.- 原因? 在 - =前后换行,可能混淆赋的值。- // bad const foo = 'superLongLongLongLongLongLongLongLongString'; // bad const bar = superLongLongLongLongLongLongLongLongFunctionName(); // bad const fullHeight = borderTop + innerHeight + borderBottom; // bad const anotherHeight = borderTop + innerHeight + borderBottom; // bad const thirdHeight = ( borderTop + innerHeight + borderBottom ); // good - max-len 会忽略字符串,直接写后面即可。 const foo = 'superLongLongLongLongLongLongLongLongString'; // good const bar = ( superLongLongLongLongLongLongLongLongFunctionName() ); // good const fullHeight = borderTop + innerHeight + borderBottom; // good const anotherHeight = borderTop + innerHeight + borderBottom; // good const thirdHeight = ( borderTop + innerHeight + borderBottom );
- 13.8 【必须】 禁止定义了变量却不使用它。 eslint: - no-unused-vars- 原因? 在代码里到处定义变量却没有使用它,不完整的代码结构看起来像是个代码错误。即使没有使用,但是定义变量仍然需要消耗资源,并且对阅读代码的人也会造成困惑,不知道这些变量是要做什么的。 - // bad let some_unused_var = 42; // bad,定义了变量不意味着就是使用了 let y = 10; y = 5; // bad,对自身的操作并不意味着使用了 let z = 0; z = z + 1; // bad, 未使用的函数参数 function getX(x, y) { return x; } // good function getXPlusY(x, y) { return x + y; } let x = 1; let y = a + 2; alert(getXPlusY(x, y)); /* * 有时候我们想要提取某个对象排除了某个属性外的其他属性,会用 rest 参数解构对象 * 这时候 type 虽然未使用,但是仍然被定义和赋值,这也是一种空间的浪费 * type 的值是 'a' * coords 的值是 data 对象,但是没有 type 属性 { example1: 'b', example2: 'c' } */ let data = { type: 'a', example1: 'b', example2: 'c' } let { type, ...coords } = data; /* * 有时候如果定义了一个基类或者接口(Typescript) * 基类或接口中该方法需要定义这个参数,但消费参数是由子类自行实现的 * 可以通过在变量名前加下划线的方式来表示该变量是未使用的 */ export class Model { toList(_isCompact) { // 待子类实现 } } export class MyModel extends Model { data = []; toList(isCompact) { return isCompact ? this.data.flat() : this.data; } }
14. 变量提升
- 14.1 【可选】 - var定义的变量会被提升到函数作用域范围内的最顶部,但是对它的赋值是不会被提升的,因此在函数顶部相当于定义了变量,但是值是- undefined。- const和- let声明的变量受到一个称之为 "暂时性死区" (Temporal Dead Zones ,简称 TDZ) 的新概念保护,因此在 "暂时性死区" 内部的- const和- let变量,都需要先声明再使用,否则会报错。详情可以阅读 typeof 不再安全 这篇文章。- // notDefined 未定义 (假设没有定义的同名全局变量) function example() { // => throws a ReferenceError console.log(notDefined); } /* * 函数体内部因存在 var 变量声明,因此在引用变量的语句之前,变量提升就已经起作用了 * 注意: 真正的值 `true` 不会被提升。 */ function example() { // => undefined console.log(declaredButNotAssigned); var declaredButNotAssigned = true; } /* * 解释器将变量提升到函数的顶部 * 这意味着我们可以将上边的例子重写为: */ function example() { let declaredButNotAssigned; // => undefined console.log(declaredButNotAssigned); declaredButNotAssigned = true; } // 使用 const 和 let function example() { // => throws a ReferenceError console.log(declaredButNotAssigned); // => throws a ReferenceError console.log(typeof declaredButNotAssigned); const declaredButNotAssigned = true; }
- 14.2 【可选】 匿名函数赋值表达式提升变量名,而不是函数赋值。 - function example() { // => undefined console.log(anonymous); // => TypeError anonymous is not a function anonymous(); var anonymous = function () { console.log('anonymous function expression'); }; }
- 14.3 【可选】 命名函数表达式提升的是变量名,而不是函数名或者函数体。 - function example() { // => undefined console.log(named); // => TypeError named is not a function named(); // => ReferenceError superPower is not defined superPower(); var named = function superPower() { console.log('Flying'); }; } // 当函数名和变量名相同时也是如此。 function example() { // => undefined console.log(named); // => TypeError named is not a function named(); var named = function named() { console.log('named'); }; }
- 14.4 【可选】 函数声明提升其名称和函数体。 - function example() { // => Flying superPower(); function superPower() { console.log('Flying'); } }- 更多信息请参考 Ben Cherry 的 JavaScript Scoping & Hoisting。
 
15. 比较运算符和等号
- 15.1 【推荐】 使用 - ===和- !==而不是- ==和- !=。 eslint:- eqeqeq- 原因? - ==和- !=存在类型转换,会得到和- ===、- !==不一样的结果- // bad, true undefined == null // good, false undefined === null // bad, true '0' == 0 // good, false '0' === 0 // bad, true 0 == false // good, false 0 === false // bad, true '' == false // good, false '' === false
- 15.2 【可选】 条件语句,例如 - if语句使用- ToBoolean的抽象方法来计算表达式的结果,并始终遵循以下简单的规则:- Objects 的取值为: true,{} 和 [] 的取值也为: true
- Undefined 的取值为: false
- Null 的取值为: false
- Booleans 的取值为: 布尔值的取值
- Numbers 的取值为:如果为 +0, -0, or NaN 值为 false 否则为 true
- Strings 的取值为: 如果是一个空字符串 ''值为 false 否则为 true
 - if ([0] && []) { // true, 数组(即使是空的)是一个对象,对象的取值为 true }
- 15.3 【推荐】 对于布尔值(在明确知道是布尔值的情况下)使用简写,但是对于字符串和数字进行显式比较。 - // bad if (isValid === true) { // ... } // good if (isValid) { // ... } // bad if (name) { // ... } // good if (name !== '') { // ... } // bad if (collection.length) { // ... } // good if (collection.length > 0) { // ... }
- 15.4 【可选】 关于布尔值转换和条件语句,详细信息可参阅 Angus Croll 的 Truth Equality and JavaScript 这篇文章。 
- 15.5 【必须】 在 - case和- default的子句中,如果存在声明 (例如.- let,- const,- function, 和- class),使用大括号来创建块级作用域。 eslint:- no-case-declarations- 原因? 变量声明的作用域在整个 switch 语句内,但是只有在 case 条件为真时变量才会被初始化。 当多个 - case语句定义相同的变量时,就会导致变量覆盖的问题。- // bad switch (foo) { case 1: let x = 1; break; case 2: const y = 2; break; case 3: function f() { // ... } break; default: class C {} } // good switch (foo) { case 1: { let x = 1; break; } case 2: { const y = 2; break; } case 3: { function f() { // ... } break; } case 4: bar(); break; default: { class C {} } }
- 15.6 【推荐】 三元表达式不应该嵌套,通常是单行表达式,如果确实需要多行表达式,那么应该考虑使用条件语句。 eslint: - no-nested-ternary- // bad const foo = maybe1 > maybe2 ? "bar" : value1 > value2 ? "baz" : null; // 分离为两个三元表达式 const maybeNull = value1 > value2 ? 'baz' : null; // better const foo = maybe1 > maybe2 ? 'bar' : maybeNull; // best const foo = maybe1 > maybe2 ? 'bar' : maybeNull;
- 15.7 【推荐】 避免不必要的三元表达式。 eslint: - no-unneeded-ternary- // bad const foo = a ? a : b; const bar = c ? true : false; const baz = c ? false : true; // good const foo = a || b; const bar = !!c; const baz = !c;
- 15.8 【必须】 使用混合运算符时,使用小括号括起来需要一起计算的部分,只要觉得有必要,那么尽可能地用括号让代码的优先级更明显。大家都能理解的运算符 - +、- -、- **不要求添加括号。但我们建议在- *、- /之间添加括号,因为乘除运算符混写写的比较长的时候容易产生歧义。 eslint:- no-mixed-operators- 原因? 这能提高可读性并且表明开发人员的意图。 - // bad const foo = a && b < 0 || c > 0 || d + 1 === 0; // bad const bar = a ** b - 5 % d; // bad, 可能陷入一种 (a || b) && c 的思考 if (a || b && c) { return d; } // bad const bar1 = a + b / c * (d / e); // good const foo = (a && b < 0) || (c > 0) || (d + 1 === 0); // good const bar = (a ** b) - (5 % d); // good if (a || (b && c)) { return d; } // good const bar1 = a + (b / c) * (d / e);
16. 代码块
- 16.1 【必须】 当有多行代码块的时候,应使用大括号包裹。 eslint: - nonblock-statement-body-position- // bad if (test) return false; // bad let condition = true; let test = 1; // 在缩进不规范的时候,容易造成误解 if (condition) condition = false; test = 2; // good if (test) return false; // good if (test) { return false; } // bad function foo() { return false; } // good function bar() { return false; }
- 16.2 【必须】 如果你使用的是 - if和- else的多行代码块,则将- else语句放在- if块闭括号同一行的位置。 eslint:- brace-style- // bad if (test) { thing1(); thing2(); } else { thing3(); } // good if (test) { thing1(); thing2(); } else { thing3(); }
- 16.3 【推荐】 如果一个 - if块总是会执行 return 语句,那么接下来的- else块就没有必要了。 如果一个包含- return语句的- else if块,在一个包含了- return语句的- if块之后,那么可以拆成多个- if块。 eslint:- no-else-return- // bad function foo() { if (x) { return x; } else { return y; } } // bad function cats() { if (x) { return x; } else if (y) { return y; } } // bad function dogs() { if (x) { return x; } else { if (y) { return y; } } } // good function foo() { if (x) { return x; } return y; } // good function cats() { if (x) { return x; } if (y) { return y; } } // good function dogs(x) { if (x) { if (z) { return y; } } else { return z; } }
17. 控制语句
- 17.1 【推荐】 如果你的控制语句 ( - if,- while等) 太长或者超过了一行最大长度的限制,则可以将每个条件(或组)放入一个新的行。 逻辑运算符应该在行的开始。- 原因? 在行的开头要求运算符保持对齐,并遵循类似于方法链的模式。这提高了可读性,并且使更复杂的逻辑更容易直观的被理解。 - // bad if ((foo === 123 || bar === 'abc') && doesItLookGoodWhenItBecomesThatLong() && isThisReallyHappening()) { thing1(); } // bad if (foo === 123 && bar === 'abc') { thing1(); } // bad if (foo === 123 && bar === 'abc') { thing1(); } // bad if ( foo === 123 && bar === 'abc' ) { thing1(); } // good if ( foo === 123 && bar === 'abc' ) { thing1(); } // good if ( (foo === 123 || bar === 'abc') && doesItLookGoodWhenItBecomesThatLong() && isThisReallyHappening() ) { thing1(); } // good if (foo === 123 && bar === 'abc') { thing1(); }
- 17.2 【必须】 不要使用选择操作符代替控制语句。 - // bad !isRunning && startRunning(); // good if (!isRunning) { startRunning(); }
18. 注释
- 18.1 【必须】 使用 - /* ... */来进行多行注释。- // bad // make() returns a new element // based on the passed in tag name // // @param {String} tag // @return {Element} element function make(tag) { // ... return element; } // good /* * make() returns a new element * based on the passed-in tag name */ function make(tag) { // ... return element; }
- 18.2 【推荐】 使用 - //进行单行注释。 将单行注释放在需要注释的行的上方新行。 建议在注释之前放一个空行,除非它在块的第一行。但如果一段代码每行都包含注释,允许不加分行。- // bad const active = true; // is current tab // good // is current tab const active = true; // bad function getType() { console.log('fetching type...'); // set the default type to 'no type' const type = this.type || 'no type'; return type; } // good function getType() { console.log('fetching type...'); // set the default type to 'no type' const type = this.type || 'no type'; return type; } // also good function getType() { // set the default type to 'no type' const type = this.type || 'no type'; return type; }
- 18.3 【必须】 用一个空格开始所有的注释,使它更容易阅读。 eslint: - spaced-comment- // bad //is current tab const active = true; // good // is current tab const active = true; // bad /** *make() returns a new element *based on the passed-in tag name */ function make(tag) { // ... return element; } // good /** * make() returns a new element * based on the passed-in tag name */ function make(tag) { // ... return element; }
- 18.4 【推荐】 使用 - FIXME或者- TODO开始你的注释可以帮助其他开发人员快速了解相应代码,如果你提出了一个需要重新讨论的问题,或者你对需要解决的问题提出的解决方案。 这些不同于其他普通注释,因为它们是可操作的。 这些操作是- FIXME: -- 需要解决这个问题或者- TODO: -- 需要被实现。
- 18.5 【推荐】 使用 - // FIXME:注释问题。- class Calculator extends Abacus { constructor() { super(); // FIXME: 这里不应该使用全局变量 total = 0; } }
- 18.6 【推荐】 使用 - // TODO:注释解决问题的方法。- class Calculator extends Abacus { constructor() { super(); // TODO: total 应该由一个 param 的选项配置 this.total = 0; } }
- 18.7 【必须】 - /** ... */风格(首行有两个 *)的块级多行注释仅能被用于 JSDoc。- class Calculator { /** * 用于将多个数字相加求和 * * @param {number[]} nums 一系列用于相加的数字 * @returns {number} 最终的和 */ add(...nums) { return nums.reduce((res, num) => res + num, 0); } }
19. 空白
- 19.1 【推荐】 使用 tabs (空格字符) 设置为2个空格。 eslint: - indent- // bad function foo() { ∙∙∙∙let name; } // bad function bar() { ∙let name; } // good function baz() { ∙∙let name; }
- 19.2 【必须】 在花括号前放置一个空格。 eslint: - space-before-blocks- // bad function test(){ console.log('test'); } // good function test() { console.log('test'); } // bad dog.set('attr',{ age: '1 year', breed: 'Bernese Mountain Dog', }); // good dog.set('attr', { age: '1 year', breed: 'Bernese Mountain Dog', });
- 19.3 【必须】 在控制语句中的左括号前放置1个空格(if,while等)。在函数调用和声明中,参数列表和函数名之间不能留空格。 - keyword-spacing- // bad if(isJedi) { fight (); } // good if (isJedi) { fight(); } // bad function fight () { console.log ('Swooosh!'); } // good function fight() { console.log('Swooosh!'); }
- 19.4 【必须】 运算符左右设置各设置一个空格 eslint: - space-infix-ops- // bad const x=y+5; // good const x = y + 5;
- 19.5 【必须】 在文件的结尾需要保留一个空行 eslint: - eol-last- // bad import { es6 } from './AirbnbStyleGuide'; // ... export default es6;- // bad import { es6 } from './AirbnbStyleGuide'; // ... export default es6;↵ ↵- // good import { es6 } from './AirbnbStyleGuide'; // ... export default es6;↵
- 19.6 【推荐】 在编写多个方法链式调用(超过两个方法链式调用)时。 使用前导点,强调这行是一个方法调用,而不是一个语句。 eslint: - newline-per-chained-call- no-whitespace-before-property- // bad $('#items').find('.selected').highlight().end().find('.open').updateCount(); // bad $('#items'). find('.selected'). highlight(). end(). find('.open'). updateCount(); // good $('#items') .find('.selected') .highlight() .end() .find('.open') .updateCount(); // bad const leds = stage.selectAll('.led').data(data).enter().append('svg:svg').classed('led', true) .attr('width', (radius + margin) * 2).append('svg:g') .attr('transform', `translate(${radius + margin},${radius + margin})`) .call(tron.led); // good const leds = stage.selectAll('.led') .data(data) .enter().append('svg:svg') .classed('led', true) .attr('width', (radius + margin) * 2) .append('svg:g') .attr('transform', `translate(${radius + margin},${radius + margin})`) .call(tron.led); // good const leds = stage.selectAll('.led').data(data);
- 19.7 【推荐】 在块和下一个语句之前留下一空白行。 - // bad if (foo) { return bar; } return baz; // good if (foo) { return bar; } return baz; // bad const obj = { foo() { }, bar() { }, }; return obj; // good const obj = { foo() { }, bar() { }, }; return obj; // bad const arr = [ function foo() { }, function bar() { }, ]; return arr; // good const arr = [ function foo() { }, function bar() { }, ]; return arr;
- 19.8 【必须】 不要在块的开头使用空白行。 eslint: - padded-blocks- // bad function bar() { console.log(foo); } // bad if (baz) { console.log(qux); } else { console.log(foo); } // bad class Foo { constructor(bar) { this.bar = bar; } } // good function bar() { console.log(foo); } // good if (baz) { console.log(qux); } else { console.log(foo); }
- 19.9 【必须】 不要使用多个空行填充代码。 eslint: - no-multiple-empty-lines- // bad class Person { constructor(fullName, email, birthday) { this.fullName = fullName;
  this.email = email;
  this.setAge(birthday);
}
setAge(birthday) {
  const today = new Date();
  const age = this.getAge(today, birthday);
  this.age = age;
}
getAge(today, birthday) {
  // ..
}}
// good class Person { constructor(fullName, email, birthday) { this.fullName = fullName; this.email = email; this.setAge(birthday); }
setAge(birthday) {
  const today = new Date();
  const age = getAge(today, birthday);
  this.age = age;
}
getAge(today, birthday) {
  // ..
}}
- [19.10](#whitespace--in-parens) **【必须】** 不要在括号内添加空格。 eslint: [`space-in-parens`](https://eslint.org/docs/rules/space-in-parens.html)
```javascript
// bad
function bar( foo ) {
  return foo;
}
// good
function bar(foo) {
  return foo;
}
// bad
if ( foo ) {
  console.log(foo);
}
// good
if (foo) {
  console.log(foo);
}- 19.11 【必须】 不要在中括号中添加空格。 eslint: - array-bracket-spacing- // bad const foo = [ 1, 2, 3 ]; console.log(foo[ 0 ]); // good const foo = [1, 2, 3]; console.log(foo[0]);
- 19.12 【推荐】 在花括号内添加空格。 eslint: - object-curly-spacing- // bad const foo = {clark: 'kent'}; // good const foo = { clark: 'kent' };
- 19.13 【可选】 避免让你的代码行超过120个字符(包括空格)。 注意:根据上边的规则,长字符串编写可不受该规则约束,不应该被分解。 eslint: - max-len- 原因? 这样能够提升代码可读性和可维护性。 - // bad const foo = jsonData && jsonData.foo && jsonData.foo.bar && jsonData.foo.bar.baz && jsonData.foo.bar.baz.quux && jsonData.foo.bar.baz.quux.xyzzy; // bad $.ajax({ method: 'POST', url: 'https://airbnb.com/', data: { name: 'John' } }).done(() => console.log('Congratulations!')).fail(() => console.log('You have failed this city.')); // good const foo = jsonData && jsonData.foo && jsonData.foo.bar && jsonData.foo.bar.baz && jsonData.foo.bar.baz.quux && jsonData.foo.bar.baz.quux.xyzzy; // better const foo = jsonData ?.foo ?.bar ?.baz ?.quux ?.xyzzy; // good $.ajax({ method: 'POST', url: 'https://airbnb.com/', data: { name: 'John' }, }) .done(() => console.log('Congratulations!')) .fail(() => console.log('You have failed this city.'));
- 19.14 【必须】 要求打开的块标志和同一行上的标志拥有一致的间距。此规则还会在同一行关闭的块标记和前边的标记强制实施一致的间距。 eslint: - block-spacing- // bad function foo() {return true;} if (foo) { bar = 0;} // good function foo() { return true; } if (foo) { bar = 0; }
- 19.15 【必须】 逗号之前避免使用空格,逗号之后需要使用空格。eslint: - comma-spacing- // bad const arr = [1 , 2]; // good const arr = [1, 2];
- 19.16 【推荐】 不要在计算属性括号内插入空格。eslint: - computed-property-spacing- // bad obj[foo ] obj[ 'foo'] var x = {[ b ]: a} obj[foo[ bar ]] // good obj[foo] obj['foo'] var x = { [b]: a } obj[foo[bar]]
- 19.17 【必须】 避免在函数名及其入参括号之间插入空格。 eslint: - func-call-spacing- // bad func (); func (); // good func();
- 19.18 【必须】 在对象的属性和值之间的冒号前不加空格,冒号后加空格。 eslint: - key-spacing- // bad var obj = { foo : 42 }; var obj2 = { foo:42 }; // good var obj = { foo: 42 };
- 19.19 【必须】 避免在行尾添加空格。 eslint: - no-trailing-spaces
- 19.20 【必须】 在代码开始处不允许存在空行,行间避免出现多个空行,而结尾处必须保留一个空行。 eslint: - no-multiple-empty-lines- // bad const x = 1;
const y = 2;
// good const x = 1;
const y = 2;
- [19.21](#linebreak-style) **【推荐】** 推荐使用 Unix 的 LF 作为换行符,而不是 Windows 的 CRLF,这样可以统一文件的换行符,避免因为换行符导致的格式混乱。
## 20. 逗号
- [20.1](#commas--leading-trailing) **【必须】** 逗号**不能**前置 eslint: [`comma-style`](https://eslint.org/docs/rules/comma-style.html)
```javascript
// bad
const story = [
    once
  , upon
  , aTime
];
// good
const story = [
  once,
  upon,
  aTime,
];
// bad
const hero = {
    firstName: 'Ada'
  , lastName: 'Lovelace'
  , birthYear: 1815
  , superPower: 'computers'
};
// good
const hero = {
  firstName: 'Ada',
  lastName: 'Lovelace',
  birthYear: 1815,
  superPower: 'computers',
};- 20.2 【推荐】 添加尾随逗号: 可以 eslint: - comma-dangle- 原因? 在 git diff 时能够更加清晰地查看改动。 另外,像Babel这样的编译器,会在转译时删除代码中的尾逗号,这意味着你不必担心旧版浏览器中的尾随逗号问题 。 - // bad - 没有尾随逗号的 git 差异 const hero = { firstName: 'Florence', - lastName: 'Nightingale' + lastName: 'Nightingale', + inventorOf: ['coxcomb chart', 'modern nursing'] }; // good - 有尾随逗号的 git 差异 const hero = { firstName: 'Florence', lastName: 'Nightingale', + inventorOf: ['coxcomb chart', 'modern nursing'], };- // bad const hero = { firstName: 'Dana', lastName: 'Scully' }; const heroes = [ 'Batman', 'Superman' ]; // good const hero = { firstName: 'Dana', lastName: 'Scully', }; const heroes = [ 'Batman', 'Superman', ]; // bad function createHero( firstName, lastName, inventorOf ) { // does nothing } // good function createHero( firstName, lastName, inventorOf, ) { // does nothing } // good (注意逗号不能出现在 "rest" 元素后边) function createHero( firstName, lastName, inventorOf, ...heroArgs ) { // does nothing } // bad createHero( firstName, lastName, inventorOf ); // good createHero( firstName, lastName, inventorOf, ); // good (注意逗号不能出现在 "rest" 元素后边) createHero( firstName, lastName, inventorOf, ...heroArgs );
21. jQuery
- 21.1 【推荐】 对于 jQuery 对象一律使用 - $符作为前缀。- // bad const sidebar = $('.sidebar'); // good const $sidebar = $('.sidebar'); // good const $sidebarBtn = $('.sidebar-btn');
- 21.2 【推荐】 缓存 jQuery 查询,节省 DOM 查询开销。 - // bad function setSidebar() { $('.sidebar').hide(); // ... $('.sidebar').css({ 'background-color': 'pink', }); } // good function setSidebar() { const $sidebar = $('.sidebar'); $sidebar.hide(); // ... $sidebar.css({ 'background-color': 'pink', }); }
- 21.3 【推荐】 能通过一次调用查找到的,不要分多次;在已有对象内查询,使用 - find函数,减少重复查询。- // bad $('ul', '.sidebar').hide(); // bad $('.sidebar').find('ul').hide(); // good $('.sidebar ul').hide(); // good $('.sidebar > ul')