Changeset 19690

Timestamp:

2010/11/30 02:50:21 (13 years ago)

Author:

AMUAMU

Message:

#564 MDB2化による各種問題の改善
#835　郵便番号辞書の登録でエラーになる(MySQL)の問題改善
#515 SC_Query内のPHP5.3非推奨関数の除去(PHP5.3対応)
#781 INSERT/UPDATEの速度向上(規格のデータベースを木構造に)

SC_Queryがメモリリークしている問題の改善を全面的に行った、副次的に #835 が改善。
テストデータ作成スクリプトがMySQLでデータ作成が出来ない問題も改善。
これによりmemory_limitが低い環境や、DBのプロセスあたりのメモリ最大使用量が低い環境でも動くようになったはず。

UPDATE/INSERT/DELETE時に正しい返値を返さない問題の修正（反映された数が正しく帰るようにした。queryに残っていたDML実行時のTODOも解決。

File:

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

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

@@ -55 +55 @@
         }
 
-        // Debugモード指定
-        $options['debug'] = PEAR_DB_DEBUG;
+        // Debugモード指定 
+        // 常時ONにするとメモリが解放されない。
+        // 連続クエリ実行時に問題が生じる。
+        if(DEBUG_MODE) {
+            $options['debug'] = PEAR_DB_DEBUG;
+        } else {
+            $options['debug'] = 0;
+        }
+
         // 持続的接続オプション
         $options['persistent'] = PEAR_DB_PERSISTENT;
+
+        // バッファリング trueにするとメモリが解放されない。
+        // 連続クエリ実行時に問題が生じる。
+        $options['result_buffering'] = false;
 
         if ($new) {
@@ -411 +422 @@
 
         if(count($sqlval) <= 0 ) return false;
-
         foreach ($sqlval as $key => $val) {
             $strcol .= $key . ',';
-            if(eregi("^Now\(\)$", $val)) {
+            if(strcasecmp("Now()", $val) === 0) {
                 $strval .= 'Now(),';
+            } else if(strpos($val, '~') === 0) {
+                $strval .= preg_replace("/^~/", "", $val);
             } else {
                 $strval .= '?,';
@@ -426 +438 @@
         }
         // 文末の","を削除
-        $strcol = ereg_replace(",$","",$strcol);
-        // 文末の","を削除
-        $strval = ereg_replace(",$","",$strval);
+        $strcol = preg_replace("/,$/", "", $strcol);
+        $strval = preg_replace("/,$/", "", $strval);
         $sqlin = "INSERT INTO $table(" . $strcol. ") VALUES (" . $strval . ")";
         // INSERT文の実行
-        $ret = $this->query($sqlin, $arrval);
+        $ret = $this->query($sqlin, $arrval, false, null, MDB2_PREPARE_MANIP);
 
         return $ret;
@@ -451 +462 @@
         $arrVal = array();
         $find = false;
+
         foreach ($sqlval as $key => $val) {
-            if (eregi("^Now\(\)$", $val)) {
+            if (strcasecmp("Now()", $val) === 0) {
                 $arrCol[] = $key . '= Now()';
+            } else if(strpos('~', $val) === 0) {
+                $arrCol[] = $key . '= ' . $val;
             } else {
                 $arrCol[] = $key . '= ?';
@@ -487 +501 @@
 
         // UPDATE文の実行
-        return $this->query($sqlup, $arrVal);
+        return $this->query($sqlup, $arrVal, false, null, MDB2_PREPARE_MANIP);
     }
 
@@ -626 +640 @@
             $sqlde = "DELETE FROM $table WHERE $where";
         }
-        $ret = $this->query($sqlde, $arrval);
+        $ret = $this->query($sqlde, $arrval, false, null, MDB2_PREPARE_MANIP);
         return $ret;
     }
@@ -667 +681 @@
      * SQL を実行する.
      *
-     * XXX 更新系には exec() を使用するべき
+     * FIXME $ignore_errが無視されるようになっているが互換性として問題が無いか確認が必要
      *
      * @param string $n 実行する SQL 文
-     * @param array $arrval ブレースホルダに挿入する値
+     * @param array $arr ブレースホルダに挿入する値
+     * @param boolean $ignore_err MDB2切替で無効化されている (エラーが発生しても処理を続行する場合 true)
+     * @param mixed $types プレースホルダの型指定 デフォルトnull = string
+     * @param mixed $result_types 返値の型指定またはDML実行(MDB2_PREPARE_MANIP)
      * @return array SQL の実行結果の配列
      */
-    function query($n ,$arr = array(), $ignore_err = false){
+    function query($n ,$arr = array(), $ignore_err = false, $types = null, $result_types = MDB2_PREPARE_RESULT ){
 
         $n = $this->dbFactory->sfChangeMySQL($n);
 
-        $sth =& $this->prepare($n);
+        $sth =& $this->prepare($n, $types, $result_types);
         if (PEAR::isError($sth) && $this->force_run) {
             return;
@@ -686 +703 @@
             return;
         }
-
+        
+        //PREPAREの解放
+        $sth->free();
+        
         return $result;
     }
@@ -775 +795 @@
      * @access private
      * @param string $sql プリペアドステートメントを構築する SQL
+     * @param mixed $types プレースホルダの型指定 デフォルト null
+     * @param mixed $result_types 返値の型指定またはDML実行(MDB2_PREPARE_MANIP)、nullは指定無し
      * @return MDB2_Statement_Common プリペアドステートメントインスタンス
      */
-    function prepare($sql) {
-        $sth =& $this->conn->prepare($sql);
+    function prepare($sql, $types = null, $result_types = MDB2_PREPARE_RESULT) {
+        $sth =& $this->conn->prepare($sql, $types, $result_types);
         if (PEAR::isError($sth)) {
             if (!$this->force_run) {