javascript(node.js)のクラス設計:playerにroleを持たせたい

Question

実現したいこと

node.jsの初心者です。
オンラインゲームを作っており、その中で参加者(player)に役割(role)を持たせたいです。
参加者それぞれに好きな役割を選ばせる形です。

これの実現として、次のような形を考えています（実際のコードではなく、例です）

class Player{
  consutructor(name){
    this.name = name
    this.role = null
  }

  takeRole(role){
    //??
  }

  solo(){
    if(this.role.canSolo){
       console.log("俺の歌を聞け！")
    }
  }
}

class Role{
  constructor(type){
    this.type = type
    this.vocal = 300
    this.dance = 300
    this.visual = 300
    this.canSolo = false
  }

  totalAppeal(){
    return this.vocal + this.dance + this.visual
  }
}

class Vocalist extends Role{
  constructor(){
    super("vocalist")
    this.vocal = 700
    this.canSolo = true
  }
}

//Roleは他にも複数存在

roleはほぼ構造体です。

疑問

問題はtakeRoleの部分です。
playerに与えるroleが決まっていれば

//player内
takerole(role){
   this.role = role
}


var uduki = new Player("uduki")
uduki.takeRole(new Vocalist())

などとすれば良いように思うのですが、
udukiから希望するroletypeを受け取ってroleを設定しようとすると、

uduki.takeRole("Vocalist")

のような形でコールせざるを得ないと考えています。
この時takeRole内部でどう処理をすればいいのかがわかりません。

考えたこと

・roleManagerのようなオブジェクト、あるいはクラスを作る
すなわち、

var roleManager = {
  "Vocalist": Vocalist,
  "Dancer": Dancer,
  "Performer": Performer
}

//player内
takeRole(roleName){
  this.role = new roleManager[roleName]()
}


var uduki = new Player("uduki")
uduki.taleRole("Vocalist")

のような形にすればやりたいことは達成できますが、
この形だと、roleの種類だけroleManagerの行数を増やさなくてはならず、あまりスマートに思えません。
(roleは将来的に増やしていくことを考えています。)

あるいは、

class Role{
  constructor(type){
    this.type = type
    this.vocal = 300
    this.dance = 300
    this.visual = 300
    this.canSolo = false

    switch(type){
      case "Vocalist":
        this.canSolo = true
        this.vocal = 700
        break
      case "Dancer":
        //以下延々と処理
    }
  }
}

uduki.takeRole(new Role(roleName))

のような方法も考えましたが、これはもっと筋が悪い気がします。
evalは…さすがに論外ですよね。

質問

①この方針で行く場合、takeRoleの実装はどのような方法がスマート、あるいはスタンダードですか？

②そもそも、現状の設計は理にかなっているものですか？
ほぼメソッドを持たないRoleクラスに違和感を感じていて、その情報をplayerに持たせれば十分なような気もしています。
その場合は、どのような形にすべきですか？

Answer 1

コードを書くと、こちらが開発してしまうことになりそうなので、考え方、構成のヒントだけ回答できればと思います。
①というより、②の部分での話になるかと思います。

• Playerクラスをオーバーライドしたクラスを用意する

各ロールを、Playerクラスをオーバーライドしたクラスにて実装をまずすべきと思います。
もしくは、Playerクラスに、roleメンバを用意し、Roleクラスを別で用意し、各ロールを、Roleクラスをオーバーライドしたクラスとし、Playerの初期化時に、指定されたロールのクラスのインスタンスを、roleメンバに代入（初期化）するとかですね。
これで一元管理しつつ、各ロール別の特性の定義もしやすくなるでしょう。

と、既にそのような実装をしていたようなので、下記の実装の仕方で、大方解決できるかと。

• 上記を踏まえ列挙型の定義をする

多くの静的型付け言語には、列挙型という、便利なデータ型がありますので、
それを使うと、より効率的な開発ができるかと思います。
（詳しくはお調べを）
ただし、JavaScriptは動的型付けなので、列挙型は存在しませんので、自身で実装するか、TypeScriptの使用をお勧めします。
Node.jsでの開発なら、容易に導入できるかと思いますよ。
むしろ、列挙型を使わなくても、TypeScriptの方が、ゲーム開発ならばかなり優位になるかと思います。

• constructor.nameないし、instanceofキーワードを使い、クラスを特定

JavaScriptにおいて、そのインスタンスが、どのクラスで初期化されたかどうかは、上記のプロパティ、キーワードを使えば確かめることができるので、その情報を使い、処理を分けるという方法もあります。

また、下記に、JavaScriptにての、列挙型の実装例を記載いたしますので、
参考にしてください。

const EnumCaseStruct = (() => {
  const key = Symbol();

  return class EnumCaseStruct {
    constructor(val) {
      this[key] = val;
    }
    // Swiftを参考に、rawValueで、値を取得できるようにしている
    get rawValue() {
      return this[key];
    }
  };
})();

class Enum {
  constructor(...cases) {
    if (!cases.every(v => {
      switch (typeof v) {
        case 'object':
          // オブジェクトの場合、各キー値にて、上のクラスのインスタンスとして定義
          Object.keys(v).forEach(ve => {
            Object.defineProperty(this, ve, {value: new EnumCaseStruct(v[ve])});
          });
          return true;
        case 'string':
          // 文字列型なら、そのまま初期化する
          Object.defineProperty(this, v, {value: new EnumCaseStruct(v)});
          return true;
        defalult:
          // オブジェクトか、文字列型以外は受け付けないようにする
          return false;
      }
    })) {
      // オブジェクトか文字列以外は、エラーをスローする
      throw new Error('Type error!');
    }
  }
}

// 使い方
const fruits = new Enum(
  {apple: 25},
  {melon: 125}
);

const apple01 = fruits.apple;
const apple02 = fruits.apple;
console.log(apple01 == apple02); // true

// 可変長引数のため、下記のようにも定義可能
const fruits02 = new Enum(
  {
    apple: 25,
    melon: 125
  },
  'pine'
);
// 各列挙型は各値が一意となるため、比較時は偽となる。
console.log(fruits.apple == fruits02.apple); // false
// Swiftを参考に、rawValueにて、設定した値を返す
console.log(fruits.apple.rawValue); // 25
// 文字列をそのまま入れた場合は、rawValueはその文字列を返す
console.log(fruits.pine.rawValue); // pine

一例なので、もっといい、もっと列挙型っぽい定義はできるかと思います。

大雑把な概念などの話で恐縮ですが、以上になります。
開発のヒントになれば幸いです。

Comments

[udoP_]

素早い回答ありがとうございます。

列挙型については知りませんでした。調べてみるとまさに求めていたものでした。
ですがjsにはないとのことで、現状のプログラムに組み込もうとすると、
結局質問に書いたような辞書的なものを作ることになるのかな、と考えています。

TypeScriptを採用すればいいのかな、とも思うのですが、
型がかっちりしている言語に苦手意識があり(慣れればそちらのほうが楽なのでしょうが）
いきなり採用はハードルが高いので、学ぶことから始めようと思います。

[miyabi_pudding]

確かに、JavaScriptとか動的型付け慣れしてると、静的型付けやるのはハードル高いですよね　汗

Set型（配列のようなものだが、違うデータ型）、Symbol型値（そのスレッド内では完全に一意なデータ）を上手く使うと、列挙型みたいなものを実装できるかと思いますよー

[miyabi_pudding]

JavaScriptにての列挙型っぽいものを回答に追記してみたので、
ご参考いただければ幸いです。

[udoP_]

ご丁寧にありがとうございました。頂いたコードも読んで、理解を深めようと思います。
BAとさせていただきました。