PowerShell使用ForEach循环统计安全日志4624事件ID出现次数的问题

• 变量名冲突导致比对逻辑完全失效：原脚本将存储目标事件ID的变量命名为$id，同时foreach循环的迭代变量也使用了$id，遍历过程中迭代值会直接覆盖预先设置的4624，全程没有任何匹配校验，所有日志条目都会触发计数累加，最终得到的就是安全日志总条数。
• 计数器缺少明确初始化：原脚本未在循环外定义计数器$count的初始值，容易受当前会话遗留的同名变量值干扰，导致统计结果偏差。
• 计数触发条件错误：原脚本没有增加实例ID匹配判断，不管当前遍历的条目是不是目标事件，都会执行计数器自增。
• 输出位置错误：原脚本将打印语句放在循环内部，每遍历一条日志就会输出一次中间累计值，无法在遍历完成后展示最终准确结果。

全程未使用Measure-Object、.Count属性做统计，完全依赖foreach循环完成计数：

# 定义目标匹配的事件实例ID
$targetEventId = 4624
# 获取安全日志所有条目的实例ID集合
$allInstanceIds = (Get-EventLog security).instanceid
# 显式初始化计数器，在循环外定义避免干扰
$matchCount = 0

foreach ($currentInstanceId in $allInstanceIds) {
    # 仅当当前遍历的实例ID匹配目标值时，计数器才累加
    if ($currentInstanceId -eq $targetEventId) {
        $matchCount++
    }
}
# 遍历完成后输出最终统计结果
Write-Host "The amount of 4624's in the security log is $matchCount"

• 所有变量统一做语义化命名，彻底避免循环迭代变量与存储目标值的变量重名覆盖，确保匹配校验逻辑正常生效。
• 为每个匹配目标值的条目单独触发计数器自增，从逻辑上保证只统计符合要求的事件数量，而非全量日志数量。
• 输出语句移至循环外部，等所有日志遍历完成、计数准确后再展示结果，减少循环过程中的无效打印。

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