[RESOLVED] My Calendar Issues

KingWylim

Dear Coders,

I am having a slight problem with the following calendar code. What I would like it to do is start the 1st on the corresponding day of the week. So far I have not been able to get my code to correspond. If anyone here is able to assist with this small issue I would greatly appreciate it. Here is the glorious code.

<?php

/**
Calendar Program
This program was designed for the use of PRA International Only
@version 1.0
@copyright Alex Autrey August, 31 2005
*/

$day_num = date('j');
$year = date('Y');
$wday = date('w');
$day_n_month = date('t');
$month = date('n');

$text_wday = array("Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat");
$text_month = array("", "January", "Febuary", "March", "April", "May", "June", "July", "August", "September", "October", "November", "December");

?>
<table border="1">
<tr>
<td colspan="7" align="center"><a href="?prev"><.</a> <? echo "$text_month[$month]"; ?><a href="?next">.></a></td>
</tr>
<?
echo '<tr>';
for($i=0; $i <= 6; $i++){
echo '<td align=center>';
echo "$text_wday[$i]";
echo '</td>';
}
echo '</tr>';

//Find out what day the first day of the month fell on.
//This gives us a starting location.
if($day_num==1){
$first_day = $wday;
} else {
for($i=$day_num; $i==1; $i--){
$wday--;
if($wday == -1){
$wday = 6;
}
}
$first_day = $wday;
}

echo '<tr>';
for($i = 0; $i <= $day_n_month; $i++){
if($wday == $first_day){
if($wday == 7){
echo '</tr><tr>';
echo '<td>';
echo $i;
echo '</td>';
$wday = 0;
} else {
echo '<td>';
echo $i;
echo '</td>';
}
} else {
echo '<td> </td>';
}
$wday++;
}
echo '</tr>';
?>

</table>

Weedpacket

//Find out what day the first day of the month fell on.
//This gives us a starting location. 
$first_day = date('w', mktime(0,0,0, $month, 1));

Should be easier to debug than the loop.

KingWylim

I will try this out and let you know how it goes. Thank you.

KingWylim

Well that didn't fix it exactly. I'm trying to figure out how to get the 1st to fall on Thursday which is what it did for this month. But then when I switch months it has to do the same thing. I'm not sure how to accomplish this. Any help on this issue would be greatly appreciated. Thank you.

KingWylim

If anyone could give me a little more input I would appreciate it. This issue has not yet been resolved.

kburger

KingWylim wrote:
I'm trying to figure out how to get the 1st to fall on Thursday

Huh?

You have no control over whether or not the 1st day of the month is Thursday. Maybe you should explain a little more what you're trying to do. Obviously, it's not what we thought. What are we missing?

Weedpacket

Awfully sorry. I selfishly decided to go to sleep for a few hours.
You may want to think your logic here (reformatted from the original to make things clearer. If your code is already indented like this, consider using [php]..[/code] tags around your code),

if($wday == $first_day)
{
	if($wday == 7)
	{
		echo '</tr><tr>';
		echo '<td>';
		echo $i;
		echo '</td>';
		$wday = 0;
	}
	else
	{
		echo '<td>';
		echo $i;
		echo '</td>';
	}
}
else
{
	echo '<td> </td>';
}

For one thing, you only print the date for the first day of the month (which should be 1, not 0, but that's because your loop starts at 0, not 1). Also, $wday is never 7 (date('w') returns 0 to 6).

You could start by working out how many blank cells to insert at the start, and put those in the table before looping through all the days in the month (which is presumably what you want to do anyway). If you want the month to start on a Sunday, you add 0 empty cells, on a Monday, you add 1 empty cell, on a Tuesday you add 2 empty cells.... You will need to need to keep count of how many cells you've added so that when you are going through the actual days of the month you remember to start a new row at the right time; and then afterwards add a few more empty cells to take the total number in the table up to a multiple of seven (just to be tidy).

Looking at some already-existing code that does this (e.g., PEAR's Date class) might give you some further ideas; such as putting the dates into an array instead of trying to build a big HTML string right from the beginning.