Changes between Version 1 and Version 2 of リファクタリングガイドライン

Timestamp:

2011/01/28 10:34:20 (13 years ago)

Author:

nanasess

Comment:

--

リファクタリングガイドライン

@@ -4 +4 @@
 
 == init 関数 ==
-
-変数の初期化のみを行い, ビジネスロジックは記述しないこと
+init 関数は, クラスの初期化を目的とします.
+ビジネスロジックの記述をしてはいけません.
 
 {{{
@@ -11 +11 @@
 <?php
 function init() {
-     // 変数の初期化は OK
-     $this->tpl_mainpage = 'index.tpl';
-     $this->arrDISP = $masterData->getMasterData("mtb_disp");
-
-     // ビジネスロジックの記述は NG
-     if (count($this->arrPayment) > 0) {
-         $i = 0;
-         foreach ($this->arrPayment as $val) {
-             $this->payment[$i] = $val;
-             $i++;
-         }
-     }
-}
-?>
-}}}
+
+    /**
+     * NG ビジネスロジックの記述はしない
+     */
+    if (count($this->arrPayment) > 0) {
+        $i = 0;
+        foreach ($this->arrPayment as $val) {
+            $this->payment[$i] = $val;
+            $i++;
+        }
+    }
+
+    /**
+     * OK クラスの初期化のみ行う
+     */
+    $this->tpl_mainpage = 'index.tpl';
+    $this->arrDISP = $masterData->getMasterData("mtb_disp");
+}
+}}}
+
 == action 関数 ==
 
@@ -203 +208 @@
 
 {{{
-誤)
-$arrRet
-
-正)
-$arrProducts
+#!php
+<?php
+
+/** 違反サンプル */
+$arrRet = getProducts();
+
+/** 修正サンプル */
+$arrProducts = getProducts();
 }}}