PowerShell含多检查字符串的For循环未正常运行，求排查

核心问题点 #

• 未读取目标日志内容：脚本直接使用$fileContent但未通过Get-Content读取目标文件，导致匹配逻辑无数据可用。
• 目录类型参数拼写错误：创建归档目录时ItemType写为director，正确值是directory。
• 未定义日期变量：备份路径引用$date但未赋值，会导致备份文件名缺失日期部分。
• 变量名误用：select-string中使用未定义的$checkString，应改为循环变量$element；且原循环逻辑会在匹配每个检查字符串时重复执行重启、归档，不符合“只要有任意匹配就执行一次操作”的需求。
• 日志时间固定：$logString在脚本启动时生成，无法反映实际操作的时间。
• 重复执行操作：原循环会在每个匹配项触发一次服务重启和日志处理，导致不必要的重复操作。

修正后的脚本 #

$serviceName = "MySQL80"
$file = "test.txt"
$pwd = "C:\tmp\"

$checkStrings = @(
    "Unhandled error. Error message: Error retrieving response.",
    "Unhandled error. Error message: Error retrieving response. Second"
)

$logName = "ServiceCheck.log"
$backupFolder = "Archive"

Set-Location -Path $pwd

if (Test-Path -Path $file) {
    # 确保归档目录存在
    if (-not (Test-Path -Path $backupFolder)) {
        New-Item -Path $pwd -Name $backupFolder -ItemType "directory" | Out-Null
    }

    # 读取日志文件内容
    $fileContent = Get-Content $file

    # 检查是否存在任意匹配的字符串
    $hasMatch = $checkStrings | ForEach-Object { $fileContent -match $_ } | Where-Object { $_ }
    
    if ($hasMatch) {
        # 获取当前操作时间
        $currentDate = (Get-Date).ToString("ddMMyyyyHHmmss")
        $logString = "$currentDate - The service has been reset and the log moved to backup"

        # 重启服务
        Restart-Service -Name $serviceName -Force

        # 归档日志文件
        $backupPath = Join-Path -Path (Join-Path -Path $pwd -ChildPath $backupFolder) -ChildPath "$currentDate.log"
        $currentFile = Join-Path -Path $pwd -ChildPath $file
        Copy-Item -Path $currentFile -Destination $backupPath

        # 过滤掉所有匹配检查字符串的行，重置日志
        $fileContent | Select-String -Pattern $checkStrings -NotMatch | Set-Content -Path $currentFile -Encoding UTF8

        # 记录操作日志
        if (Test-Path -Path $logName) {
            Add-Content -Path $logName -Value $logString
        }
        else {
            Set-Content -Path $logName -Value $logString
        }
    }
}

关键优化说明 #

• 使用数组字面量@()创建检查字符串列表，比ArrayList更简洁高效。
• 先一次性检查所有字符串是否存在匹配，避免重复执行服务重启和日志操作。
• 使用Join-Path处理路径拼接，避免手动拼接时的分隔符错误。
• 操作时实时获取当前时间，确保日志和备份文件名的时间准确对应操作时间。
• 用Set-Content替代Out-File并指定编码，避免文件编码变更导致的日志乱码问题。

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