No need to calculate the length of the array. Just set

rowsource=this.JohnArray
rowsourcetype=array
numberofelements=0 (default)

in the combobox init
select somestuff from somewhere into array this.JohnArray

Voila!

>I tried this and still had to do a ReQuery() to get it to show up...

>>Someone else got the answer for you, but here are some tips:
>>- combobox should almost always have bounto=.t. (set this is your base class)
>>- consider making a combobox class that has an array as a property. This solves having to requery after loading the array and then releasing the array.