Check if UserForm is loaded?
Charlotte E
I have a macro, which under some conditions load an UserForm, and under
other conditions don't.
And, on some of these conditions, the UserForm is only loaded, but not shown
(for extraction certain information)
To make sure, that I clean up my act, I have an Unload UserForm statement at
the very end of the macro.
But, for some weird obscure reason the Unload statement actually LOADS the
userform before unloading it!!!
So, my question is, how to test if the UserForm is already opened, so I only
Unload it, if it is open???
TIA,
GS
You should check if it's already loaded and if so then unload it,
otherwise do nothing.
Code...
If Not UserForm Is Nothing Then Unload UserForm
--
Garry
Free usenet access at http://www.eternal-september.org
ClassicVB Users Regroup! comp.lang.basic.visual.misc
Charlotte E
> You should check if it's already loaded and if so then unload it,
> otherwise do nothing.
That was the whole idea :-)
> Code...
> If Not UserForm Is Nothing Then Unload UserForm
Problem is that every call to the UserForm forces it to load...
But, I've solved it by setting/clearing a global variable, upon
loading/unloading the UserForm - works :-)
CE
Peter T
this -
Sub UnloadAllForms()
Dim i As Long
For i = UserForms.Count To 1 Step -1
Unload UserForms(i - 1)
Next
End Sub
If a particular form is loaded you might want to run any clean up code
before unloading the form.
Regards,
Peter T
"Charlotte E" <s...@m.kills.it> wrote in message
news:ar-dneR3NIu...@giganews.com...
Charlotte E
userform first before unloading it, if the userform is not loaded upon
calling the unload-statement, thus your solution cannot be used, since it is
important not to load the userform, because this sets off a chain of event,
which is not wanted when unloading it.
But, I really find it a lack of effencicy that that the unload statement
actually loads the userform first!!!
But, as said: I found a work around...
CE
"Peter T" <pet...@discussions.com> wrote in message
news:i5ijvr$hjd$1...@news.eternal-september.org...
GS
While that works well enough, it means you are loading an instance of
the userform rather than the userform object. This requires memory
overhead and more coding to manage it. The single line of code I posted
doesn't call the userform nor cause it to load. It simply checks if
it's in memory and unloads if it is. Otherwise, it does nothing which,
according to you, "was the whole idea".<g>
Dave Peterson
Try creating a new userform1 and make sure it has a procedure like:
Option Explicit
Private Sub UserForm_Initialize()
MsgBox "hi"
End Sub
Then step through your suggestion -- once with the userform loaded and once
without the userform in memory.
On 09/02/2010 17:46, GS wrote:
> Charlotte E wrote :
>>> You should check if it's already loaded and if so then unload it,
>>> otherwise do nothing.
>>
>> That was the whole idea :-)
>>
>>
>>> Code...
>>> If Not UserForm Is Nothing Then Unload UserForm
>>
>> Problem is that every call to the UserForm forces it to load...
>> But, I've solved it by setting/clearing a global variable, upon
>> loading/unloading the UserForm - works :-)
>>
>> CE
>
> While that works well enough, it means you are loading an instance of the
> userform rather than the userform object. This requires memory overhead and more
> coding to manage it. The single line of code I posted doesn't call the userform
> nor cause it to load. It simply checks if it's in memory and unloads if it is.
> Otherwise, it does nothing which, according to you, "was the whole idea".<g>
>
--
Dave Peterson
GS
> You may want to do some more testing.
>
> Try creating a new userform1 and make sure it has a procedure like:
>
> Option Explicit
> Private Sub UserForm_Initialize()
> MsgBox "hi"
> End Sub
>
> Then step through your suggestion -- once with the userform loaded and once
> without the userform in memory.
Well that was educational! Thanks for that. Seems using an object
variable is actually the solution. Fact is I usually do load userforms
that way (Set myform as new userform1) and so ass-u-me-d loading the
object (userform1) gave the same behavior. I learned something here, so
thanks for the lesson!
>
> On 09/02/2010 17:46, GS wrote:
>> Charlotte E wrote :
>>>> You should check if it's already loaded and if so then unload it,
>>>> otherwise do nothing.
>>>
>>> That was the whole idea :-)
>>>
>>>
>>>> Code...
>>>> If Not UserForm Is Nothing Then Unload UserForm
>>>
>>> Problem is that every call to the UserForm forces it to load...
>>> But, I've solved it by setting/clearing a global variable, upon
>>> loading/unloading the UserForm - works :-)
>>>
>>> CE
>>
>> While that works well enough, it means you are loading an instance of the
>> userform rather than the userform object. This requires memory overhead and
>> more
>> coding to manage it. The single line of code I posted doesn't call the
>> userform
>> nor cause it to load. It simply checks if it's in memory and unloads if it
>> is.
>> Otherwise, it does nothing which, according to you, "was the whole
>> idea".<g>
--
Peter T
> Thanks, Peter, but once again: The Unload statement actually loads the
> userform first before unloading it, if the userform is not loaded upon
> calling the unload-statement, thus your solution cannot be used, since it
> is important not to load the userform, because this sets off a chain of
> event, which is not wanted when unloading it.
No, the example I posted only unloads Forms that are already loaded. If no
Forms are loaded UserForms.Count will be zero, so the loop will not even
start.
> But, I really find it a lack of effencicy that that the unload statement
> actually loads the userform first!!!
Whenever you attempt to reference a Form it will load if it is not alrady
loaded. It's not a question of efficiency or lack of it.
Regards,
Peter T
GS
> loaded. It's not a question of efficiency or lack of it.
As you can see by my previous post, I've gotten used to using
Load/Unload events with forms since using COM and VB6. As Dave aptly
points out, the Initialize event fires whenever a form is referenced.
In VB6, the Load event doesn't fire unless you execute it in code.
Either way, being a Rob Bovey student I've grown accustomed to using
object variables when creating instances of a form/userform because
that's what's exampled in most his books. Referencing the variable has
different behavior than referencing a VBA userform directly is the
lesson I've learned from Dave's suggestion. The problem lies where
there's code in the Initialize event, which I rarely use in VB6
projects.
Just another interesting difference between VB6 and VBA...
Peter T
reference a VB6 Form also loads the form into memory, even say Unload Form1
as with VBA. Also, perhaps counter intuitively, the Unload statement does
not fully Unload the VB6 form.
If you are not using a variable you might want to do say -
Unload Form1 ' to fire the QueryUnload event
Set Form1 = Nothing ' trigger the Terminate event
Regards,
Peter T
"GS" <gesa...@netscape.net> wrote in message
news:i5r4jd$s0h$1...@news.eternal-september.org...
GS
> There are several differences with VB6 forms, however any attempt to
> reference a VB6 Form also loads the form into memory, even say Unload Form1
> as with VBA. Also, perhaps counter intuitively, the Unload statement does not
> fully Unload the VB6 form.
>
> If you are not using a variable you might want to do say -
>
> Unload Form1 ' to fire the QueryUnload event
> Set Form1 = Nothing ' trigger the Terminate event
>
cleanup for me. What I was trying to emphasize is that the Initialize
event fires whenever a form/userform is referenced, and so using code
in that event is where the problem lies in this OP's case. Looking back
at some old VBA code before using an object var to hold an instance of
a userform, I was using my own 'Initialize' procedure rather than the
event, wherein I loaded a userform, did whatever setup tasks I needed
to do, then used Show to display it. Therefore, whenever I used the
following code to query the userform I never had any problems because
the Initialize event was never used.<g>
If Not Userform1 Is Nothing Then Unload Userform1