PowerShell脚本Break未生效及循环、Record功能异常问题排查

修复后的完整脚本 #

Clear-Host
Write-Host "Welcome to QuickScripter! Quickly script files! Type run to start, clear to restart, and exit to exit and remove script."

# 初始化script.ps1文件
if (Test-Path .\script.ps1) {
    Remove-Item .\script.ps1
}
New-Item .\script.ps1 -ErrorAction SilentlyContinue | Out-Null

function Enter-Loop {
    $in = Read-Host

    if ($in -eq "run") {
        .\script.ps1
    }
    elseif ($in -eq "exit") {
        Remove-Item .\script.ps1 -ErrorAction SilentlyContinue
        Clear-Host
        exit
    }
    elseif ($in -eq "clear") {
        Clear-Content .\script.ps1 -ErrorAction SilentlyContinue
    }
    elseif ($in -eq "load") {
        $i = 0
        $a = 0

        # 修复循环递增逻辑，终止无限循环
        :loop while ($true) {
            if (Test-Path "./tape/$i") {
                Write-Host $i
                $a++
                $i++  # 关键：每次检查后递增索引
            }
            else {
                break loop
            }
        }

        Write-Host "Choose a script to load:"
        # 修复for循环语法错误
        for ($i=0; $i -lt $a; $i++) {
            Write-Host $i
        }

        $t = Read-Host -Prompt "Tape:"
        # 转换输入为数字，避免字符串比较错误
        $t = [int]$t
        if ($t -ge $a) {
            zenity --error --text="404: This tape is missing"
        }
        else {
            Copy-Item "./tape/$t" .\script.ps1 -ErrorAction SilentlyContinue
        }
    }
    elseif ($in -eq "record") {
        $i = 0
        # 找到可用索引后创建文件并退出循环
        while ($true) {
            if (Test-Path "./tape/$i") {
                $i++
            }
            else {
                # 确保tape目录存在
                New-Item -Path "./tape/" -ItemType Directory -ErrorAction SilentlyContinue | Out-Null
                # 复制script.ps1内容到tape文件，而非写入路径字符串
                Get-Content .\script.ps1 | Set-Content "./tape/$i" -ErrorAction SilentlyContinue
                # 终止循环
                break
            }
        }
        Write-Host "Script recorded to tape $i"
    }
    else {
        Add-Content -Path .\script.ps1 -Value $in
    }
}

while ($true) {
    Enter-Loop
}

问题排查与修复说明 #

• Load模块无限循环：
  原循环只检查./tape/$i但从未递增$i，导致永远重复检查./tape/0，只要该路径存在就无限执行$a++。修复时在if (Test-Path)分支内添加$i++，让索引逐个递增，直到找不到对应路径时终止循环。

• Load模块for循环语法错误：
  PowerShell的for循环必须遵循for(初始化; 条件; 递增)的语法，原脚本for ($i -lt $a)缺少初始化和递增逻辑，导致无限循环。修复为for ($i=0; $i -lt $a; $i++)，正确遍历所有可用的tape索引。

• Record功能失效：

  1. 原循环未添加break，创建tape文件后会无限循环生成新文件；修复时在创建完成后添加break终止循环。
  2. 原脚本用Set-Content写入的是字符串./script.ps1，而非目标脚本的内容；修复为用Get-Content .\script.ps1 | Set-Content "./tape/$i"读取原脚本内容并写入新tape文件。
  3. 未确保tape目录存在，添加目录创建逻辑避免文件写入失败。

• 其他语法修复：
  原脚本load分支末尾缺少闭合}，导致后续elseif结构语法错误；补全括号后代码结构恢复正常。添加[int]$t将用户输入转为数字，避免字符串比较导致的逻辑错误。

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