How to prevent loop's last iteration

[maxwell_]

I am trying place each paragraph from a block of text into its own form field textarea.
I have managed to do this with the following code, however the code always leaves
one unwanted empty teaxtarea field at the end of the page

$textArray = explode("</p>", $text);

for ($i = 0; $i < sizeof($textArray); $i++) {
    $textArray[$i] = strip_tags($textArray[$i]);
	 echo "<textarea>$textArray[$i]</textarea><br /><br />";
	}

I tried changing the code to the following without success -

$textArray = explode("</p>", $text);
$len = count($textArray);

for ($i = 0; $i < sizeof($textArray); $i++) {
    $textArray[$i] = strip_tags($textArray[$i]);
	 echo "<textarea>$textArray[$i]</textarea><br /><br />";

if($len == -1) { 
    break; 
}

}

[sneakyimp]

I think the solution in this case would be to dispense with the last array element, not adjust the bounds of the while loop.

$textArray = explode("</p>", $text);
// pops the last element off the array
$empty_element = array_pop($textArray);
if (strlen(trim($empty_element)) > 0) {
  echo "oops, maybe that element is not actually empty";
}

[maxwell_]

That's fixed it -

$textArray = explode("</p>", $text);

$empty_element = array_pop($textArray);

if (strlen(trim($empty_element)) > 0) {
echo "oops, maybe that element is not actually empty";
}  

for ($i = 0; $i < sizeof($textArray); $i++) {
    $textArray[$i] = strip_tags($textArray[$i]);
	 echo "<div class='aligned'><textarea>$textArray[$i]</textarea></div><br /><br />";			
}

Many thanks for helping.

[dalecosp]

Just curious ... would this also fix it without array_pop() ?

$textArray = explode("</p>", trim( $text ));

I was thinking it likely that your text file had a spare newline at the end. It's sometimes helpful for your future to know why something was broken, and not just that you had some magic bullet to fix it.

Although I love dem magic bullets, and have plenty of array_pop()s lying around in scripts, to be sure! 🙂