Notice: Undefined index carrello

[lukezz]

Salve a tutti, come da titolo mi sono scontrato con questa notice, precisamente è un carrello della spesa che memorizza gli acquisti in una sessione.
In questo script vi sono varie funzioni, nel mio caso quella che fa i capricci è usaCarello:

function usaCarrello()
{
$carrello = $_SESSION['carrello'];
  if($carrello)
  {
    return 'Il carrello è vuoto.
';
  }else{
    $prodotti = @explode(',',$carrello);
    return 'Ci sono <a href="carrello.php">'.
    @count($prodotti). ' prodotti nel carrello.</a>
';
  }
} 

che ho corretto con un isset cosi:

function usaCarrello()
{
  if(!isset($_SESSION['carrello']))
  {
    return 'Il carrello è vuoto.
';
  }else{
    $prodotti = @explode(',',$carrello);
    return 'Ci sono <a href="carrello.php">'.
    @count($prodotti). ' prodotti nel carrello.</a>
';
  }
} 

In questo modo la notice sparisce, però il numero di prodotti non si incrementa ad esempio metto due, tre prodotti nel carrello ma il count me ne legge sempre 1, sapete aiutarmi?

Un'altra cosa non meno importante è che la notice appare solo se incremento il sistema di login, senza, il carrello funziona benissimo.

Grazie anticipatamente

[oly1982]

se avessi tolto quelle @ avresti forse avresti individuato da solo l'errore...

function usaCarrello() 
{ 
  if(!isset($_SESSION['carrello'])) 
  { 
    return 'Il carrello è vuoto.
'; 
  }else{ 
$carrello = $_SESSION['carrello']; 

    $prodotti = explode(',',$carrello); 
    return 'Ci sono <a href="carrello.php">'. 
    count($prodotti). ' prodotti nel carrello.</a>
'; 
  } 
} 

[RoTeam]

semmai volevi fare cosi

function usaCarrello() 
{ 
  if(!isset($_SESSION['carrello'])) 
  { 
    return 'Il carrello è vuoto.
'; 
  }else{ 
    $prodotti = explode(',',$_SESSION['carrello']); 
    return 'Ci sono <a href="carrello.php">'. 
    count($prodotti). ' prodotti nel carrello.</a>
'; 
  } 
} 

[lukezz]

Togliendo la @ non mi da errori!

[lukezz]

Risolto! ignorantemente avevo risolto in parte, il count lo faceva sulla variabile carrello! grazie Roteam ^^

[lukezz]

Altro problema!
Se cancello i prodotti dal carrello mi rimane sempre che ho 1 prodotto come mai?

[RoTeam]

Originariamente inviato da lukezz
Altro problema!
Se cancello i prodotti dal carrello mi rimane sempre che ho 1 prodotto come mai?

Posta la funzione con cui cancelli

[lukezz]

$carrello = $_SESSION['carrello'];
if(@isset($_GET['action']))
{
  $action = $_GET['action'];
  
  switch ($action)
  {
    case 'cancella':
    if ($carrello)
    {
      $prodotti = @explode(',',$carrello);
      $acquisto = '';
      foreach ($prodotti as $prodotto)
      {
        if ($_GET['id'] != $prodotto)
        {
          if ($acquisto != '')
          {
            $acquisto .= ','.$prodotto;
          }else{
            $acquisto = $prodotto;
          }
        }
      }
      $carrello = $acquisto;
    }
    break; 

[oly1982]

Hai la pessima abitudine di disseminare @ nel tuo codice.... cmq sia

$carrello = $_SESSION['carrello']; 
if(@isset($_GET['action'])) 
{ 
  $action = $_GET['action']; 
   
  switch ($action) 
  { 
    case 'cancella': 
    if ($carrello) 
    { 
      $prodotti = @explode(',',$carrello); 
      $acquisto = ''; 
      foreach ($prodotti as $prodotto) 
      { 
        if ($_GET['id'] != $prodotto) 
        { 
          if ($acquisto != '') 
          { 
            $acquisto .= ','.$prodotto; 
          }else{ 
            $acquisto = $prodotto; 
          } 
        } 
      } 
      $_SESSION['carrello'] = $acquisto; 
    } 
    break; 

... ma ci sarebbe molto altro da sistemare...

[RoTeam]

Beh quella funzione non elimina tutti gli elementi perciò se ne hai altri è ovvio che rimangano