ssk score 286
2017/03/02 15:32 投稿
| PHPの書き方を効率的に書き換えたいです。 |
| ######説明 |
| 日付の比較を行っています。 |
| コード量を減らして可読性を上げたいのですが、皆さんでしたら以下のコードをどのように書きますか？ |
| ■条件 |
| <両方とも値がある> |
| `$dnw05_afterDate` = '2017/02/01' |
| `$dnw05_beforeDate` = '2017/02/28' |
| <afterDateのみ値がある> |
| `$dnw05_afterDate` = '2017/02/01' |
| `$dnw05_beforeDate` = '' |
| <beforeDateのみ値がある> |
| `$dnw05_afterDate` = '' |
| `$dnw05_beforeDate` = '2017/02/01' |
| ```PHP |
| if(is_string($dnw05_afterDate) && $dnw05_afterDate !== ''){ |
| $query->where(function ($exp, $q) use($dnw05_afterDate,$dnw05_beforeDate){ |
| return $exp->addCase( |
| [ |
| $q->newExpr()->lte('dnw05', $dnw05_beforeDate), |
| $q->newExpr()->between('dnw05', $dnw05_afterDate, $dnw05_beforeDate), |
| $q->newExpr()->gte('dnw05', $dnw05_afterDate), |
| ] |
| ); |
| }); |
| } |
| ``` |
| ######自分のレベルで考えられるコード |
| ```PHP |
| if(is_string($dnw05_afterDate) && $dnw05_afterDate !== '' && is_string($dnw05_beforeDate) && $dnw05_beforeDate !== ''){ |
| //両方に値がある場合の処理 |
| }elseif{ |
| ・ |
| ・ |
| ・ |
| } |
| ``` |
| と何と不恰好な方法しか考えられず、、、 |
| と何と不恰好な方法しか考えられず、、、 |
| #####追記 |
| 現状のコードから生成されるのがこちらのSQL文です。 |
| ```SQL |
| WHEN dnw05 <= '2017/01/31' THEN 'SMALL' |
| WHEN dnw05 BETWEEN '2017/01/31' AND '2017/02/18' THEN 'MEDIUM' |
| WHEN dnw05 >= '2017/02/18' THEN 'LARGE' |
| ``` |
| 2つとも値がある場合 |
| ```SQL |
| WHEN dnw05 BETWEEN '2017/01/31' AND '2017/02/18' THEN 'MEDIUM' |
| ``` |
| afterDateのみ値がある場合 |
| ```SQL |
| WHEN dnw05 <= '2017/01/31' THEN 'SMALL' |
| ``` |
| beforeDateのみ値がある場合 |
| ```SQL |
| WHEN dnw05 >= '2017/02/18' THEN 'LARGE' |
| ``` |
| と値によって、出力を制御したいです。 |
