Este codigo tiene un posible defecto. Observo que pasas una var por referencia (lo cual en si no es malo)y luego la asignas a la propiedad del objeto... el problema es que si llegas a tener un codigo como:

for i=1 to ObjetoPadre.ObjetoHijo.Contador

next

estaras creando el objeto UNA Y OTRA VEZ sin necesidad... mas bien chequea primero si la propiedad es nula (por lo tanto no hay objeto) y entonces si creas el objeto, de lo contrario lo dejas tal cual...

Adicionalmente, seria bueno asignar un back-reference al objeto hijo... esto es colocar Hijo.Parent=TuParent, veras rapidamente que es necesario porque eventualmente necesitaras realizar notficaciones y demas cosas por el estilo...

Por ultimo, ten una buena disciplina con la destruccion de objetos... mejor aun pega un mensaje en la pared que diga "No olvides destruir los malditos objetos" y nunca olvides en los destroy limpiarlos... Te ahorrar un par de errores C00005...

Por ultimo, aprovecha los parametros en el init para cargar los datos filtrados o usa un metodo load para ello...

Por ejemplo si tienes un Objeto de cuentas, las cuales son varias para un cliente, podrias usar:

Cuentas.Load('Cliente=1') y asi evitas traer datos innecesarios. (o de forma mas orientada a objetos Cuentas.Load(oCliente))

Adicionalmente, investiga sobre Design Patterns, especialmente el Iterator, Builder, Factory y Singleton... son muy utiles en estos casos!