TypeScriptで、特定のクラスからのみアクセス可能なメソッドの作成はできますでしょうか

Question

下記「★質問ポイント★」の場所です。

class Manager {
   mem1: Member; // メンバへのアクセスは可能
   mem2: Member;
   mem3: Member;
   constructor() {
       this.mem1 = new Member("yama");
       this.mem2 = new Member("naga");
       this.mem1 = new Member("koku");
   }
   allSing() {
       mem1.sing();
       mem2.sing();
       mem3.sing();
   }
}

class Member {
    name: string;
    constructor(n: string) {
         this.name = n;
    }
    /** みんな呼べる */
    greet() {
        alert(`はい${this.name}`);
    }
    /** 
      ★質問ポイント★Managerからのみ呼び出されるようにしたい。Otherからは呼び出されたくない。
      publicではみんな呼べてしまう
      privateでは誰も呼べない
      protectedでもManagerから呼べない
      なんかうまい方法ない？
    */
    sing() {
        alert(`♪${this.name}`);
    }
}

class Other {
    hey() {
         mngr = new Manager();
         mngr.allSing(); // できてほしい
         mngr.mem1.greet(); // できてほしい
         mngr.mem1.sing(); // ★ビルドエラーになってほしい
    }
}

Answer 1

ちょっとがっさいですが、引数から、特定のクラスのインスタンスかどうかを取得し、
特定のクラスでなければ、処理をしない、とか・・・。

class Manager {
   mem1: Member; // メンバへのアクセスは可能
   mem2: Member;
   mem3: Member;
   constructor() {
       this.mem1 = new Member("yama");
       this.mem2 = new Member("naga");
       this.mem1 = new Member("koku");
   }
   allSing() {
       mem1.sing(this);
       mem2.sing(this);
       mem3.sing(this);
   }
}

class Member {
    name: string;
    constructor(n: string) {
         this.name = n;
    }
    /** みんな呼べる */
    public greet() {
        alert(`はい${this.name}`);
    }
    /** 
      ★質問ポイント★Managerからのみ呼び出されるようにしたい。Otherからは呼び出されたくない。
    */
    public sing(instance: any): Error | boolean {
        if (instance.constructer.name === 'Manager') {
          alert(`♪${this.name}`);
          return true;
        } else {
          throw new Error('Method call error, Cannot be called from that class.');
        }
    }
}

class Other {
    hey() {
         mngr = new Manager();
         mngr.allSing(); // できてほしい
         mngr.mem1.greet(); // できてほしい
         mngr.mem1.sing(this); // ★ビルドエラーになってほしい
    }
}

大変申し訳ございませんが、動作検証はしておりません。
クラス名の取得と、エラーを返す部分はもしかしたら、構文が間違っているかもです。
なので、その場合は、考え方のみご参考いただければと思います。
（しかもビルドでエラーはでないかも）

Answer 2

直接メソッドにするのではなく、singを別ファイルに関数として用意して、使う場所でだけimportすることで、無関係な箇所では使わせない、という方法はどうでしょうか。

Answer 3

中を読んでないですけど
https://stackoverflow.com/questions/45847399/friend-class-in-typescript

Answer 4

下記のように、MemberをPrivateにして、各Menberのgreetだけ個別に呼べるよう、Managerにpublic関数を定義したらどうてしょうか？

「特定のクラスからのみアクセス可能なメソッドの作成」とはちょっと違うので質問の意図と違う回答だったらすみません。

class Manager {
   private mem1: Member; // メンバへのアクセスは可能
   private mem2: Member;
   private mem3: Member;
   constructor() {
       this.mem1 = new Member("yama");
       this.mem2 = new Member("naga");
       this.mem1 = new Member("koku");
   }
   allSing() {
       mem1.sing();
       mem2.sing();
       mem3.sing();
   }
   mem1Greet() {this.mem1.greet()}
   mem2Greet() {this.mem2.greet()}
   mem3Greet() {this.mem3.greet()}
}

class Member {
    name: string;
    constructor(n: string) {
         this.name = n;
    }
    /** みんな呼べる */
    greet() {
        alert(`はい${this.name}`);
    }
    /** 
      ★質問ポイント★Managerからのみ呼び出されるようにしたい。Otherからは呼び出されたくない。
      publicではみんな呼べてしまう
      privateでは誰も呼べない
      protectedでもManagerから呼べない
      なんかうまい方法ない？
    */
    sing() {
        alert(`♪${this.name}`);
    }
}

class Other {
    hey() {
         mngr = new Manager();
         mngr.allSing(); // できてほしい
         mngr.mem1Greet(); // 変更
         mngr.mem1.sing(); // ビルドエラー
    }
}

Comments

[退会済みユーザー]

ありがとうございます。やはりManagerが全て面倒見るスタイルになってしまいますかね。。

[gauss0322]

「Manager」なので、OtherはManager経由でアクセスするでいいかなーと思います。
Member側で「Managerクラスからのアクセスのみ許可する」みたいなコードを無理矢理書くと、今度はMenberがManagerに依存することになりますので、そちらの方が不自然かと思いました。

Answer 5

Member クラスに sign() 関数を実装しなければいいのではないでしょうか。
Manager クラスの allSign() ですべてやるようにすればいいと思います。

allSing() {
    alert(`♪${this.mem1.name}`);
    alert(`♪${this.mem2.name}`);
    alert(`♪${this.mem3.name}`);
}

Comments

[退会済みユーザー]

ありがとうございます。Memberの挙動はMemberの関心事としたいので(ManagerはMemberが”どう歌うか”は興味ない)、singはMemberに置きたいんですよね。