syntax error in case 'remove' when decrementing [closed]

Question

Closed. This question is not reproducible or was caused by typos. It is not currently accepting answers.

---

This question was caused by a typo or a problem that can no longer be reproduced. While similar questions may be on-topic here, this one was resolved in a way less likely to help future readers.

Closed 9 years ago.

Improve this question

I get a syntax error on the case "remove". I've been trying to fix it but I can't figure it out. Can anyone tell me how to fix it?Thanks

The code is for a online shop shopping cart

switch($action)
{
    case "add":
    if (isset($_SESSION['cart'][$id]))
        $_SESSION['cart'][$id]++;
    else
        $_SESSION['cart'][$id]=1;
break;

case "remove":
    if (isset($_SESSION['cart'][$id]))
    (
        $_SESSION['cart'][$id]--; (ERROR HERE)
        if ($_SESSION['cart'][$id]==0)
            unset($_SESSION['cart'][$id]);
    )
break;

case "empty":
    unset($_SESSION['cart']);
break;

}

Answer 1

You are using parenthesis instead of brackets for your IF statement:

if (isset($_SESSION['cart'][$id]))
( <-- HERE
    $_SESSION['cart'][$id]--; (ERROR HERE)
    if ($_SESSION['cart'][$id]==0)
        unset($_SESSION['cart'][$id]);
) <-- HERE

Answer 2

Please replace ( and ) with { and }

Corrected code:

case "remove":
    if (isset($_SESSION['cart'][$id]))
    {
        $_SESSION['cart'][$id]--; (ERROR HERE)
        if ($_SESSION['cart'][$id]==0)
            unset($_SESSION['cart'][$id]);
    }
break;

Answer 3

// Syntactical Errors, Find the corrected code!!

  switch($action)
  {
case "add":
    if (isset($_SESSION['cart'][$id])) {
        $_SESSION['cart'][$id]++;
    }
    else {
          $_SESSION['cart'][$id]=1;
     }
    break;

case "remove":
    if (isset($_SESSION['cart'][$id])) {
            $_SESSION['cart'][$id]--; //(ERROR HERE)
        }
        if ($_SESSION['cart'][$id]==0) {
            unset($_SESSION['cart'][$id]);
        }
    break;

case "empty":
    unset($_SESSION['cart']);
    break;

  }