リファクタリング提案その１

[pm-yosuke]

responsive-iframe-googlemap/responsive-iframe-googlemap.php

Lines 48 to 58 in 84b4419

	/**
	* Return a list of constants.
	*
	* @return array Constant list
	* @throws ReflectionException
	*/
	static function getConstants() {
	$oClass = new ReflectionClass( __CLASS__ );
	return $oClass->getConstants();
	}

プロダクトのコードでリフレクションは使用しないほうが良いかと。
理由として、

1. 普通に配列を作る場合と比べて、 15倍 以上遅くなる
2. @throws にあるとおり、例外が発生する可能性を無駄に増やす
3. 71〜77、86〜92行目でこんだけ列挙してるなら115行目で同じように全て列挙しても変わらんよね

※私の経験上、リフレクションは基本的にデバッグやテスト用と割り切ったほうが良いかと思います。

[pm-yosuke]

ちなみに下記スクリプトを実行すると15倍以上遅いのが確認できます。

<?php

class CLS
{
    const FOO = 'foo';
    const BAR = 'bar';
    const BAZ = 'baz';
    const HOGE = 'hoge';
    const FUGA = 'fuga';
    const PIYO = 'piyo';

    public static function get1()
    {
        $cls = new ReflectionClass(__CLASS__);

        return $cls->getConstants();
    }

    public static function get2()
    {
        return [
            'FOO' => self::FOO,
            'BAR' => self::BAR,
            'BAZ' => self::BAZ,
            'HOGE' => self::HOGE,
            'FUGA' => self::FUGA,
            'PIYO' => self::PIYO
        ];
    }
}

$num = 1000000;

echo 'START---------------------------------------------------------------------', PHP_EOL;
$s = microtime(true);

for ($i = 0; $i < $num; $i++) {
    CLS::get1();
}

echo 'Reflection: ', $ref = microtime(true) - $s, PHP_EOL;
$s = microtime(true);


for ($i = 0; $i < $num; $i++) {
    CLS::get2();
}

echo 'Normal: ', $nor = microtime(true) - $s, PHP_EOL;
echo 'END-----------------------------------------------------------------------', PHP_EOL;
echo 'RESULT: ', ($ref / $nor), ' times, ', round(($nor - $ref) / $nor * 100, 2), '%', PHP_EOL;

↓実行結果

$ php test.php
START---------------------------------------------------------------------
Reflection: 0.25741696357727
Normal: 0.016985177993774
END-----------------------------------------------------------------------
RESULT: 15.155388049017 times, -1415.54%