ガード節を意識する

else文を使わずにソースコードを書く努力する。

return で必要な処理がおわったら返却することで、return以降を意識しなくてもいい。

またネストが浅くなるため読みやすい。

よく書いていた感じのPHPソース

function calc( a , b, type){
    ret = 0;
    if( is_numeric( a ) && is_numeric( b ) ){
        if( type == 1 ) {
            ret = a + b;
        } else if ( type == 2 ){
            ret = a - b;
        }
    }
    return ret;
}

ガード節を意識したPHPソース

function calc( a, b, type ){
    if( ! is_numeric( a ) || ! is_numeric( b ) ) return 0;
    if( type == 1 ) return a + b;
    if( type == 2 ) return a - b; 
    return 0;
}

値チェックをよく関数の先頭に書くようになり、ネストが浅くなったのが実感できています。

if文が長くなるのを防げるので、インデントで閉じカッコまで探す手間が減ったのもありがたいです。

表示でもっと関数を利用

どうもちょっとした処理であればそのまま書いていたが、見た目的に関数化して値を変換したほうが見やすいことを理解してきました。

str = "";
if ( a == 1 ) {
    str = "aです。";
} else if ( a == 2 ) {
   str = "bです。";
} 
echo str;

calendar_date_str = "";
if( ! empty( calendar_date ) ) calendar_date_str = date( 'Y年m月d日', strtotime( calendar_date ) ); 
echo $calendar_date;

echo dispString( a );
echo dispCalendar( calendar_date );

function dispString( a ){
    if( a == 1 ) return "aです。";
    if( a == 2 ) return "bです。";
    return "";
} 
function dispCalendar( calendar_date ){
    if( empty( calendar_date ) ) return "";
    
    return date( 'Y年m月d日', strtotime( calendar_date ) );
}

単純に表示部分がまとまっているため、見やすいかなと思っています。