[PHP] __callStaticで詰んでいます。

Question

お世話になってます。
PHPの__callStaticで定義された関数を呼ぶにあたり、

publicとprivateの中でそれぞれ呼ぼうとしたら「Cannot redeclare」の定義済みエラーになってしまいました。

しかし先に呼ばれる箇所にのみ書くようにすると、今度は2番目以降で「Call to a member function xxx on a non-object in」と、未定義扱いになってしまいます。。

PHPに詳しくないので基本的なミスかもしれませんが、
些細な点でもかまわないので、アドバイスいただきたく、お願いいたします。
情報が不足していたら追記します。取り急ぎ、対象のソースです。

class libCrypt
{
	public static function __callStatic($class, $args)
	{
		function exeClassAutoLoad($class) {
			require_once("$class"."php");
		}
		spl_autoload_register("exeClassAutoLoad");
		return new $class($args[0]);
	}
}

class Point extends My_Controller
{

	public function Hoge()
	{
		$objCrypt = libCrypt::Aes256Cbc('sec_key'); # Aes256Cbc が上の$classにあたります。	
		$strDeCrypt = $objCrypt->exeEnCrypt($fuga); #exeDeCrypt はAes256Cbcにpublicで定義しているメソッドです。
		・・・
		$this->_bar($data);
	}

	private function _bar($data)
	{
		$objCrypt = libCrypt::Aes256Cbc('sec_key'); # これを書くとPHP Fatal error:  Cannot redeclare exeClassAutoLoad()
		$strDeCrypt = $objCrypt->exeDeCrypt($data); # 上を書かないとここで Call to a member function exeDeCrypt() on a non-object
	}
	

Answer 1

spl_autoload_register を使う必要ないと思います。
__callStatic が直接 require_once すれば大丈夫です。

class libCrypt
{
    public static function __callStatic($class, $args)
    {
        require_once("$class"."php"); // ちなみにこれ、ドットが入らないと思いますが大丈夫ですか
        return new $class($args[0]);
    }
}

Comments

[yoshiky]

早速のご回答ありがとうございます！

ご指摘のとおりspl_autoload_registerを外したら2回呼んでも
エラーになりませんでした。

これで確認してみます。助かりました。ありがとうございました。

> // ちなみにこれ、ドットが入らないと思いますが大丈夫ですか
こちらは require_once($class.".php"); でした。質問にあたり間引いた箇所ですので問題なしです。
ご指摘ありがとうございます。

[tozjp]

あ、いやでも spl_autoload_register は、正しく使えば便利です。
使い方を学んでおくに越したことはありません。

spl_autoload_register はご提示のコードのように、クラスをインクルードしたい時にインクルードするための処理を「自分で」呼び出す必要が無いことがメリットです。
つまり、libCrypt::__callStatic というクラスをロードするための処理をいちいち実行しなければならないようにしてしまうと、全然有り難みを享受できません。

// 一旦関数宣言せずに書くこともできます。
spl_autoload_register(function($className) {
$className = str_replace('\\', DIRECTORY_SEPARATOR, $className); // 一応 \ は / にしておく。
include $className . ".php";
});

これをすべてのコードの前で１回だけ実行すれば、普通にクラスを作ろうとした時点で勝手にインクルードされます。

ストレートに $objCrypt = new Aes256Cbc('sec_key'); と書けます。
うまくいかない場合は、インクルードパスの問題かもしれません。
http://php.net/manual/ja/function.get-include-path.php

[yoshiky]

アドバイスありがとうございます！
__callStaticとspl_autoload_registerはともに「未定義関数にアクセスした際に呼ばれる」という程度の理解なので、使い方までご提示いただけて助かります。

> これをすべてのコードの前で１回だけ実行すれば、普通にクラスを作ろうとした時点で勝手にインクルードされます。

ほかのクラスでも使う箇所が多そうなので、簡単に使えるようにしておいたほうがいいですね。検討します！

[tozjp]

あ、細かいのですが、
> __callStaticとspl_autoload_registerはともに「未定義関数にアクセスした際に呼ばれる」という程度の理解
__callStatic の対象は未定義ではなく、アクセス不能なメソッドです。
private 宣言されたものを外から呼んだ時が含まれます。

spl_autoload_register は関数とは無関係です。未定義のクラスのみです。