you could built a class based on container in which you put a textbox and a label, than in its refresh you could write
this.visible = this.textbox.value > 0
so you have less code and less objects to manage

>hi,
>thank you for reply,
>this code under ,it works as what i need exactly
>

>thisform.text4.visible  =Thisform.text4.value>0
>thisform.label1.Visible=Thisform.text4.value>0
>thisform.text5.visible=Thisform.text5.value>0
>thisform.label2.Visible=Thisform.text5.value>0
>thisform.text6.visible=Thisform.text6.value>0
>thisform.label3.Visible=Thisform.text6.value>0
>thisform.text7.visible=Thisform.text7.value>0
>thisform.label4.Visible=Thisform.text7.value>0
>.
>.
>.
>

>best regards
>m.qasem
>>try this under form refresh
>>
>> for each oTxtBox in this.objects
>> if lower(oTxtBox.baseclass) = 'textbox'
>> oTxtBox.visible = oTxtBox.value > 0
>> endif
>> endfor
>>
>>if you have other textboxes in the form you should test also the object name
>>
>>HTH
>>
>>
>>>hi all,
>>>any idea,help .
>>>
>>>if i have 10 textboxes at my form as thisform.text1.value to thisform.text10.value or mor,
>>>
>>>if any textbox >0 visible=.t
>>>else
>>>visible=.f
>>>endif
>>>
>>>thanks.