特定クラスからの呼び出しを禁止する実装方法はどうすればいいですか。

Question

現在、セッション情報を取得するSessionクラスがあり色々なところで利用されています。

ですが、特定クラスからSessionクラスを操作してセッション情報のデータを変えたり、取得するのを禁止したいです。
特定クラスからSessionクラスにアクセスされた場合は、例外をなげるようにできればと思っています。

実装をする人が気がつければいいのですが、特定クラスは不特定多数の人がさわり、たまにしか触らないので
忘れて間違えてSessionクラスから情報を操作するのを防ぎたいです。

ちなみに最初に思いついた実装方法は、Sessionクラスのすべてのメソッドの引数にクラスを渡して、特定クラスだったら例外を出すように考えましたが、Sessionクラスが使われているところが多くてバグにつながるのと、Sessionクラスのメソッド(public,static)の数も多いのでチェックの関数を入れるのも大変なのであきらめました。

もっとシンプルで既存のソースに影響がない方法でSessionクラスのメソッドが呼び出されたら際に自動でチェックする実装方法がありましたら、教えて頂けますでしょうか。

よろしくお願いします。

Answer 1

Kosuke_Shibuyaさんの回答から丸パクリですが・・・

<?php

class Session {
  private function check() {
    $classes = ['test2']; //禁止したいclass
    $info = debug_backtrace();
    $class = $info[count($info) - 1]['class'];
    if (in_array($class, $classes)) {
      throw new Exception('You may not use this method.');
    }
  }

  public static function test() {
    self::check(); // 制限を掛けたいmethodにこれを書く。
    echo "OK\n";
  }
}

class test1 {
  function aa() {
    Session::test();
  }
}

class test2 {
  function aa() {
    Session::test();
  }
}

(new test1)->aa();
(new test2)->aa();

-----出力
OK
Fatal error: Uncaught exception 'Exception' with message 'You may not use this object.' in ～～

```ちょっと実証コードを載せます。
`foo`クラスの`check`メソッドの`$classes`に禁止したいclassを入れます。
そして制限したいメソッドの頭に`self::check()`を入れてチェックさせます。
これだと、新しいstaticメソッドが一個と、それぞれのメソッドに一行追加するだけなので、やりやすいのではないでしょうか？

Comments

[matsujin_]

細かく解説して頂きありがとうございます。
今回、対応しそうなstaticメソッドが20近くありそうだったので、すべて入れるのが大変そうなると、次にSessionにstaticメソッドを追加する人が入れ忘れる場合があるのを懸念いたしました。
でも、今回は自分の技量を考えるとこれがベストだと感じました。
そもそも、Sessionクラスの設計がここまでの用途に作られていないんだなと気づけ、今後の設計にとても勉強になりました。
ありがとうございました。

Answer 2

コンストラクタでフィルタリングすればいいのでは？

get_called_class()
debug_backtrace()

---

class Sample
{

	private static $forbidenClass = [
		'Sample2'
	];

	private static function init()
	{
		$trace = debug_backtrace();
		foreach ($trace as $dbg) {
			if ($dbg['class'] != __CLASS__ &&
				in_array($dbg['class'], self::$forbidenClass)) {
				$msg = sprintf('%s クラスから呼ぶことは禁止されています。', $dbg['class']);
				throw new \Exception($msg);
			}
		}
	}

	public static function method()
	{
		self::init();
		return true;
	}

}

class Sample2
{

	public static function method()
	{
		Sample::method();
	}

}

try {
	Sample2::method();
} catch (Exception $e) {
	var_dump($e);
}

Comments

[shi_ue]

横から失礼。get_called_classは Session が返って来そうです。

[matsujin_]

返答ありがとうございます。
ほとんどのメソッドがstaticで実装されていたので、コンストラクタのことが完全に考えからぬけておりました。
イニシャライザを使ってフィルタリングを試してみたいと思います。

[退会済みユーザー]

Sessionクラスの実装次第ですね。static メソッドしかないようなクラスなら Session::method() になると思います。

[shi_ue]

debug_backtrace !! そんな手があったか・・・

[退会済みユーザー]

autoload でゴニョゴニョする方法もあるかもしれませんんが、__construct() で debug_backtrace() を使う方が、シンプルにかけると思います。

[matsujin_]

publicメソッドについては確認できました。
ありがとうございます。
PHPにstaticイニシャライザがないのですね。
はじめて知りました・・・。
イニシャライザ作成は自分の技量とコストを考えて今回は見送りました。
Session::method()についてですが、コンストラクタの代わりという意味なのでしょうか？
そうすると別のstaticメソッドが呼ばれてしまうと駄目だしと思っていて実装が見えていないのですが、教えて頂けますでしょうか。

[matsujin_]

細かく書いて頂いてありがとうございます。
とても参考になりました。
やはり、staticメソッドすべてにこのチェックをいれなければいけなさそうですね。
ありがとうございましたm_ _m