Esta bien hecho esto?, necesito una opinión

[mariano_donati]

Hola, estoy cambiando toda mi aplicación para optimizarla, una de las cosas que estoy haciendo es usar el método getRows, pero para utilizarlo, tnego que cambiar algunas cosas. La que particularmente me interesaría uqe me dijeran si esta bien o mal llo que estoy haciendo es esta:
En un formulario tengo toda la lista de productos de un negocio, cada producto tiene un checkbox, y cada producto que es seleccionado lo puede empezar a modificar. Ese formulario lo procesaba antes de esta forma:

set RS = Server.CreateObject("ADODB.RecordSet")
Rs.Open "Productos", conexion

For each Id in Request.Form("products")
RS.Filter= "IdProd = "&Id&""

----- muestro todos los campos ----

next

RS.Close
set RS = nothing

Ahora, lo que estoy haciendo es esto: primero meto en un array todos los valores que me pasa por la coleccion Request.Form("products"). Lo segundo que hago es armar una cadena interior SQL (la verdad que no se como llamarle) para que me seleccione aquellos productos cuyos Id's sean iguales los pasados por la cadena interior. El código resumido sería algo así:

arr = split(Request.Form("products"),",")
sub_SQL = join(arr," OR IdProd = ")

SQL = "SELECT * FROM Productos WHERE IdProd = "&sub_SQL&""
set init_RS = Server.CreateObject("ADODB.RecordSet")
init_RS.Open SQL, conexion

RS = init_RS.getRows()

RS.Close
set RS = nothing

Espero que puedan decirme que método les parece mejor, porque cambié recién un cuarto de la aplicación, y me gustaría antes de seguir avanzando en esto, leer sus comentarios.
Les dejo un saludo y mcuhas gracias como siempre.