Changeset 20326


Ignore:
Timestamp:
2011/02/22 15:57:30 (10 years ago)
Author:
coelacanth
Message:

#964 リファクタリング 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない

Location:
branches/version-2_5-dev/data/class/pages/admin/basis
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Holiday.php

    r20323 r20326  
    8484            $this->lfInitParam($mode, $objFormParam); 
    8585            $objFormParam->setParam($_POST); 
     86            $objFormParam->convParam(); 
    8687 
    8788            $this->arrErr = $this->lfCheckError($mode, $objFormParam); 
     
    9091                return; 
    9192            } 
     93 
     94            $post = $objFormParam->getHashArray(); 
    9295        } 
    9396 
     
    97100        case 'edit': 
    98101            // POST値の引き継ぎ 
    99             $this->arrForm = $_POST; 
     102            $this->arrForm = $this->arrForm = $_POST; 
    100103 
    101104            if(count($this->arrErr) <= 0) { 
    102                 if($_POST['holiday_id'] == "") { 
     105                if($post['holiday_id'] == "") { 
    103106                    $this->lfInsertClass($this->arrForm, $_SESSION['member_id']);    // 新規作成 
    104107                } else { 
    105                     $this->lfUpdateClass($this->arrForm, $_POST['holiday_id']); // 既存編集 
     108                    $this->lfUpdateClass($this->arrForm, $post['holiday_id']);  // 既存編集 
    106109                } 
    107110                // 再表示 
     
    109112            } else { 
    110113                // POSTデータを引き継ぐ 
    111                 $this->tpl_holiday_id = $_POST['holiday_id']; 
     114                $this->tpl_holiday_id = $post['holiday_id']; 
    112115            } 
    113116            break; 
    114117        // 削除 
    115118        case 'delete': 
    116             $objDb->sfDeleteRankRecord("dtb_holiday", "holiday_id", $_POST['holiday_id'], "", true); 
     119            $objDb->sfDeleteRankRecord("dtb_holiday", "holiday_id", $post['holiday_id'], "", true); 
    117120            // 再表示 
    118121            $this->objDisplay->reload(); 
     
    121124        case 'pre_edit': 
    122125            // 編集項目を取得する。 
    123             $arrHolidayData = $this->lfGetHolidayDataByHolidayID($_POST['holiday_id']); 
     126            $arrHolidayData = $this->lfGetHolidayDataByHolidayID($post['holiday_id']); 
    124127 
    125128            // 入力項目にカテゴリ名を入力する。 
     
    128131            $this->arrForm['day'] = $arrHolidayData[0]['day']; 
    129132            // POSTデータを引き継ぐ 
    130             $this->tpl_holiday_id = $_POST['holiday_id']; 
     133            $this->tpl_holiday_id = $post['holiday_id']; 
    131134        break; 
    132135        case 'down': 
    133             $objDb->sfRankDown("dtb_holiday", "holiday_id", $_POST['holiday_id']); 
     136            $objDb->sfRankDown("dtb_holiday", "holiday_id", $post['holiday_id']); 
    134137            // 再表示 
    135138            $this->objDisplay->reload(); 
    136139            break; 
    137140        case 'up': 
    138             $objDb->sfRankUp("dtb_holiday", "holiday_id", $_POST['holiday_id']); 
     141            $objDb->sfRankUp("dtb_holiday", "holiday_id", $post['holiday_id']); 
    139142            // 再表示 
    140143            $this->objDisplay->reload(); 
     
    189192 
    190193    /* DBへの更新 */ 
    191     function lfUpdateClass($arrData, $holiday_id) { 
     194    function lfUpdateClass($arrData) { 
    192195        $objQuery =& SC_Query::getSingletonInstance(); 
    193196        // UPDATEする値を作成する。 
     
    198201        $where = "holiday_id = ?"; 
    199202        // UPDATEの実行 
    200         $ret = $objQuery->update("dtb_holiday", $sqlval, $where, array($holiday_id)); 
     203        $ret = $objQuery->update("dtb_holiday", $sqlval, $where, array($arrData['holiday_id'])); 
    201204        return $ret; 
    202205    } 
  • branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Kiyaku.php

    r20297 r20326  
    7676 
    7777        if (!empty($_POST)) { 
    78             $this->arrErr = $this->lfCheckError($mode, $_POST); 
     78            $objFormParam = new SC_FormParam(); 
     79            $this->lfInitParam($mode, $objFormParam); 
     80            $objFormParam->setParam($_POST); 
     81            $objFormParam->convParam(); 
     82 
     83            $this->arrErr = $this->lfCheckError($mode, $objFormParam); 
    7984            if (!empty($this->arrErr['kiyaku_id'])) { 
    8085                SC_Utils_Ex::sfDispException(); 
    8186                return; 
    8287            } 
     88            $post = $objFormParam->getHashArray(); 
    8389        } 
    8490 
     
    9197 
    9298            if(count($this->arrErr) <= 0) { 
    93                 if($_POST['kiyaku_id'] == "") { 
     99                if($post['kiyaku_id'] == "") { 
    94100                    $this->lfInsertClass($this->arrForm, $_SESSION['member_id']);    // 新規作成 
    95101                } else { 
    96                     $this->lfUpdateClass($this->arrForm, $_POST['kiyaku_id']);    // 既存編集 
     102                    $this->lfUpdateClass($this->arrForm, $post['kiyaku_id']);    // 既存編集 
    97103                } 
    98104                // 再表示 
     
    100106            } else { 
    101107                // POSTデータを引き継ぐ 
    102                 $this->tpl_kiyaku_id = $_POST['kiyaku_id']; 
     108                $this->tpl_kiyaku_id = $post['kiyaku_id']; 
    103109            } 
    104110            break; 
    105111        // 削除 
    106112        case 'delete': 
    107             $objDb->sfDeleteRankRecord("dtb_kiyaku", "kiyaku_id", $_POST['kiyaku_id'], "", true); 
     113            $objDb->sfDeleteRankRecord("dtb_kiyaku", "kiyaku_id", $post['kiyaku_id'], "", true); 
    108114            // 再表示 
    109115            $this->objDisplay->reload(); 
     
    112118        case 'pre_edit': 
    113119            // 編集項目を取得する。 
    114             $arrKiyakuData = $this->lfGetKiyakuDataByKiyakuID($_POST['kiyaku_id']); 
     120            $arrKiyakuData = $this->lfGetKiyakuDataByKiyakuID($post['kiyaku_id']); 
    115121 
    116122            // 入力項目にカテゴリ名を入力する。 
     
    118124            $this->arrForm['kiyaku_text'] = $arrKiyakuData[0]['kiyaku_text']; 
    119125            // POSTデータを引き継ぐ 
    120             $this->tpl_kiyaku_id = $_POST['kiyaku_id']; 
     126            $this->tpl_kiyaku_id = $post['kiyaku_id']; 
    121127        break; 
    122128        case 'down': 
    123             $objDb->sfRankDown("dtb_kiyaku", "kiyaku_id", $_POST['kiyaku_id']); 
     129            $objDb->sfRankDown("dtb_kiyaku", "kiyaku_id", $post['kiyaku_id']); 
    124130            // 再表示 
    125131            $this->objDisplay->reload(); 
    126132            break; 
    127133        case 'up': 
    128             $objDb->sfRankUp("dtb_kiyaku", "kiyaku_id", $_POST['kiyaku_id']); 
     134            $objDb->sfRankUp("dtb_kiyaku", "kiyaku_id", $post['kiyaku_id']); 
    129135            // 再表示 
    130136            $this->objDisplay->reload(); 
     
    190196    } 
    191197 
    192     /* 取得文字列の変換 */ 
    193     function lfConvertParam($array) { 
    194         // 文字変換 
    195         $arrConvList['kiyaku_title'] = "KVa"; 
    196         $arrConvList['kiyaku_text'] = "KVa"; 
    197  
    198         foreach ($arrConvList as $key => $val) { 
    199             // POSTされてきた値のみ変換する。 
    200             if(isset($array[$key])) { 
    201                 $array[$key] = mb_convert_kana($array[$key] ,$val); 
    202             } 
    203         } 
    204         return $array; 
    205     } 
    206  
    207     /** 
    208      * 入力エラーチェック 
    209      * 
    210      * @param string $mode 
    211      * @return array 
    212      */ 
    213     function lfCheckError($mode, $post) { 
    214         $arrErr = array(); 
    215  
     198    function lfInitParam($mode, &$objFormParam) { 
    216199        switch ($mode) { 
    217200            case 'edit': 
    218                 $_POST = $this->lfConvertParam($post); 
    219  
    220                 $objErr = new SC_CheckError(); 
    221                 $objErr->doFunc(array("規約タイトル", "kiyaku_title", SMTEXT_LEN), array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
    222                 $objErr->doFunc(array("規約内容", "kiyaku_text", MLTEXT_LEN), array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
    223                 if(!isset($objErr->arrErr['name'])) { 
    224                     $objQuery =& SC_Query::getSingletonInstance(); 
    225                     $arrRet = $objQuery->select("kiyaku_id, kiyaku_title", "dtb_kiyaku", "del_flg = 0 AND kiyaku_title = ?", array($post['kiyaku_title'])); 
    226                     // 編集中のレコード以外に同じ名称が存在する場合 
    227                     if ($arrRet[0]['kiyaku_id'] != $post['kiyaku_id'] && $arrRet[0]['kiyaku_title'] == $post['kiyaku_title']) { 
    228                         $objErr->arrErr['name'] = "※ 既に同じ内容の登録が存在します。<br>"; 
    229                     } 
    230                 } 
     201                $objFormParam->addParam('規約タイトル', 'kiyaku_title', SMTEXT_LEN, 'KVa', array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
     202                $objFormParam->addParam('規約内容', 'kiyaku_text', MLTEXT_LEN, 'KVa', array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
    231203            case 'delete': 
    232204            case 'pre_edit': 
    233205            case 'down': 
    234206            case 'up': 
    235                 $this->objFormParam = new SC_FormParam(); 
    236                 $this->objFormParam->addParam('規約ID', 'kiyaku_id', INT_LEN, 'n', array('NUM_CHECK', 'MAX_LENGTH_CHECK')); 
    237                 $this->objFormParam->setParam($post); 
    238                 $this->objFormParam->convParam(); 
    239                 $arrErr = $this->objFormParam->checkError(); 
    240  
     207                $objFormParam->addParam('規約ID', 'kiyaku_id', INT_LEN, 'n', array('NUM_CHECK', 'MAX_LENGTH_CHECK')); 
    241208                break; 
    242209            default: 
    243210                break; 
    244211        } 
    245         return array_merge((array)$objErr->arrErr, (array)$arrErr); 
     212    } 
     213 
     214    /** 
     215     * 入力エラーチェック 
     216     * 
     217     * @param string $mode 
     218     * @return array 
     219     */ 
     220    function lfCheckError($mode, $objFormParam) { 
     221        $arrErr = $objFormParam->checkError(); 
     222        if(!isset($arrErr['name']) && $mode == 'edit') { 
     223            $post = $objFormParam->getHashArray(); 
     224            $objQuery =& SC_Query::getSingletonInstance(); 
     225            $arrRet = $objQuery->select("kiyaku_id, kiyaku_title", "dtb_kiyaku", "del_flg = 0 AND kiyaku_title = ?", array($post['kiyaku_title'])); 
     226            // 編集中のレコード以外に同じ名称が存在する場合 
     227            if ($arrRet[0]['kiyaku_id'] != $post['kiyaku_id'] && $arrRet[0]['kiyaku_title'] == $post['kiyaku_title']) { 
     228                $arrErr['name'] = "※ 既に同じ内容の登録が存在します。<br>"; 
     229            } 
     230        } 
     231        return $arrErr; 
    246232    } 
    247233} 
  • branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Mail.php

    r20297 r20326  
    7474        SC_Utils_Ex::sfIsSuccess($objSess); 
    7575 
     76        $mode = $this->getMode(); 
     77 
     78        if (!empty($_POST)) { 
     79            $objFormParam = new SC_FormParam(); 
     80            $this->lfInitParam($mode, $objFormParam); 
     81            $objFormParam->setParam($_POST); 
     82            $objFormParam->convParam(); 
     83 
     84            $this->arrErr = $objFormParam->checkError(); 
     85            if (!empty($this->arrErr['template_id'])) { 
     86                SC_Utils_Ex::sfDispException(); 
     87                return; 
     88            } 
     89            $post = $objFormParam->getHashArray(); 
     90        } 
     91 
    7692        $this->arrMailTEMPLATE = $masterData->getMasterData("mtb_mail_template"); 
    77         switch ($this->getMode()) { 
     93 
     94        switch ($mode) { 
    7895        case 'id_set': 
    79             // テンプレートプルダウン変更時 
    80  
    81             if ( SC_Utils_Ex::sfIsInt( $_POST['template_id']) ){ 
    82                 $result = $this->lfGetMailTemplateByTemplateID($_POST['template_id']); 
    83                 if ( $result ){ 
     96                $result = $this->lfGetMailTemplateByTemplateID($post['template_id']); 
     97                if ($result){ 
    8498                    $this->arrForm = $result[0]; 
    8599                } else { 
    86                     $this->arrForm['template_id'] = $_POST['template_id']; 
     100                    $this->arrForm['template_id'] = $post['template_id']; 
    87101                } 
    88             } 
    89102            break; 
    90103        case 'regist': 
    91             if (SC_Utils_Ex::sfIsInt( $_POST['template_id']) ){ 
    92104 
    93                 // POSTデータの引き継ぎ 
    94                 $this->arrForm = $this->lfConvertParam($_POST); 
    95                 $this->arrErr = $this->fnErrorCheck($this->arrForm); 
    96  
    97                 if ( $this->arrErr ){ 
     105                $this->arrForm = $post; 
     106                if ($this->arrErr){ 
    98107                    // エラーメッセージ 
    99108                    $this->tpl_msg = "エラーが発生しました"; 
     
    101110                } else { 
    102111                    // 正常 
    103                     $this->lfRegist($this->arrForm, $_POST['template_id'], $_SESSION['member_id']); 
     112                    $this->lfRegistMailTemplate($this->arrForm, $_SESSION['member_id']); 
    104113 
    105114                    // 完了メッセージ 
     
    107116                    unset($this->arrForm); 
    108117                } 
    109             } 
    110118            break; 
    111119        default: 
     
    130138    } 
    131139 
    132     function lfRegist($data, $template_id, $member_id){ 
     140    function lfRegistMailTemplate($post, $member_id){ 
    133141        $objQuery =& SC_Query::getSingletonInstance(); 
    134142 
    135         $data['creator_id'] = $member_id; 
     143        $post['creator_id'] = $member_id; 
    136144 
    137145        $sql = "SELECT * FROM dtb_mailtemplate WHERE template_id = ?"; 
    138         $result = $objQuery->getAll($sql, array($template_id)); 
    139         if ( $result ){ 
     146        $template_data = $objQuery->getAll($sql, array($post['template_id'])); 
     147        if ($template_data){ 
    140148            $sql_where = "template_id = ?"; 
    141             $objQuery->update("dtb_mailtemplate", $data, $sql_where, array(addslashes($template_id))); 
     149            $objQuery->update("dtb_mailtemplate", $post, $sql_where, array(addslashes($post['template_id']))); 
    142150        }else{ 
    143             $objQuery->insert("dtb_mailtemplate", $data); 
     151            $objQuery->insert("dtb_mailtemplate", $post); 
    144152        } 
    145153 
    146154    } 
    147155 
    148  
    149     function lfConvertParam($array) { 
    150  
    151         $new_array["template_id"] = $array["template_id"]; 
    152         $new_array["subject"] = mb_convert_kana($array["subject"] ,"KV"); 
    153         $new_array["header"] = mb_convert_kana($array["header"] ,"KV"); 
    154         $new_array["footer"] = mb_convert_kana($array["footer"] ,"KV"); 
    155  
    156         return $new_array; 
    157     } 
    158  
    159     /* 入力エラーのチェック */ 
    160     function fnErrorCheck($array) { 
    161  
    162         $objErr = new SC_CheckError($array); 
    163  
    164         $objErr->doFunc(array("テンプレート",'template_id'), array("EXIST_CHECK")); 
    165         $objErr->doFunc(array("メールタイトル",'subject',MTEXT_LEN,"BIG"), array("EXIST_CHECK", "MAX_LENGTH_CHECK")); 
    166         $objErr->doFunc(array("ヘッダー",'header',LTEXT_LEN,"BIG"), array("MAX_LENGTH_CHECK")); 
    167         $objErr->doFunc(array("フッター",'footer',LTEXT_LEN,"BIG"), array("MAX_LENGTH_CHECK")); 
    168  
    169         return $objErr->arrErr; 
     156    function lfInitParam($mode, &$objFormParam) { 
     157        switch ($mode) { 
     158            case 'regist': 
     159                $objFormParam->addParam('メールタイトル', 'subject', MTEXT_LEN, 'KVa', array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
     160                $objFormParam->addParam('ヘッダー', 'header', LTEXT_LEN, 'KVa', array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
     161                $objFormParam->addParam('フッター', 'footer', LTEXT_LEN, 'KVa', array("EXIST_CHECK","SPTAB_CHECK","MAX_LENGTH_CHECK")); 
     162            case 'id_set': 
     163                $objFormParam->addParam('テンプレート', 'template_id', INT_LEN, 'n', array('NUM_CHECK', 'MAX_LENGTH_CHECK')); 
     164                break; 
     165            default: 
     166                break; 
     167        } 
    170168    } 
    171169} 
Note: See TracChangeset for help on using the changeset viewer.