[RESOLVED] Array created from mysql not being recognize as an array

[tha_ripmasta]

Hello everyone,

This is my first post, so i'm new here.
I have a coding problem i've been trying to solve for awhile now:

I'm created an attendance script, where the user chooses a date from a calender.

What i'm trying to do is display all the dates for example from March where the person was present.

Here's my problem:

I retrieve the date from the database like this, the date is stored in DATE format :

while ($row=mysql_fetch_array($result)) {

   $attend_date = stripslashes($row['attend_date']); // date

   $day = date("d", strtotime($attend_date));
   $month = date("m", strtotime($attend_date));
   $year = date("Y", strtotime($attend_date));

   $Values = array();
   $Values[] = $day;

  }

And when I run this code:

// Get the first day of the month
$month_start = $month."-"."1"."-".$year;

// determine how many days are in the last month.
if($month == 1){
   $num_days_last = cal_days_in_month(0, 12, ($year -1));
} else {
    $num_days_last = cal_days_in_month(0, ($month -1), $year);
} 

// determine how many days are in the current month.
   $num_days_current = cal_days_in_month(0, $month, $year); 

// Build an array for the current days
// in the month
for($i = 1; $i <= $num_days_current; $i++){
   $num_days_array[] = $i;
   //echo "$i<br>";
} 


while (list ($key, $val) = each ($Values)) {

if (in_array ($val, $num_days_array)) {


echo "$val"; // to display present days in the month

$diff =array_merge(array_diff($val, $num_days_array),array_diff($num_days_array,$val));

echo "$diff"; // to display absent days in the month

 }

}

When I run this I get this Error:

Warning: array_diff(): Argument #1 is not an array in \htdocs\suaca2\modules\attendance.php on line 235

Warning: array_diff(): Argument #2 is not an array in \htdocs\suaca2\modules\attendance.php on line 235
Array15

I think Arrays is my problem ... maybe this $Values[] = $day;

CAN SOMEONE HELP ME PLEASE.. THANKS IN ADVANCE

[sneakyimp]

i think your prob is with mysql_fetch_array. this results in arrays that have numeric indices but no associative (i.e, plain text) keys.

try mysql_fetch_assoc instead.

[tha_ripmasta]

sneakyimp thank you for replying, but mysql_fetch_assoc didn't do anything, i still have the same problem....

CAN ANYONE TRY AND HELP ME AGAIN!!!

[tsinka]

Hi,

how does the SQL query look like that fetches the attend_date info ?

If it just selects the rows for a given month then the $Values array already contains all the days of the given month where the person was present.

In order to get all the days where the person wasn't present, do something like:

$arrDaysNotPresent = array_diff($num_days_array, $Values);

The resulting array should contain all the days where the person was absent.

To print the info do something like:

// present days:
foreach ($Values as $Day) {
   echo $day."<br>\n";
}

// absent days:
foreach ($arrDaysNotPresent as $Day) {
   echo $day."<br>\n";
}

I hope I didn't completely misunderstand you.

Thomas

[Installer]

"$val" is not an array. "array_diff()" takes arrays as its arguments. So your error message is precise and accurate..

"$Values" will contain at most one element, since you're reinitializing it on every iteration of the "while" loop.

...mysql_fetch_array...results in arrays that have numeric indices but no associative...

That's incorrect. mysql_fetch_array()

[Houdini]

i think your prob is with mysql_fetch_array. this results in arrays that have numeric indices but no associative

Installer is correct. As a matter of fact the mysql_fetch_array() function given no second parameter will default to both (meaning it returns both an indexed (or numeric array) and an associative array) Using

mysql_fetch_array_($query),MYSQL_ASSOC)

wil produce the same thing as

mysql_fetch_assoc($query)

only real difference being the length of the function but the result would be the same.

[Installer]

...it returns both an indexed (or numeric array) and an associative array...

Not quite. 🙂 It returns one array which contains both numeric and associative indexes.

[tha_ripmasta]

Thank you tsinka, Your code did exactly what I wanted. You understood me correctly and made my code more simple then what I was trying.

Also thanks to Installer and Houdini for the knowledge.

[tha_ripmasta]

I thought i was finish with this post, but i got another problem with the script. Adding the code from tsinka to my script worked great, but now my results keeping looping the dates.

HERE IS MY FULL CODE NOW.

                              $_GET['month'] = intval($_GET['month']);
			  $_GET['year'] = intval($_GET['year']);

			  $nid = $_GET['nid'];

			  if ($_GET['month'] != "" AND $_GET['year'] != "") {
			   $query = "SELECT * FROM attendance WHERE                MONTH(attend_date) = {$_GET['month']} AND YEAR(attend_date) = {$_GET['year']} AND studentid = '$nid' ORDER BY attend_date DESC;";
			   $result = mysql_query($query);
			   $num = mysql_numrows($result);



			   $query2 = "SELECT COUNT(*) FROM attendance WHERE MONTH(attend_date) = {$_GET['month']} AND YEAR(attend_date) = {$_GET['year']} AND studentid = '$nid';";
			   $result2 = mysql_query($query2) or die(mysql_error());

			   while ($row=mysql_fetch_assoc($result)) {

				$attend_date = stripslashes($row['attend_date']);
			    $pre_abs = stripslashes($row['pre_abs']);
			    $attend_option = stripslashes($row['attend_option']);

				$day = date("d", strtotime($attend_date));
			    $month = date("m", strtotime($attend_date));
			    $year = date("Y", strtotime($attend_date));

				$Values = array();
				$Values[] = $day;

				//reset($Values);
				//sort($Values);



				// Get the first day of the month
				$month_start = $month."-"."1"."-".$year;

				// determine how many days are in the last month.
				if($month == 1){
				   $num_days_last = cal_days_in_month(0, 12, ($year -1));
				} else {
				   $num_days_last = cal_days_in_month(0, ($month -1), $year);
				} 

				// determine how many days are in the current month.
				$num_days_current = cal_days_in_month(0, $month, $year); 

				// Build an array for the current days
				// in the month
				for($i = 1; $i <= $num_days_current; $i++){
					$num_days_array[] = $i;
					//echo "$i<br>";
				} 


				$arrDaysNotPresent = array_diff($num_days_array, $Values);


			    $totpresent = count($Values);
			    echo "<b>$totpresent day(s) present</b><br><br>";
				// present days: 
				foreach ($Values as $Day) {
				   echo $Day.",";

				 } 

				echo "<br><br>\n";
				$totabsent = count($arrDaysNotPresent);
			    echo "<b>$totabsent day(s) absent</b><br><br>";
				// absent days:
				foreach ($arrDaysNotPresent as $Day) {
					echo $Day.",";
			      } 

			    }
			   }

WHEN I RUN THIS SCRIPT AND MY DATABASE HAS 1 DATE FOR MONTH JULY IT OUTPUTS THIS:

1 day(s) present

15,

27 day(s) absent

1,2,3,4,5,6,7,8,9,10,11,12,13,14,16,17,18,19,20,21,22,23,24,25,26,27,28,

AND WHEN MY DATABASE HAS 2 DATES FOR MONTH JULY IT OUTPUT THIS:

1 day(s) present

23,

30 day(s) absent

1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,24,25,26,27,28,29,30,31,1 day(s) present

03,

62 day(s) absent

1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,

DOES THIS PROBLEM HAS TO DO WITH THE WHILE LOOP, PLEASE HELP ME.

[tsinka]

There are several problems:

1. move the complete code where you calculate the $num_days_array out of the while loop because for n present dates the array will be filled n times
2. remove the $Values = array(); from inside the while loop because that resets the $Values array each time so you'll have exactly one value in that array no matter how many matching records you have in the tables.
3. Move the code where you print the results out of the while loop to the end.

=> calculate $num_days_array in front of the while loop
=> just calculate $Values inside the while loop
=> print the results after the while loop
=> put $Values = array(); somewhere at the beginning of the script to preset that variable.

Thomas

[tha_ripmasta]

Thanks alot tsinka. My script is working now.