Cita:
Iniciado por dashtrash 
Eso es lo que no es eficiente.Haz 1 query para obtener los datos de todos los productos a la vez.Las queries dentro de bucles son una plaga....Si te preocupas por la eficiencia, el no meter una query en un bucle es lo más básico.
La segunda opción que planteas (cachear los datos dentro de la sesion) es peligrosa, a menos que puedas asegurar que nada ni nadie va a tocar ese producto durante el tiempo de vida de la sesión (lo cual, si lo que estás haciendo es un carrito de compra, no puedes asegurar).
Gracias por la respuesta.
Es la pescadilla que se muerde la cola.
Efectivamente es como un carrito de la compra.
No puedo asegurar 100% que los datos de la bd no se modifiquen mientras dura la sesion.
Por tu comentario entiendo, cosa que ya intuia y tenia asumido que una consulta dentro del bucle era/es un desperdicio de recursos y que lo mejor es utilizar el bucle para construir la consulta (en la medida de lo posible) y posteriormente ejecutarla.