Grouping $_Session outputs

[deadman6]

So i have 3 form fields. Each time they are submitted the inputs are display as well as the previous inputs are display. Each time the set of inputs are submitted, each output is put into a div with the same name. That name is increased by one each time. So the first time submit is hit there is 3 <div id="id1"></div> then second submit is hit there are 3 <div id="id1"></div> and 3 <div id="id2"></div> .

What i need to do is wrap each group of <div id="id(X)"></div>. So if the submit button has been hit twice there should be :

<div id="wrap"> <div id="id1"></div> <div id="id1"></div> <div id="id1"></div> </div> 
<div id="wrap"> <div id="id2"></div> <div id="id2"></div> <div id="id2"></div> </div>

Let me know if it would be easier if i show you have i am getting each <div id="id(X)"></div>

[deadman6]

Here is my code.. I will make a picture in just a minute...

<?php session_start(); ?>
<?php
if(!isset($_SESSION['counter'])){
	$_SESSION['counter'] ='1';
} 
if($_POST)
	$_SESSION['name'] .= '<div id="id'.$_SESSION['counter'].'">'.$_POST['name'].'</div>'; 
	echo $_SESSION['name'] ;
?>
<?php if($_POST)
	$_SESSION['email'] .= '<div id="id'.$_SESSION['counter'].'">'.$_POST['email'].'</div>';
	echo $_SESSION['email'] ;  

	 ?> 
<?php if($_POST)
	$_SESSION['phone'] .= '<div id="id'.$_SESSION['counter'].'">'.$_POST['phone'].'</div>';  

	echo $_SESSION['phone'] ;
	 ?>    

<?php if($_POST)
	$_SESSION['counter']++;
?>
<?php if($_POST[reset]) {
	unset($_SESSION['counter']);
	unset($_SESSION['name']);
	unset($_SESSION['email']);
	unset($_SESSION['phone']);
 } ?>


<form method="post" action="index.php">
<div id="name">Name:<input type="text" name="name" size="10" /></div>
<div id="email">Armor:<input type="text" name="email" size="10" /></div>
<div id="phone">Strength:<input type="text" name="phone" size="10" /></div>

<input type="submit" name="submitted" value="Submit"/>
<input type="submit" name="reset" value="reset" />
</form>

And here is a img of what i want to happen...

[johanafm]

First off, since you are

$_SESSION['name'] .= '...'; 

You will
1. Get an error the on the first post request, since $_SESSION['name'] (goes for the others as well) does not yet exist, so you can't use the concatenation operator - you'd need assignment the first time. This should produce error messages. If it doesn't, you really should configure PHP to report error messages. See http://php.net, search for ini directives, then click list of ini directives and check error_reporting, display_errors and log_errors.

2. Keep all previous posted data (and the div tags that you add) concatenated in the session variables, which means that you'd need to add your div wrappers when you add stuff to the session variables along the lines of

    .= '<div id="wrap">' . 'previous code here' .'</div>'
   

   which will produce an HTML error! Use the w3 validator to make sure that you create valid html. The problem here is that you will end up with the same id value several times, and ids have to be unique across each page.

3. Make things more complicated (in my opinion) than they need to be. Use arrays instead of string concatenation. Add div wrappers and whatever else you need while looping over that data. That way, you will also store less data, since everything which is static will be stored as static data in the script, rather than as ever-constant dynamic data.