lpmen

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

6 posts in this topic

#1 ·  Posted (edited)

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 by lpmen

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

this is your problem:

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

EDIT:

Welcome to the forum.

mLipok

Edited by mLipok

Signature beginning:   Wondering who uses AutoIT and what it can be used for ?
* GHAPI UDF - modest begining - comunication with GitHub REST API *
ADO.au3 UDF     POP3.au3 UDF     XML.au3 UDF    How to use IE.au3  UDF with  AutoIt v3.3.14.x  for other useful stuff click the following button

Spoiler

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

My contribution (my own projects): * Debenu Quick PDF Library - UDF * Debenu PDF Viewer SDK - UDF * Acrobat Reader - ActiveX Viewer * UDF for PDFCreator v1.x.x * XZip - UDF * AppCompatFlags UDF * CrowdinAPI UDF * _WinMergeCompare2Files() * _JavaExceptionAdd() * _IsBeta() * Writing DPI Awareness App - workaround * _AutoIt_RequiredVersion() * Chilkatsoft.au3 UDF * TeamViewer.au3 UDF * JavaManagement UDF * VIES over SOAP * WinSCP UDF * GHAPI UDF - modest begining - comunication with GitHub REST API *

My contribution to others projects or UDF based on  others projects: * _sql.au3 UDF  * POP3.au3 UDF *  RTF Printer - UDF * XML.au3 - BETA * ADO.au3 UDF SMTP Mailer UDF *

Useful links: * Forum Rules * Forum etiquette *  Forum Information and FAQs * How to post code on the forum * AutoIt Online Documentation * AutoIt Online Beta Documentation * SciTE4AutoIt3 getting started * Convert text blocks to AutoIt code * Games made in Autoit * Programming related sites * Polish AutoIt Tutorial * DllCall Code Generator * 

Wiki: Expand your knowledge - AutoIt Wiki * Collection of User Defined Functions * How to use HelpFile * Best coding practices * 

IE Related:  * How to use IE.au3  UDF with  AutoIt v3.3.14.x * Why isn't Autoit able to click a Javascript Dialog? * Clicking javascript button with no ID * IE document >> save as MHT file * IETab Switcher (by LarsJ ) * HTML Entities * _IEquerySelectorAll() (by uncommon) * 

I encourage you to read: * Global Vars * Best Coding Practices * Please explain code used in Help file for several File functions * OOP-like approach in AutoIt * UDF-Spec Questions *  EXAMPLE: How To Catch ConsoleWrite() output to a file or to CMD *

"Homo sum; humani nil a me alienum puto" - Publius Terentius Afer
"Program are meant to be read by humans and only incidentally for computers and execute" - Donald Knuth, "The Art of Computer Programming"
:naughty:  :ranting:, be  :) and       \\//_.

Anticipating Errors :  "Any program that accepts data from a user must include code to validate that data before sending it to the data store. You cannot rely on the data store, ...., or even your programming language to notify you of problems. You must check every byte entered by your users, making sure that data is the correct type for its field and that required fields are not empty."

Signature last update: 2017-06-04

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

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

it says line 18 sleep(1)

Edited by lpmen

Share this post


Link to post
Share on other sites
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

Signature beginning:   Wondering who uses AutoIT and what it can be used for ?
* GHAPI UDF - modest begining - comunication with GitHub REST API *
ADO.au3 UDF     POP3.au3 UDF     XML.au3 UDF    How to use IE.au3  UDF with  AutoIt v3.3.14.x  for other useful stuff click the following button

Spoiler

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

My contribution (my own projects): * Debenu Quick PDF Library - UDF * Debenu PDF Viewer SDK - UDF * Acrobat Reader - ActiveX Viewer * UDF for PDFCreator v1.x.x * XZip - UDF * AppCompatFlags UDF * CrowdinAPI UDF * _WinMergeCompare2Files() * _JavaExceptionAdd() * _IsBeta() * Writing DPI Awareness App - workaround * _AutoIt_RequiredVersion() * Chilkatsoft.au3 UDF * TeamViewer.au3 UDF * JavaManagement UDF * VIES over SOAP * WinSCP UDF * GHAPI UDF - modest begining - comunication with GitHub REST API *

My contribution to others projects or UDF based on  others projects: * _sql.au3 UDF  * POP3.au3 UDF *  RTF Printer - UDF * XML.au3 - BETA * ADO.au3 UDF SMTP Mailer UDF *

Useful links: * Forum Rules * Forum etiquette *  Forum Information and FAQs * How to post code on the forum * AutoIt Online Documentation * AutoIt Online Beta Documentation * SciTE4AutoIt3 getting started * Convert text blocks to AutoIt code * Games made in Autoit * Programming related sites * Polish AutoIt Tutorial * DllCall Code Generator * 

Wiki: Expand your knowledge - AutoIt Wiki * Collection of User Defined Functions * How to use HelpFile * Best coding practices * 

IE Related:  * How to use IE.au3  UDF with  AutoIt v3.3.14.x * Why isn't Autoit able to click a Javascript Dialog? * Clicking javascript button with no ID * IE document >> save as MHT file * IETab Switcher (by LarsJ ) * HTML Entities * _IEquerySelectorAll() (by uncommon) * 

I encourage you to read: * Global Vars * Best Coding Practices * Please explain code used in Help file for several File functions * OOP-like approach in AutoIt * UDF-Spec Questions *  EXAMPLE: How To Catch ConsoleWrite() output to a file or to CMD *

"Homo sum; humani nil a me alienum puto" - Publius Terentius Afer
"Program are meant to be read by humans and only incidentally for computers and execute" - Donald Knuth, "The Art of Computer Programming"
:naughty:  :ranting:, be  :) and       \\//_.

Anticipating Errors :  "Any program that accepts data from a user must include code to validate that data before sending it to the data store. You cannot rely on the data store, ...., or even your programming language to notify you of problems. You must check every byte entered by your users, making sure that data is the correct type for its field and that required fields are not empty."

Signature last update: 2017-06-04

Share this post


Link to post
Share on other sites

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

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

Share this post


Link to post
Share on other sites

#6 ·  Posted (edited)

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:

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 by SmOke_N

[center]Common sense plays a role in the basics of understanding AutoIt... If you're lacking in that, do us all a favor, and step away from the computer.[/center]

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