Troubling fitting in the unset function

Benny007

Hi,

I have 2 files below which work fine - though where the "Unset" function is - at present when ran - it unsets the whole session rather than individual items.
(Basically it's to work like a shopping cart - removing items individually)

First Part

<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>

<body bgcolor="#FFFFFF">
<form method="post" action="basket.php" name="test">
  <p> 
    <input type="checkbox" name="1" value="Company 1">
    <input type="checkbox" name="2" value="Company 2">
    <input type="checkbox" name="3" value="Company 3">
    <input type="checkbox" name="4" value="Company 4">
    <input type="checkbox" name="5" value="Company 5">
  </p>
  <p>
    <input type="submit" name="Submit" value="Submit">
  </p>
</form>
</body>
</html>

Second Page - This is where the unset function dwells (near bottom) - any idea as to why it's unsetting? - could someone please perhaps donwload and try this?

[

<?php 
// start the session 
session_start(); 
header("Cache-control: private"); //IE 6 Fix 

echo "<strong>Franchise Selection</strong><br />"; 

// Get the user's input from the form
   $Co1 = $_POST['1'];

// Establish if anything returned
   IF ($Co1 == TRUE) {

// Register session key with the value 
   $_SESSION[] = $Co1;
   }

   $Co2 = $_POST['2'];
   IF ($Co2 == TRUE) {
   $_SESSION[] = $Co2;
   }

   $Co3 = $_POST['3'];
   IF ($Co3 == TRUE) {
   $_SESSION[] = $Co3;
   }

   $Co4 = $_POST['4'];
   IF ($Co4 == TRUE) {
   $_SESSION[] = $Co4;
   }

   $Co5 = $_POST['5'];
   IF ($Co5 == TRUE) {
   $_SESSION[] = $Co5;
   }

// Display the sssion information: 

$num_elements = count ($_SESSION);
// count how many entries in the array
//
echo "<form method='post' action='$PHPSELF' name='test1'>";
for ($idx = 0; $idx < $num_elements; ++$idx){
unset($_SESSION['idx']);
echo ("$_SESSION[$idx]");
echo "<input type='checkbox' name='$_SESSION[$idx]' value='$_SESSION[$idx]'><br>";
}
echo "<input type='submit' name='Submit' value='Submit'>";
echo "</form>";




//echo $_SESSION['1'];

echo $num_elements;





?> 


Welcome to my website <strong><? echo $_SESSION['name']; ?></strong>!<br /> 
Let's see what happens on the <a href="page3.php">next page.</a><br /><br />

If someone could test this script 1st page index.php 2nd page basket.php that would be great then you can see my problem - or if you can spot it just by looking that too would be great thank you.

devinemke

Benny007 wrote:
any idea as to why it's unsetting?

because you are telling it to. you are unsetting every numerated array index in your for loop. i still don't undertand what you are ultimately trying to do here. what do you wish to unset?

toplay

devinemke wrote:
because you are telling it to. you are unsetting every numerated array index in your for loop. i still don't undertand what you are ultimately trying to do here. what do you wish to unset?

I also don't fully understand what Benny007 is trying to do, however, Benny007 is not unsetting every array index devinemke.

Benny007's unset line is this:
unset($_SESSION['idx']);

That doesn't even use the $idx variable and even if it had the dollar sign, it's inside single quotes and it's value wouldn't get parsed out. So, it's just unsetting an index called 'idx' which probably doesn't even exist.

Benny007, figure out which index(es) you want to unset and use this sample syntax:

unset($_SESSION[0]); // removes index zero as an example!

By having a loop and putting the unset() within the index loop, will unset all the corresponding indexes. This will delete them all:

for ($idx = 0, $num_elements = count ($SESSION); $idx < $num_elements; ++$idx){
unset($SESSION[$idx]); // Use $idx variable!
}

But if you have nothing else stored in $_SESSION, then you can simply do:

$_SESSION = array();

to delete all values all at once.

hth.

Benny007

Thank you for your input, i'v tried using the principles you listed in your post above - but i'm obviously going wrong somewhere! - if you could identify where that would be great!

As you both seem a little unsure of my problem I have created a diagram to help get my message across.

Diagram of Page2

Page1: people can make a selection of what companies they want (using buttons)

Page2: (Diagram) it lists the companies requested from page1 - and add's a delete button next to each one so they can remove that specific company in case they made a mistake.

Page3 - when that user clicks sumbit - they are taken to a form which they fill in their details when completed we are emailed the companies requested and then we would contact them back with specific information (I can do part1&3 - using form and mail functions - just need help with page2)

Perhaps my code is way off?

<?php  

// start the session 
session_start();  

header("Cache-control: private"); //IE 6 Fix 

echo "<strong>Franchise Selection</strong><br />";  

// Display the sssion information: 

// Get the user's input from the form 
if(isset($_POST['check1'])) { 
$_SESSION['check'][] = $_POST['check1']; 
} 
if(isset($_POST['check2'])) { 
$_SESSION['check'][] = $_POST['check2']; 
} 
if(isset($_POST['check3'])) { 
$_SESSION['check'][] = $_POST['check3']; 
} 
if(isset($_POST['check4'])) { 
$_SESSION['check'][] = $_POST['check4']; 
} 
if(isset($_POST['check5'])) { 
$_SESSION['check'][] = $_POST['check5']; 
} 

// Display the sssion information: 

$num_elements = count ($_SESSION['check']); 
// count how many entries in the array 
// 
echo "<form method='post' action='basket2.php' name='test1'>"; 
for ($idx = 0, $num_elements = count ($_SESSION); $idx < $num_elements; ++$idx){ 
unset($_SESSION[$idx]); // Use $idx variable!
echo ("$_SESSION[$check]"); 
echo "<input type='checkbox' name='$_SESSION[$idx]' value='$_SESSION[$idx]'><br>"; 
} 
echo "<input type='submit' name='Submit' value='Submit'>"; 
echo "</form>"; 

//echo $_SESSION['check']; 
echo $num_elements; 

?>

This code still won't display each company selected , or add a delete option (at present the delete option is a checkbox but can substitute it for a button like in the diagram)

Thank's for your time.