if文をternaryにすると可読性があがる(こともある

2022/11/04に公開約2,200字

基本

if($priceA > $price B){
    $message = 'Aの方が高い';
}else{
    #message = 'Bと同じかBの方が高い';
}

というコードがあったら

$price = $price > $price 
    ? 'Aの方が高い'
    : 'Bと同じかBの方が高い';

と書けるよというのがこのお話の基本
(?とか:を先頭にしておくと、三項演算子の処理が続くよ、とわかるのでおすすめ)

循環複雑度が下がるので不具合の発生度が下がるはず

応用

基本があれば応用があるのが世の習い
もっと複雑な処理でもこんなふうに綺麗になる

汚いif文


if ($paymentTotal > $salesTotal) {
            $differenceBetweenPaymentsAndSales = $paymentTotal - $salesTotal;
            if ($price1) {
                $result1 -= $differenceBetweenPaymentsAndSales;
                if ($differenceBetweenPaymentsAndSales > $price1) {
                    $result2 += $differenceBetweenPaymentsAndSales - $price1;
                }
            } else {
                $result2 += $differenceBetweenPaymentsAndSales;
            }
        } elseif ($salesTotal > $paymentTotal) {
            $differenceBetweenSalesAndPayments = $salesTotal - $paymentTotal;
            if ($price2) {
                $result2 -= $differenceBetweenSalesAndPayments;
                if ($differenceBetweenSalesAndPayments > $price2) {
                    $result1 += $differenceBetweenSalesAndPayments - $price2;
                }
            } else {
                $result1 += $differenceBetweenSalesAndPayments;
            }
        }
return [
    'result1' => $result1,
    'result2' => $result2
];

読む気も失せたら、あなたの感覚は正しい
これを引き継いで改修する人は苦労することが明白
このコードを書いた人も、仕様を必死に再現しようとしただけ、きっと誰も悪くない😪

綺麗なternary

return $paymentTotal > $salesTotal 
        ?  [
            'result1' => $price1 
                ? - $differenceBetweenPaymentsAndSales
                : 0,
            'result2' => $price1 
                ? max(0, $differenceBetweenPaymentsAndSales - $price1)
                : $differenceBetweenPaymentsAndSales
        ] 
        : [
            'result1' => $price2 
                ? -$differenceBetweenSalesAndPayments 
                : 0,
            'result2' => $price2 
                ? max(0, $differenceBetweenSalesAndPayments - $price2) 
                : $differenceBetweenSalesAndPayments
        ];

行数も減ってだいぶ良くなったのでは無いでしょうかね
ちなみにmax(), min()を使うとternary自体の個数も減らせるんです。これはまた今度。

(一番いいのは、きちんとモデリングしてクラス切ることだということは忘れないで生きていこ

Discussion

ログインするとコメントできます