dynamically declaring variables in a do while loop

kat_deery

can anyone tell me what's wrong with the following code

if($myrow = mysql_fetch_array($result)){
	$i=1;
	do{
		$size="size".$i;
		$height="height".$i;
		$width="width".$i;
		$$size = getimagesize("$myrow["image"]);
		$$height = $$size[1];
		$$width = $$size[0];
		echo "<img src='".$myrow["image"]."' height='".$$height."' width='".$$width."'/>";
			$i++;
	} while ($myrow = mysql_fetch_array($result));
}

I know that the variables $size1, $size2, etc. are being created but when I try to echo $$height and $$width I just get the value for $i, rather than the height and width of the image.
Any insight would be greatly appreciated.

vaaaska

$n_size = $size[$i];

Shouldn't $i = 0 since array keys start with zero?

kat_deery

it doesn't matter what i starts as, it could start at 100, it's just to create a differently named variable for each iteration of the loop

Installer

If what you want is to loop through the results for display, this code will do that:

while ($myrow = mysql_fetch_array($result)) {
    $size = getimagesize($myrow['image']);
    echo '<img src="' . $myrow['image'] . '" height="' . $size[1] . '" width="' . $size[0] . '" />';
}

If you really want the variable names with numbers (for later use?) you could do something like this:

$i = 0;
while ($myrow = mysql_fetch_array($result)) {
    $img_size = getimagesize($myrow['image']);
    ${'size' . $i} = $img_size;
    ${'height' . $i} = $img_size[1];
    ${'width' . $i} = $img_size[0];
    echo '<img src="' . $myrow['image'] . '" height="' . $img_size[1] . 
         '" width="' . $img_size[0] . '" />';
    $i++;
}

although it would probably be better to use arrays:

while ($myrow = mysql_fetch_array($result)) {
    $img_size = getimagesize($myrow['image'];
    $size[] = $img_size;
    $height[] = $img_size[1];
    $width[] = $img_size[0];
    echo '<img src="' . $myrow['image'] . '" height="' . $img_size[1] . 
         '" width="' . $img_size[0] . '" />';
}

kat_deery

that's exactly what I needed - thanks for your help!