浅谈JavaScript 代码整洁之道(5)

日期：2020-05-11 栏目：程序人生浏览：次

好:

function createTempFile(name) {
 fs.create('./temp/' + name);
}

function createFile(name) {
 fs.create(name);
}

避免副作用

如果一个函数不是获取一个输入的值并返回其它值，它就有可能产生副作用。这些副作用可能是写入文件、修改一些全局变量，或者意外地把你所有钱转给一个陌生人。

现在你确实需要在程序中有副作用。像前面提到的那样，你可能需要写入文件。现在你需要做的事情是搞清楚在哪里集中完成这件事情。不要使用几个函数或类来完成写入某个特定文件的工作。采用一个，就一个服务来完成。

关键点是避免觉的陷阱，比如在没有结构的对象间共享状态，使用可以被任意修改的易变的数据类型，没有集中处理发生的副作用等。如果你能做到，你就能比其他大多数程序员更愉快。

不好:

// 下面的函数使用了全局变量。
// 如果有另一个函数在使用 name，现在可能会因为 name 变成了数组而不能正常运行。
var name = 'Ryan McDermott';

function splitIntoFirstAndLastName() {
 name = name.split(' ');
}

splitIntoFirstAndLastName();

console.log(name); // ['Ryan', 'McDermott'];

好:

function splitIntoFirstAndLastName(name) {
 return name.split(' ');
}

var name = 'Ryan McDermott'
var newName = splitIntoFirstAndLastName(name);

console.log(name); // 'Ryan McDermott';
console.log(newName); // ['Ryan', 'McDermott'];

不要写入全局函数

JavaScript 中全局污染是一件糟糕的事情，因为它可能和另外库发生冲突，然而使用你 API 的用户却不会知道——直到他们在生产中遇到一个异常。来思考一个例子：你想扩展 JavaScript 的原生 Array，使之拥有一个 diff 方法，用来展示两数据之前的区别，这时你会怎么做？你可以给 Array.prototype 添加一个新的函数，但它可能会与其它想做同样事情的库发生冲突。如果那个库实现的 diff 只是比如数组中第一个元素和最后一个元素的异同会发生什么事情呢？这就是为什么最好是使用 ES6 的类语法从全局的 Array 派生一个类来做这件事。

不好:

Array.prototype.diff = function(comparisonArray) {
 var values = [];
 var hash = {};

 for (var i of comparisonArray) {
 hash[i] = true;
 }

 for (var i of this) {
 if (!hash[i]) {
  values.push(i);
 }
 }

 return values;
}

好:

class SuperArray extends Array {
 constructor(...args) {
 super(...args);
 }

 diff(comparisonArray) {
 var values = [];
 var hash = {};

 for (var i of comparisonArray) {
  hash[i] = true;
 }

 for (var i of this) {
  if (!hash[i]) {
  values.push(i);
  }
 }

 return values;
 }
}

转载注明出处：http://www.heiqu.com/417.html

浅谈JavaScript 代码整洁之道(5)

相关推荐