It seems like it is ok to me, it looks like you are dynamically selecting the option based on what is in the recordset though, not sure if this is necessary when a user is making the selections before adding to the cart.
If you are getting the rite result in your cart for the options you are selecting then I would say that it is fine and you are ok to move on to your checkout. Let us know if you have any issues or questions about any of this.