giorno

so già che è una stupidata ma mi serve una mano.

mi ritrovo stamattina a mettere di nuovo mano su un codice non mio, che gestisce una transazione di e-commerce..

uno stralcio del codice originale in cui l'utente X sceglie semplicemente di pagare n prodotti è
codice:
...
// CALCOLO IMPORTO (da n_prodotti)
// n_prodotti è il numero di prodotti scelti in un form
$pagamento->set("importo",$pagamento->f("n_prodotti")*312);;
...
...
// Formato importo per ecommerce
$importo=$pagamento->f("importo")*100;
...
ora devo modificare questa cosa perché da un campo frequenza già esistente
codice:
     $this->field["frequenza"] = new Field(array("name" => "frequenza",
     	"form_control" => array("type" => "select", 
                 "class"=>"testo8",
                  "data_elements" => array("Annuale: unico versamento" => "annuale",
                                     "Semestrale: due versamenti" => "semestrale",
                                     "Trimestrale: quattro versamenti" => "trimestrale"
)
)
)
);
l'utente deve poter scegliere se pagare tutto insieme ("annuale") o se dividere il pagamento in "semestrale" o "trimestrale".

io sono intervenuto semplicemente così:
codice:
...
//Numero versamenti in base a frequenza scelta
if ($pagamento->f("frequenza")="annuale")
	{ $numero_versamenti="1"; }
elseif ($pagamento->f("frequenza")="semestrale")
	{ $numero_versamenti="2"; }
elseif ($pagamento->f("frequenza")="trimestrale")
	{ $numero_versamenti="4"; }

// Formato importo per ecommerce
	 $importo=$pagamento->f("importo")*100/$numero_versamenti;
...
secondo voi, visto che non posso fare molte prove ed è la prima volta che mi cimento in qualcosa di simile, potrebbe funzionare o c'è qualcosa di evidentemente sbagliato?

grazie