[SOLVED] implode function

[timmah1]

I'm setting up a newsletter system.

 

I have a form that selects everybody from the database and puts the users email address into a hidden field like so

<input type="hidden" name="user[]" value="<?=$row['email']; ?>" />

 

I'm testing to make sure it has all of the emails, when I click on the 'Submit' button, it should display every email that the newsletter is going to get sent to for testing purposes.

 

What happens is, it displaying every email 13 different times.

There are 128 different emails in the system, so logically it should only show 128 emails, but instead it's showing me 1,664 emails.

Here is the code:

$subscriber=$_POST['user'];

	$together = implode(", ", $subscriber);
	for($i = 0; $i < count($subscriber); $i++){

		echo "$together<br>";
	}

 

Can anybody tell me why it's showing every email multiple times, and not just once?

 

Thanks in advance

I

[GingerRobot]

You're imploding the data, which creates (in this case) a comma delimited string containing all of the values in the array. You then loop through the array echoing the string each time.

 

You should either be looping or imploding. Not both.

[timmah1]

I see now

So the correct coding would be

 

$together = implode(", ", $subscriber);
echo "$together<br>";