Hide or Destroy the previous Instance of a Gui Window Created Through a Function Topic is solved

[jsong55]

Code: Select all

QuickCopyLaunch(QuickCopyContents)                                                                                               
{
    ; if (qc.Hwnd)
    ;     qc.Destroy()
    qc:=Gui()
    qc.Add("Edit","xm y+0" " w500 r10" " vQuickCopyContents",QuickCopyContents)
    qc.show()
}

Running the function repeatedly causes endless Guis

[RussF]

MyGui.Destroy

Russ

[jsong55]

How to destroy old instance and create a new instance within the same function creating it?

[RussF]

Perhaps it would be easier to post exactly what it is that you're trying to accomplish. Why would you call the function multiple times without closing the previous one first?

I'm usually not a fan of creating GUIs from within a function. I will typically define them outside the function within the autoexec section of the script and then call MyGui.Show() and/or MyGui.Hide() from within the function. That way, there is only one copy of the GUI in memory at any one time.

Russ

[swagfag]

u can keep the existing reference around in a static variable

Code: Select all

#Requires AutoHotkey v2.0.2

QuickCopyLaunch(QuickCopyContents)                                                                                               
{
	static qc

	if IsSet(qc)
		qc.Destroy()

    qc:=Gui()
    qc.Add("Edit","xm y+0" " w500 r10" " vQuickCopyContents",QuickCopyContents)
    qc.show()
}

[RussF]

Thanks @swagfag, I wasn't sure if you could do that with an object and didn't have time to test it (I'm supposed to be working, you know )

Russ

[jsong55]

@swagfag thank you it works great!