Help us understand the problem. What is going on with this article?

クラスメソッドのプロパティはインスタンスで共有されてしまう

インスタンスごとに同じ値が取得できてしまう

他の人のソースを触っていて、以下のような現象に遭遇。
hoge.val()の結果が最後に作成したインスタンスの値になってしまう。

class Test {
  constructor(commonVal) {
    this.commonVal = commonVal;
    this.hoge.val = () => this.commonVal;
  }

  hoge() {
    // do something....
    return;
  }
}

const fuga = new Test('aaaaaaaaaaa');
const fuga2 = new Test('bbbbbbbbbbb');

fuga2.hoge.val() // ← "bbbbbbbbbbb"
fuga.hoge.val()  // ← "bbbbbbbbbbb" !?

原因

classのメソッドはprototypeに宣言されることが原因でした。
hogeはメソッド定義されていますのでprototypeに宣言され、その中にvalというプロパティが作成されてしまいます。
その結果、何個インスタンスを生成しようとも、最後に生成されたインスタンスで戻り値が上書きされてしまうわけです:helmet_with_cross:
上記はchromeのコンソールで確認しましたが、babelでプリコンパイルしても同様の挙動になります。
babelもクラスメソッドはprototypeに宣言するから

そもそも上記のようなコンストラクタの記述が好ましくないと思うのですが、他人のソースなので意図が読めず・・・(grepしたけど、hoge.valはどこにも使用されていなかった)

以上、わかればどうということはないのですが、挙動やソースが意味わからなさすぎて2時間くらいかかりました:santa:

Why do not you register as a user and use Qiita more conveniently?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
Comments
Sign up for free and join this conversation.
If you already have a Qiita account
Why do not you register as a user and use Qiita more conveniently?
You need to log in to use this function. Qiita can be used more conveniently after logging in.
You seem to be reading articles frequently this month. Qiita can be used more conveniently after logging in.
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away