Jump to content
Sign in to follow this  
HiNoTora

"Wend" statement with no matching "While" statement.:

Recommended Posts

HiNoTora
MsgBox(4096, "AutoTrade started", "Press ESC to exit")
HotKeySet("{ESC}", "Terminate")
while 1
$coord = PixelSearch(17, 753, 37, 765, 0x016702)
If Not @error Then
    MouseClick("left", 1265, 738, 1)
            Sleep ( 300 )
            EndIf       
        If @error Then         
    Sleep ( 100 )
 EndIf
  wend 
  
while 1
$coord = PixelSearch(22, 749, 49, 760, 0xFFFFFF)
If Not @error Then
    $coord = PixelSearch(280, 123, 350, 157, 0xD12222)
 If Not @error Then
    MouseClick("primary", $coord[0], $coord[1], 1, 0)
    If @error Then  
       Sleep ( 100 )
    EndIf
    WEnd   
    
     Func Terminate()
    Exit
EndFunc

 

I get this error while i try test my script.
 
>"C:Program Files (x86)AutoIt3SciTE..autoit3.exe" /ErrorStdOut "C:UsersderDocumentsTest.au3"
C:UsersderDocumentsTest.au3 (23) : ==> "Wend" statement with no matching "While" statement.:
WEnd
 
>Exit code: 1    Time: 0.219
 
 
can u help me fix it, and since i havent test it, if u see any mistakes, can u help me?
Edited by HiNoTora

Share this post


Link to post
Share on other sites
kaotkbliss

you are missing a couple endif's before your 2nd wend

each 'if then' must have a matching endif


010101000110100001101001011100110010000001101001011100110010000

001101101011110010010000001110011011010010110011100100001

My Android cat and mouse game
https://play.google.com/store/apps/details?id=com.KaosVisions.WhiskersNSqueek

We're gonna need another Timmy!

Share this post


Link to post
Share on other sites
Melba23

HiNoTora,

You are missing 2 EndIf statements before the second WEnd. :)

Do you use the full SciTE4Autoit3 package? It includes a couple of useful utilties which would have found this very quickly - it is how I did it. You can download it from here. ;)

M23


Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind

Open spoiler to see my UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites
HiNoTora

you are missing a couple endif's before your 2nd wend

each 'if then' must have a matching endif

got it fixed thank you

Share this post


Link to post
Share on other sites
HiNoTora

Got fixed thanks for help =)

Share this post


Link to post
Share on other sites
HiNoTora

for some reason i cant get second part of script to work.

Share this post


Link to post
Share on other sites
Colyn1337

Same deal as before..... You're missing your endif

Thing to remember is that all loops and conditionals need terminating.

The ONLY exception to this that I can think of is the one line if/then statement like so:

If $anyVar = 1 Then ConsoleWrite("blabla" & @LF)

EDIT:

Something else you'll find useful is the "Tidy" app inside of AutoIt.  Hit CTRL+T and it'll execute.  It puts your code in order for humans and does some basic script checking, like making sure everything is terminated and closed.  When it finds an error, it tells you where and usually what to do to fix it.

Edited by Colyn1337

Share this post


Link to post
Share on other sites
BrewManNH

You've got 2 While loops, and there's no way for it to exit the first one. The second loop will never execute.

When does it need to execute the second loop?


If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Share this post


Link to post
Share on other sites
Colyn1337

Good Catch BrewManNH, I missed that one...

Share this post


Link to post
Share on other sites
HiNoTora

You've got 2 While loops, and there's no way for it to exit the first one. The second loop will never execute.

When does it need to execute the second loop?

exactly that what i ment.

kinda they shold loop both at same time or... 

it has to start work after 8th line of script.

Share this post


Link to post
Share on other sites
HiNoTora

MsgBox(4096, "AutoTrade started", "Press ESC to exit")
HotKeySet("{ESC}", "Terminate")
while 1
$coord = PixelSearch(17, 753, 37, 765, 0x90F393); wait till positive position
If Not @error Then
    MouseClick("left", 1265, 738, 1); closing position
            Sleep ( 300 )
            EndIf       
        If @error Then         
    Sleep ( 100 )
 EndIf
  wend 
 
while 1
$coord = PixelSearch(22, 749, 49, 760, 0xFFFFFF); is position opend?
If Not @error Then
    $coord = PixelSearch(280, 123, 350, 157, 0xD32424); waiting for sale mark
    If @error Then  
       Sleep ( 100 )
 If Not @error Then
    MouseClick("primary", $coord[0], $coord[1], 1, 0); opening new position
    If @error Then  
       Sleep ( 100 )
    EndIf
 EndIf
 EndIf
 EndIf
    wend   
    
     Func Terminate()
    Exit
EndFunc

Share this post


Link to post
Share on other sites
Melba23

HiNoTora,

 

AutoTrade started

Now that has set my antennae quivering. Would you care to explain in what app you are "autotrading"? I take it you have read the Forum rules (there is also a link at bottom right of each page)? :huh:

M23


Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind

Open spoiler to see my UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites
Melba23

JLogan3o13,

No need to apologise - I would have expected no less. ;)

M23

Edited by Melba23
Typo

Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind

Open spoiler to see my UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites
HiNoTora

HiNoTora,

 

Now that has set my antennae quivering. Would you care to explain in what app you are "autotrading"? I take it you have read the Forum rules (there is also a link at bottom right of each page)? :huh:

M23

its not a game, its forex program, is that banned to in this forum? if so then so sry >.<

Share this post


Link to post
Share on other sites
Melba23

HiNoTora,

 

its not a game, its forex program

That is fine and my apologies for having asked, but I hope you understand why I did. :)

M23


Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind

Open spoiler to see my UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites
kaotkbliss

you would need to combine both while loops into 1

maybe something like

MsgBox(4096, "AutoTrade started", "Press ESC to exit")
HotKeySet("{ESC}", "Terminate")
while 1
$coord = PixelSearch(17, 753, 37, 765, 0x016702)
$coord2 = PixelSearch(22, 749, 49, 760, 0xFFFFFF)
If IsArray($coord) Then
    MouseClick("left", 1265, 738, 1)
    Sleep ( 300 )
EndIf
If IsArray($coord2) Then
    $coord3 = PixelSearch(280, 123, 350, 157, 0xD12222)
    If IsArray($coord3) Then
        MouseClick("primary", $coord3[0], $coord3[1], 1, 0)
    Else
        Func Terminate()
    EndIf
EndIf

Sleep ( 100 )

wend 

Func Terminate()
    Exit
EndFunc
Edited by kaotkbliss

010101000110100001101001011100110010000001101001011100110010000

001101101011110010010000001110011011010010110011100100001

My Android cat and mouse game
https://play.google.com/store/apps/details?id=com.KaosVisions.WhiskersNSqueek

We're gonna need another Timmy!

Share this post


Link to post
Share on other sites
HiNoTora

HiNoTora,

 

That is fine and my apologies for having asked, but I hope you understand why I did. :)

M23

i do, btw this program has automatication allowed, it has even script builder inside, but it has different script language, so cant use this

Share this post


Link to post
Share on other sites
HiNoTora

 

you would need to combine both while loops into 1

maybe something like

MsgBox(4096, "AutoTrade started", "Press ESC to exit")
HotKeySet("{ESC}", "Terminate")
while 1
$coord = PixelSearch(17, 753, 37, 765, 0x016702)
$coord2 = PixelSearch(22, 749, 49, 760, 0xFFFFFF)
If IsArray($coord) Then
    MouseClick("left", 1265, 738, 1)
    Sleep ( 300 )
EndIf
If IsArray($coord2) Then
    $coord3 = PixelSearch(280, 123, 350, 157, 0xD12222)
    If IsArray($coord3) Then
        MouseClick("primary", $coord3[0], $coord3[1], 1, 0)
    Else
        Func Terminate()
    EndIf
EndIf

Sleep ( 100 )

wend 

Func Terminate()
    Exit
EndFunc
>"C:Program Files (x86)AutoIt3SciTE..autoit3.exe" /ErrorStdOut "C:UsersderDocumentsForex.au3"
C:UsersderDocumentsForex.au3 (23) : ==> "Func" statement has no matching "EndFunc".:
Func Terminate()
 
>Exit code: 1    Time: 0.295

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

×