Changeset 21903

Timestamp:

2012/06/09 20:48:43 (12 years ago)

Author:

Seasoft

Message:

#1467 (SC_Query#getSingletonInstance DB問い合わせ用パラメータの保持・初期化が不適当)
#1858 (SC_Query#getSingletonInstance グローバル変数の使用を避ける)
#1859 (無駄な処理を改善する)

• SC_Query#getSingletonInstance isset() と is_null() の AND 実行は無意味

#1860 (SC_Query#getSingletonInstance 第1引数($dsn)の差を適切に処理できない)

File:

• branches/version-2_12-dev/data/class/SC_Query.php (modified) (2 diffs)

branches/version-2_12-dev/data/class/SC_Query.php

@@ -39 +39 @@
     var $order = '';
     var $force_run = false;
+    static $arrInstance = array();
 
     /**
@@ -98 +99 @@
      */
     function getSingletonInstance($dsn = '', $force_run = false, $new = false) {
-        if (!isset($GLOBALS['_SC_Query_instance'])
-            || is_null($GLOBALS['_SC_Query_instance'])) {
-            $GLOBALS['_SC_Query_instance'] =& new SC_Query_Ex($dsn, $force_run, $new);
-        }
-        $GLOBALS['_SC_Query_instance']->where = '';
-        $GLOBALS['_SC_Query_instance']->arrWhereVal = array();
-        $GLOBALS['_SC_Query_instance']->order = '';
-        $GLOBALS['_SC_Query_instance']->groupby = '';
-        $GLOBALS['_SC_Query_instance']->option = '';
-        return $GLOBALS['_SC_Query_instance'];
+        if (!isset(SC_Query_Ex::$arrInstance[$dsn])) {
+            SC_Query_Ex::$arrInstance[$dsn] =& new SC_Query_Ex($dsn, $force_run, $new);
+        }
+        /*
+         * 歴史的な事情で、このメソッドの呼び出し元は参照で受け取る確率がある。
+         * 退避しているインスタンスをそのまま返すと、退避している SC_Query の
+         * プロパティを直接書き換えることになる。これを回避するため、クローンを返す。
+         * 厳密な意味でのシングルトンではないが、パフォーマンス的に大差は無い。
+         */
+        return clone SC_Query_Ex::$arrInstance[$dsn];
     }