Changeset 20369

Timestamp:

2011/02/24 11:54:27 (15 years ago)

Author:

adachi

Message:

#1031 商品CSVアップロードリファクタリング

• $this->objFormParamをローカル変数化
• $this->objUpFileをローカル変数化

File:

• branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php (modified) (14 diffs)

branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php

@@ -43 +43 @@
     var $objFormParam;
 
-    /** SC_UploadFile インスタンス */
-    var $objUpfile;
-
     /** TAGエラーチェックフィールド情報 */
     var $arrTagCheckItem;
@@ -83 +80 @@
         $this->arrMaker = SC_Helper_DB_Ex::sfGetIDValueList("dtb_maker", "maker_id", "name");
         $this->arrPayments = SC_Helper_DB_Ex::sfGetIDValueList("dtb_payment", "payment_id", "payment_method");
+        $this->arrInfo = SC_Helper_DB_Ex::sfGetBasisData();
         $this->arrAllowedTag = $masterData->getMasterData("mtb_allowed_tag");
         $this->arrTagCheckItem = array();
@@ -104 +102 @@
     function action() {
         $this->objDb = new SC_Helper_DB_Ex();
-
-        // ファイル管理クラス
-        $this->objUpFile = new SC_UploadFile(IMAGE_TEMP_REALDIR, IMAGE_SAVE_REALDIR);
-        // サイト基本情報 (ポイントレート初期値用)
-        $this->arrInfo = $this->objDb->sfGetBasisData();
+        
         // CSV管理ヘルパー
         $objCSV = new SC_Helper_CSV_Ex();
@@ -124 +118 @@
 
         // CSVファイルアップロード情報の初期化
-        $this->lfInitFile();
-        // パラメータ管理クラス
-        $this->objFormParam = new SC_FormParam();
+        $objUpFile = new SC_UploadFile(IMAGE_TEMP_REALDIR, IMAGE_SAVE_REALDIR);
+        $this->lfInitFile($objUpFile);
+
         // パラメータ情報の初期化
-        $this->lfInitParam($arrCSVFrame);
-
-        $this->objFormParam->setHtmlDispNameArray();
-        $this->arrTitle = $this->objFormParam->getHtmlDispNameArray();
+        $objFormParam = new SC_FormParam();
+        $this->lfInitParam($objFormParam, $arrCSVFrame);
+
+        $objFormParam->setHtmlDispNameArray();
+        $this->arrTitle = $objFormParam->getHtmlDispNameArray();
 
         switch($this->getMode()) {
         case 'csv_upload':
-            $this->doUploadCsv();
+            $this->doUploadCsv($objFormParam, $objUpFile);
             break;
         default:
@@ -169 +164 @@
      * @return void
      */
-    function doUploadCsv() {
+    function doUploadCsv(&$objFormParam, &$objUpFile) {
         // ファイルアップロードのチェック
-        $this->objUpFile->makeTempFile('csv_file');
-        $arrErr = $this->objUpFile->checkExists();
+        $objUpFile->makeTempFile('csv_file');
+        $arrErr = $objUpFile->checkExists();
         if (count($arrErr) > 0) {
             $this->arrErr = $arrErr;
@@ -178 +173 @@
         }
         // 一時ファイル名の取得
-        $filepath = $this->objUpFile->getTempFilePath('csv_file');
+        $filepath = $objUpFile->getTempFilePath('csv_file');
         // CSVファイルの文字コード変換
         $enc_filepath = SC_Utils_Ex::sfEncodeFile($filepath, CHAR_CODE, CSV_TEMP_REALDIR);
@@ -192 +187 @@
         
         // 登録フォーム カラム情報
-        $this->arrFormKeyList = $this->objFormParam->getKeyList();
+        $this->arrFormKeyList = $objFormParam->getKeyList();
 
         $err = false;
 
         // 登録対象の列数
-        $col_max_count = $this->objFormParam->getCount();
+        $col_max_count = $objFormParam->getCount();
         // 行数
         $line_count = 0;
@@ -226 +221 @@
             }
             // シーケンス配列を格納する。
-            $this->objFormParam->setParam($arrCSV, true);
-            $arrRet = $this->objFormParam->getHashArray();
-            $this->objFormParam->setParam($arrRet);
+            $objFormParam->setParam($arrCSV, true);
+            $arrRet = $objFormParam->getHashArray();
+            $objFormParam->setParam($arrRet);
             // 入力値の変換
-            $this->objFormParam->convParam();
+            $objFormParam->convParam();
             // <br>なしでエラー取得する。
-            $arrCSVErr = $this->lfCheckError();
+            $arrCSVErr = $this->lfCheckError($objFormParam);
 
             // 入力エラーチェック
@@ -243 +238 @@
             }
 
-            $this->lfRegistProduct($objQuery, $line_count);
-            $arrParam = $this->objFormParam->getHashArray();
+            $this->lfRegistProduct($objQuery, $line_count, $objFormParam);
+            $arrParam = $objFormParam->getHashArray();
 
             $this->addRowResult($line_count, "商品ID：".$arrParam['product_id'] . " / 商品名：" . $arrParam['name']);
@@ -281 +276 @@
      * @return void
      */
-    function lfInitFile() {
-        $this->objUpFile->addFile("CSVファイル", 'csv_file', array('csv'), CSV_SIZE, true, 0, 0, false);
+    function lfInitFile(&$objUpFile) {
+        $objUpFile->addFile("CSVファイル", 'csv_file', array('csv'), CSV_SIZE, true, 0, 0, false);
     }
 
@@ -291 +286 @@
      * @return void
      */
-    function lfInitParam(&$arrCSVFrame) {
+    function lfInitParam(&$objFormParam, &$arrCSVFrame) {
         // 固有の初期値調整
         $arrCSVFrame = $this->lfSetParamDefaultValue($arrCSVFrame);
@@ -319 +314 @@
             }
             // パラメーター登録
-            $this->objFormParam->addParam(
+            $objFormParam->addParam(
                     $item['disp_name']
                     , $col
@@ -336 +331 @@
      * @return void
      */
-    function lfCheckError() {
+    function lfCheckError(&$objFormParam) {
         // 入力データを渡す。
-        $arrRet =  $this->objFormParam->getHashArray();
+        $arrRet =  $objFormParam->getHashArray();
         $objErr = new SC_CheckError($arrRet);
-        $objErr->arrErr = $this->objFormParam->checkError(false);
+        $objErr->arrErr = $objFormParam->checkError(false);
         // HTMLタグチェックの実行
         foreach($this->arrTagCheckItem as $item) {
@@ -372 +367 @@
      * @return void
      */
-    function lfRegistProduct($objQuery, $line = "") {
+    function lfRegistProduct($objQuery, $line = "", &$objFormParam) {
         $objProduct = new SC_Product();
         // 登録データ対象取得
-        $arrList = $this->objFormParam->getHashArray();
+        $arrList = $objFormParam->getHashArray();
         // 登録時間を生成(DBのnow()だとcommitした際、すべて同一の時間になってしまう)
         $arrList['update_date'] = $this->lfGetDbFormatTimeWithLine($line);