Error else with no matching if

I don’t know what Scan is but it is interpreted as a keyword if you remove it from ScanScreen, your code will probably work.

^q::Screen(930,430,960,460)

Screen(x1="", y1="", x2="", y2="") {
	x1 += 9 ; These are adjustments to the area, the script I use to find my x/y coordinates is off by 9 pixels for both x and y.
	y1 += 9
	x2 += 9
	y2 += 9
	
	if(x2 = 9 || y2 = 9){
		MouseMove, %x2%, %y2%
	} else {
		MouseMove, %x1%, %y1%
		x := x1
		y := y1
		loop {
			if(x = x2) {
				break
			} else if(y = y2){
				x += 1
				y := y1
			} else {
				y += 1
				MouseMove, %x%, %y%
				sleep, 500
			}
		}
	}
}

Esc::ExitApp

Edit: It seems to assume it is a remap, I’m gussing it’s a key I don’t know, although I could not find it in the key list.

002: SetKeyDelay,-1
002: Send,{Blind}{ScanScreen(930,430,960,460) DownTemp}
002: Return (0.17)
002: SetKeyDelay,-1
002: Send,{Blind}{ScanScreen(930,430,960,460) Up}
002: Return (1.31)

Edit 2: or just do this.

^q::
	ScanScreen(930,430,960,460)
return

ScanScreen(x1="", y1="", x2="", y2="") {
	x1 += 9 ; These are adjustments to the area, the script I use to find my x/y coordinates is off by 9 pixels for both x and y.
	y1 += 9
	x2 += 9
	y2 += 9
	
	if(x2 = 9 || y2 = 9){
		MouseMove, %x2%, %y2%
	} else {
		MouseMove, %x1%, %y1%
		x := x1
		y := y1
		loop {
			if(x = x2) {
				break
			} else if(y = y2){
				x += 1
				y := y1
			} else {
				y += 1
				MouseMove, %x%, %y%
				sleep, 500
			}
		}
	}
}

Esc::ExitApp

And I think your problem is that you are making a function hotkey that takes parameters, to my knowledge the function is called without any parameters when the hotkey is trigured, and the hotkey execution stops with the function meaning the call with parameters is never reached.

Script Error: ELSE with no matching IF

I am just starting to use AutoHotKey scripts ..

Can you explain the basic syntax error reported in this test script ..
intended to send Ctrl+C, Ctrl+C or Ctrl+V when *.exe is run
.. where .exe argument %1% = 1 to 3.

I get an error message when running the script.ahk ..

Error: ELSE with no matching IF

• Members
• 278 posts

• Last active: Dec 31 2011 10:53 PM
• Joined: 07 Sep 2004

Your problem seems to come from the «return» commands

not tested but I think that’s OK

• Members
• 21 posts

• Last active: Jul 27 2008 09:33 PM
• Joined: 09 Jun 2005

Thank you .. that works .. sans error message.
On reflection, probably an easier method is to pass two arguments?

• Members
• 278 posts

• Last active: Dec 31 2011 10:53 PM
• Joined: 07 Sep 2004

I think the first option is good. But it’s my own opinion.

• Members
• 26 posts

• Last active: Mar 19 2016 05:59 PM
• Joined: 04 Jun 2005

The problem was caused by the returns, but only while the set of instructions to be excuted for any of the conditions, was not enclosed in braces. These are needed to group the instructions. Without braces, the first return would always return, and the first else would be an orphan.
Although the returns are superfluous as shown by Nemroth, following would have worked:

Источник

ELSE with no matching IF, error? :S

This is the code I’m trying to make for a autologin for the game SRO, the decapcha and everything works just fine, the main problem in the code is in the region labled ;—ImageSearch— , I can’t seem to figure out why «ELSE with no matching IF» error keeps occuring, hopefully a trained eye can spot it. 😆 thanks!

• Administrators
• 2709 posts

• Hates playing Janitor

• Last active: Jan 15 2016 11:37 PM
• Joined: 09 Mar 2008

if statements need < >‘s if there is more then 1 line of code that goes with the if.

rawr. be very afraid
*poke*
. Populate the AutoHotkey city . Pointless but somewhat fun. .

• Members
• 101 posts

• Last active: Jan 22 2010 12:24 AM
• Joined: 13 Apr 2009

• 
• Change Theme
  • IP.Board
  • IP.Board Mobile
  • Ortem
• English (USA)
  • English (USA)
  • Deutsch
• Privacy Policy
• Help

Источник

Else with no matching if error, need help

Hello,
this is a part of an script that would have to verify the status of an application screen, but I´m getting the Else with no matching if error, where is the error?

What are the mistakes here? because I´m sure there are more than one
Thanks,

• Members
• 726 posts

• Last active: Apr 23 2010 02:50 PM
• Joined: 21 Jul 2008

I went through and properly indented your code. Right off the bat, I noticed one problem.

• Moderators
• 3165 posts

• Last active: Apr 01 2014 01:46 AM
• Joined: 07 Oct 2006

• Guests

• Last active:
• Joined: —

• Guests

• Last active:
• Joined: —

• Members
• 251 posts

• Last active: Jan 28 2015 08:33 PM
• Joined: 08 Dec 2006

• Moderators
• 3622 posts

• Last active: Dec 24 2015 02:21 AM
• Joined: 07 Oct 2006

• Guests

• Last active:
• Joined: —

This is your original code reduced to the minimum and indented

Apart from the mixing of braces/no-braces: If you have another condition, you use else if directly. Else-only is used for the last «anything else case», when you have no conditions left.

Источник

Error else with no matching if

Here is my first program, I had to use a lot of if/ else statements. For some reason the last «else» (bolded) is giving me an error and not letting me run the program. whats going on?

and yes, the if else’s are indented properly, because there are no errors for the first 3 unit types, only for the last one ( ‘F’ ), and i have them all exactly in the same format.

If you want to use multiple statements within an if statement, you need to use braces.

Without braces, you can only have one statement that follows, and you have two right before that bolded else.

PS: Use code tags next time. It’s the <> button to the right ->

Thank you! it worked.

and do the code tags reserve the spacing/ indenting?

To improve your program a bit, make use of the following:

A ShowMenu function, which prints out the options with descriptions.

A switch statement that has a case for each option in the menu, possibly calling a function for each one. the switch also has a default : clause that catches bad input.

Enclose the whole thing in a loop like this:

Branflakes91093 wrote:

If you want to use multiple statements within an if statement, you need to use braces.

In fact, I’d go further: as a matter of good practice, you should always put braces around the code that is executed conditionally, even if it is only a single statement. It is frighteningly easy to introduce bugs by going back and adding a second statement, and forgetting to add the braces. I’ve seen this happen in every single one of the jobs I’ve worked at, and I’ve done it a few times myself.

It’s much safer to protect yourself from this by putting in the braces every time.

Источник

[Need Help] Else with no matching IF; IfMsgbox woes.

Hi everyone, this is day one for me in scripting. I have read the tutorials and searched endlessly on the Msgbox, Else, If/Else and all other related hotlinks in the beginners section. Unfortunately, I have not solved it solo. Here is the script. It keeps telling me the Else line has no matching IF. I can comprend why this is occurring; however, I cannot understand how to implement this with the IFMsgbox command.

Ultimately, I need the script to «Goto» either the «Top» or «Bottom» script depending on how the User inputs their selection.

It should be noted that I copy pasted this first area from http://www.autohotke. ButtonNames.htm and modified it for my own use.

SetTimer, ChangeButtonNames, 50
MsgBox, 4, Eel/Octopus or Hatch/Flycatcher, Which Fish are you turning in?
IfMsgBox, YES
MsgBox, Turning in Eel/Octopus.
Goto, Top
Else,
MsgBox, Turning in Hatchling/Flycatcher.
Goto, Bottom

ChangeButtonNames:
IfWinNotExist, Eel/Octopus or Hatch/Flycatcher
return ; Keep waiting.
SetTimer, ChangeButtonNames, off
WinActivate
ControlSetText, Button1, &Eel/Octopus
ControlSetText, Button2, &Hatch/Fly
return

;Top-Loop: ;**********Top Selection*************
<
;Sleep, 200
;MouseClick, right, xNpc, yNpc ;RIGHT CLICK NPC
;Sleep, 700
;MouseClick, left, xTop, yTop ;Eel or Octopus (Top)
;Sleep, 700
;MouseClick, left, xRep, yRep ;Rep Reward
;Sleep, 700
;MouseClick, left, xFin, yFin ;Finish Button
;Sleep, 700
;MouseClick, left, xClo, yClo ;Close Box via «X»
;Sleep, 700
;Goto, Top
Goto, Start
>

Bottom-Loop: ;**********Bottom Selection***********
<
Sleep, 200
MouseClick, right, xNpc, yNpc ;RIGHT CLICK NPC
Sleep, 700
;MouseClick, left, xBot, yBot ;Hatchling or Flycatcher (Bottom)
;Sleep, 700
;MouseClick, left, xRep, yRep ;Rep Reward
;Sleep, 700
;MouseClick, left, xFin, yFin ;Finish Button
;Sleep, 700
;MouseClick, left, xClo, yClo ;Close Box via «X»
;Sleep, 700
Goto, Bottom
>

::Pause
]::Reload
Esc::ExitApp

Источник

#RequireAdmin
#include <GUIConstantsEx.au3>
#include <Date.au3>
#include <WindowsConstants.au3>
#include <Timers.au3>
#include <StaticConstants.au3>
#include <WinAPIProc.au3>
#Include <ScreenCapture.au3>
#include <WinAPI.au3>
#include <Clipboard.au3>
#include <GDIPlus.au3>

Global  $sPS, $sPD
Global $iPid, $iTM
Global $Counter = IniRead("Setting.ini","Statistika","Count","0")

Global $GP = IniRead("Setting.ini","GPat","Pat","")

$hGui = GUICreate('Секундомер', 700, 680, -1, -1, -1, $WS_EX_TOPMOST)
Local $btStart = GUICtrlCreateButton('Старт', 392, 8, 75, 25)
Local $sPathSS = GUICtrlCreateInput('', 8, 96, 325, 21)
Local $sPathS = GUICtrlCreateButton('Откуда', 392, 96, 75, 25)
GUISetState()
While 1
    Switch GUIGetMsg()
        Case $GUI_EVENT_CLOSE

            ProcessClose($iPid)
            Exit
        Case $sPathS
            GUICtrlSetData($sPathSS, '')
            $sPS = FileSelectFolder('Выбор копируемой папки', '', 0, '', $hGui)
            If Not @error Then GUICtrlSetData($sPathSS, $sPS)
        Case $btStart
            _GetPath()
EndSwitch
WEnd

Func _GetPath()
Local $GPath1 =IniWrite("Setting.ini","Путь1","Путь",'')
Local $GPath2 =IniWrite("Setting.ini","Путь2","Путь",'')
Local $GPath3 =IniWrite("Setting.ini","Путь3","Путь",'')
Local $GPath4 =IniWrite("Setting.ini","Путь4","Путь",'')
  If $GPath1 ='' Then $GPath1 = $sPS
IniWrite("Setting.ini","Путь1","Путь",$sPS)
  ElseIf $GPath2 ='' Then $GPath2 = $sPS
IniWrite("Setting.ini","Путь2","Путь",$sPS)
  Else $GPath3 ='' Then $GPath3 = $sPS
IniWrite("Setting.ini","Путь3","Путь",$sPS)
   EndFunc   ;==>_GetPath

I noticed this behaviour and it did not seem very intuitive to me:

This script will generate an error @ line 4 Error: «Else» statement with no matching «If» statement

Global $iTest = 1

If $iTest = 1 Then MsgBox(0, "Debug", "Test was true")
ElseIf $iTest = 0 Then MsgBox(0, "Debug", "Test was false")
Else 
    MsgBox(0, "Debug", "Test was naughty")
EndIf

This script runs without error

Global $iTest = 1

If $iTest = 1 Then 
    MsgBox(0, "Debug", "Test was true")
ElseIf $iTest = 0 Then MsgBox(0, "Debug", "Test was false")
Else 
    MsgBox(0, "Debug", "Test was naughty")
EndIf

So after an If conditional, Then must be followed by a newline, if the conditional statement is to be followed by an ElseIf.

But after an ElseIf conditional, Then does not need to be followed by a newline, even if an Else follows its conditional

This script also runs without error:

Global $iTest = 1

If $iTest = 1 Then 
    MsgBox(0, "Debug", "Test was true")
ElseIf $iTest = 0 Then MsgBox(0, "Debug", "Test was false")
ElseIf $iTest = 5 Then MsgBox(0, "Debug", "Test was crazy")
Else 
    MsgBox(0, "Debug", "Test was naughty")
EndIf

But this will cause an error:

Global $iTest = 1

If $iTest = 1 Then 
    MsgBox(0, "Debug", "Test was true")
ElseIf $iTest = 0 Then MsgBox(0, "Debug", "Test was false")
ElseIf $iTest = 5 Then MsgBox(0, "Debug", "Test was crazy")
Else MsgBox(0, "Debug", "Test was naughty")
EndIf

Else needs a new line before its statement.

For me it is a little surprising that I must have a newline after the first Then, if I am going to have ElseIf’s following it. Perhaps it would be useful to remove the dependence on the newline?