Security News
Research
Data Theft Repackaged: A Case Study in Malicious Wrapper Packages on npm
The Socket Research Team breaks down a malicious wrapper package that uses obfuscation to harvest credentials and exfiltrate sensitive data.
eslint-config-imweb-t
Advanced tools
[TOC]
为前端开发提供良好的基础编码风格修行指南。
基于 Airbnb 编码规范。所有规范分为三个等级:必须、推荐、可选。
必须(Mandatory) 级别要求员工必须严格按照规范的编码格式编写,否则会在代码扫描和自动化构建中报错。
推荐(Preferable) 级别希望员工尽量按照规范编写,但如有特殊情况,可以不采用。
可选(Optional) 级别并不对员工编码提出要求,一般是JavaScript常识以及ES6、ES7的新语法,但仍希望员工按参考规范编写。委员会将定期review编码规范,不断提高规范等级要求。
安装:
javascript
tnpm install eslint @tencent/eslint-config-tencent --save-dev
.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'],
}
发布方法详见: #Publish
增量检查
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
全量检查
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 【可选】 基本类型: 当你访问一个基本类型时,直接操作它的值。
string
number
boolean
null
undefined
symbol
const foo = 1;
let bar = foo;
bar = 9;
console.log(foo, bar); // => 1, 9
1.2 【可选】 复杂类型: 当你访问一个复杂类型时,直接操作其值的引用。
object
array
function
const foo = [1, 2];
const bar = foo;
bar[0] = 9;
console.log(foo[0], bar[0]); // => 9, 9
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.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)来获得一个新对象,该对象省略了某些属性。
// 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.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 【推荐】 如果数组有多行,则在数组开始括号 [
的时候换行,然后在数组结束括号 ]
的时候换行。
// 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.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.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.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
对象。
// 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 【推荐】 总是把默认参数放在最后。
// 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.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.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.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 【必须】 将所有的 import
s 语句放在其他语句之前。
eslint: import/first
原因? 将所有的
import
s 提到顶部,可以防止某些诡异行为的发生。
// 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.1 【推荐】 不要使用迭代器。 推荐使用 JavaScript 的高阶函数代替 for-in
或者 for-of
。 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.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.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;
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');
}
}
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
的抽象方法来计算表达式的结果,并始终遵循以下简单的规则:
''
值为 false 否则为 trueif ([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.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.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.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;
}
}
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 【必须】 不要在括号内添加空格。 eslint: space-in-parens
// 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;
// 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 【推荐】 推荐使用 Unix 的 LF 作为换行符,而不是 Windows 的 CRLF,这样可以统一文件的换行符,避免因为换行符导致的格式混乱。
20.1 【必须】 逗号不能前置 eslint: comma-style
// 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.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').hide();
// good
$sidebar.find('ul').hide();
22.1 【推荐】 使用 String()
函数将变量转成字符串,比较保险: eslint: no-new-wrappers
// => this.reviewScore = 9;
// bad
const totalScore = new String(this.reviewScore); // typeof totalScore is "object" not "string"
// bad
const totalScore = this.reviewScore + ''; // invokes this.reviewScore.valueOf()
// bad
const totalScore = this.reviewScore.toString(); // isn’t guaranteed to return a string
// good
const totalScore = String(this.reviewScore);
22.2 【推荐】 数字类型转换推荐用 Number()
或者 parseInt()
函数,其中 parseInt()
需显式标明底数。 eslint: radix
no-new-wrappers
const inputValue = '4';
// bad
const val = new Number(inputValue);
// bad
const val = +inputValue;
// bad
const val = inputValue >> 0;
// bad
const val = parseInt(inputValue);
// good
const val = Number(inputValue);
// good
const val = parseInt(inputValue, 10);
22.3 【可选】 如果你对性能有极高的要求,觉得 parseInt
性能太低 ,可以使用位运算,但请用注释说明代码含义,否则很难看懂。
// good
/**
* parseInt 使我的代码变慢。
* 位运算将一个字符串转换成数字更快。
*/
const val = inputValue >> 0;
22.4 注意: 【可选】 谨慎使用位运算符。 在js里,数字的最大值是64位 ,但位运算只能返回32位的整数 (来源)。 对于大于 32 位的数值,位运算无法得到预期结果。讨论。 最大的 32 位整数是: 2,147,483,647。
2147483647 >> 0; // => 2147483647
2147483648 >> 0; // => -2147483648
2147483649 >> 0; // => -2147483647
22.5 【推荐】 用两个叹号来转换布尔类型: eslint: no-new-wrappers
const age = 0;
// bad
const hasAge = new Boolean(age);
// good
const hasAge = Boolean(age);
// best
const hasAge = !!age;
23.1 【可选】 避免用单个字母来命名函数或变量。 尽量让名字具有可读性。
补充说明:由于eslint的
id-length
规则会把for(let i=0; i < len; i++)这种情况也报错,这不太符合我们的习惯,因此在配置中暂时去掉id-length检查。
// bad
function q() {
// ...
}
// good
function query() {
// ...
}
23.2 【必须】 使用驼峰命名法(camelCase)命名对象、函数和实例。 eslint: camelcase
// bad
const OBJEcttsssss = {};
const this_is_my_object = {};
function c() {}
// good
const thisIsMyObject = {};
function thisIsMyFunction() {}
23.3 【必须】 只有在命名构造器或者类的时候,才用帕斯卡命名法(PascalCase),即首字母大写。 eslint: new-cap
// bad
function user(options) {
this.name = options.name;
}
const bad = new user({
name: 'nope',
});
// good
class User {
constructor(options) {
this.name = options.name;
}
}
const good = new User({
name: 'yup',
});
23.4 【推荐】 变量命名时不要使用前置或者后置下划线。 eslint: no-underscore-dangle
因为在 Javascript 里属性和方法没有私有成员一说。 虽然前置下划线通常表示这是私有成员,但实际上还是公开的。 你无法阻止外部调用这类方法。 并且,开发人员容易误以为修改这类函数不需要知会调用方或者不需要测试。 简而言之,如果你想定义私有成员,必须使其不可见。
// bad
this.__firstName__ = 'Panda';
this.firstName_ = 'Panda';
this._firstName = 'Panda';
// good
this.firstName = 'Panda';
// 好,在 WeakMap 可用的环境中。 参考 https://kangax.github.io/compat-table/es6/#test-WeakMap
const firstNames = new WeakMap();
firstNames.set(this, 'Panda');
23.5 【推荐】 不要保存 this
的引用,请使用箭头函数或者 函数#bind。
// bad
function foo() {
const self = this;
return function () {
console.log(self);
};
}
// bad
function foo() {
const that = this;
return function () {
console.log(that);
};
}
// good
function foo() {
return () => {
console.log(this);
};
}
23.6 【可选】 文件名应该和默认导出的名称保持一致(文件名建议使用 kebab-case,后缀为小写)。
// file 1 contents
class CheckBox {
// ...
}
export default CheckBox;
// file 2 contents
export default function fortyTwo() { return 42; }
// file 3 contents
export default function insideDirectory() {}
// in some other file
// bad
import CheckBox from './checkBox'; // PascalCase import/export, camelCase filename
import FortyTwo from './FortyTwo'; // PascalCase import/filename, camelCase export
import InsideDirectory from './InsideDirectory'; // PascalCase import/filename, camelCase export
// bad
import CheckBox from './check_box'; // PascalCase import/export, snake_case filename
import forty_two from './forty_two'; // snake_case import/filename, camelCase export
import inside_directory from './inside_directory'; // snake_case import, camelCase export
import index from './inside_directory/index'; // requiring the index file explicitly
import insideDirectory from './insideDirectory/index'; // requiring the index file explicitly
// good
import CheckBox from './check-box'; // kebab-case export/import/filename
import fortyTwo from './forty-two'; // kebab-case export/import/filename
import insideDirectory from './inside-directory'; // kebab-case export/import/directory name/implicit "index"
// ^ supports both inside-directory.js and inside-directory/index.js
23.7 【必须】 导出默认函数时使用驼峰命名法,并且文件名应该和方法名相同。文件名建议使用 kebab-case,后缀为小写。
function makeStyleGuide() {
// ...
}
export default makeStyleGuide;
23.8 【必须】 当导出构造器 / 类 / 单例 / 函数库 / 对象时应该使用帕斯卡命名法(首字母大写)。
const TencentStyleGuide = {
es6: {
},
};
export default TencentStyleGuide;
23.9 【推荐】 缩略词和缩写都必须是全部大写或者全部小写,可读性更好。
// bad
import SmsContainer from './containers/SmsContainer';
// bad
const HttpRequests = [
// ...
];
// good
import SMSContainer from './containers/SMSContainer';
// good
const HTTPRequests = [
// ...
];
// also good
const httpRequests = [
// ...
];
// best
import TextMessageContainer from './containers/TextMessageContainer';
// best
const requests = [
// ...
];
23.10 【可选】 对于export的常量,可以用全大写命名,但模块内部的常量名不需要全大写(用驼峰试命名可读性更好)。
UPPERCASE_VARIABLES 全大写变量可以让开发者知道这是个常量。 但注意在常量对象内的属性名不需要全大写(如
EXPORTED_OBJECT.key
)。
// bad
const PRIVATE_VARIABLE = 'should not be unnecessarily uppercased within a file';
// bad
export const THING_TO_BE_CHANGED = 'should obviously not be uppercased';
// bad
export let REASSIGNABLE_VARIABLE = 'do not use let with uppercase variables';
// ---
// 允许,但是不提供语义值
export const apiKey = 'SOMEKEY';
// 多数情况下,很好
export const API_KEY = 'SOMEKEY';
// ---
// bad - 不必要大写 key 没有增加语义值
export const MAPPING = {
KEY: 'value'
};
// good
export const MAPPING = {
key: 'value'
};
[24.1] 【可选】 如果没有特殊需要,类属性存取器其实是没有必要的。
24.2 【可选】 不要使用 JavaScript 的 getters/setters 方法,因为它们会导致意外的副作用,并且更加难以测试、维护和推敲。 相应的,如果你需要处理存取过程的时候可以使用函数 getVal()
和 setVal('hello')
实现。
// bad
class Dragon {
get age() {
// ...
}
set age(value) {
// ...
}
}
// good
class Dragon {
getAge() {
// ...
}
setAge(value) {
// ...
}
}
24.3 【推荐】 如果属性/方法是一个 boolean
值,使用 isVal()
或者 hasVal()
。
// bad
if (!dragon.age()) {
return false;
}
// good
if (!dragon.hasAge()) {
return false;
}
24.4 【可选】 可以创建 get()
和 set()
方法,但是要保证一致性。
class Jedi {
constructor(options = {}) {
const lightsaber = options.lightsaber || 'blue';
this.set('lightsaber', lightsaber);
}
set(key, val) {
this[key] = val;
}
get(key) {
return this[key];
}
}
25.1 【推荐】 无论原生 DOM 事件还是像 Backbone 事件这样的自定义事件,在给事件传递参数时,参数应该使用对象而不是单个值。这样未来如果参数有所变化,就不用修改事件回调函数本身。
// bad
$(this).trigger('listingUpdated', listing.id);
// ...
$(this).on('listingUpdated', (e, listingID) => {
// do something with listingID
});
// good
$(this).trigger('listingUpdated', { listingID: listing.id });
// ...
$(this).on('listingUpdated', (e, data) => {
// do something with data.listingID
});
原因? 当 JavaScript 解析器解析到没有分号的单行代码时,它会使用一个叫做 自动分号插入算法(Automatic Semicolon Insertion) 来确定是否应该以换行符视为语句的结束,如果判断为语句结束,会在代码中断前插入一个分号到代码中。 但是,ASI 包含了一些奇怪的行为,如果 JavaScript 错误的解释了你的换行符,你的代码将会中断。 随着越来越多的新特性成为 JavaScript 的一部分,这些规则将变得更加复杂。明确地终止你的语句,并配置你的 linter 以捕获缺少分号的代码行,将有助于预防此类问题。
// bad - 可能异常
const jedis = {}
['luke', 'leia'].some((name) => {
jedis[name] = true
return true
})
// bad - 可能异常
const reaction = "No! That's impossible!"
(async function meanwhileOnTheFalcon() {
// handle `leia`, `lando`, `chewie`, `r2`, `c3p0`
// ...
}())
// bad - 返回 `undefined` 而不是下一行的值 - 当 `return` 单独一行的时候 ASI 总是会发生
function foo() {
return
'search your feelings, you know it to be foo'
}
// good
const jedis = {};
['luke', 'leia'].some((name) => {
jedis[name] = true;
return true;
});
// good
const reaction = "No! That's impossible!";
(async function meanwhileOnTheFalcon() {
// handle `leia`, `lando`, `chewie`, `r2`, `c3p0`
// ...
}());
// good
function foo() {
return 'search your feelings, you know it to be foo';
}
1.1 【必须】 类的只读属性若是一个字面量,则必须使用只读属性而不是 getter
//bad
class Foo1 {
public get bar() {
return 1;
}
}
//good
class Foo2 {
public readonly bar = 1;
}
1.2 【可选】 必须设置类的成员的可访问性
原因? 1、将不需要公开的成员设为私有的,可以增强代码的可理解性,对文档输出也很友好 2、将暴露出去的设置为public的,类和外界的联系一目了然
//bad
class Foo2 {
static foo = 'foo';
static getFoo() {
return Foo2.foo;
}
constructor() {}
bar = 'bar';
getBar() {}
get baz() {
return 'baz';
}
set baz(value) {
console.log(value);
}
}
//good
class Foo2 {
private static foo = 'foo';
public static getFoo() {
return Foo2.foo;
}
public constructor() {}
protected bar = 'bar';
public getBar() {}
public get baz() {
return 'baz';
}
public set baz(value) {
console.log(value);
}
}
1.3 【必须】 指定类成员的排序规则
优先级:
//bad
class Foo1 {
private getBar3() {
return this.bar3;
}
protected getBar2() {}
public getBar1() {}
public constructor() {
console.log(Foo1.getFoo3());
console.log(this.getBar3());
}
private bar3 = 'bar3';
protected bar2 = 'bar2';
public bar1 = 'bar1';
private static getFoo3() {
return Foo1.foo3;
}
protected static getFoo2() {}
public static getFoo1() {}
private static foo3 = 'foo3';
protected static foo2 = 'foo2';
public static foo1 = 'foo1';
}
//good
class Foo2 {
public static foo1 = 'foo1';
protected static foo2 = 'foo2';
private static foo3 = 'foo3';
public static getFoo1() {}
protected static getFoo2() {}
private static getFoo3() {
return Foo2.foo3;
}
public bar1 = 'bar1';
protected bar2 = 'bar2';
private bar3 = 'bar3';
public constructor() {
console.log(Foo2.getFoo3());
console.log(this.getBar3());
}
public getBar1() {}
protected getBar2() {}
private getBar3() {
return this.bar3;
}
}
1.4 【可选】 禁止给类的构造函数的参数添加修饰符
原因?强制所有属性都定义到类里面,比较统一
//bad
class Foo1 {
constructor(private name: string) {}
}
//good
class Foo2 {
constructor(name: string) {}
}
1.5 【推荐】 禁止出现没必要的 constructor
//bad
class Foo1 {
constructor() {}
}
class Bar1 extends Foo1 {
constructor() {
super();
}
}
//good
class Foo2 {
constructor() {
this.doSomething();
}
doSomething() {}
}
class Bar2 extends Foo1 {
constructor() {
super();
this.doSomething();
}
doSomething() {}
}
2.1 【必须】 重载的函数必须写在一起,增强可读性。
// bad
declare namespace NSFoo1 {
export function foo(s: string): void;
export function foo(n: number): void;
export function bar(): void;
export function foo(sn: string | number): void;
}
type TypeFoo1 = {
foo(s: string): void;
foo(n: number): void;
bar(): void;
foo(sn: string | number): void;
};
interface IFoo1 {
foo(s: string): void;
foo(n: number): void;
bar(): void;
foo(sn: string | number): void;
}
// good
declare namespace NSFoo2 {
export function foo(s: string): void;
export function foo(n: number): void;
export function foo(sn: string | number): void;
export function bar(): void;
}
type TypeFoo2 = {
foo(s: string): void;
foo(n: number): void;
foo(sn: string | number): void;
bar(): void;
};
interface IFoo2 {
foo(s: string): void;
foo(n: number): void;
foo(sn: string | number): void;
bar(): void;
}
2.2 【必须】 函数重载时,若能通过联合类型将两个函数的类型声明合为一个,则使用联合类型而不是两个函数声明
//bad
function foo1(x: number): void;
function foo1(x: string): void;
function foo1(x: any): any {
return x;
}
//good
function foo2(x: number | string): void;
function foo2(x: any): any {
return x;
}
3.1 【必须】 类型断言必须使用 as Type,禁止使用 <Type>,禁止对对象字面量进行类型断言(断言成 any 是允许的)
原因?避免Type被理解为 jsx
//bad
let bar1: string | number;
const foo1 = <string>bar1;
//good
let bar2: string | number;
const foo2 = bar2 as string;
//bad
const baz1 = {
bar: 1
} as object;
//good
bar: 1
} as object;
let bar2: string | number;
const foo2 = bar2 as string;
const baz2 = {
bar: 1
} as any;
3.2 【推荐】 禁止给一个初始化时直接赋值为 number, string 的变量显式的声明类型, 可以简化代码
//bad
let foo1: number = 1;
let bar1: string = '';
//good
const foo2 = 1;
const bar2 = '';
3.3 【必须】 interface 和 type 定义时必须声明成员的类型
//bad
type Foo1 = {
bar;
baz;
};
//good
type Foo2 = {
bar: boolean;
baz: string;
};
3.4 【推荐】 使用函数类型别名替代包含函数调用声明的接口
//bad
interface Foo1 {
(): string;
}
//good
type Foo2 = () => string;
4.1 【可选】 优先使用 interface 而不是 type
需要被implement, extend 或者merge的建议定义为interface
type Foo1 = {
foo: string;
};
//或者
interface Foo2 {
foo: string;
}
4.2 【可选】 接口中的方法必须用属性的方式定义
原因?配置了 strictFunctionTypes 之后,用属性的方式定义方法可以获得更严格的检查
//bad
interface Foo1 {
bar(): number;
}
//good
interface Foo1 {
bar: () => number;
}
4.3 【必须】 禁止定义空的接口
//bad
interface Foo1 {}
//good
interface Foo2 {
foo: string;
}
5.1 【必须】 禁止使用 namespace 来定义命名空间
原因?使用 es6 引入模块,才是更标准的方式。 但是允许使用 declare namespace ... {} 来定义外部命名空间
//bad
namespace foo1 {}
//good
declare namespace foo1 {}
5.2 【必须】 禁止使用 module 来定义命名空间
原因?module 已成为 js 的关键字
//bad
module Foo1 {}
//good
namespace Foo2 {}
6.1 【必须】 禁止在 optional chaining 之后使用 non-null 断言(感叹号)
原因?optional chaining 后面的属性一定是非空的
//bad
let foo1: { bar: string } | undefined;
console.log(foo1?.bar!);
//good
let foo2: { bar: string } | undefined;
console.log(foo2?.bar);
6.2 【必须】 禁止将 this 赋值给其他变量,除非是解构赋值
//bad
function foo() {
const self = this;
setTimeout(function () {
self.doWork();
});
}
//good
function foo() {
const { bar } = this;
setTimeout(() => {
this.doWork();
});
}
6.3 【必须】 禁止无用的表达式
//bad
declare const foo1: any;
declare const bar1: any;
declare const baz1: any;
1;
foo1;
('foo1');
foo1 && bar1;
foo1 || bar1;
foo1 ? bar1 : baz1;
`bar1`;
//good
'use strict';
declare const foo2: any;
declare const bar2: any;
declare const baz2: any;
foo2 && bar2();
foo2 || bar2();
foo2 ? bar2() : baz2();
foo2`bar2`;
6.4 【必须】 使用 optional chaining 替代 &&
原因?简化代码
//bad
let foo1: any;
console.log(foo1 && foo1.a && foo1.a.b && foo1.a.b.c);
//good
let foo2: any;
console.log(foo2?.a?.b?.c);
7.1 【必须】 禁止使用三斜杠导入文件
//bad
/// <reference path="./Animal">
//good
import Animal from './Animal';
7.2 【必须】 禁止使用 require
原因?统一使用 import 来引入模块,特殊情况使用单行注释允许 require 引入
//bad
const fs = require('fs');
//good
import * as fs from 'fs';
8.1 【推荐】 使用 for 循环遍历数组时,如果索引仅用于获取成员,则必须使用 for of 循环替代 for 循环
//bad
const arr1 = [1, 2, 3];
for (let i = 0; i < arr1.length; i++) {
console.log(arr1[i]);
}
//good
const arr2 = [1, 2, 3];
for (const x of arr2) {
console.log(x);
}
for (let i = 0; i < arr2.length; i++) {
// i is used to write to arr, so for-of could not be used.
arr2[i] = 0;
}
for (let i = 0; i < arr2.length; i++) {
// i is used independent of arr, so for-of could not be used.
console.log(i, arr2[i]);
}
8.2 【必须】 在类型注释周围需要一致的间距
//bad
let foo:string = "bar";
let foo :string = "bar";
let foo : string = "bar";
function foo():string {}
function foo() :string {}
function foo() : string {}
class Foo {
name:string;
}
class Foo {
name :string;
}
class Foo {
name : string;
}
type Foo = ()=> {};
//good
let foo: string = "bar";
function foo(): string {}
class Foo {
name: string;
}
type Foo = () => {};
FAQs
ESLint Config for Tencent
We found that eslint-config-imweb-t demonstrated a not healthy version release cadence and project activity because the last version was released a year ago. It has 1 open source maintainer collaborating on the project.
Did you know?
Socket for GitHub automatically highlights issues in each pull request and monitors the health of all your open source dependencies. Discover the contents of your packages and block harmful activity before you install or update your dependencies.
Security News
Research
The Socket Research Team breaks down a malicious wrapper package that uses obfuscation to harvest credentials and exfiltrate sensitive data.
Research
Security News
Attackers used a malicious npm package typosquatting a popular ESLint plugin to steal sensitive data, execute commands, and exploit developer systems.
Security News
The Ultralytics' PyPI Package was compromised four times in one weekend through GitHub Actions cache poisoning and failure to rotate previously compromised API tokens.