編集履歴

回答編集履歴

コードの修正

2018/04/20 17:47

投稿

スコア210

test CHANGED Viewed

@@ -8,9 +8,7 @@
 $option_month_list = range(1, 12);
-if(!empty($_GET['show_submit'])) {
+if($_SERVER['REQUEST_METHOD'] === 'GET' && !empty($_GET['show_submit'])) {
   $choosed_year = (int)$_GET['year'];
@@ -50,7 +48,7 @@
           <?php foreach ($option_year_list as $option_year): ?>
-            <option value="<?=h($option_year)?>" <?php if($option_year === $choosed_year) echo('selected="selected"'); ?>>
+            <option value="<?=h($option_year)?>" <?php if(!empty($choosed_year) && ($option_year === $choosed_year)) echo('selected="selected"'); ?>>
               <?=h($option_year)?>
@@ -64,7 +62,7 @@
           <?php foreach ($option_month_list as $option_month): ?>
-          <option value="<?=h($option_month)?>" <?php if($option_month === $choosed_month) echo('selected="selected"'); ?>>
+          <option value="<?=h($option_month)?>" <?php if(!empty($choosed_month) && ($option_month === $choosed_month)) echo('selected="selected"'); ?>>
             <?=h($option_month)?>
@@ -82,7 +80,7 @@
       ini_set('display_errors',1);    //エラー表示
-      if(!empty($_GET)) {
+      if($_SERVER['REQUEST_METHOD'] === 'GET' && !empty($_GET)) {
         $year=$_GET['year'];
@@ -162,8 +160,6 @@
 </html>
 ```

追記: htmlspecialcharsの説明

2018/04/20 17:47

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -18,6 +18,14 @@
 }
+function h($str) {
+  return htmlspecialchars($str);
+}
 ?>
@@ -42,15 +50,11 @@
           <?php foreach ($option_year_list as $option_year): ?>
-          <?php if($option_year === $choosed_year): ?>
-            <option value="<?=$option_year?>" selected="selected"><?=$option_year?></option>
+            <option value="<?=h($option_year)?>" <?php if($option_year === $choosed_year) echo('selected="selected"'); ?>>
-          <?php else: ?>
-            <option value="<?=$option_year?>"><?=$option_year?></option>
-          <?php endif; ?>
+              <?=h($option_year)?>
+            </option>
           <?php endforeach; ?>
@@ -60,15 +64,11 @@
           <?php foreach ($option_month_list as $option_month): ?>
-          <?php if($option_month === $choosed_month): ?>
-            <option value="<?=$option_month?>" selected="selected"><?=$option_month?></option>
+          <option value="<?=h($option_month)?>" <?php if($option_month === $choosed_month) echo('selected="selected"'); ?>>
-          <?php else: ?>
-            <option value="<?=$option_month?>"><?=$option_month?></option>
-          <?php endif; ?>
+            <?=h($option_month)?>
+          </option>
           <?php endforeach; ?>
@@ -232,6 +232,14 @@
+サンプルコード内で、
+h()で囲んである部分がありますが、
+phpをhtmlに出力する際は、いつも囲む必要があるようです。
 後、$_GETや$_POSTを利用する際は、!emptyで確認してから、
 処理しないと、初回読み込み時にエラーが出ます。

追記：注意点

2018/04/19 16:33

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -229,3 +229,15 @@
 <?php endforeach; ?>
 ```
+後、$_GETや$_POSTを利用する際は、!emptyで確認してから、
+処理しないと、初回読み込み時にエラーが出ます。
+後、表示ボタンを押すと、年月の数字が戻ってしまうようになっていたので、
+入力数字を維持するようにしてあります。見てみてください。

不必要な英語を日本語へ修正

2018/04/19 16:18

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -188,7 +188,7 @@
 **>>(3) プルダウンメニューの[年]をfor文で入れたいが方法がわからない。**
-sample codeでは、foreachを使用しました。
+私が書いたサンプルコードでは、foreachを使用しました。
 phpはhtmlに埋め込めるので、
@@ -198,7 +198,7 @@
-sample codeでは、短く書ける書き方を使用しています。
+サンプルコードでは、短く書ける書き方を使用しています。

追記：質問への回答

2018/04/19 16:07

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -165,3 +165,67 @@
 ```
+**>>(1) （日数を考慮した部分を除外）動作が不安定であるので、どの部分がどうおかしいのか見てほしい。**
+うるう年判定の部分で構文エラーが出てました。
+自動構文チェック機能付きのエディターを使えば、解決すると思います。
+おすすめは、vsc(visual studio code)です!
+**>>(2) 指定した月の日数の記述方法がわからず、if文でやってみたがうまくいかない。うるう年も考慮されない。**
+これは(1)と同じ問題だと思いますので、(1)を解決すれば、いいと思います。
+**>>(3) プルダウンメニューの[年]をfor文で入れたいが方法がわからない。**
+sample codeでは、foreachを使用しました。
+phpはhtmlに埋め込めるので、
+埋め込み方を知れば、解決すると思います。
+https://www.flatflag.nir87.com/for-273
+sample codeでは、短く書ける書き方を使用しています。
+```php
+<?php echo('hello!'); ?>
+<?='hello'?>//上と動きは、同じです。echo短縮の書き方です。
+```
+```php
+<?php foreach($sample_arr as $sample_val) { ?>
+<?=$sample_val?>
+<?php } ?>
+//上のコードは、下のようにも書くことが出来ます。
+<?php foreach($sample_arr as $sample_val): ?>
+<?=$sample_val?>
+<?php endforeach; ?>
+```

コードの修正

2018/04/19 16:04

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -17,26 +17,6 @@
   $choosed_month = (int)$_GET['month'];
 }
-class CalenderUtilities
-{
-  public function isLeapYear($year): bool
-  {
-    return (($year % 4 === 0) && ($year % 100 !== 0)) || ($year % 400 === 0);
-  }
-}
-$calender_utilities = new CalenderUtilities();
 ?>
@@ -114,7 +94,7 @@
         if($month==2){
-          if($calender_utilities->isLeapYear($year)) {
+          if((($year % 4 === 0) && ($year % 100 !== 0)) || ($year % 400 === 0)) {
             $lastday=29;
@@ -182,4 +162,6 @@
 </html>
 ```

コード修正

2018/04/19 15:47

投稿

fjaiofjawiefjaw

スコア210

test CHANGED Viewed

@@ -19,6 +19,24 @@
 }
+class CalenderUtilities
+{
+  public function isLeapYear($year): bool
+  {
+    return (($year % 4 === 0) && ($year % 100 !== 0)) || ($year % 400 === 0);
+  }
+}
+$calender_utilities = new CalenderUtilities();
 ?>
@@ -48,9 +66,11 @@
             <option value="<?=$option_year?>" selected="selected"><?=$option_year?></option>
-          <?php endif; ?>
+          <?php else: ?>
             <option value="<?=$option_year?>"><?=$option_year?></option>
+          <?php endif; ?>
           <?php endforeach; ?>
@@ -64,9 +84,11 @@
             <option value="<?=$option_month?>" selected="selected"><?=$option_month?></option>
-          <?php endif; ?>
+          <?php else: ?>
             <option value="<?=$option_month?>"><?=$option_month?></option>
+          <?php endif; ?>
           <?php endforeach; ?>
@@ -92,7 +114,7 @@
         if($month==2){
-          if((($year % 4 === 0) && ($year % 100 !== 0)) || ($year % 400 === 0)) {
+          if($calender_utilities->isLeapYear($year)) {
             $lastday=29;