need help Recursion level has been exceeded - AutoIt will quit to prevent stack overflow

[lpmen]

i am trying to run my program but it will crash after some minuets, i have search the forum and did not found any useful answer.

Recursion level has been exceeded - AutoIt will quit to prevent stack overflow.:

sleep(1)

sleep(500)

$i=0
$g=2
change($i)
func change($i) ; function changing
      if $i >= $g Then
      $i=0
   MouseClick( "left",1,1)
  Tooltip("1", 0, 0)
      sleep(1)
      send("{enter}")
         sleep(1)
         $i=$i+1
          search($i)
      EndIf
      MouseClick("left" ,1,1)
      sleep(1)
      send("{down}")
      sleep(1)
      send("{enter}")
      $i=$i+1
      Tooltip($i, 0, 0)
      search($i)

   EndFunc

   Func search($i)
      sleep(1)
      change($i)
   EndFunc

Edited March 17, 2015 by lpmen

[mLipok]

this is your problem:

Func search($i)
      sleep(1)
      change($i) ; <<<---===<  exactly HERE
   EndFunc

EDIT:

Welcome to the forum.

mLipok

Edited March 17, 2015 by mLipok

[lpmen]

can you help me to edit this script that it will not crush ?

it says line 18 sleep(1)

Edited March 17, 2015 by lpmen

[mLipok]

With your script is not clear what it is designed to do.

I do not know why many times you call the function again.

 

You need to be more detailed in the description of what you want to achieve.

 

mLipok

[lpmen]

this is the 2 functions that are part of the program the most of the code he is doing this :

expandcollapse popup

func change($i) ; function changing
      if $i >= $g Then
      $i=0
   MouseClick( "left",91,72)
  Tooltip("1", 0, 0)
      sleep(100)
      send("{enter}")
         sleep(5000)
         $i=$i+1
          search($i)
      EndIf
      MouseClick("left" ,102,900)
      sleep(100)
      send("{down}")
      sleep(100)
      send("{enter}")
      $i=$i+1
      Tooltip($i, 0, 0)
      search($i)

   EndFunc

func search($i) ; function checking
      $te=0
      $p=0
      $f=0
      $m=0
      $z=0
      sleep(1200)
      while True
         sleep(100)
  $p = PixelSearch( 250,401,750,900,0xFFAA21,4);search counti
   if IsArray ($p) Then
      $f = PixelSearch( $p[0]+5,$p[1]-45,$p[0]+6,$p[1]-20,0xFFEFCE);search problem
   if IsArray ($f) Then
       $m = PixelSearch( 250,300,750,800,0x737573,4);search for the radiobuton color.
    if IsArray ($m) Then
      sleep(200)
           $m[1]=$m[1]+145
       MouseClick( "left" ,$m[0],$m[1]) ; radio button
       sleep(3000)
   $m = PixelSearch( 250,300,750,800,0xFFAA21,4);search for cotniue after radio button fixed
    if IsArray ($m) Then
        MouseClick( "left" ,$m[0],$m[1])
        change($i)
        EndIf
    EndIf
 EndIf
      MouseClick( "left" ,$p[0],$p[1])
      sleep($time)
      Tooltip($i, 0, 0)
       $z = PixelSearch( 650,400,1150,900,0xFFAA21,4) ;search completet after review
      if IsArray ($z) Then
         sendd($i) ; send to other function and end the program.
      EndIf
      change($i)
   EndIf

[SmOke_N]

You're calling search() from change() and change() from search().  So they keep going into one another over and over without the ability to get back to the first time they were called.  This is recursion (where the first instance of the call is seamlessly lost and has little to no chance to ever get back).

Your code was incomplete (missing WEnd and EndFunc I presume), but I would change the calls to change() from search() to return:

expandcollapse popup

Func change($i) ; function changing
    If $i >= $g Then
        $i = 0
        MouseClick("left", 91, 72)
        ToolTip("1", 0, 0)
        Sleep(100)
        Send("{enter}")
        Sleep(5000)
        $i = $i + 1
        search($i)
    EndIf
    MouseClick("left", 102, 900)
    Sleep(100)
    Send("{down}")
    Sleep(100)
    Send("{enter}")
    $i = $i + 1
    ToolTip($i, 0, 0)
    search($i)

EndFunc   ;==>change

Func search($i) ; function checking
    $te = 0
    $p = 0
    $f = 0
    $m = 0
    $z = 0
    Sleep(1200)
    While True
        Sleep(100)
        $p = PixelSearch(250, 401, 750, 900, 0xFFAA21, 4);search counti
        If IsArray($p) Then
            $f = PixelSearch($p[0] + 5, $p[1] - 45, $p[0] + 6, $p[1] - 20, 0xFFEFCE);search problem
            If IsArray($f) Then
                $m = PixelSearch(250, 300, 750, 800, 0x737573, 4);search for the radiobuton color.
                If IsArray($m) Then
                    Sleep(200)
                    $m[1] = $m[1] + 145
                    MouseClick("left", $m[0], $m[1]) ; radio button
                    Sleep(3000)
                    $m = PixelSearch(250, 300, 750, 800, 0xFFAA21, 4);search for cotniue after radio button fixed
                    If IsArray($m) Then
                        MouseClick("left", $m[0], $m[1])
                        Return;change($i)
                    EndIf
                EndIf
            EndIf
            MouseClick("left", $p[0], $p[1])
            Sleep($time)
            ToolTip($i, 0, 0)
            $z = PixelSearch(650, 400, 1150, 900, 0xFFAA21, 4) ;search completet after review
            If IsArray($z) Then
                sendd($i) ; send to other function and end the program.
            EndIf
            Return;change($i)
        EndIf
    WEnd
EndFunc

If this doesn't work, then you'll need to write the code more simplistic and put change/search into one func or create a separate function that does the leg work that doesn't call other functions you've created.

Edited March 17, 2015 by SmOke_N