Changeset 18487 for branches/comu-ver2/data/class/SC_DbConn.php

Timestamp:

2010/01/10 10:55:55 (16 years ago)

Author:

Seasoft

Message:

• 使用するフェッチモードの指定を可能とした。
  • getAll#getAll
  • SC_Query#getall
  • SC_Query#select
• PHP_Document 追加
• 引数の変数名を分かりやすいものに変更

File:

• branches/comu-ver2/data/class/SC_DbConn.php (modified) (2 diffs)

branches/comu-ver2/data/class/SC_DbConn.php

@@ -147 +147 @@
     }
 
-    // SELECT文の実行結果を全て取得
-    function getAll($n, $arr = ""){
-
-        // mysqlの場合にはビュー表を変換する
-        if (DB_TYPE == "mysql") $n = $this->dbFactory->sfChangeMySQL($n);
-
-        if(PEAR::isError($this->conn)) {
-            if(ADMIN_MODE){
+    /**
+     * クエリを実行し、全ての行を返す
+     *
+     * @param string $sql SQL クエリ
+     * @param array $arrVal プリペアドステートメントの実行時に使用される配列。配列の要素数は、クエリ内のプレースホルダの数と同じでなければなりません。 
+     * @param integer $fetchmode 使用するフェッチモード。デフォルトは DB_FETCHMODE_ASSOC。
+     * @return array データを含む2次元配列。失敗した場合に 0 または DB_Error オブジェクトを返します。
+     */
+    function getAll($sql, $arrVal = "", $fetchmode = DB_FETCHMODE_ASSOC) {
+
+        // mysqlの場合にはビュー表を変換する
+        if (DB_TYPE == "mysql") $sql = $this->dbFactory->sfChangeMySQL($sql);
+
+        // XXX このエラー処理はここで行なうべきなのか疑問。また、戻り値も疑問(なお、変更時はドキュメントも変更を)。
+        if (PEAR::isError($this->conn)) {
+            if (ADMIN_MODE) {
                 SC_Utils_Ex::sfErrorHeader("DBへの接続に失敗しました。:" . $this->dsn);
-            }else{
+            } else {
                 SC_Utils_Ex::sfErrorHeader("DBへの接続に失敗しました。:");
             }
@@ -162 +170 @@
         }
 
-        if ( $arr ){
-            $result = $this->conn->getAll($n, $arr, DB_FETCHMODE_ASSOC);
-        } else {
-            $result = $this->conn->getAll($n, DB_FETCHMODE_ASSOC);
-        }
-
-        if ($this->conn->isError($result)){
-            $this->send_err_mail($result, $n);
+        if ($arrVal) { // FIXME 判定が曖昧
+            $result = $this->conn->getAll($sql, $arrVal, $fetchmode);
+        } else {
+            $result = $this->conn->getAll($sql, $fetchmode);
+        }
+
+        if ($this->conn->isError($result)) {
+            $this->send_err_mail($result, $sql);
         }
         $this->result = $result;