Ich hätte Dir noch ein paar Anregungen als Optimierungsmaßnahmen :-) Wenn ich Denkfehler habe, wird mich der BOfH_666 sicher gerne korrigieren.
$targetPath = "d:\temp\!Archiv"
Nicht ganz ideal, wenn das Ziel innerhalb der zu untersuchenden Struktur liegt - ich würde das woanders unterbringen. Aber ok.
$Folders = (Get-ChildItem $sourcePath -Recurse | Where {$_.PSIsContainer -eq $True} | Where {$_.GetFiles().Count -ne 0} | Where-Object { -not ($_.FullName -like "$targetPath*")}).FullName
Filtern immer so weit links wie möglich. "Get-ChildItem -Directory -Exclude $( Split-Path $TargetPath -Leaf)" erspart Dir schon mal 2 dieser Where-Filter. Und warum steckst Du in $Folders nicht die Ordner-Objekte, sondern nur Strings? ( .FullName
ganz hinten...)
foreach ($Folder in $Folders) {
$File = @(Get-ChildItem $Folder -File | Sort-Object $Sort)[-1]
if ($File){
$FilePath = @($File.DirectoryName | Where {$File.LastWriteTime -lt $Stichtag})
}
Damit suchst Du ja nur, ob im jeweiligen Ordner eine Datei älter als Stichtag enthalten ist, oder? Das Sort-Object kannst Du weglassen - $Sort ist leer, damit holst Du halt eine Datei - und daß eine Datei drin ist, weißt Du schon ($_.GetFiles().Count
-ne 0). Und unnötig ist es eigentlich auch, denn: Bei Ordnern ist LastWriteTime die LastWriteTime der neuesten Datei darin. Damit wäre das zu ersetzen durch
If ( $Folder.LastWriteTime -lt $Stichtag ) { ... }
> if ($FilePath){
Get-ChildItem -Path $sourcePath -Recurse -File |
Warum fängst Du hier die Suche wieder oben an? Du ackerst Dich doch grad durch Unterordner durch, -Path $Filepath wäre geeigneter.
Where-Object { -not ($_.FullName -like "$targetPath*")} |
Kann entfallen, die sind in $Folders schon aussortiert.
Where-Object { ($_.FullName -like "$FilePath*")} |
Kann entfallen, wenn Du 2 Zeilen weiter oben nicht $sourcePath verwendest, sondern $FilePath.
"$Get-ChildItem | Foreach-Object { #Do something }" ist immer langsamer als "$Items = Get-ChildItem; Foreach ($Item in $Items) { #Do something }". Pipeline ist ok für manuell ausgeführte Commands, in Skripts solltest Du sie
meist vermeiden.
ForEach-Object {
$Release = Split-Path -Path (Split-Path -Path $_.FullName) -Leaf
$Kunde = Split-Path -Path (Split-Path -Path (Split-Path -Path $_.FullName)) -Leaf
Da würde ich nicht Split-Path verwenden, sondern direkt Split (oder einen Match). Ist eine Zeile mehr, aber übersichtlicher:
$Parts = $_.Fullname -Split '\\' (doppelt, weil das escaped werden muß)
$Release = $Parts[-2]
$Kunde = $Parts[-3]
Geht mit RegEx bestimmt noch eleganter, aber das krieg ich auswendig nicht hin, RegEx ist nicht so meins :-))
$Destination = Join-Path -Path (Join-Path -Path $targetPath -ChildPath $Kunde ) -ChildPath $Release
...der Lesbarkeit halber nicht Join-Path verwenden, sondern den String direkt zusammenbauen:
$Destination = '{0}\{1}\{2}' -f $TargetPath, $Kunde, $Release
$Destination
if (-not (Test-Path -Path $Destination) ) {New-Item -Path $Destination -ItemType Directory}
Move-Item -Path $_.FullName -Destination $Destination #-WhatIf
}
}
}
[/code]
warum der Ausgangsordner leer zurück bleibt und nicht mit verschoben wird.
Weil Du das nigends machst. Du verschiebst nur Dateien.
Und dann noch ein Skript um leere Unterordner aufzuspüren und zu löschen.
Das hast doch in der $Folders-Ermittlung oben quasi schon andersrum drin.
