wordpressでの関数の書き方が正しいのかどうか

Question

wordpressで期間で表示を分けるphpを作って、それを関数にしてみました。
動くは動くのですが書き方としてこの書き方は正しいかどうか判断できずもっといい書き方があったら教えてもらえると幸いです。

作った関数（function.php）

function f_period($before, $after){
  date_default_timezone_set('Asia/Tokyo');
  $today = date('Ymd');
  $branch = ($before <= $today && $today <= $after);
  return $branch;
}

テンプレートファイルに書くコード

<?php if(f_period('20181001', '20181031')): ?>
10月中の表示
<?php elseif(f_period('20181101', '20181130')): ?>
11月中の表示
<?php endif; ?>

追記

ご指摘ありがとうございます。
目的としては、「任意の期間を指定」できる条件分岐を関数にする事が目的です。
ただ、条件分岐を関数にする記事が見つからずとりあえず作ってみたら動くは動くのですが上記記述のようなコードで大丈夫なのか不安になったので質問させていただきました。

気になっているのは以下のような書き方だと、不格好な気がして他の書き方があるのかどうか知りたいです。
「$branch = ($before <= $today && $today <= $after);」

Comments

[papinianus]

正しいというのはどういう観点ででしょうか？

[m.ts10806]

要件は具体的に書いてください。「コード見て理解して」というのは横着しすぎているように思います。仕様を把握しているのは質問者さんだけなので。

[m.ts10806]

「その月を期間とする」なのか「任意の期間を指定できる」なのかでも作りが全く変わってきますので。

Answer 1

気になっているのは以下のような書き方だと、不格好な気がして他の書き方があるのかどうか知りたいです。
「$branch = ($before <= $today && $today <= $after);」

どちらかというと呼び出し側の方が気になります。
テンプレートのif文が仮に12パターンあったら、12回f_period関数をコールするのでしょうか？

仮に私が実装するとしたら、以下のようにします。

function get_month() {
    date_default_timezone_set('Asia/Tokyo');
    return date('m');
}

function load_contents($month) {
    // おそらく定期的に書き換えるので、テンプレートに組み込まないほうがよさそう
    if ($month == 10) {
        // 10月中の表示
    }
    if ($month == 11) {
        // 11月中の表示
    }
}

$month = get_month();
load_contents($month);

1つの参考としてどうぞ。

Comments

[hatao]

ありがとうございます。
とても参考になりました。