Control Structures: Simple, I'm sure!

[bubblenut]

I started off with this code

do {
  $line=fgets($fh,4096);
  $linesize=strlen($line);
    list($blank,$blank,$blank,$model2,$checkloop,$blank,$blank,$blank,$image2,$description)=csv_explode($line,',','"');
  $model[]=$model2;
  $image[]=$image2;
} while($checkloop=='');

and then quickly realised that I would get an extra entry in the $model and $image arrays because the check on the recieved data is at the end.
I have changed it to the following but I'm sure there must be a simple construct for achieving this without repeating code (as I see it eather repeating the code for getting the data as below or repeating the condition in the while in an if within the while)

$linesize=strlen($line);
list($blank,$blank,$blank,$model2,$checkloop,$blank,$blank,$blank,$image2,$description)=csv_explode($line,',','"');
while($checkloop=='') {
  $model[]=$model2;
  $image[]=$image2;
  $line=fgets($fh,1048576);
  $linesize=strlen($line);
  list($blank,$blank,$blank,$model2,$checkloop,$blank,$blank,$blank,$image2,$description)=csv_explode($line,',','"');
}

Any ideas?

Thanks
Bubble

[laserlight]

One way would be to use a well documented infinite loop

//this loop terminates when $checkloop == ''
for (;;) {
	$line = fgets($fh, 4096);
	$linesize = strlen($line);
	list($blank, $blank, $blank,
		$model2, $checkloop,
		$blank, $blank, $blank,
		$image2, $description) = csv_explode($line, ',', '"');
	if ($checkloop == '')
		break; //INFINITE LOOP ENDS HERE!
	$model[] = $model2;
	$image[] = $image2;
}

Another interesting way might be:

function doStuff(&$fh, &$model2, &$image2) {
	$line = fgets($fh, 4096);
	$linesize = strlen($line);
	list($blank, $blank, $blank,
		$model2, $checkloop,
		$blank, $blank, $blank,
		$image2, $description) = csv_explode($line, ',', '"');
	return $checkloop;
}
//assume $fh, $model2, $image2 initialised
while (doStuff($fh, $model2, $image2) == '') {
	$model[] = $model2;
	$image[] = $image2;
}

Though personally I'm not sure if it will work as expected.
You'll also have to deal with $linesize, probably making it an argument passed by reference.
I'll prefer the infinite loop though, or simply doing it in your current code.

[bubblenut]

Of course! Thanks Laser, I went with the infinite loop option.