实例：消除魔术字符串

魔术字符串指的是，在代码之中多次出现、与代码形成强耦合的某一个具体的字符串或者数值。风格良好的代码，应该尽量消除魔术字符串，改由含义清晰的变量代替。

  1. function getArea(shape, options) {
  2.   let area = 0;
  4.   switch (shape) {
  5.     case 'Triangle': // 魔术字符串
  6.       area = .5 * options.width * options.height;
  7.       break;
  8.     /* ... more code ... */
  9.   }
  11.   return area;
  12. }
  14. getArea('Triangle', { width: 100, height: 100 }); // 魔术字符串

上面代码中，字符串Triangle就是一个魔术字符串。它多次出现，与代码形成“强耦合”，不利于将来的修改和维护。

常用的消除魔术字符串的方法，就是把它写成一个变量。

  1. const shapeType = {
  2.   triangle: 'Triangle'
  3. };
  5. function getArea(shape, options) {
  6.   let area = 0;
  7.   switch (shape) {
  8.     case shapeType.triangle:
  9.       area = .5 * options.width * options.height;
  10.       break;
  11.   }
  12.   return area;
  13. }
  15. getArea(shapeType.triangle, { width: 100, height: 100 });

上面代码中，我们把Triangle写成shapeType对象的triangle属性，这样就消除了强耦合。

如果仔细分析，可以发现shapeType.triangle等于哪个值并不重要，只要确保不会跟其他shapeType属性的值冲突即可。因此，这里就很适合改用 Symbol 值。

  1. const shapeType = {
  2.   triangle: Symbol()
  3. };

上面代码中，除了将shapeType.triangle的值设为一个 Symbol，其他地方都不用修改。