那些年被吐槽的JavaScript代码风格你写过多少？

现在写代码比以前好多了，代码的格式都有 eslint、prettier、babel(写新版语法) 这些来保证，然而，技术手段再高端都不能解决代码可读性（代码能否被未来的自己和同事看懂）的问题，因为这个问题只有人自己才能解决。我们写代码要写到下图中左边这样基本上就功德圆满了。

（1）变量数量的定义

拒绝：滥用变量

正确姿势：: 数据只使用一次或不使用就无需装到变量中

(2) 变量的命名

拒绝：自我感觉良好的缩写

正确姿势：无需对每个变量都写注释，从名字上就看懂

（3) 特定的变量

拒绝：无说明的参数

正确姿势：添加变量

（4）变量的命名

拒绝：命名过于啰嗦

YES：做到简洁明了

（5）使用说明性的变量 (即有意义的变量名)

正确姿势：长代码不知道啥意思

YES：用变量名来解释长代码的含义

（6）避免使用太多的全局变量

拒绝：在不同的文件不停的定义全局变量

name.js
window.name = 'a';
hello.js
window.name = 'b';
time.js
window.name = 'c'; //

 三个文件的先后加载顺序不同，都会使得 window.name 的值不同，同时，你对 window.name 的修改了都有可能不生效，因为你不知道你修改过之后别人是不是又在别的说明文件中对其的值重置了。所以随着文件的增多，会导致一团乱麻。

正确姿势：少用或使用替代方案

你可以选择只用局部变量。通过 (){}的方法。

如果你确实用很多的全局变量需要共享，你可以使用 vuex,redux 或者你自己参考 flux 模式写一个也行。

(7) 变量的赋值

拒绝：对于求值获取的变量，没有兜底。

这样你就给你的代码成功的埋了一个坑，你有考虑过如果 fullName = ['jackie'] 这样的情况吗？这样程序一跑起来就爆炸。要不你试试。

正确姿势：对于求值变量，做好兜底。

const MIN_NAME_LENGTH = 8;
let lastName = fullName[1] || ''; //

做好兜底，fullName[1] 中取不到的时候，不至于赋值个 undefined, 至少还有个空字符，从根本上讲，lastName 的变量类型还是 String，String 原型链上的特性都能使用，不会报错。不会变成

undefined。
if(lastName.length > MIN_NAME_LENGTH) {
 ....
}

其实在项目中有很多求值变量，对于每个求值变量都需要做好兜底。
let propertyValue = Object.attr || 0; // 因为 Object.attr 有可能为空，所以需要兜底。
但是，赋值变量就不需要兜底了。
let a = 2; // 因为有底了，所以不要兜着。
let myName = 'Tiny'; // 因为有底了，所以不要兜着。

二、函数相关

（1）函数命名

拒绝：从命名无法知道返回值类型

function showFriendsList() {....} // 现在问，你知道这个返回的是一个数组，还是一个对象，还是 true or false。你能答的上来否？

正确姿势：对于返回 true or false 的函数，最好以 should/is/can/has 开头

function shouldShowFriendsList() {...}
function isEmpty() {...}
function canCreateDocuments() {...}
function hasLicense() {...}

(2) 功能函数最好为纯函数

拒绝: 不要让功能函数的输出变化无常

function plusAbc(a, b, c) { // 这个函数的输出将变化无常，因为 api 返回的值一旦改变，同样输入函数的 a，b,c 的值，但函数返回的结果却不一定相同。

var c = fetch('../api');
 return a+b+c;
}

正确姿势：功能函数使用纯函数，输入一致，输出结果永远唯一

function plusAbc(a, b, c) { // 同样输入函数的 a，b,c 的值，但函数返回的结果永远相同。
 return a+b+c;
}

（3）函数传参

拒绝：传参无说明

page.getSVG(api, true, false); // true 和 false 啥意思，一目不了然

正确姿势: 传参有说明

page.getSVG({
 imageApi: api,
 includePageBackground: true, // 一目了然，知道这些 true 和 false 是啥意思
 compress: false,
})

（4）动作函数要以动词开头

拒绝: 无法辨别函数意图

function emlU(user) {
 ....
}

正确姿势：动词开头，函数意图就很明显

function sendEmailToUser(user) {
 ....
}

（5）一个函数完成一个独立的功能，不要一个函数混杂多个功能

这是软件工程中最重要的一条规则，当函数需要做更多的事情时，它们将会更难进行编写、测试、理解和组合。当你能将一个函数抽离出只完成一个动作，他们将能够很容易的进行重构并且你的代码将会更容易阅读。如果你严格遵守本条规则，你将会领先于许多开发者。

拒绝：函数功能混乱，一个函数包含多个功能。最后就像能以一当百的老师傅一样，被乱拳打死（乱拳（功能复杂函数）打死老师傅（老程序员））

function sendEmailToClients(clients) {
 clients.forEach(client => {
 const clientRecord = database.lookup(client)
 if (clientRecord.isActive()) {
 email(client)
 }
 })
}

正确姿势：功能拆解

function sendEmailToActiveClients(clients) { // 各个击破，易于维护和复用
 clients.filter(isActiveClient).forEach(email)
}
function isActiveClient(client) {
 const clientRecord = database.lookup(client)
 return clientRecord.isActive()
}

（6）优先使用命令式编程

拒绝: 使用 for 循环编程

for(i = 1; i <= 10; i++) { // 一看到 for 循环让人顿生不想看的情愫
 a[i] = a[i] +1;
}

正确姿势：使用命令式编程

let b = a.map(item => ++item)

 怎么样，是不是很好理解，就是把 a 的值每项加一赋值给 b 就可以了。现在在 javascript 中几乎所有的 for 循环都可以被 map,filter,find,some,any,forEach 等命令式编成取代。

(7) 函数中过多的采用 if else …

拒绝： if else 过多

if (a === 1) {
 ...
} else if (a ===2) {
 ...
} else if (a === 3) {
 ...
} else {
 ...
}

正确姿势: 可以使用 switch 替代或用数组替代

switch(a) {
 case 1:
 ....
 case 2:
 ....
 case 3:
 ....
 default:
 ....
}
Or
let handler = {
 1: () => {....},
 2: () => {....}.
 3: () => {....},
 default: () => {....}
}
handler[a]() || handler['default']()

尽量使用 ES6

（1）尽量使用箭头函数

拒绝：采用传统函数

function foo() {
 // code
}

正确姿势：使用箭头函数

let foo = () => {
 // code
}

（2）连接字符串

拒绝：采用传统 + 号

var message = 'Hello ' + name + ', it\'s ' + time + ' now'

正确姿势：采用模板字符

var message = `Hello ${name}, it's ${time} now`

(3) 使用解构赋值

拒绝：使用传统赋值

var data = { name: 'dys', age: 1 };
var name = data.name;
var age = data.age;
var fullName = ['jackie', 'willen'];
var firstName = fullName[0];
var lastName = fullName[1];

正确姿势：使用结构赋值

const data = {name:'dys', age:1};
const {name, age} = data; // 怎么样，是不是简单明了
var fullName = ['jackie', 'willen'];
const [firstName, lastName] = fullName;

（4）尽量使用类 class

拒绝: 采用传统的函数原型链实现继承

典型的 ES5 的类 (function) 在继承、构造和方法定义方面可读性较差，当需要继承时，优先选用 class。代码太多，就省略了。

正确姿势：采用 ES6 类实现继承

class Animal {
 constructor(age) {
 this.age = age
 }
 move() {
 /* ... */
 }
}
class Mammal extends Animal {
 constructor(age, furColor) {
 super(age)
 this.furColor = furColor
 }
 liveBirth() {
 /* ... */
 }
}
class Human extends Mammal {
 constructor(age, furColor, languageSpoken) {
 super(age, furColor)
 this.languageSpoken = languageSpoken
 }
 speak() {
 /* ... */
 }
}

注：除了上述这些人为习惯之外，就像前面提到的，对于机械性的，你可以使用 Babel、Eslint、Prettier 这些工具来保证代码的格式一致。

我自己是一名从事了5年web前端开发的老程序员，辞职目前在做线上教育讲师，每晚都会直播，今年年初我花了一个月整理了一份最适合2018年学习的web前端干货，从最基础的HTML+CSS+JS到移动端HTML5都有整理，送给每一位前端小伙伴，这里是前端学习者聚集地，欢迎初学和进阶中的小伙伴。web前端交流扣扣裙：884975954。