Changeset 18571 for branches/comu-ver2

Timestamp:

2010/02/18 11:06:04 (13 years ago)

Author:

Seasoft

Message:

マスタデータキャッシュに関する改訂

• キャッシュ生成のロジックをPHP関数に依存させることで高速化。
• 表示順を任意とすることで一部の処理を高速化。
• パラメータ更新時の個別ロジックを極力単純にして共通化。

Location:

branches/comu-ver2/data/class

Files:

• db/SC_DB_MasterData.php (modified) (6 diffs)
• pages/admin/design/LC_Page_Admin_Design_Template.php (modified) (1 diff)
• pages/admin/system/LC_Page_Admin_System_Parameter.php (modified) (1 diff)

branches/comu-ver2/data/class/db/SC_DB_MasterData.php

@@ -59 +59 @@
      *
      * 以下の順序でマスタデータを取得する.
-     * 1. MASTER_DATA_DIR のマスタデータキャッシュを include() で読み込む
-     * 2. 1 で読み込んだ値をチェックし, 値が変数定義されていれば値を返す.
-     *    されていなければ, 次の処理を行う.
-     * 3. 値が未定義の場合は, DBからマスタデータを取得する.
-     * 4. 取得した後, マスタデータのキャッシュを生成し, 値を返す.
+     * 1. MASTER_DATA_DIR にマスタデータキャッシュが存在しない場合、
+     *    DBからマスタデータを取得して、マスタデータキャッシュを生成する。
+     * 2. マスタデータキャッシュを読み込み、変数に格納し返す。
      *
      * 返り値は, key => value 形式の配列である.
@@ -76 +74 @@
         $columns = $this->getDefaultColumnName($columns);
 
-        // 可変変数を定義
-        $valiable = "_" . $name . "_master";
+        $filepath = MASTER_DATA_DIR . $name . '.serial';
+
+        if (!file_exists($filepath)) {
+            // キャッシュ生成
+            $this->createCache($name, $columns);
+        }
+
         // キャッシュを読み込み
-        @include(MASTER_DATA_DIR . $name . ".php");
-
-        // キャッシュがあれば, キャッシュの値を返す.
-        if (!empty($$valiable)) {
-            return $$valiable;
-        }
-        // マスタデータを取得
-        $masterData = $this->getDbMasterData($name, $columns);
-        // キャッシュ生成
-        $this->createCache($name, $masterData);
+        $masterData = unserialize(file_get_contents($filepath));
+
         return $masterData;
     }
@@ -253 +248 @@
      * @return bool キャッシュの生成に成功した場合 true
      */
-    function createCache($name, $masterData, $isDefine = false,
+    function createCache($name, $columns = array(), $isDefine = false,
                          $commentColumn = array()) {
 
+        // マスタデータを取得
+        $masterData = $this->getDbMasterData($name, $columns);
+
         // マスタデータを文字列にする
-        $data = "<?php\n";
         // 定数を生成する場合
         if ($isDefine) {
-
+            $path = MASTER_DATA_DIR . $name . '.php';
+
+            $data = "<?php\n";
             // 定数コメントを生成する場合
             if (!empty($commentColumn)) {
@@ -268 +267 @@
                 $data .= $this->getMasterDataAsDefine($masterData);
             }
+            $data .=  "?>\n";
 
         // 配列を生成する場合
         } else {
-            $data .= $this->getMasterDataAsString($name, $masterData);
-        }
-        $data .=  "?>\n";
+            $path = MASTER_DATA_DIR . $name . '.serial';
+            $data = serialize($masterData);
+        }
 
         // ファイルを書き出しモードで開く
-        $path = MASTER_DATA_DIR . $name . ".php";
         $handle = fopen($path, "w");
         if (!$handle) {
@@ -305 +304 @@
 
         $this->objQuery = new SC_Query();
-        $this->objQuery->setorder($columns[2]);
+        if (isset($columns[2]) && strlen($columns[2]) >= 1) {
+            $this->objQuery->setorder($columns[2]);
+        }
         $results = $this->objQuery->select($columns[0] . ", " . $columns[1], $name);
 
@@ -337 +338 @@
             return $this->columns;
         }
-    }
-
-    /**
-     * マスタデータの配列を配列定義の文字列として出力する.
-     *
-     * @access private
-     * @param string $name マスタデータ名
-     * @param array $masterData マスタデータの配列
-     * @return string 配列定義の文字列
-     */
-    function getMasterDataAsString($name, $masterData) {
-        $data = "\$_" . $name . "_master = array(\n";
-        $i = count($masterData);
-        foreach ($masterData as $key => $val) {
-            $data .= "'" . $key . "' => '" . $val . "'";
-            if ($i > 1) {
-                $data .= ",\n";
-            }
-            $i--;
-        }
-        $data .= ");\n";
-        return $data;
     }
 

branches/comu-ver2/data/class/pages/admin/design/LC_Page_Admin_Design_Template.php

@@ -228 +228 @@
     }
 
-    /**
-     * 使用するテンプレートをDBへ登録する
-     */
-    function lfRegisterTemplate($template_code) {
-        $objQuery = new SC_Query();
-        $sqlval['name'] = "\"" . $template_code . "\"";
-        $objQuery->update("mtb_constants", $sqlval, "id = ?", array('TEMPLATE_NAME'));
-        // キャッシュを生成
-        $masterData = new SC_DB_MasterData_Ex();
-        // 更新したデータを取得
-        $mtb_constants = $masterData->getDBMasterData("mtb_constants");
-        $masterData->clearCache("mtb_constants");
-        $masterData->createCache("mtb_constants", $mtb_constants, true,
-                                 array("id", "remarks", "rank"));
-    }
+    /**
+     * 使用するテンプレートをDBへ登録する
+     */
+    function lfRegisterTemplate($template_code) {
+        $masterData = new SC_DB_MasterData_Ex();
+
+        $data = array('TEMPLATE_NAME' => var_export($template_code, TRUE));
+
+        // DBのデータを更新
+        $masterData->updateMasterData('mtb_constants', array(), $data);
+
+        // キャッシュを生成
+        $masterData->createCache('mtb_constants', array(), true, array('id', 'remarks'));
+    }
 
     /**

branches/comu-ver2/data/class/pages/admin/system/LC_Page_Admin_System_Parameter.php

@@ -135 +135 @@
 
         // DBのデータを更新
-        $masterData->updateMasterData("mtb_constants", array(), $data);
-
-        // 更新したデータを取得
-        $mtb_constants = $masterData->getDBMasterData("mtb_constants");
+        $masterData->updateMasterData('mtb_constants', array(), $data);
 
         // キャッシュを生成
-        $masterData->clearCache("mtb_constants");
-        $masterData->createCache("mtb_constants", $mtb_constants, true,
-                                 array("id", "remarks", "rank"));
+        $masterData->createCache('mtb_constants', array(), true, array('id', 'remarks'));
     }