Loops Bug

[Dieuz]

Hi there!

I'm having a little problem with a part of my script

For ???????????????????
    
$ColorAttack = Pixelgetcolor(840,467)
$ColorVictory = Pixelgetcolor(507,457)
    
If $ColorAttack ="5902484" Then
MouseClick ( "left" , 505, 455 , 1 , 1  )

ElseIf $ColorVictory ="15716261" Then
MouseClick ( "left" , 512, 530 , 1 , 1  )
sleep(2000)
Call ("Winner")

Endif
Next

I want the script to loop but I dont know what to write after For. I tried with $n = 1 to $ColorAttack and the script worked fine for the first part but when it should have launched the ElseIf , nothing happened.

Thanks alot!

[mikehunt114]

How many times do you want this to loop? A For...Next loop will execute however many times you specify. If you want the script to loop indefinitely try wrapping it in:

While 1
    ;stuff here
WEnd

[Dieuz]

Well it loop alot of time until it suppose to reach my ElseIf ( when a new windows pop up). It loops like 10-15 time before it supposed to reach the ElseIf.

I tried with While 1 and Wend but the ElseIf is suppose to work, nothing happened. Maybe the problem come from there...

CODE

CODE

While 1

$ColorAttack = Pixelgetcolor(840,467)

$ColorVictory = Pixelgetcolor(507,457)

If $ColorAttack ="5902484" Then

MouseClick ( "left" , 505, 455 , 1 , 1 )

ElseIf $ColorVictory ="15716261" Then

MouseClick ( "left" , 512, 530 , 1 , 1 )

sleep(2000)

Call ("Winner")

Endif

Wend

Edited December 15, 2006 by Dieuz

[mikehunt114]

Perhaps a Do...Until loop would better suit your needs. The examples in the helpfile show how each loop can be utilized.

Do    
    $ColorAttack = Pixelgetcolor(840,467)
    $ColorVictory = Pixelgetcolor(507,457)      
    If $ColorAttack ="5902484" Then MouseClick ( "left" , 505, 455 , 1 , 1  )
Until $ColorVictory ="15716261"
MouseClick ( "left" , 512, 530 , 1 , 1  )
sleep(2000)
Call ("Winner")

Is that more like what you are trying to do?

[Dieuz]

Yes I want something like this. I tried your code but the same thing happened again, the ElseIf doesnt work...

[mikehunt114]

Yes I want something like this. I tried your code but the same thing happened again, the ElseIf doesnt work...

First of all, there are no ElseIfs in the snippet I posted. Secondly, just saying something "doesn't work" is useless. If you want more help, you're going to have to explain what is happening, ie. describe what you expect to happen and the errors you are getting. Perhaps your color code is incorrect.

[Dieuz]

Yes sorry about that but I did not know how to explain my problem. But you was right, my color code was incorrect. I changed that and it's working now. Thanks , greatly apreciate !

[mikehunt114]

Welcome.

[Dieuz]

New Issue: Count Variable

I want to set a splash text everytime my character won a combat and the numbers on the splash text will change. The number will change everytime the script reach this point. Wich variables should I use to achieve that? The variable start at 0 have to change +1 each time.

$TimesWon = 

; Displays a splash text to show the number of combats won
SplashTextOn ("", "Combats Won: $TimesWon" , 290, 20, 0, 30, 1, "", 10)

Edited December 16, 2006 by Dieuz

[Jos]

$TimesWon = 0
;
;
;
; Displays a splash text to show the number of combats won
$TimesWon += 1
SplashTextOn ("", "Combats Won: " & $TimesWon , 290, 20, 0, 30, 1, "", 10)

[Dieuz]

$TimesWon = 0
;
;
;
; Displays a splash text to show the number of combats won
$TimesWon += 1
SplashTextOn ("", "Combats Won: " & $TimesWon , 290, 20, 0, 30, 1, "", 10)

I tried your code and Im getting this error:

C:\Documents and Settings\XXXXX : ==> Expected a "=" operator in assignment statement.: 
$Trophyswon += 1 
$Trophyswon ^ ERROR

My script close instantly.

Anyone else know how to set my splash text? The variable start at 0 and +1 is added everytime the script pass over it.

Edited December 17, 2006 by Dieuz

[Jos]

What version of AutoIt3 are you using ?

On older versions:

$Trophyswon = $Trophyswon + 1

[Dieuz]

Thanks , it's working ! I have an older version, I will upgrade it.

Quick question: What is the operator used to express " is not equal to"?

I tried with $ColorBug NOT "7567456" but im getting an error.

Edited December 17, 2006 by Dieuz

[Dieuz]

I made some tests and found that finally it wasnt working.

The number always stay 1 because:

$Trophyswon =0

$Trophyswon = $Trophyswon + 1 
SplashTextOn ("", "Combats Won: " & $TimesWon , 290, 20, 0, 30, 1, "", 10)

It's always 0 +1, so always 1. $Trophyswon should add + 1 everytime the script pass this code but now it isnt.

Example of what I want:

First time the script reach this part : $Trophyswon = 1

Second time the script reach this part : $Trophyswon = 2

Third time the script reach this part : $Trophyswon = 3

Fourth time the script reach this part : $Trophyswon = 4

etc...

Edited December 17, 2006 by Dieuz

[Jos]

I made some tests and found that finally it wasnt working.

The number always stay 1 because:

$Trophyswon =0

$Trophyswon = $Trophyswon + 1 
SplashTextOn ("", "Combats Won: " & $TimesWon , 290, 20, 0, 30, 1, "", 10)
oÝ÷ ØwöÆ¥Á¬¬Óíl¡©pk+5Ó~®aÊÌ(ÈhºWZußµz÷«ÊئzØ^±Êâ¦ÚZ²ËaÇ(uæî¶z0جÑ1jjezð«HÁ©í*ì¶Ø¦zØ^±Êâ¦ÚÞiÈm+)j»tߤë¢r³
'

Edited December 17, 2006 by JdeB

[Dieuz]

I know, I have 150 lines of script. I put it somewhere in my script but everytime the script pass over it, the number stay 1. It's why im wondering if this code is correct.

[Jos]

I know, I have 150 lines of script. I put it somewhere in my script but everytime the script pass over it, the number stay 1. It's why im wondering if this code is correct.

Yea guess my code is wrong somewhere ...

[GEOSoft]

Yea guess my code is wrong somewhere ...

OK smartguy, You made me take a second look. I should have known better but for a split second I thought maybe it was you tha put the $Trophyswon = 0 inside the loop. In that case of course $Trohpyswon would always = 1.

Time to take a break. If I was a drinking man I'd have a stiff belt right about now.

[Dieuz]

Ok I feel stupid because I cant correct this simple problem. Have a look at my core script and see if you can find where's the problem come from.

expandcollapse popup

;Hotkeys
HotKeySet("{HOME}", "Terminate") ;
HotKeySet("{F2}", "Start") ; 
HotKeySet("{F4}", "Pause") ; 

;Variables

Global $Paused
$ColorAttack = Pixelgetcolor(837,464)
$ColorVictory = Pixelgetcolor(402,169)
$Color5wins = Pixelgetcolor(595,387)
$Trophyswon = 0
$ColorBug = Pixelgetcolor(840,467)



While 1
    sleep(100)
Wend

Func Start()

;Stuff here

Call ("Restart")
EndFunc

Func Restart()
    
;Stuff here

;5 WINS
If $Color5wins ="5911585" Then
    Call ("Winner")

;Stuff here

Call ("Restart")
EndFunc



Func Winner()

;stuff here
        
$Trophyswon = $Trophyswon + 1 
SplashTextOn ("", "Trophys Won: " & $Trophyswon, 290, 20, 0, 30, 1, "", 10) 

;stuff here

Call ("Restart")
EndFunc

Func Pause()

;stuff here
    
EndFunc

Func Terminate()
  Exit(0)
  EndFunc

With this script, im always getting $Trohpyswon=1

Edited December 17, 2006 by Dieuz

[GEOSoft]

Ok I feel stupid because I cant correct this simple problem. Have a look at my core script and see if you can find where's the problem come from.

expandcollapse popup

;Hotkeys
HotKeySet("{HOME}", "Terminate") ;
HotKeySet("{F2}", "Start") ; 
HotKeySet("{F4}", "Pause") ; 

;Variables

Global $Paused
$ColorAttack = Pixelgetcolor(837,464)
$ColorVictory = Pixelgetcolor(402,169)
$Color5wins = Pixelgetcolor(595,387)
$Trophyswon = 0
$ColorBug = Pixelgetcolor(840,467)
While 1
    sleep(100)
Wend

Func Start()

;Stuff here

Call ("Restart")
EndFunc

Func Restart()
    
;Stuff here

;5 WINS
If $Color5wins ="5911585" Then
    Call ("Winner")

;Stuff here

Call ("Restart")
EndFunc
Func Winner()

;stuff here
        
$Trophyswon = $Trophyswon + 1 
SplashTextOn ("", "Trophys Won: " & $Trophyswon, 290, 20, 0, 30, 1, "", 10) 

;stuff here

Call ("Restart")
EndFunc

Func Pause()

;stuff here
    
EndFunc

Func Terminate()
  Exit(0)
  EndFunc

With this script, im always getting $Trohpyswon=1

a splash won't update automatically

$Trophyswon = $Trophyswon + 1

SplashTextOn ("", "Trophys Won: " & $Trophyswon, 290, 20, 0, 30, 1, "", 10)

See ControlSetText in the help file under Window Management / Controls

Doing it without ControlSetText tou would hve to do

SplashTextOn ("", "Trophys Won: " & $Trophyswon, 290, 20, 0, 30, 1, "", 10)

Sleep(5000) ;; Use a time long enough to read the message and then shut it off.

SplashOff()

You are better off with ControlSetText.

A better answer yet is to get rid of the Splash altogether and create a GUI the same size as your splash. Then create a lable the full size of the gui. With that arrangement you could use

$Trophyswon = $trophyswon +1

GuiCtrlSetData($my_Label("Trophies won " & $Trophyswon)

Edited December 17, 2006 by GEOSoft