Changeset 22064

Timestamp:

2012/11/02 15:48:19 (11 years ago)

Author:

pineray

Message:

#1903 (無駄な処理を改善する) 冗長な処理を短縮。

Location:

branches/version-2_12-dev/data/class/pages/admin/products

Files:

• LC_Page_Admin_Products_Product.php (modified) (3 diffs)
• LC_Page_Admin_Products_ReviewEdit.php (modified) (2 diffs)

branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Product.php

@@ -326 +326 @@
         $objFormParam->addParam('商品ステータス', 'product_status', INT_LEN, 'n', array('NUM_CHECK', 'MAX_LENGTH_CHECK'));
 
-        if ($this->lfGetProductClassFlag($arrPost['has_product_class']) == false) {
+        if (!$arrPost['has_product_class']) {
             // 新規登録, 規格なし商品の編集の場合
             $objFormParam->addParam('商品種別', 'product_type_id', INT_LEN, 'n', array('EXIST_CHECK', 'NUM_CHECK', 'MAX_LENGTH_CHECK'));
@@ -949 +949 @@
 
     /**
-     * 入力フォームパラメーターの規格ありなしフラグを判定
-     *
-     * @param string $has_product_class 入力フォームパラメーターの規格ありなしフラグ
-     * @return boolean true: 規格あり, false: 規格なし
-     */
-    function lfGetProductClassFlag($has_product_class) {
-        if ($has_product_class == '1') {
-            return true;
-        } else {
-            return false;
-        }
-    }
-
-    /**
      * ページonload用JavaScriptを取得する
      * - 入力画面
@@ -1035 +1021 @@
             // 複製商品の場合には規格も複製する
             if ($arrList['copy_product_id'] != '' && SC_Utils_Ex::sfIsInt($arrList['copy_product_id'])) {
-                if ($this->lfGetProductClassFlag($arrList['has_product_class']) == false) {
+                if (!$arrList['has_product_class']) {
                     //規格なしの場合、複製は価格等の入力が発生しているため、その内容で追加登録を行う
                     $this->lfCopyProductClass($arrList, $objQuery);

branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_ReviewEdit.php

@@ -86 +86 @@
             // 登録
             case 'complete':
-                $this->arrErr = $this->lfCheckError($objFormParam);
+                $this->arrErr = $objFormParam->checkError();
                 // エラー無し
-                if (!$this->arrErr) {
+                if (!SC_Utils_Ex::isBlank($this->arrErr)) {
                     // レビュー情報の更新
                     $this->lfRegistReviewData($this->arrForm['review_id'], $objFormParam);
@@ -137 +137 @@
 
     /**
-     * フォーム入力パラメーターエラーチェック
-     *
-     * @param array $objFormParam フォームパラメータークラス
-     * @return array エラー配列
-     */
-    function lfCheckError(&$objFormParam) {
-        $arrErr = $objFormParam->checkError();
-        if (!SC_Utils_Ex::isBlank($arrErr)) {
-            return $arrErr;
-        }
-    }
-
-    /**
      * レビュー情報のDB取得
      *