ป้องกัน foreach loop เมื่อรายการว่าง

ใช้ Powershell v2.0 ฉันต้องการลบไฟล์ใด ๆ ที่เก่ากว่า X วัน:

$backups = Get-ChildItem -Path $Backuppath | 
                Where-Object {($_.lastwritetime -lt (Get-Date).addDays(-$DaysKeep)) -and (-not $_.PSIsContainer) -and ($_.Name -like "backup*")}

foreach ($file in $backups)
{
    Remove-Item $file.FullName;
}

อย่างไรก็ตามเมื่อการสำรองข้อมูล $ ว่างเปล่าฉันได้รับ: Remove-Item : Cannot bind argument to parameter 'Path' because it is null.

ฉันได้พยายาม:

1. ปกป้องด้านหน้าด้วย if (!$backups)
2. การป้องกันการลบรายการด้วย if (Test-Path $file -PathType Leaf)
3. การป้องกันการลบรายการด้วย if ([IO.File]::Exists($file.FullName) -ne $true)

สิ่งเหล่านี้ดูเหมือนจะไม่ทำงานถ้าหากวิธีที่แนะนำในการป้องกันการวนลูป foreach ไม่ทำงานถ้ารายการว่างเปล่า

ด้วย PowerShell 3 foreachคำสั่งจะไม่ซ้ำไปซ้ำมา$nullและปัญหาที่อธิบายโดย OP จะไม่เกิดขึ้นอีก

จากบล็อก Windows PowerShellโพสต์คุณสมบัติภาษาใหม่ V3 :

คำสั่ง ForEach ไม่ได้วนซ้ำเกิน $ null

ใน PowerShell V2.0 ผู้คนมักจะประหลาดใจโดย:

PS> foreach ($i in $null) { 'got here' }

got here

สถานการณ์นี้มักจะเกิดขึ้นเมื่อ cmdlet ไม่ส่งคืนวัตถุใด ๆ ใน PowerShell V3.0 คุณไม่จำเป็นต้องเพิ่มคำสั่ง if เพื่อหลีกเลี่ยงการวนซ้ำเกิน $ null เราดูแลสิ่งนั้นเพื่อคุณ

สำหรับ PowerShell $PSVersionTable.PSVersion.Major -le 2ดูคำตอบต้นฉบับต่อไปนี้

คุณมีสองตัวเลือกฉันส่วนใหญ่จะใช้ตัวที่สอง

ตรวจสอบไม่ได้$backups รอบวงที่$nullเรียบง่ายIfสามารถตรวจสอบได้$null

if ( $backups -ne $null ) {

    foreach ($file in $backups) {
        Remove-Item $file.FullName;
    }

}

หรือ

เริ่มต้น$backupsเป็นอาร์เรย์ว่าง นี้หลีกเลี่ยงความคลุมเครือของ "ย้ำอาร์เรย์ว่างเปล่า" ปัญหาที่คุณถามเกี่ยวกับในคำถามสุดท้ายของคุณ

$backups = @()
# $backups is now a null value array

foreach ( $file in $backups ) {
    # this is not reached.
    Remove-Item $file.FullName
}

ขออภัยฉันละเลยที่จะให้ตัวอย่างการรวมรหัสของคุณ บันทึกGet-ChildItemcmdlet ที่อยู่ในอาร์เรย์ $nullนอกจากนี้ยังจะทำงานร่วมกับฟังก์ชั่นที่สามารถกลับมาเป็น

$backups = @(
    Get-ChildItem -Path $Backuppath |
        Where-Object { ($_.lastwritetime -lt (Get-Date).addDays(-$DaysKeep)) -and (-not $_.PSIsContainer) -and ($_.Name -like "backup*") }
)

foreach ($file in $backups) {
    Remove-Item $file.FullName
}

ฉันรู้ว่านี่เป็นโพสต์เก่า แต่ฉันต้องการจะชี้ให้เห็นว่า cmdlet ForEach-Object ไม่ประสบปัญหาเดียวกับการใช้คำสำคัญ ForEach ดังนั้นคุณสามารถไพพ์ผลลัพธ์ของ DIR ไปยัง ForEach และเพียงแค่อ้างอิงไฟล์โดยใช้ $ _ เช่น:

$backups | ForEach{ Remove-Item $_ }

คุณสามารถส่งต่อคำสั่ง Dir ผ่านท่อและหลีกเลี่ยงการกำหนดตัวแปรเช่น:

Get-ChildItem -Path $Backuppath | 
Where-Object {
             ($_.lastwritetime -lt (Get-Date).addDays(-$DaysKeep)) -and `
             (-not $_.PSIsContainer) -and ($_.Name -like "backup*")
             } |
ForEach{ Remove-Item $_ }

ฉันเพิ่มตัวแบ่งบรรทัดเพื่อให้สามารถอ่านได้

ฉันเข้าใจบางคนเช่น ForEach / In เพื่อความสะดวกในการอ่าน บางครั้ง ForEach-object อาจมีขนดกเล็กน้อยโดยเฉพาะถ้าคุณทำรังเพราะยากต่อการอ้างอิง $ _ ไม่ว่าในกรณีใดก็ตามสำหรับการใช้งานขนาดเล็กเช่นนี้มันสมบูรณ์แบบ หลายคนยืนยันว่ามันเร็วกว่า แต่ฉันก็พบว่ามันเป็นเพียงเล็กน้อยเท่านั้น

ฉันได้พัฒนาวิธีแก้ปัญหาโดยการเรียกใช้แบบสอบถามสองครั้งหนึ่งครั้งเพื่อรับไฟล์และอีกครั้งเพื่อนับจำนวนไฟล์โดยการส่ง get-ChilItem เพื่อส่งกลับอาร์เรย์ .
อย่างน้อยก็ใช้งานได้ตามที่คาดไว้ (ประสิทธิภาพไม่ควรเป็นปัญหาเนื่องจากจะไม่มีไฟล์มากกว่าหนึ่งโหล) หากใครรู้วิธีแก้ปัญหาแบบสอบถามเดียวโปรดโพสต์

$count = @(Get-ChildItem -Path $zipFilepath | 
                Where-Object {($_.lastwritetime -lt (Get-Date).addDays(-$DaysKeep)) -and (-not $_.PSIsContainer) -and ($_.Name -like $partial + "*")}).count;

if ($count -gt 0)
{
    $backups = Get-ChildItem -Path $zipFilepath | 
                Where-Object {($_.lastwritetime -lt (Get-Date).addDays(-$DaysKeep)) -and (-not $_.PSIsContainer) -and ($_.Name -like $partial + "*")};

    foreach ($file in $backups)
    {
        Remove-Item $file.FullName;
    }
}

ใช้สิ่งต่อไปนี้เพื่อประเมินว่าอาร์เรย์มีเนื้อหาใด ๆ หรือไม่:

if($backups.count -gt 0) { echo "Array has contents" } else { echo "Array is empty" }

หากตัวแปรไม่มีอยู่ Powershell จะประเมินว่าเป็นเท็จดังนั้นไม่จำเป็นต้องตรวจสอบว่ามีอยู่หรือไม่