Script not repeating itself

[sebbulon]

I have a problem with this script after it finishes the pause, after that it does one repetition of "dowork" and then i have to press ''space'' for it to continue, i want it to just continue without me pressing anything.

Code: Select all

#Singleinstance force
#persistent 


Loop
    {

Random, WorkTime, 20, 23  ; seconds
minpause:=14     ; seconds
maxpause:=15      ; seconds



	;;RANDOM VARIABLES;;,

		;;ITEM AND HIGH ALCH;;
			Random x1, 730, 735
			Random y1, 285, 289
		
		;;RANDOM SLEEPS;;
			Random a150, 133, 238
			Random a2432, 2398, 2879



    SetTimer, startpause, % -Worktime*1000
    working:=True
    While working
         gosub dowork     
    random, rand, minpause, maxpause	
    sleep % rand*1000
    random, rand, WorkTime	
    sleep % rand*1000


startpause:


dowork:
Mousemove, x1, y1, 13
	sleep a150
	click
	sleep a150
	click
	sleep a2432
	

return

}

[mikeyww]

After your Gosub, the script continues straight through to the bottom because you have no intervening Return. Putting a subroutine inside a loop is also tricky-- not usually helpful, and sometimes leading to unintended effects.

[sebbulon]

After your Gosub, the script continues straight through to the bottom because you have no intervening Return. Putting a subroutine inside a loop is also tricky-- not usually helpful, and sometimes leading to unintended effects.

Ok so any tips on what i can do to fix this?

[Xtra]

Code: Select all

return

}

should be:

Code: Select all

}

return

[sebbulon]

Code: Select all

return

}

should be:

Code: Select all

}

return

It says return has to be before {

[Xtra]

That is not the same script in your original post.

See below:

Code: Select all

#Singleinstance force
#persistent 


Loop
{
    Random, WorkTime, 20, 23  ; seconds
    minpause:=14     ; seconds
    maxpause:=15      ; seconds

        ;;RANDOM VARIABLES;;,

        ;;ITEM AND HIGH ALCH;;
            Random x1, 730, 735
            Random y1, 285, 289
        
        ;;RANDOM SLEEPS;;
            Random a150, 133, 238
            Random a2432, 2398, 2879

        SetTimer, startpause, % -Worktime*1000
        working:=True
        While working
             gosub dowork     
        random, rand, minpause, maxpause	
        sleep % rand*1000
        random, rand, WorkTime	
        sleep % rand*1000


    startpause:

    dowork:
    Mousemove, x1, y1, 13
        sleep a150
        click
        sleep a150
        click
        sleep a2432
}         ; <--------
return    ; <--------

[sebbulon]

That is not the same script in your original post.

See below:

Code: Select all

#Singleinstance force
#persistent 


Loop
{
    Random, WorkTime, 20, 23  ; seconds
    minpause:=14     ; seconds
    maxpause:=15      ; seconds

        ;;RANDOM VARIABLES;;,

        ;;ITEM AND HIGH ALCH;;
            Random x1, 730, 735
            Random y1, 285, 289
        
        ;;RANDOM SLEEPS;;
            Random a150, 133, 238
            Random a2432, 2398, 2879

        SetTimer, startpause, % -Worktime*1000
        working:=True
        While working
             gosub dowork     
        random, rand, minpause, maxpause	
        sleep % rand*1000
        random, rand, WorkTime	
        sleep % rand*1000


    startpause:

    dowork:
    Mousemove, x1, y1, 13
        sleep a150
        click
        sleep a150
        click
        sleep a2432
}         ; <--------
return    ; <--------

The one i posted was then i tried to put return after } as you said in the same script and i got that error message. It is the same script except that return is after }, instead of before

[sebbulon]

That is not the same script in your original post.

See below:

Code: Select all

#Singleinstance force
#persistent 


Loop
{
    Random, WorkTime, 20, 23  ; seconds
    minpause:=14     ; seconds
    maxpause:=15      ; seconds

        ;;RANDOM VARIABLES;;,

        ;;ITEM AND HIGH ALCH;;
            Random x1, 730, 735
            Random y1, 285, 289
        
        ;;RANDOM SLEEPS;;
            Random a150, 133, 238
            Random a2432, 2398, 2879

        SetTimer, startpause, % -Worktime*1000
        working:=True
        While working
             gosub dowork     
        random, rand, minpause, maxpause	
        sleep % rand*1000
        random, rand, WorkTime	
        sleep % rand*1000


    startpause:

    dowork:
    Mousemove, x1, y1, 13
        sleep a150
        click
        sleep a150
        click
        sleep a2432
}         ; <--------
return    ; <--------

The only difference is the last thing which only occurs when im done, and is not suppose to happend more than once so thats not the problem i think? But i still cant put return after as you wanted