PowerShell foreach innerhalb der Funktion

Question

Ich schreibe eine Funktion, um Protokolle für die Analyse zu sammeln.

Ich versuche, Variablen zu verwenden, um es so eng wie möglich zu halten, aber aus irgendeinem Grund wird die Funktion nicht erstellen, stöhnt über Klammern und Variablen.

Jede Hilfe wäre willkommen.

FUNCTION GetLogs ($CompName) 
{ 
$LOGS = "CcmExec.log,Scheduler.log,WUAHandler.log" 
$CCMPath = "C$\Windows\CCM\Logs" 
$Target = "C:\Temp\Logs" 

foreach (file$ IN $LOGS) {file$ = Copy-Item \\$CompName\$CCMPath\$LOGS $Target\$CompName-$LOGS}  
} 

Answer 1

Dieser Code wird nicht funktionieren.

Ich denke, Sie wollten ein Array in $ LOGS definieren, aber Sie definieren tatsächlich nur einen String.

Try this: $LOGS = "CcmExec.log", "Scheduler.log", "WUAHandler.log"

Auch sieht es aus wie Sie irgendwo jene drei Dateien zu kopieren versuchen, aber was Sie tun, ist die var $ file auf etwas anderes jeder Runde setzen. Die Anweisung $file In $LOGS bedeutet, dass die Variable $ file beim ersten Mal der Schleife "CcmExec.log" ist, dann "Scheduler.log" und so weiter.

Also, was ich denke, was Sie tun wollen, ist dies:

Function GetLogs ($CompName) { 

$LOGS = "CcmExec.log","Scheduler.log","WUAHandler.log" 
$CCMPath = "C$\Windows\CCM\Logs" 
$Target = "C:\Temp\Logs" 

foreach ($file In $LOGS) { 
     Copy-Item -Path \\$CompName\$CCMPath\$file -Destination $Target\$CompName-$file 
    }  
} 

ich auch einige Fehler beseitigt.

Ist das ungefähr richtig?

Answer 2

Sie $Logs ist nur eine Zeichenfolge, kein String Array daher kann man nicht über sie iterieren. Sie können beheben, dass die Verwendung:

$LOGS = "CcmExec.log", "Scheduler.log","WUAHandler.log" 

Auch das Ergebnis des CopyItem auf eine ungültige Variable zuweisen und nicht den aktuellen Schleifenvariable verwenden. Ihre Funktion wahrscheinlich sollte wie folgt aussehen:

function Get-Logs 
{ 
    Param 
    (
     [string]$CompName 
    ) 

    $logs = "CcmExec.log", "Scheduler.log","WUAHandler.log" 
    $ccmPath = "C$\Windows\CCM\Logs" 
    $target = "C:\Temp\Logs" 

    $logs | ForEach-Object { 
     Copy-Item -Path "\\$CompName\$ccmPath\$_" -Destination "$target\$CompName-$_" 
    } 
} 

Hinweis: Sie sollten wahrscheinlich benennen Sie Ihre Funktion Copy-LogsToLocal oder etwas ...