Gui and MosGetPos

[j123258]

Hellow,I wrote a simple script,
I want to when I choose radio1 and press F2 (MosGetPos) ,the Gui can get the mosuse's x and y into the edit (x1,y1).
when I choose radio2 and press F2 (MosGetPos) , I also can get the information of mouse's and show it in the edit (x2,y2).

but it doesn't work as I excepted,how can I improve this script.

Code: Select all

Gui, Add, edit, vx1 ghotkey22
Gui, Add, edit, vy1 ghotkey22
Gui, Add, Radio ,  gHotkey22 ,Radio1
Gui, Add, edit, vx2 ghotkey33
Gui, Add, edit, vy2 ghotkey33
Gui, Add, Radio ,  gHotkey33, Radio2
Gui,show, w200 h200

F2::
{
	CoordMode ,Mouse,Relative
	MouseGetPos get_mousex, get_mousey
	varx = %get_mousex%
	vary = %get_mousey%
}
Hotkey22:
{
	 Gui,Submit,NoHide
	 GuiControl,,x1,%varx%
	 GuiControl,,y1,%vary%
	 return
}
Hotkey33:
{
	 Gui,Submit,NoHide
	 GuiControl,,x2,%varx%
	 GuiControl,,y2,%vary%
	 return
}

[Odlanir]

Try this:

Code: Select all

Gui, Add, edit, x10 y010 vx1
Gui, Add, edit, x10 y035 vy1
Gui, Add, edit, x10 y080 vx2
Gui, Add, edit, x10 y105 vy2
Gui, Add, Radio , x10 y60  Checked gRadio, Radio1
Gui, Add, Radio , x10 y130 gRadio, Radio2
Gui,show, w200 h200
rd:=1
F2::
	CoordMode ,Mouse,Relative
	MouseGetPos get_mousex, get_mousey
	GuiControl,,% "x" rd,%get_mousex%
	GuiControl,,% "y" rd,%get_mousey%
return

Radio:
	Gui,Submit,NoHide
	rd := substr(A_GuiControl, -0)
return

[j123258]

@Odlanir that's what i want, thank you very much !!

[divanebaba]

Try this modified version.
I have deleted not needed g-Labels and added some return to prevent accessing parts of codes, which were desired to execute by g-labels.

Code: Select all

Gui, Add, edit, vx1 
Gui, Add, edit, vy1 
Gui, Add, Radio, checked vRadio1, Radio1
Gui, Add, Radio, vRadio2, Radio2
Gui, Add, edit, vx2 
Gui, Add, edit, vy2 

Gui, show, w200 h200
return ; <--- Needed to stop executing script

F2::
{
	OutputVar =
	CoordMode, Mouse,Relative
	MouseGetPos, get_mousex, get_mousey
	
	GuiControlGet, OutputVar,, Radio1
	ifequal, OutputVar, 1
	goto Hotkey22
	else
	goto Hotkey33
	
}
return

Hotkey22:
{
	 GuiControl,,x1,%get_mousex%
	 GuiControl,,y1,%get_mousey% 
}
return

Hotkey33:
{
	 GuiControl,,x2,%get_mousex%
	 GuiControl,,y2,%get_mousey%
}

return

[wolf_II]

Code: Select all

return ; <--- Needed to stop executing script

That return is not needed to stop executing script. It adds clarity. When unclear why AHK auto-adds this return, refer to the helpfile.

[divanebaba]

... That return is not needed to stop executing script. ...

In the original script, he creates the Gui and don't stop his script.

Code: Select all

Gui, Add, Radio ,  gHotkey33, Radio2
Gui,show, w200 h200
<---- Here is a return needed, otherwise the script executes the F2-label
F2::

I think the return at shown line is urgent needed. Or do I deceive again?

[wolf_II]

No, your are overlooking the fact that AHK auto-inserts a Return. See https://autohotkey.com/docs/Scripts.htm#auto
There it says Auto-execute section stops at a hotkey label. Somewhere else it says why .. b/c of auto-inserting a Return
Maybe I learned about the auto-insert portion in the forum.

BTW: I don't suggest to omit the Return in question, I advise to always use it for clarity.

[divanebaba]

The help-file says:

After the script has been loaded, it begins executing at the top line, continuing until a Return, Exit, hotkey/hotstring label, or the physical end of the script is encountered (whichever comes first).

Thanks. You're right. This information is new for me.

[wolf_II]

Try this:

Code: Select all

;-------------------------------------------------------------------------------
AutoExecute: ; auto-execute section of the script
;-------------------------------------------------------------------------------
    #NoEnv
    #SingleInstance, Force

; Return ; end of auto-execute section (commented out)



;-------------------------------------------------------------------------------
F12:: ; do stuff
;-------------------------------------------------------------------------------
    MsgBox, Hi, there!

Return

1. Run the script.
2. Right click tray icon to see most recent lines
3. Press F12 (the hotkey from script)
4. Press F5 (to refresh the most recent lines)

Notice the return there? I commented out my own to disable. I wanted to try the script not to stop autoexecuting (sort of).
The script will stop the auto-execute section when running into a hotkey label.

sorry for poor English, I hope you understand anyway.