Aumentar eficiencia-reutilización a este código

[jossss]

Te en cuenta que incias $cupcakes[] (al poner los corchetes) como un array. Entoces la salida de foreach deberia ser

if($cupcake>0)
$cupcakes[un_indice][$I]=$cupcake;