Question

我想知道使用try块来测试文件是否被锁定是否是错误的形式。这是背景。我需要同时将应用程序的文本输出发送到两个串行打印机。我的解决方案是使用MportMon和Powershell脚本。它应该工作的方式是应用程序默认打印到MportMon虚拟打印机端口，它实际上在＆＃34; dropbox＆＃34;中生成一个唯一命名的文件。夹。 powershell脚本使用filesystemwatcher来监视文件夹，当创建新文件时，它会获取文本内容并将其推出两个串行打印机，然后删除该文件，以免填满该文件夹。尝试从虚拟打印机创建的文件中读取文本时遇到问题。我发现由于文件仍然被锁定，我收到了错误。为了解决这个问题，我使用FSM来实现逻辑，而不是每次尝试从文件中获取内容之前都检查锁，我使用了一个尝试从文件中读取内容的try块，如果失败， catch block只是重申了FSM所处的状态，并且该过程重复进行直到成功。它似乎工作正常，但我已经在某处读到了它的不良做法。这种方法有危险，还是安全可靠？以下是我的代码。

$fsw = New-Object system.io.filesystemwatcher
$q = New-Object system.collections.queue
$path = "c:\DropBox"
$fsw.path = $path
$state = "waitforQ"
[string]$tempPath = $null

    Register-ObjectEvent -InputObject $fsw -EventName created -Action {
    $q.enqueue( $event.sourceeventargs.fullpath )
    }

    while($true) {
    switch($state)
    {
        "waitforQ" {
                    echo "waitforQ"
                    if ($q.count -gt 0 ) {$state = "retrievefromQ"}
                    }
        "retrievefromQ"  {
                    echo "retrievefromQ"
                    $tempPath = $q.dequeue()
                    $state = "servicefile"
                    }
        "servicefile"  {
                        echo " in servicefile "

                    try
                        {
                        $text = Get-Content -ErrorAction stop $tempPath 
                        #echo "in try"
                        $text | out-printer db1
                        $text | out-printer db2
                         echo " $text "
                        $state = "waitforQ"
                        rm $tempPath
                        }
                    catch
                        {
                        #echo "in catch" 
                        $state = "servicefile"
                        }
                    }

           Default {    $state = "waitforQ"  }
    }
    }

Answer 1

我不会说测试文件是否被锁定是不好的做法，但它并不像检查其他进程使用的句柄一样干净。就个人而言，我会像你一样测试文件，但我会调整一些部分以使其更安全/更好。

那个switch语句看起来很复杂（对我来说），我用一个简单的if-test替换它。＆＃34;如果文件在队列中，请继续，如果没有，请等待＆＃34;。
您需要放慢速度..在锁定文件时，您会尝试尽可能多地阅读该文件。这是浪费资源，因为当前应用程序需要一些时间才能将数据保存到HDD。添加一些暂停。你不会注意到它们，但你的CPU会喜欢它们。当队列中没有文件时也是如此。
您可能会受益于添加超时，例如最多50次尝试读取文件，以避免在永远不会释放某个特定文件时脚本卡住。

尝试：

$fsw = New-Object system.io.filesystemwatcher
$q = New-Object system.collections.queue
$path = "c:\DropBox"
$fsw.path = $path
$MaxTries = 50  #50times * 0,2s sleep = 10sec timeout
[string]$tempPath = $null

Register-ObjectEvent -InputObject $fsw -EventName created -Action {
    $q.enqueue( $event.sourceeventargs.fullpath )
}

while($true) {
    if($q.Count -gt 0) {

        #Get next file in queue
        $tempPath = $q.dequeue()

        #Read file
        $text = $null
        $i = 0
        while($text -eq $null) {

            #If locked, wait and try again
            try {
                $text = Get-Content -Path $tempPath -ErrorAction Stop
            } catch {
                $i++
                if($i -eq $MaxTries) {
                    #Max attempts reached. Stops script
                    Write-Error -Message "Script is stuck on locked file '$tempPath'" -ErrorAction Stop
                } else {                
                    #Wait
                    Start-Sleep -Milliseconds 200
                }

            }
        }

        #Print file
        $text | Out-Printer db1
        $text | Out-Printer db2
        echo " $text "

        #Remove temp-file
        Remove-Item $tempPath

    }

    #Relax..
    Start-Sleep -Milliseconds 500

}

使用try块测试文件是否在powershell中被锁定

1 个答案: