批处理脚本FTP上传失败邮件通知功能的循环问题排查

核心问题分析 #

1. 变量延迟扩展未正确应用
   循环内通过set /a file_count+=1更新了计数器，但最后输出时使用%file_count%，这会读取脚本预解析时的初始值（0），而非循环中更新后的数值。必须改用延迟扩展语法!file_count!才能获取正确结果。

2. 空目录导致循环跳过
   若%export_dir%目录为空，for %%i in (%export_dir%\*)会直接跳过循环体，file_list和file_count保持初始值，看起来像循环“未工作”。

3. HTML转义字符的字面量问题
   set file_list=!file_list!!filename!<br>中的<br>是HTML实体，批处理会将其作为字面量拼接。如果用于HTML邮件内容这没问题，但命令行调试时会显示转义后的字符串，可能造成误解。

修复后的代码 #

setlocal enabledelayedexpansion

REM winscp script call here

set exitCode=%ERRORLEVEL%
set file_list=
set file_count=0
set export_dir=D:\data\dir1\dir2

pause

echo Checking files in %export_dir%
dir %export_dir%
pause

REM 修复后的循环逻辑
if exist "%export_dir%\*" (
    for %%i in ("%export_dir%\*") do (
        set "filename=%%~nxi"
        echo Found file: !filename!
        set "file_list=!file_list!!filename!<br>"
        set /a file_count+=1
    )
) else (
    echo No files found in %export_dir%
)

echo File List: !file_list!
echo File Count: !file_count!

pause

额外优化建议 #

• 遍历含隐藏文件的所有非目录文件：若需要包含隐藏文件，改用for /f遍历dir命令输出，避免默认for循环跳过隐藏文件：

  for /f "delims=" %%i in ('dir /b /a-d "%export_dir%"') do (
      set "filename=%%i"
      echo Found file: !filename!
      set "file_list=!file_list!!filename!<br>"
      set /a file_count+=1
  )
  

• 规避路径空格问题：将%export_dir%\*用引号包裹为"%export_dir%\*"，防止后续路径修改后因空格导致循环解析错误。
• 纯文本邮件换行处理：如果发送纯文本邮件，将<br>替换为批处理换行符，需先定义换行变量：

  set newline=^
  
  REM 此处需空一行
  set "file_list=!file_list!!filename!!newline!"
  

内容的提问来源于stack exchange，提问作者samuka