Changes between Version 3 and Version 4 of EC-CUBE標準規約/リファクタリングガイドライン

Timestamp:

2012/06/07 09:25:54 (9 years ago)

Author:

nanasess

Comment:

EC-CUBE 2.12 における内容をフィードバック

EC-CUBE標準規約/リファクタリングガイドライン

@@ -27 +27 @@
     $this->arrDISP = $masterData->getMasterData('mtb_disp');
 }
-?>
 }}}
 
@@ -78 +77 @@
 
         /** NG SQLを実行する処理を記述してはならない */
-        $objQuery =& SC_Query::getSingletonInstance();
+        $objQuery = SC_Query::getSingletonInstance();
         $arrResults = $objQuery->select('*', 'table_name');
 
@@ -92 +91 @@
     }
 }
-?>
 }}}
 
@@ -120 +118 @@
     SC_Response_Ex::sendRedirect('complete.php');
 }
-?>
 }}}
 
@@ -194 +191 @@
     $this->assertEquals($expected, $actual);
 }
-?>
 }}}
 
@@ -222 +218 @@
      return $arrResults;
 }
-?>
 }}}
 
@@ -241 +236 @@
  * OK シングルトンが保証され, トランザクションの扱いが容易になる
  */
-$objQuery =& SC_Query::getSingletonInstance();
-
-?>
+$objQuery = SC_Query::getSingletonInstance();
 }}}
 
@@ -270 +263 @@
 
     }
-    $objQuery =& SC_Query::getSingletonInstance();
+    $objQuery = SC_Query::getSingletonInstance();
     $arrResults = $objQuery->select('*', 'table_name', $where, $arrValues);
     return $arrResults;
@@ -276 +269 @@
 $arrResults = getResults();
 $arrResults2 = getResults($arrParams);
-
-?>
 }}}
 
@@ -287 +278 @@
 
 /** NG SC_Query::query() は使用しない */
-$objQuery =& SC_Query::getSingletonInstance();
+$objQuery = SC_Query::getSingletonInstance();
 $objQuery->query('INSERT INTO table_name (col1, col2) VALUES (?, ?)', arrary($col1, $col2));
 
 /** OK SC_Query::insert(), SC_Query::update(),  SC_Query::delete() を使用する */
-$objQuery =& SC_Query::getSingletonInstance();
+$objQuery = SC_Query::getSingletonInstance();
 $objQuery->insert('table_name', array('col1' => $col1,
                                       'col2' => $col2));
-
-?>
-}}}
+}}}
+
+LIMIT, OFFSET は SC_Query::setLimit(), SC_Query::setLimitOffset() を使用すること.
+
+{{{
+#!php
+<?php
+
+/** NG LIMIT, OFFSET を直接使用しない */
+$objQuery = SC_Query::getSingletonInstance();
+$arrResults = $objQuery->getAll('SELECT * FROM table_name WHERE del_flg = 0 LIMIT 10 OFFSET 5');
+
+/** OK SC_Query::setLimit(), SC_Query::setLimitOffset() を使用する */
+$objQuery = SC_Query::getSingletonInstance();
+$objQuery->setLimitOffset(10, 5);
+$arrResults = $objQuery->select('*', 'table_name', 'del_flg = ?', array('0'));
+}}}
+
+RDBMS の可搬性向上のため USING 句は使用しないこと
+
+{{{
+#!php
+<?php
+
+/** NG USING 句は使用しない */
+$objQuery = SC_Query::getSingletonInstance();
+$arrResults = $objQuery->select('T1.*', 'table_name T1 JOIN table2_name T2 USING(col)');
+
+/** OK ON 句 を使用する */
+$objQuery = SC_Query::getSingletonInstance();
+$arrResults = $objQuery->select('T1.*', 'table_name T1 JOIN table2_name T2 ON T1.col = T2.col');
+}}}
+
+
 == 入力チェック ==
 
@@ -327 +349 @@
 }
 $this->register($arrForm);
-
-?>
 }}}
 
@@ -346 +366 @@
     // 外部からは $conn という変数を使用しているのがわからない
     if (is_null($conn)) {
-        $conn =& new ClassName();
+        $conn = new ClassName();
     }
 }
@@ -358 +378 @@
     }
 }
-?>
 }}}
 
@@ -378 +397 @@
  */
 $arrProducts = getProducts();
-
-?>
 }}}
 
@@ -402 +419 @@
  */
 $product_id = getProductId();
-
-?>
 }}}
 
@@ -443 +458 @@
   * is_dst を mktime() に渡すこと。 かわりにタイムゾーン処理用の新しい関数を使用します。
 
-以下の機能も, PHP5.3.x で非推奨となっているが, PHP4互換のために使用しても良い
-
-  * new の返り値を参照で代入すること
-  * 呼び出し時の参照渡し
+~~以下の機能も, PHP5.3.x で非推奨となっているが, PHP4互換のために使用しても良い~~
+EC-CUBE 2.12.0 から PHP4 は非対応となりました.
+  * ~~new の返り値を参照で代入すること~~
+  * ~~呼び出し時の参照渡し~~
 
 参考