Cosas que veo. Espero que lo entiendas como crítica constructiva
1. El for no sirve de gran cosa... de hecho sólo lo quieres para que i=ind-1, cosa que podrías indicar directamente. También podrías utilizar la propiedad ubound (mira mi mensaje #3) para obtener el valor.
2. No sé por qué te has fijado en obtener el valor del último botonprods que has utilizado. También podrías utilizar la del primero (i=0) y sería lo mismo.
3. Me imagino que VScrollProd es un control de tipo VScroll. Si te fijas tiene una propiedad Max. Yo le asignaría algún múltiplo de filas de controles botonprods que llevas y de la altura de los mismos, teniendo en cuenta ese 49 que supongo que es la distancia entre VScrollProd. De ese modo te será más fácil controlar en qué fila estás cuando encuentres el valor actual VScrollProd.Value.
Saludos.