I've been staring at this code too long...

maxxd

And now I can't even begin to figure out what's wrong... Any help at all would be greatly appreciated. I'm fairly new at PHP coding, so it's probably something extremely simple that I'm missing.... Here's the code:

//Set the date default....
$dateArray = getdate();
$mo2Use = $dateArray['mon'];

//Initialize calendar information....
$month = array(1=>"January", "February", "March", "April", "May", "June", "July", "August", "September", "October", "November", "December");
$startDays = array(1=>3, 6, 6, 1, 4, 0, 2, 5, 1, 3, 6, 1);
$numDays = array (1=>31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);


//initialize build data...
	$moStrt=$startDays[$mo2Use];	

//Get the band data to display...
	$qry="SELECT * FROM tblbands WHERE playMonth = '".$mo2Use."'";
	$bands=mysql_query($qry, $theatre);
	$bandsRow=mysql_fetch_assoc($bands);

//build calendar headers...

//Blah-de-blah...

/*	Populate calendar...
	$cdn = Current Day Number...
	$w = week count
	42 is the most day cells in any given calendar month - 7 days per week, 6 weeks per month.... */
	$cdn=1;
	$w=1;
	for ($i=0; $i<42; $i++) {
//Open table cell...
		print ("<td valign=\"top\">");
//Check to see if the current cell count is before the first day of the month or after the last - if so, print empty cell...
		if(($i<$moStrt) || ($cdn>$numDays[$mo2Use])){
			print "&nbsp;</td>\n";
		} else {
//If cell counter is between first and last day of the month, print the day number in the cell...
			print "$cdn";
		do {
			if($bandsRow['playDay'] == $cdn) {
				print ("<br><a href=\"".$bandsRow['webLink']."\" target=\"_blank\">".$bandsRow['bandName']."<br>".$bandsRow['musicStyle']."</a></td>\n");
			}else{
				print "</td>";
			}
		} while ($bandsRow=mysql_fetch_assoc($bands));
			$cdn++;
		}
//If current week day counter is greater than 7, start a new row and reset week day counter...
		if($w==7){
			print("</tr>\n<tr class=\"calNums\" height=\"61\">\n");
			$w=1;
		} else {
//Increment week day counter...
			$w++;
		}
	}
//Close the final row and table...

Now, it seems like it should work - and it does for the first item in the database that has a playDay value that matches $cdn. Anything beyond the first record, however, it totally ignores.

Any ideas?

:edit: I realized that I had the variable incorrect in my While statement. Now that I fixed it, the code doesn't output any data, regardless of the date. I broke it by fixing the variable. That stinks.

PS - Much thanks in advance for any and all help!

m_tt

I have never seen an array setup like that, i'm curious why you put 1=> on each one of your arrays?

m_tt

I'm guessing the 1=> sets the first array element to 1 instead of 0..?

Also, I noticed you call the $bandsRow query twice.. don't think it is working when you call it a second time.

maxxd

m@tt - hadn't even noticed that I called the d@mned query twice - I'll give that a shot. Thanks!

I set up the Arrays that way basically because I'm easily distracted. 😃 I found it easier to simply begin the array at index 1 and refer to the months with the actual value than to add 1 to it later. Had a bit of an idea that the code might get a bit unruly, and figured that little bit would be forgotten more than once. Not sure if that made any sense at all. It works in the dank recesses of my mind, tho'!

Thanks for the help. I'll give a shot to killing one of the query calls.

maxxd

This is making no sense to me at all. I got it working, but I don't quite understand why. I created a class and new object to populate with a while() statement. No biggie. Here's what's wierd - I had to change this chunk of code -

        if(($i<$moStrt) || ($cdn>$numDays[$mo2Use])){
            print "&nbsp;</td>\n";
        } else {
//If cell counter is between first and last day of the month, print the day number in the cell...
            print "$cdn";
        do {
            if($bandsRow['playDay'] == $cdn) {
                print ("<br><a href=\"".$bandsRow['webLink']."\" target=\"_blank\">".$bandsRow['bandName']."<br>".$bandsRow['musicStyle']."</a></td>\n");
            }else{
                print "</td>";
            }
        } while ($bandsRow=mysql_fetch_assoc($bands));
            $cdn++;
        }

tp this -

	if(($i<$moStrt) || ($cdn>$numDays[$mo2Use])){
		print "&nbsp;";
	} else {
		print $cdn;
//Check if the current date is the date of a performance...
		for($s=0; $s<$totRows; $s++) {
			if($band[$s]->day == $cdn) {
				print ("<br>Woo-hoo!");
				continue;
			}
		}
	print ("</td>\n");
	$cdn++;
	}

😕 It's almost like the program was running both the if and the else clauses, no matter what. I kept getting multiple </td> calls at each line. Just not working in my brain right now. If anybody can explain why this happened, I'd love to hear it.

tekky

i know this doesnt answer your main question, but just a suggestion to cut down on code lines...

echo date("t", strtotime("1 $month $year")); //echo number of days in that month

echo date("w", strotime("1 $month $year")); //echo numerical value of start day of that month ie: 0=sunday 6=saturday

of course, just add 1 to your "w" date value, to get 1-7 instead of 0-6 😉 that will completely eliminate all 3 arrays... and make the code more portable between years....

more on the date() function can be found at that link

tekky

        if(($i<$moStrt) || ($cdn>$numDays[$mo2Use])){
            print "&nbsp;</td>\n";
        } else {
//If cell counter is between first and last day of the month, print the day number in the cell...
            print "$cdn";
        do {
            if($bandsRow['playDay'] == $cdn) {
                print ("<br><a href=\"".$bandsRow['webLink']."\" target=\"_blank\">".$bandsRow['bandName']."<br>".$bandsRow['musicStyle']."</a></td>\n");
            }else{
                print "</td>";
            }
        } while ($bandsRow=mysql_fetch_assoc($bands));
            $cdn++;
        }

once you determined that $i was less than the starting day of month and current day number is less than days in the month.... you loop UNTIL you run out of database query data, and each of those loops where bandPlayday != curr day number it prints another </td> for each row in the database...

and your new code, you correctly did with a for loop, instead of a misplaced do/while 😉

maxxd

tekky - thanks for the replies! Somehow I totally missed the "t" switch for number of days in the month.. could've saved lots of typing there!

Thanks for the help!

tekky

hehe no problem 😉 hope everything is solved now, if so "mark thread resolved 😉)