变量

让变量名说明，这个变量存储的内容是什么

• 代码逻辑
  • 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