リファクタリング

[napo0703]

自分ですら読むのがつらくなってきた。
コードレビューや開発環境の見直しなどアドバイスあったらお願いいたします:bow:

[shokai]

[shokai]

[napo0703]

[shokai]

「nameを与えたらtupleをwatchして、nameに基づいてgrid内から自分が変更すべきDOMを探して表示を変えるモジュール」を書くのはどうか
jqeuryでいいので

そのために「nameに基いてgrid内から自分が変更すべきDOMを探して表示を変えるモジュール」とそれを継承して「tupleを見て自分が表示すべき内容を決める具体的な実装」に分ける

gridparts/wakarite.js

export default class Wakarite {
  constructor (name) {
    this.name = name
    $('.grid').append($('<div>').addClass(this.name))
  }

  renderImage (img_url) {
    $(`.grid .${this.name} img`).attr('src', img_url)
  }
}

gridparts/delta-door.js

import Wakarite from './wakarite'
export default class DeltaDoor extends Wakarite {
  constructor () {
    super('delta_door')
  }

  start () {
    tuplespace.watch({type: door, cmd: 'open'}, (err, tuple) => {
      this.renderImage('http://ドア開いた.png')
      setTimeout(() => {
        this.renderImage('http://ドア閉じた.png')
      }, 3000)
    })
  }
}

index.js からはこう呼ぶ

import DeltaDoor from './gridparts/delta-door'
new DeltaDoor().start()

雑なスケッチだけどこんな感じでどう表示するかと、表示内容を決める役に分担して、delta-door.jsみたいなのをたくさん作ればいいのでは

[masui]

• "wakarite"ってどういう意味ですか?
• tuplespaceってグローバル変数?
• メソッドて"function"みたいなの要らないんですか

[shokai]

"wakarite"ってどういう意味ですか?

わかりを表示する人です

tuplespaceってグローバル変数?

いいえ
どこかで宣言する必要があります

メソッドて"function"みたいなの要らないんですか

class記法なのでいりません

[masui]

wakarite = 分かり手?

[shokai]

はい