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

Timestamp:

2011/01/31 12:28:21 (13 years ago)

Author:

nanasess

Comment:

レビューの内容を反映

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

@@ -27 +27 @@
     $this->arrDISP = $masterData->getMasterData('mtb_disp');
 }
+?>
 }}}
 
@@ -221 +222 @@
      return $arrResults;
 }
-
 ?>
 }}}
@@ -234 +234 @@
 
 /**
- * NG
- * PHP4 環境では, 新たなインスタンスが生成されてしまう
+ * NG PHP4 環境では, 新たなインスタンスが生成されてしまう
  */
 $objQuery = new SC_Query();
@@ -327 +326 @@
 }
 $this->register($arrForm);
+
 ?>
 }}}
@@ -357 +357 @@
     }
 }
-
 ?>
 }}}
@@ -378 +377 @@
  */
 $arrProducts = getProducts();
-}}}
-
-テーブルカラム名と共通の変数名は, キャメル式を使用せず, アンダーバーで区切ること
-
-{{{
-#!php
-<?php
-
-/**
- * テーブルカラム名と共通の変数名は, 連想配列の添字で使用されるため,
+
+?>
+}}}
+
+リテラルを格納する (つまり配列・オブジェクト以外の) 変数名は, すべて小文字を使用し, アンダーバーで区切ること
+
+{{{
+#!php
+<?php
+
+/**
+ * リテラルを格納する変数名は, 連想配列の添字で使用されることも考慮し
  * アンダーバー区切りの方が扱いやすい
  */
@@ -400 +401 @@
  */
 $product_id = getProductId();
+
+?>
 }}}
 
 == 1行の文字数/行数 ==
- * 1行の行数は, 80文字を目安に折り返すのが望ましい
+ * 1行の文字数は80文字までにすることを目指すこと. すなわち, コードの長さを現実的な範囲で80文字までにおさめるよう努力すべきです. しかしながら, 場合によっては少々長くなってしまってもかまいません.
  * 1関数の行数は多くても200行程度に留めるのが望ましい. これ以上行数が増えてしまう場合は, 関数を分割する等, コーディングの見直しを行ってください