Store .t. to ("thisform." + thisobject +".visible")

is the cleanest and also fastest way to set with a variable.

thisform.&thisobject..visible=.t.

should work as well, but IMO should be avoided (note the 2! dots before visible)

For the ***specific*** loop line best (cleanest,fastest, most succinct):

Store .t. to ("thisform.Dayjobsline" + ALLTRIM(STR(x)) +".visible")

Personal preference for logging/errormsg/debug would be always same length of numberstring, so I'd add PadL(ALLTRIM(STR(x)), "0", 7) to Dayjobsline, but that clearly is not the problem and the form probably has already handcoded elements ;-)


>I have a form with a number of the same invisible objects on. I want to make them incrementally visible according to how many records are in a table - 1 object for each record...
>
>This code generates a 1925 error - Unknown member "name".
>
>

>zrecs=RECCOUNT()
>IF zrecs>0
>	FOR x=1 TO zrecs
>		thisobject='Dayjobsline'+ALLTRIM(STR(x))
>		thisform.thisobject.visible=.t.
>	NEXT
>ENDIF
>

>
> I have checked the value and type of 'thisobject' by displaying it in a messagebox, and it is all correct, yet I still get the 1925 error.
>
>Please advise what I am missing here...
>
>Many thanks!!