batch-file - 为什么批处理文件中的 FOR/L 不能按预期工作？

Question

我有一个批处理文件:

:extract_stream
SETLOCAL ENABLEDELAYEDEXPANSION
FOR /F "delims=" %%A IN ('%media_info% "--output=%stream_count_string%" "%file%"') DO SET /A "stream_type_total=%%A"

SET /A "stream_count=1"

FOR /F "tokens=1-4 delims=," %%A IN ('%media_info% "--output=%format_string%" "%file%"') DO (
    SET /A "id=%%A-1"
    SET "title=%%B"
    SET "codec=%%C"
    SET "language=%%D"

    REM SET "id=!id:~1!"
    SET "title=!title:~1!"
    SET "codec=!codec:~1!"
    SET "language=!language:~1!"

    IF NOT DEFINED language SET "language=English"
    REM IF "!codec!" EQU "V_MPEG4/ISO/AVC" SET "language="

    SET stream_obj[!stream_count!].id=!id!

    ECHO ID:!id! Title=!title!, Codec=!codec!, Lamguage=!language!

    SET /A "stream_count+=1"
)
(  ENDLOCAL
    FOR /L  %%A IN (1,1,%stream_type_total%) DO (
        CALL SET "stream_obj[%%A].id=%stream_obj[%%A].id%"
    )
)
GOTO:EOF

但是我无法让 FOR/L 循环中的 %%A 按预期工作。

如果我使用像 1 这样的数字而不是 %%A，它工作得很好。

SET "stream_obj[1].id=%stream_obj[1].id%"

但 FOR/L 循环中的命令行不适用于:

SET "stream_obj[%%A].id=%stream_obj[%%A].id%"   

为什么？

当我用 echo 检查时，它在等号 = 处截断了奇数行。所以我不知道它是否与此有关。

Answer 1

不要使用算术表达式 为环境变量赋值。通常没有理由这样做。只需使用 SET "stream_type_total=%%A" 而不使用 /A。

命令 GOTO 和它的第一个参数 :EOF 之间应该有一个空格。

但主要问题是命令方 block :

(  ENDLOCAL
    FOR /L  %%A IN (1,1,%stream_type_total%) DO (
        CALL SET "stream_obj[%%A].id=%stream_obj[%%A].id%"
    )
)

命令 ENDLOCAL 不仅恢复延迟环境变量扩展的先前状态，它还丢弃当前事件的环境变量列表并恢复先前的环境变量列表。阅读 this answer 以了解有关命令 SETLOCAL 和 ENDLOCAL 的详细信息。

因此环境变量 stream_type_total 以及所有 stream_obj[x].id 环境变量在该命令之后不再存在。

我建议将此命令 block 修改为:

FOR /L %%A IN (1,1,%stream_type_total%) DO CALL SET "stream_obj[%%A].id=%stream_obj[%%A].id%"
ENDLOCAL

现在 FOR 循环应该按预期工作了。

但请注意，即使进行了此更改，修改后的 stream_obj[x].id 环境变量在命令 ENDLOCAL 后不再存在。

最好避免在这个子例程中使用 SETLOCAL 和 ENDLOCAL，就像使用以下代码一样:

:extract_stream
FOR /F "delims=" %%A IN ('%media_info% "--output=%stream_count_string%" "%file%"') DO SET "stream_type_total=%%A"

SET "stream_count=1"

FOR /F "tokens=1-4 delims=," %%A IN ('%media_info% "--output=%format_string%" "%file%"') DO (
    SET /A "id=%%A-1"
    SET "title=%%B"
    SET "codec=%%C"
    SET "language=%%D"

    REM SET "id=!id:~1!"
    CALL SET "title=%%title:~1%%"
    CALL SET "codec=%%codec:~1%%"
    IF DEFINED language CALL SET "language=%%language:~1%%"

    IF NOT DEFINED language SET "language=English"
    REM IF "!codec!" EQU "V_MPEG4/ISO/AVC" SET "language="

    CALL SET "stream_obj[%%stream_count%%].id=%%id%%"

    CALL ECHO ID:%%id%% Title=%%title%%, Codec=%%codec%%, Language=%%language%%

    SET /A "stream_count+=1"
)

FOR /L %%A IN (1,1,%stream_type_total%) DO CALL SET "stream_obj[%%A].id=%stream_obj[%%A].id%"
GOTO :EOF

在上面的代码中未修改用命令REM注释掉的命令。

另一种解决方案是使用一个子程序:

:extract_stream
FOR /F "delims=" %%A IN ('%media_info% "--output=%stream_count_string%" "%file%"') DO SET "stream_type_total=%%A"

SET "stream_count=1"

FOR /F "tokens=1-4 delims=," %%A IN ('%media_info% "--output=%format_string%" "%file%"') DO CALL :UpdateVariables "%%~A" "%%~B" "%%~C" "%%~D"

FOR /L %%A IN (1,1,%stream_type_total%) DO CALL SET "stream_obj[%%A].id=%stream_obj[%%A].id%"
GOTO :EOF

:UpdateVariables
SET /A "id=%~1-1"
SET "title=%~2"
SET "codec=%~3"
SET "language=%~4"

REM SET "id=%id:~1%"
SET "title=%title:~1%"
SET "codec=%codec:~1%"
IF DEFINED language SET "language=%language:~1%"

IF NOT DEFINED language SET "language=English"
REM IF "%codec%" == "V_MPEG4/ISO/AVC" SET "language="

SET "stream_obj[%stream_count%].id=%id%"

ECHO ID:%id% Title=%title%, Codec=%codec%, Language=%language%

SET /A "stream_count+=1"
GOTO :EOF

REM注释掉的命令在上面的代码中也做了修改。

最后一个提示:不要在比较两个字符串时使用比较运算符 EQU。此运算符专为比较 32 位带符号整数值而不是字符串比较而设计。阅读 Symbol equivalent to NEQ, LSS, GTR, etc. in Windows batch files 上的答案，了解有关使用命令 IF 比较字符串或整数值的详细信息。