[Solved] Help Preventing double gui

[satanttin]

Hello,

Hope someone could help me out. I'm trying to open a gui with a hotkey and when it's already open to not open a new one.
The code i have so far is:

HotKeySet("+z", "Note")

Func Note()
If $Note = False Then
$Note = GUICreate("Note", 633, 504, 346, 297)
$dEdit = IniRead (@ScriptDir & "\Data\Note.ini" , "Note", "Note", @error)

$Edit = GUICtrlCreateEdit("", 0, 0, 632, 503)
GUICtrlSetData($Edit, StringReplace(IniRead(@ScriptDir & "\Data\Note.ini" , "Note", "Note", @error), "{ENTER}", @CRLF))
GUISetState(@SW_SHOW)
GUISetOnEvent($GUI_EVENT_CLOSE, "NoteFunc")

ElseIf $Note = True Then
GUIDelete($Note)
EndIf
EndFunc

If i press shift + z it will open the gui and a second press deletes it but when i use it a 3th time it doesn't do anything.

So basicly want to toggle a gui with a hotkey.

Edited July 7, 2015 by satanttin

[Melba23]

satanttin,

I would just hide/show the GUI like this:

#include <GUIConstantsEx.au3>

HotKeySet("+z", "Note")

$bShow = True

$Note = GUICreate("Note", 633, 504, 346, 297)
$dEdit = IniRead (@ScriptDir & "\Data\Note.ini" , "Note", "Note", @error)

$Edit = GUICtrlCreateEdit("", 0, 0, 632, 503)
GUICtrlSetData($Edit, StringReplace(IniRead(@ScriptDir & "\Data\Note.ini" , "Note", "Note", @error), "{ENTER}", @CRLF))
GUISetState(@SW_SHOW)

While 1
    Switch GUIGetMsg()
        Case $GUI_EVENT_CLOSE
            Exit
    EndSwitch

WEnd



Func Note()

    $bShow = Not $bShow

    If $bShow Then
        GUISetState(@SW_SHOW, $Note)
    Else
        GUISetState(@SW_HIDE, $Note)
    EndIf

EndFunc

M23

[JohnOne]

What Melba posted is a better way to do this, but to answer your question.

ElseIf $Note = True Then
    GUIDelete($Note)
    $Note = False
EndIf

 

Edited July 7, 2015 by JohnOne
Code was incorrect

[satanttin]

Thanks Melba23 ^^

works like a charm :3 guess i will save this piece of code if i need it in the future

[satanttin]

Thanks for the comment JohnOne:)