Why doesn't this batch script execute?

Question

sc stop myService > nul

for /l %%A in (1,1,5) do (
   for /f "tokens=3 delims=: " %%H in ('sc query "myService" ^|findstr "STATE"') do (
     if /I "%%H" EQU "STOPPED" (
       exit goto :stopSuccess
     ) else (
       echo Checking if the Source Agent service has stopped successfully....
       timeout 3 > nul 
     )
   )
)

echo The service myService could not be stopped. 
goto :error

:stopSuccess
sc delete myService > nul

When I try to exectue my .bat script from cmd, it doesn't do anything and cmd closes. Obviously, the name of the service is not actually myService, but I'm using it as a placeholder for anonymity.

What I want to do is do 5 checks spaced 3 seconds apart to see if the service stopped properly. Then I use a couple goto statements to proceed.

You explicitly have an EXIT command. The GOTO :stopsuccess will not execute because you are using EXIT in front of it. — Squashman
– Squashman, Commented Mar 3, 2017 at 14:35
And just so you know, you cannot forcefully break out of a FOR /L command. It will execute all 5 times regardless of the IF statement being TRUE. — Squashman
– Squashman, Commented Mar 3, 2017 at 14:41

Magoo · Accepted Answer · 2017-03-03 14:36:32Z

1

   exit goto :stopSuccess

Means "exit cmd" so as soon as "STOPPED" is detected, the procedure exits.

Drop the exit and it will go to :stopsuccess as expected.

answered Mar 3, 2017 at 14:36

Magoo

80.8k8 gold badges68 silver badges92 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Why doesn't this batch script execute?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related