Loops Bug

[Jos]

This code is really wrong in many many ways.

You are using CAll.

You are "calling" Restart from within itself .... will cause a stack overflow.

Run Tidy and it will tell you are missing an EndIF..

First try to make some code that works .. then add the nice to haves ...

[Dieuz]

This code is really wrong in many many ways.

You are using CAll.

You are "calling" Restart from within itself .... will cause a stack overflow.

Run Tidy and it will tell you are missing an EndIF..

First try to make some code that works .. then add the nice to haves ...

Yes your right with the stack overflow. Ho yeah, and sorry if this code is wrong in many many many many ways but it's my first one and it's why I'm here to ask...How could I correct this stack overflow because I think it's the priority.

Run Tidy and it will tell you are missing an EndIF..

I just forgot it when I copied/pasted it on the forum. My script can run fine during 4-5 hours but it close due to this stack overflow. Edited December 17, 2006 by Dieuz

[mikehunt114]

How could I correct this stack overflow because I think it's the priority.

Rework your code so that functions are not calling themselves. Most likely you can use a loop to do what you desire.

[Dieuz]

Rework your code so that functions are not calling themselves. Most likely you can use a loop to do what you desire.

Do you think that those modificiations will correct the stack overflow? I changed the call for a While/Wend so the function is not calling itself.

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()    ; happen only one time

;Stuff here

Restart()
EndFunc

Func Restart()
While   
;Stuff here

;5 WINS
If 


Else


;stuff here


EndIf
Wend
EndFunc


Func Pause()

;stuff here
    
EndFunc

Func Terminate()
  Exit(0)
  EndFunc

I incorporated the winner function in the Restart function.

Edited December 18, 2006 by Dieuz

[mikehunt114]

Do you think that those modificiations will correct the stack overflow?

Yes, I do.

[Dieuz]

Yes, I do.

Cheers!