Mark Hensler — Wed, 04 Sep 2002 03:23:49 +0000

The FOR loop starts at 0, so you use <. If the loop started at 1, then you use <=.

I think part of our problem is that we're only seeing half of the code each time you post. We have no idea how $_SESSION['pid'] is being populated or what is in $cb.

To debug your FOR loop, echo everything inside it. Like so:

<?php
for ($i=0; $i<count($cb); $i++) {
        echo \"array_splice(\$_SESSION['pid'], $i, 1);\";
}
?>

Now, when count($cb) is suppose to be 2, does the script print that line twice?
If it does print it twice, are the values (like $i) what you expected?

I predict not...
$i is always going to be the offset of $cb, not necessarily the offset of $_SESSION['pid']. If the user selected to delete the second and fourth pid, $i will always be 1 and 2 in the FOR loop when it should be 2 and 4.

zollet — Mon, 02 Sep 2002 17:42:26 +0000

The reason your first code only looped once is because you need to use '<=' instead of just '<'..

<?php
for ($i=0; $i<=count($cb); $i++) 
        {
            array_splice($_SESSION['pid'], $i, 1);    
        }
?>

Shouldn't you also do session_unregister('pid'); if you're cleaning the shopping cart?

joyce — Mon, 02 Sep 2002 05:38:15 +0000

i've tried unset() but it's not working. nothing is deleted. can anyone tell me wat is the appropriate function to use for this situation??

i have all my ids stored in session and when user clicked add, then it will add into the cart. i display the user cart in a table with checkboxes beside each row of record. when user clicked the checkbox and delete, it should delete the record.

i've tried using unset(), but it doesn't work for my case. none of the records are able to delete. then i switch to array_splice. by using this array, i manage to delete, but there is a problem, which is if user clicked the middle record, not the first one, but clicked the middle record, then it will delete the first record instead of the middle record.

i really donot know which function is approriate for me to use. i really need help on this. pls advice me..

Mark Hensler — Mon, 02 Sep 2002 05:09:21 +0000

array_splice() definition:
array array_splice ( array input, int offset [, int length [, array replacement]])

It uses an offset, not the key.

Try something like this:

<?php
foreach ($cb as $tmp) {
    unset($_SESSION['pid'][$tmp]);
}
?>