phpでif文を使ってみたのですが…

西暦を入力するときに、月によって表示される日を異なるように以下のコードを書いてみました。
sample8.phpというページで、適当な数字を渡します。その数字によって表示される日付を変えたいのですが、うまくいきません。

php
1<?php
2  $i = $_POST['number'];
3?>
4
5<html>
6  <body>
7    <h1>西暦フォーム</h1>
8    <label for = 'year'>西暦</label>
9    <select name = 'year'>
10      <?php
11        $now = date('Y');
12        for($i = 1950; $i <= $now; $i++)
13        {
14      ?>
15          <option value = '<?php echo $i; ?>'><?php echo $i; ?>
16      <?php
17        }
18      ?>
19    </select>年
20    <select name = 'month'>
21      <?php
22        for($iii = 1; $iii <= 12; $iii++)
23        {
24      ?>
25          <option value = '<?php echo $iii; ?>'><?php echo $iii; ?>
26      <?php
27        }
28      ?>
29    </select>年
30    <select name = 'date'>
31      <?php
32        if($i === 1||$i === 3||$i === 5||$i === 7||$i === 8||$i === 10||$i === 12){
33        for($ii = 1; $ii <= 31; $ii++)
34        {
35      ?>
36          <option value = '<?php echo $ii; ?>'><?php echo $ii; ?>
37      <?php
38        }
39      }
40      ?>
41      <?php
42        if($i === 4||$i === 6||$i === 9||$i === 11){
43        for($ii = 1; $ii <= 30; $ii++)
44        {
45      ?>
46          <option value = '<?php echo $ii; ?>'><?php echo $ii; ?>
47      <?php
48        }
49      }
50      ?>
51    </select>年
52    <?php
53      if($i === 2){
54      for($ii = 1; $ii <= 28; $ii++)
55      {
56    ?>
57        <option value = '<?php echo $ii; ?>'><?php echo $ii; ?>
58    <?php
59      }
60    }
61    ?>
62  </body>
63</html>
64

　一番上の$iが前のページにて受けた入力です。本当は同じページで月を入力して、その数字を反映したいのですが、ちょっとレベルが高そうだったので、今回の様な形にしました。
今のままだと、日付を選択するところだけ表示されません。
どなたか、よろしくお願い致します。

thyda.eiqau

2019/09/07 15:26

どううまくいかないのですか？また、どうなってほしいのですか？修正の前後の姿がわからないと回答のしようがないです。質問本文は修正できますので、追記をお願いします。

rink

2019/09/07 15:49

了解しました。

takasima20

2019/09/07 15:52

$i を使いまわしてるからって話では?

thyda.eiqau

2019/09/07 16:06

本筋とは関係ないですが、日付を入力させるのが<select>なのは正直ユーザーから見るとつらいです。2000年を入力したいときは50個も不要な項目をずらーっと流していかないといけないですよね。 <input type="number" min="1950" max="<?php echo date('Y'); ?>"> にするとか、 <input type="date"> とかにしたほうがよくないですか？

rink

2019/09/07 16:18

@talasima20さんご回答ありがとうございます。使いまわしていることに気づいていませんでした。ですが、変えてみてもできませんでした…。 @thyda.eiqauさんご回答ありがとうございます。なるほど、そうですよね、わかりづらいですよね… 勉強に使っていたものをちょこっと変えたものなので…ご指摘ありがとうございます。

行動規範の内容に同意します

回答2件

ベストアンサー

ひとまずご提示のコードの不自然なところ・文法エラーを直したのが次のコードですが……

$i , $ii , $iii などの何を示しているのかが判断しづらい変数名を使うのはなるべく避けましょう。 $i はあくまで「整数のカウンター」以上の意味を持たせないほうがよいです。変数の値が年を示すのであれば $year とかにしたほうがよいです。

前のページ？で月を選んでいるのはいいですし、このページで年月日のフォームがあってユーザーは年と月と日を選ばせたいのはわかります。

たとえば前のページで1月を選んだとしますよね。で、このページでは選択された $_POST['number'] の値に従って、日は31日まで表示される。
もしこのページの月の欄で2月が選ばれた場合、2月31日などのありえない値が選択されうる形ですが、大丈夫なのでしょうか。

php
1<?php
2
3$i = intval($_POST['number'], 10);
4$thisyear = intval(date('Y'), 10);
5$is_leap_year = boolval(date('L'));
6
7$last_day = 31;
8if(in_array($i, [4, 6, 9, 11], true)) $last_day = 30;
9if($i === 2) {
10  $last_day = 28;
11  if(is_leap_year) $last_day = 29;
12}
13
14?><!doctype html>
15<html>
16  <head>
17    <meta charset="utf-8">
18    <title></title>
19  </head>
20  <body>
21    <h1>西暦フォーム</h1>
22
23    <label for='year'>西暦</label>
24    <select name='year'>
25      <?php for($y = 1950; $y <= $thisyear; $y++) : ?>
26      <option value="<?php echo $y; ?>"><?php echo $y; ?></option>
27      <?php endfor; ?>
28    </select>年
29
30    <select name='month'>
31      <?php for($m = 1; $m <= 12; $m++) : ?>
32      <option value="<?php echo $m; ?>"><?php echo $m; ?></option>
33      <?php endfor; ?>
34    </select>月
35
36    <select name = 'date'>
37      <?php for($d = 1; $d <= $last_day; $d++) : ?>
38      <option value="<?php echo $d; ?>"><?php echo $d; ?></option>
39      <?php endfor; ?>
40    </select>日
41  </body>
42</html>