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


Ignore:
Timestamp:
2011/01/31 12:28:21 (11 years ago)
Author:
nanasess
Comment:

レビューの内容を反映

Legend:

Unmodified
Added
Removed
Modified
  • リファクタリングガイドライン

    v5 v6  
    2727    $this->arrDISP = $masterData->getMasterData('mtb_disp'); 
    2828} 
     29?> 
    2930}}} 
    3031 
     
    221222     return $arrResults; 
    222223} 
    223  
    224224?> 
    225225}}} 
     
    234234 
    235235/** 
    236  * NG 
    237  * PHP4 環境では, 新たなインスタンスが生成されてしまう 
     236 * NG PHP4 環境では, 新たなインスタンスが生成されてしまう 
    238237 */ 
    239238$objQuery = new SC_Query(); 
     
    327326} 
    328327$this->register($arrForm); 
     328 
    329329?> 
    330330}}} 
     
    357357    } 
    358358} 
    359  
    360359?> 
    361360}}} 
     
    378377 */ 
    379378$arrProducts = getProducts(); 
    380 }}} 
    381  
    382 テーブルカラム名と共通の変数名は, キャメル式を使用せず, アンダーバーで区切ること 
    383  
    384 {{{ 
    385 #!php 
    386 <?php 
    387  
    388 /** 
    389  * テーブルカラム名と共通の変数名は, 連想配列の添字で使用されるため, 
     379 
     380?> 
     381}}} 
     382 
     383リテラルを格納する (つまり配列・オブジェクト以外の) 変数名は, すべて小文字を使用し, アンダーバーで区切ること 
     384 
     385{{{ 
     386#!php 
     387<?php 
     388 
     389/** 
     390 * リテラルを格納する変数名は, 連想配列の添字で使用されることも考慮し 
    390391 * アンダーバー区切りの方が扱いやすい 
    391392 */ 
     
    400401 */ 
    401402$product_id = getProductId(); 
     403 
     404?> 
    402405}}} 
    403406 
    404407== 1行の文字数/行数 == 
    405  * 1行の行数は, 80文字を目安に折り返すのが望ましい 
     408 * 1行の文字数は80文字までにすることを目指すこと. すなわち, コードの長さを現実的な範囲で80文字までにおさめるよう努力すべきです. しかしながら, 場合によっては少々長くなってしまってもかまいません. 
    406409 * 1関数の行数は多くても200行程度に留めるのが望ましい. これ以上行数が増えてしまう場合は, 関数を分割する等, コーディングの見直しを行ってください 
    407410