Output an array

Question

I have created a shortlist feature which acts a bit like a shopping cart. I output the items in the shortlist by:

$i = 0;
while ($i < $countArray){

echo $_SESSION['shortlistArray'][$i]." <a href='shortlistRemoveItem.php?arrayID=$i'>[x]</a><br />";

$i++;
}

and delete an item by

$arrayID = $_GET["arrayID"];
unset($_SESSION['shortlistArray'][$arrayID]);

The problem is that when I delete an item from an array such as $_SESSION['shortlistArray'][2] the output is all messed up as the array is no londer sequential. Should I recode the way my array is outputted or the way I am deleting an item from an array?

maybe you should use FOR instaad of WHILE? Also try to VAR_DUMP($_SESSION['shortlistArray']) to esure that is's ok before delete. — s.webbandit
– s.webbandit, Commented Sep 25, 2011 at 13:50

Ayman Safadi · Accepted Answer · 2011-09-25 13:56:52Z

0

The most efficient solution would be simply changing the way your array is outputted:

foreach($countArray as $key => $item){

    echo $_SESSION['shortlistArray'][$key]." <a href='shortlistRemoveItem.php?arrayID=$key'>[x]</a><br />";
}

If you insist on changing the way you are deleting an item from the array, consider this alternative:

$arrayID   = $_GET["arrayID"];
$tempArray = array();

foreach($countArray as $key => $item){

    if($arrayID == $key) continue;
    $tempArray[] = $item;
}

$_SESSION['shortlistArray'] = $tempArray;

I' recommend the first option though.

answered Sep 25, 2011 at 13:56

Ayman Safadi

11.6k1 gold badge30 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Output an array

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related