phpでカレンダー作成を行っています

Question

phpで今年のカレンダー作成をしています。
月末日の取得等書き込んでいきましたが、bodyで出そうとすると"Notice: Undefined offset:"と出てしまいます。
bodyの数字を変更したり、-1を試してみたりとしてみましたがそれでもうまくいきません。
どこかのコードが間違っていて$tableがうまく作動できていないのでしょうか。ご教示お願いします。

$table = array();
for($i = 1; $i <= 12; $i++){
	if(($i + 1) == 2){
		$lastday = 29;
	}
	elseif((($i + 1) == 4) || (($i + 1) == 6) || (($i + 1) == 9) || (($i + 1) == 11)){
		$lastday = 30;
	}
	else{
		$lastday = 31;
	}
}

$date = mktime(0,0,0,$i,1,2016);
$day = date('w',$date);

$table[$i] = '<tr>';
$cnt = 0;
for($d = 1; $d <=($day-1); $d++){
	$table[$i].= '<td></td>';
	$cnt++;
}

if($day ==0){
	for($d = 1; $d <= 6; $d++){
		$table[$i].= '<td></td>';
	$cnt++;
	}
}

for($c = 1; $c <= $lastday - 1; $c++){
	if($cnt == 0){
		$table[$i].= '<tr>';
	}
	$table[$i].='<td>'. $c .'</td>';
	$cnt++;
	if($cnt == 7){
		$table[$i].= '</tr>';
		$cnt = 0;
	}
}

while($cnt <= 7){
	$table[$i].='<td></td>';
	$cnt++;
}

$table[$i].= '</tr>';

<table>
<tr><th>1月</th></tr>
	<tr>
    <th>月</th>
    <th>火</th>
    <th>水</th>
    <th>木</th>
    <th>金</th>
    <th>土</th>
    <th>日</th>
</tr>
<?php echo $table[1]; ?>   
</table>

Answer 1

おそらく$iが各月になっていて、1~12月までforで回そうとしているのだと思うのですが、$iのforは下記で完結してしまっているので、その後のtableを作っている部分が12ヶ月分動作してないのだと思います、echo $table[13];としてtableが表示されるならこれでアタリだと思います。

for($i = 1; $i <= 12; $i++){
    if(($i + 1) == 2){
        $lastday = 29;
    }
    elseif((($i + 1) == 4) || (($i + 1) == 6) || (($i + 1) == 9) || (($i + 1) == 11)){
        $lastday = 30;
    }
    else{
        $lastday = 31;
    }
}

長くなるとfor等のループ処理を行うネストを間違え易いので、例えばtableを作成する部分は関数にしてしまう等コードの可読性を考慮してみてはいかがでしょうか。

Comments

[nakaoti-maguro]

ご回答ありがとうございます。$table[13]とやってみたら出ました！しかし<td>がずれてしまっているので、関数は少々苦手なのですが、視野に入れて計算式を直していこうと思います。
ありがとうございました。

[退会済みユーザー]

コードを書く際に
for(){//1~12loop
}//1~12loop end
のようにコメントを入れるようにするとネスト間違いの防止になりますよ

[nakaoti-maguro]

ネスト間違いの防止方法があったのですか！参考にさせていただきます！ありがとうございました！

[退会済みユーザー]

防止方法というほどの効果はないのですが・・・単純にコメントを入れて置くと現在行がどこのループの中なのかがわかりやすいという程度のことなので

tdがずれる件ですがとりあえず1箇所カレンダーの先頭を空白セルで埋めている部分ですが1日の曜日を求める際にdate('w',$date);を使っていますがこれは日曜0～土曜6を返すので、月曜日から始まるカレンダーの場合はdate('N',$date);を使って月曜1～日曜7の値を取得し
for($d = 1; $d <$day $d++){
$table[$i].= '<td></td>';
$cnt++;
}
としたほうがいいと思います。

Answer 2

うるう年の判断は
「西暦が４で割り切れ かつ １００で割り切れない、または４００で割り切れる」だと思います。
１０００年先、５０００年先は分かりません。が＿

munyaguさんがおっしゃるように、ｐｈｐに任せたほうが良いと途中で思った次第です。

Comments

[nakaoti-maguro]

ご回答ありがとうございます。今回は今年は29日があるからと言って省略しすぎたところがあったかもしれません。閏年ならず末日の判断の式を入れたカレンダーも作ってみようと思います。
ありがとうございました。

Answer 3

エラーとは関係ありませんが、閏年の判断が抜けています。
末日の判断はphpに任せた方が良いでしょう。

$date->format('t');

出来れば曜日の判断も、ループごとに日付変数を１日ずつ加算し、その変数から取得した方がロジックはすっきりしそうです。

$date->format('w')

日曜日からで0から始まりますので注意が必要ですが。

Comments

[nakaoti-maguro]

ご回答ありがとうございます。今年は29日閏年があったためその判断計算を抜いたカレンダーを作っていまいた。しかし、実際には閏年の計算式を入れなければいけないため省略しすぎたところはあったかもしれません。閏年の計算式を入れたパターンも作ってうまく動作させてみようと思います。
ありがとうございました。

Answer 4

ぱっと気づいたところを指摘します。
書かれているコードの下記部分は「12回変数 $lastday に別の値を代入（上書き）し続ける」というコードになります。なので、$lastday は必ず31になります（12月が31日までだから）。

for($i = 1; $i <= 12; $i++){
    if(($i + 1) == 2){
        $lastday = 29;
    }
    elseif((($i + 1) == 4) || (($i + 1) == 6) || (($i + 1) == 9) || (($i + 1) == 11)){
        $lastday = 30;
    }
    else{
        $lastday = 31;
    }
}

「$day = date('w',$date);」の後に続けて下記のように入れればよいのでは。

$month = date( 'n', $date );
if ( $month == 2 ) {
	$lastday = 29;
} elseif ( ( $month == 4 ) || ( $month == 6 ) || ( $month == 9 ) || ( $month == 11 ) ) {
	$lastday = 30;
} else {
	$lastday = 31;
}

Comments

[nakaoti-maguro]

ご回答ありがとうございます。$day = date('w',$date)の下に、とおっしゃいましたがなぜ下なのでしょうか。$tableの式を消して$monthのように入れるということでしょうか。

[kei344]

いえ、「for($i = 1; $i <= 12; $i++){」が月の日数を取得するところで終わっていたため、全月を出力していると思っていなかったので、確実に「$date」に値がある「$day = date('w',$date)」の下でやればいいのでは、と回答しました。
ということで勘違いなので、hide2e3rさん、munyaguさん、kamonohashiさんの回答を参考にしてください。全体のコードを見ていればよかったのですが、すいません。

[nakaoti-maguro]

いえいえ、ご回答くださっただけでも感謝です！ありがとうございました！