while文で365日を書き並べたい

Question

学習2日目。while文を学んでいます。
1/1,1/2,1/3……………12/28,12/29,12/30,12/31
と365日分書き並べたいです。
2,4,6,9,11月は30日まででそれ以外は31日までとしました。

<?php
$month=1;
while($month<=12){
    $day=1;
    If ($month==2 or $month==4 or $month==6 or $month==9 or $month==11){
        while($day<=30){
            print $month."/".$day.",";
            $day+=1;
        }
    $month+=1;
    }
    else while($day<=30){
        print $month."/".$day.",";
        $day+=1;
    }
    $month+=1;
    
}
?>

結果は、３０日月の翌月がなぜか消えてしまいました。
monthに１足す動作が二回行われてしまっているのかと思われます。
なぜ動作が二重に起こってしまったのか、どうしたら修正できるのか。
粘ったのですがどうしてもわからず、ご助言お願いします。

Comments

[maisumakun]

「2月30日」が出るままで大丈夫ですか？

Answer 1

とりあえず結果から。
かっこも調整しています。
気になった箇所コメントで書いておきました。

<?php
$month=1;
while($month<=12){
	$day=1;
	If ($month==2 or $month==4 or $month==6 or $month==9 or $month==11){
		while($day<=30){
			print $month."/".$day.",";
			$day+=1;
		}
	    // $month+=1;//ここいらない（翌月が消える原因）
	}
	else{//else while？
		while($day<=31){//ここ31だと思います
			print $month."/".$day.",";
			$day+=1;
		}
	}
	$month+=1;
}
?>

備考

自分だったら、月の限度は決まってますし、分岐するほどでもないかなと思うので、こう書いちゃいますが、参考程度に。

<?php
$mArr = array(31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
$month=1;
while($month<=count($mArr)){
	$day=1;
	while($day<=$mArr[$month-1]){
		print "$month/$day<br>";
		$day++;
	}
	$month++;
}
?>

Comments

[kszk311]

ありゃ解決済みになってる

[Tensaka]

ご丁寧な回答ありがとうございます。
なんとコメントまでつけていただいて！
この方法だとmonth足す１の文は２つ書かずに済みますね。
いろんな方法があるだなんて、元理系マンとして感激です。

Answer 2

最後の$month+=1; がifブロックの外にあるから、$month==2の時は二回加算されてしまう。

else whileなどと{}を省略してしまっているのが敗因です。省略せずに書きましょう。

Comments

[KojiDoi]

あと二回目の条件は30じゃなくて31だよね。

[Tensaka]

うわああ！できました！！！！
Ifの外側になっちゃってました！
ありがとうございます。
おっしゃり通りで二回目の条件も訂正しました。
敗因まで書いて頂いて、あたたかい回答とても参考になりました。

Answer 3

PHPでorは||を使います。

また、こんなコードでも実現できます。

<?php
$month=1;
while($month<=12){
    $day=1;
    while($day<=30){
        print $month."/".$day.",";
        $day+=1;
    }
    If (in_array($month, array(1,3,5,7,8,10,12))){
        print $month."/".$day.",";
    }
    $month+=1;
}

Comments

[Tensaka]

うひゃー！みやすいっっ！
こんなシンプルになっちゃうんですね！
もしかしてこれが最適解なのでしょうか。
arrayですか、いろんな技名みたいのが出てきますね。
ご回答ありがとうございます！！

[salud]

最適解が何かは分かりませんが、、

これは、arrayの配列の中に$monthの月が無いか調べてます。

ただコードを短くするのであれば、while文よりfor文ですかね、
3行が1行にまとまります。

あと、メンテナンスの観点でいうと
print文も関数とか使って一箇所にまとめたいですね。
修正が入ったときに、片方だけし忘れるなんて事も起こりえますから。。。

がんばってください！！