js 定义属性奇怪的问题。

[cc7756789]

function Keygrip(keys) {
  if (arguments.length > 1) {
    console.warn('as of v2, keygrip() only accepts a single argument.')
    console.warn('set keygrip().hash= instead.')
    console.warn('keygrip() also now only supports buffers.')
  }

  if (!Array.isArray(keys) || !keys.length) throw new Error("Keys must be provided.")
  if (!(this instanceof Keygrip)) return new Keygrip(keys)

  this.keys = keys
}


 Keygrip.prototype = {
  _hash: 'sha256',
}


Object.defineProperties(Keygrip.prototype, {
  // modified Keygrip.prototype.constructor's enumerable = false
  constructor: {
    value: Keygrip,
    writable: true,
    configurable: true
  },
  
  hash: {
    get: () => {
      console.log(this);  // {} 空对象
      return this._hash;  // undefined
    },
    set: (val) => {
      if (!util.supportedHash(val)) throw new Error('unsupported hash algorithm: ' + val);
      this._hash = val;   
    }
  },
 
});

>>> const KeyGrip = require('keygrip');
>>> var s = KeyGrip(['fuck off my house', 'I never want to see you again']);
>>> console.log(s.hash)
{}
undefined

这是keygrip模块，我看到它原本的 constructor 只是定义了 value ，是可枚举的。手贱 fork 后修改了下，发现 get 和 set 中的 this 都是指向了一个空对象{}，所以这样写的话，无论获取还是设置，都没有成功，想不明白到底是为什么。

[ETiV]

我擦，吓到宝宝了，居然有代码加色

[ggiiss]

目测是 箭头函数的坑，
解决方法是：不用箭头函数

[cc7756789]

@ggiiss 我去我一着急，就一路用箭头装逼下去，现在一换过来，果然，这个坑搞得我测试了很长时间，一路以为是作用域的问题

[ggiiss]

@cc7756789 es6 增加了 let, => 特性后我发现之前的作用域， this 指向 问题又复杂了，泪。

[Wenwei]

你这箭头函数用错了。

```javascript
hash: {
get() {
console.log(this); // {} 空对象
return this._hash; // undefined
},
set(val) {
if (!util.supportedHash(val)) throw new Error('unsupported hash algorithm: ' + val);
this._hash = val;
}
},
```

[haozhang]

arrow function 的 this 是来自外部域的，你调用 xxx.hash = yyy 的时候，是调用了 set 函数，但是 set 箭头函数中的 this 并不指向 xxx ，而是指向 xxx.hash = yyy 这个语句所在域的 this 。我估计这个域是个最外层域，所以 this 就变成了 undefined ， get 同理。这是我的推断不知道对不对。

[cc7756789]

@haozhang 是的，我用 arraw 的时候没有去看 es6 ，以为和普通的匿名函数是一样的，后来才知道 es6 的 arraw 的作用域只会是调用方法的对象，而没有运行时这个坑，之所以只在 arraw 规则这样实现，我认为是为了做到向后兼容吧，普通的函数 this 运行时依旧不改变。

[morethansean]

@cc7756789 “之所以……” function 里的 this 本来就是语言的特性之一，谈不上什么向后兼容。

[acthtml]

keygrip 这个模块看不懂有什么用，

只是做一个 hash 加密的话，为何参数要设置成数组，直接指定一个字符串作为秘钥不就行了。

不明白。