Changeset 21007

Timestamp:

2011/07/03 03:36:24 (12 years ago)

Author:

Seasoft

Message:

#1388 (SC_FormParam#checkError 不明な第2引数、不明な処理)

• 利用されていないことを確認。削除。

#642 (共通ロジックの機能向上)

• SC_FormParam#getHashArray 要素番号の取り方を変更。対象キーの複数指定対応。
• SC_FormParam#getSwapArray 主な処理をSC_FormParam#getHashArrayとSC_Utils_Ex#sfSwapArrayに依存させた。
• SC_FormParam#getKeyList 無駄な変換を省略。

File:

• branches/version-2_11-dev/data/class/SC_FormParam.php (modified) (4 diffs)

branches/version-2_11-dev/data/class/SC_FormParam.php

@@ -174 +174 @@
 
     // エラーチェック
-    function checkError($br = true, $keyname = "") {
-        // 連想配列の取得
-        $arrRet = $this->getHashArray($keyname);
+    function checkError($br = true) {
         $objErr->arrErr = array();
 
@@ -323 +321 @@
     }
 
-    // 連想配列の作成
-    function getHashArray($keyname = "") {
+    /**
+     * 連想配列で返す
+     *
+     * @param array $arrKey 対象のキー
+     * @return array 連想配列
+     */
+    function getHashArray($arrKey = array()) {
         $arrRet = array();
-        $cnt = 0;
-        foreach($this->keyname as $val) {
-            if($keyname == "" || $keyname == $val) {
-                $arrRet[$val] = isset($this->param[$cnt]) ? $this->param[$cnt] : "";
-                $cnt++;
+        foreach($this->keyname as $index => $keyname) {
+            if (empty($arrKey) || in_array($keyname, $arrKey)) {
+                $arrRet[$keyname] = isset($this->param[$index]) ? $this->param[$index] : '';
             }
         }
@@ -348 +349 @@
     }
 
-    // 配列の縦横を入れ替えて返す
-    function getSwapArray($arrKey) {
-        $arrRet = array();
-        foreach ($arrKey as $keyname) {
-            $arrRet[$keyname] = $this->getValue($keyname);
-        }
-        return SC_Utils_Ex::sfSwapArray($arrRet);
+    /**
+     * 配列の縦横を入れ替えて返す
+     *
+     * @param array $arrKey 対象のキー
+     * @return array 縦横を入れ替えた配列
+     */
+    function getSwapArray($arrKey = array()) {
+        $arrTmp = $this->getHashArray($arrKey);
+        return SC_Utils_Ex::sfSwapArray($arrTmp);
     }
 
@@ -393 +396 @@
     }
 
-    // キー名の一覧を返す
+    /**
+     * キー名の一覧を返す
+     *
+     * @return array キー名の一覧
+     */
     function getKeyList() {
-        foreach($this->keyname as $val) {
-            $arrRet[] = $val;
-        }
-        return $arrRet;
+        return $this->keyname;
     }