Array Does not print....sigh.

Anon

Hi folks...Here's a couple of code clips. Trying to print the contents of an array two different ways...although I get the proper size with a sizeof() I can't seem to get any output... I would appreciate any help you all could throw my way.

Thanks

function Get_Array($subcat, $main_cat)
{
$cat_query = ("SELECT sub_cat FROM shoot_cat WHERE cat='$main_cat'");
$cat_res = mysql_query($cat_query);

while ($myrow=mysql_fetch_array($cat_res))
{
$temp=$myrow[sub_cat]."<br>";
$cats=explode(",",$temp);
}

$cats[]="test";
$ar_len=sizeof($cat_array);
		while (list ($value) = each ($cats))
	{	
		echo $value."<br>";
	}

echo "this worked";
return $cats;

}

function Delete_Sub($sub_cat, $main_cat)
{
$cat_array=Get_Array($subcat, $main_cat);
$ar_len=sizeof($cat_array);
for ($i=0; $i < $ar_len; $i++)
{
echo $i." ";
echo $cat_arrary[$i]." ";
}

Anon

First, you pass a variable into Get_array() called $subcat that doesn't appear to be used?

Then, you select one field (sub_cat) that you later explode() into your $cats array. I will take your word for it that the sub_cat field contains a comma seperated list of values. I also note that you concatenate " " onto the $temp string prior to the explode(); so I assume that you want this to end up concatenated to the last element of your array? It also looks like you only expect to get one row back from your querry, otherwise $cats will end up only having the values from the last row returned.

I am making these observation only so that you can refute or clarify my clumsy assumptions as to what this script is supposed to be doing; please bear with me...

You then add "test" as the last value to your array (presumably for exactly that reason) and set $ar_len to the size of $cat_array which has not been previously initialized or populated anywhere in the function. This doesn't matter because you don't use it anywhere else...

You then use the list() and each() functions in a while loop to output the contents of the $cats array. If $cats has values in it, the only possible output for this loop will be a list of numbers from 0 to the number of elements in the array. This is because the each() function returns an array of key/value pairs, not the individual values of your array. change "list($value)" to "list($key, $value)" to get the array values into $value. If nothing is printing at this point, you do not have a properly populated array. In any case, you should at least get the "test" to print...

The second function is fine, except for the fact that the closing "}" is missing for the function. If your first function is populating $cats correctly, then this function should print the list of values.

Let me know if my assumptions were correct and exactly what output you are getting from the script and we'll take it from there.

-- Rich Rijnders
-- Irvine, CA US

Anon

$cat_res = mysql_query($cat_query);

and what about the link_id ??? I assume it should look like this:

$cat_res = mysql_query($cat_query, $link_id);

Maybe thats the reason why you don't have any output ...

cya Rob

Anon

The link ID is not necessary. $temp is definitely receiving information. I can print the comma deliminated string that results from the query. The explode function used is almost verbatim compared to an earlier instance from a different function which adds to the array. I can add and delete array elements, but to this point have not been able to print any of those array elements.

Regards.

Rob.D wrote:

$cat_res = mysql_query($cat_query);

and what about the link_id ??? I assume it should look like this:

$cat_res = mysql_query($cat_query, $link_id);

Maybe thats the reason why you don't have any output ...

cya Rob

Anon

Key/ value pairing worked. I still do not understand why I cannot simply do this:

echo "$foo_arrary[0]";

Thanks for the help....

Regards.

Anon

Nother question -- I get a leading comma when I implode the array...

$subcat=implode($cat_array,",");

which results in a string like this:
$subcat=,foo1,foo2,foo3,foo4;

Then when I print with the while for the key/value, the first element of the array is blank. Which is a bit awkward since I am populating a select box with it.

Anyway, thanks for the assistance...

Regards.

Anon

Accessing an array element with $foo_array[0] is perfectly acceptable. Are you saying that your second function doesn't produce the expected output?

Anon

The implode() produces a string with a leading comma becuase the first element of your array is blank. The two problems are related.

Following the code snipets you posted, I can only conclude that there is a leading comma in your database column "sub_cat"; but since you didn't post the entire script I can't be sure about that.

Anon

I cannot print the contents of the array using echo $foo[1]; . There is no result.

Anon

Post or email me the script and I will look at it, because referencing the array via vector should always work.

rich@1stsys.com

-- Rich --

Anon

This is turning out to be quite a thread...

OK, removed leading comma by checking result of the query. Leading comma was getting added when I started the arrary because I pull from the database without checking...basically adding "" to the array. Good catch. The whole script is at www.tempestdigital.com/shoot/admin/add_cats.php. Truly a lot of effort to produce menu structures...

Still confused about referencing arrarys...
while (list ($key, $value) = each ($cat_array))
{
echo "Key: ".$key." ";
echo "Value: ".$value." ";

if ($value == $sub_cat)
{
echo "YES, Value Matches";
unset ($cat_arrary[$i]);
}
echo $nctr++;
}

sort ($cat_array);
$subcat=implode(",",$cat_array);

$query = ("UPDATE shoot_cat SET sub_cat='$subcat' WHERE cat='$main_cat'");

Output:
Subcat: Gloves
Key: 0
Value: Gloves
YES, Value Matches

(Only one item in the array...) This does not unset the value for the array element...

I do appreciate all the assistance...

Richard A. Rijnders wrote:

The implode() produces a string with a leading comma becuase the first element of your array is blank. The two problems are related.

Following the code snipets you posted, I can only conclude that there is a leading comma in your database column "sub_cat"; but since you didn't post the entire script I can't be sure about that.