Question

Could somebody tell me why the following isnt working…

:PHASE_ONE
SET /P input_text=   ENTER PASSWORD:

for /f "delims=1234567890" %%A in ("%input_text%") GOTO PHASE_ONE

Im trying to avoid numerical values

Answer 1

Your syntax is wrong. You are missing a DO before your GOTO.

However, your logic is also the wrong way round – you are actually ensuring that the password must only be numbers.

The following will restrict it to letters, for example:

@ECHO OFF

:PHASE_ONE
SET /P input_text=   ENTER PASSWORD:

for /f "delims=abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ" %%A in ("%input_text%") DO GOTO PHASE_ONE

ECHO.

However, you would be better off using the FINDSTR command, such as this:

@ECHO OFF

:PHASE_ONE
SET /P input_text=   ENTER PASSWORD:

ECHO %input_text% | FINDSTR /I /R [0-9][0-9]*

REM If Errorlevel is 1, then no matching string has been found,
REM so we have a valid password (i.e. non-numeric).
IF ERRORLEVEL 1 GOTO Passed
GOTO PHASE_ONE

:Passed
ECHO Hooray - your password is fine.