变量
让变量名说明，这个变量存储的内容是什么
- 代码逻辑
- candidates 应征人员，这里表示参加考试的学生，是一个数组
- answerSheets 答题卡，这里表示这次考试收到的所有答题卡，是一个数组
- 这段代码想做的事情是，将答题卡发给相应的考生
- 问题
- 问题，出现在236行，tempFormatMap这个变量，没有表达出这个变量到底存储的是什么内容
- 改成id2CandidateMap,会让代码更加易懂一些
- 最后，为了让变量更加容易理解一些，建议尽量不采用缩写
不要增加没有用的名词
// DON'T
let nameString
let theUsers
// DO
let name
let users
- 在上面的例子中，提到尽量不要用缩写单词，这会产生一个副作用，就是，变量名过长
- 为了不让变量名长到丧心病狂的地步，在命名变量的时候不要出现任何无效的单词。如：
- 冠词
- 数据类型
- 。。。
函数
函数名称应该是一个动词或者动宾短语，通过函数名称和参数名称以及顺序表达出这个函数的目的，注释是一个比较low的做法
// DON'T
/**
* Invite a new user with its email address
* @param {String} user email address
*/
function inv (user) { /* implementation */ }
// DO
function inviteUser (emailAddress) { /* implementation */ }
- 上面两段代码表达的意思差不多，但是明显后者更优美一些，也更加实用一些
- 方法2，可以在上层逻辑中直接看出函数的作用，而不需要跟进一步，打断原有的思路
不要使用长的参数列表，使用es6的解构来声明函数参数
// DON'T
function getRegisteredUsers (fields, include, fromDate, toDate) { /* implementation */ }
// DO
function getRegisteredUsers ({ fields = [], include = [], fromDate = '2017-12-1', toDate = '2017-12-1' }) {
/* implementation */
}
- ES6的解构语法名用于函数列表的优点有两个
- 未来调整参数的顺序，不会对污染调用方
- 避免在函数开始处，写一大堆恶心的默认值赋值
减少函数的副作用
// DON'T
function addItemToCart (cart, item, quantity = 1) {
const alreadyInCart = cart.get(item.id) || 0
cart.set(item.id, alreadyInCart + quantity)
return cart
}
// DO
// not modifying the original cart
function addItemToCart (cart, item, quantity = 1) {
const cartCopy = new Map(cart)
const alreadyInCart = cartCopy.get(item.id) || 0
cartCopy.set(item.id, alreadyInCart + quantity)
return cartCopy
}
- 上述函数，将参数修改了，调用的小伙伴就会很困惑
- 函数内部，不应该影响对上层调用者的行为做任何假设
让函数在文件中排列的顺序合理一些，需要先读的放在前面
- 在排列函数的时候，应该注意函数的顺序
- 比如：react的组件，起码应该生命周期的顺序排练函数
引用：https://blog.risingstack.com/javascript-clean-coding-best-practices-node-js-at-scale