parameter parsen mit Leerzeichen im Pfad

casi4712

Hallo an alle, insbesindere unsere Syntax Experten. Wie muss man es anstellen um einen Filenamen mit Leerzeichen in der Args Zeile einzufügen, ich hatte es auch schon mit "'" pfad "'" versucht hat auch nichts geholfen, sofern das Programm ohne Leerzeuchen ist funktioneirt es.

AutoIt

Func GetExecutionParameter($filePath)
.
.
 ElseIf StringRight($filePath, 4) = ".jpg" Then
  Return $params = ' -s -i -d "Path\with spaces\Viewer.exe" '
.
EndFunc

Func ExecuteParallelBatch($batchFilePath)
   Local $parameter = GetExecutionParameter($batchFilePath)
.
.
$psScriptContent = '$hostFilePath = "' & $hostFilePath & '"' & @CRLF & _
    .
.
.
.
    '      $psexecArgs = "-u $username -p $password \\" + $computer + " ' & $parameter & ' \\"" + $batchFilePath + "\\""' & @CRLF & _
    '
.
.
.
EndFunc

Alles anzeigen

Recht herzlichen Dank schonmal

lg und schönen Sonntag noch

casi4712

ok dann formulier ich es mal ausführlicher, es gibt 2 Funktionen, eine die PSEXEC Parameter generiert und eine die ein Powershell Script ausführt: Die restlichen Dateitypen funktionieren alle, nur dass mit JPG und mp3 macht probleme wegen den "" vermute ich mal:

AutoIt

Func GetExecutionParameter($filePath)
    ; PSEXEC Parameter
    Local $params = ""
    Local $viewer = 'C:\Program Files (x86)\Google\Picasa3\PicasaPhotoViewer.exe'
    Local $splayer = 'c:\Program Files\VideoLAN\VLC\vlc.exe'
    
    If BitAND(GUICtrlRead($check_c), $GUI_CHECKED) Then $params &= " -f -c"
    If BitAND(GUICtrlRead($check_i), $GUI_CHECKED) Then $params &= " -i"
    If BitAND(GUICtrlRead($check_h), $GUI_CHECKED) Then $params &= " -h"
    If BitAND(GUICtrlRead($check_l), $GUI_CHECKED) Then $params &= " -l"
    If BitAND(GUICtrlRead($check_e), $GUI_CHECKED) Then $params &= " -e"
    If BitAND(GUICtrlRead($check_s), $GUI_CHECKED) Then $params &= " -s"
    If BitAND(GUICtrlRead($check_x), $GUI_CHECKED) Then $params &= " -x"
    If BitAND(GUICtrlRead($check_v), $GUI_CHECKED) Then $params &= " -v"
    If BitAND(GUICtrlRead($check_d), $GUI_CHECKED) Then $params &= " -d"

    If StringRight($filePath, 4) = ".ps1" Then
        Return $params & ' powershell -NoProfile -ExecutionPolicy Bypass -File '
    ElseIf StringRight($filePath, 4) = ".bat" Then
        Return $params & ' -i cmd /c '
    ElseIf StringRight($filePath, 4) = ".cmd" Then
        Return $params & ' -i cmd /c '
    ElseIf StringRight($filePath, 4) = ".exe" Then
        Return $params & ' -i '
    ElseIf StringRight($filePath, 4) = ".vbs" Then
        Return $params & ' cscript.exe '
    ElseIf StringRight($filePath, 4) = ".jpg" Then
			Return $params & ' start "" '
	ElseIf StringRight($filePath, 4) = ".mp3" Then ;Audiofile
       Return $params & ' "' & $splayer & '" -Idummy'

    Else
        Return $params & ' cmd /c '
    EndIf
EndFunc

Func ExecuteParallelBatch($batchFilePath)
    Global $username, $password, $PsExecPath, $hostFilePath
    Local $psScriptPath = @TempDir & "\temp_script.ps1"
    Local $psexecArgs
    Local $psScriptContent
	Local $player = "C:\Program Files (x86)\Google\Picasa3\PicasaPhotoViewer.exe"
    Local $splayer = 'c:\Program Files\VideoLAN\VLC\vlc.exe'
    ; Bestimme den Parameter für den Ausführungstyp basierend auf der Dateiendung
    Local $parameter = GetExecutionParameter($batchFilePath)
MsgBox(64, "Info", $Parameter)
    ; Powershell-Skript für parallele Jobausführung
    $psScriptContent = '$hostFilePath = "' & $hostFilePath & '"' & @CRLF & _
        '$computers = Get-Content -Path $hostFilePath' & @CRLF & _
        '$username = "' & $username & '"' & @CRLF & _
        '$password = "' & $password & '"' & @CRLF & _
        '$securePassword = ConvertTo-SecureString $password -AsPlainText -Force' & @CRLF & _
        '$credential = New-Object System.Management.Automation.PSCredential($username, $securePassword)' & @CRLF & _
        '$batchFilePath = "' & $batchFilePath & '"' & @CRLF & _
        '$psexecCommand = "' & $PsExecPath & '"' & @CRLF & _
        '$jobs = foreach ($computer in $computers) {' & @CRLF & _
        '    if ($computer -ne $env:COMPUTERNAME) {' & @CRLF & _
        '        $psexecArgs = "-u $username -p $password \\" + $computer + "' & $parameter & '" + $batchFilePath' & @CRLF & _
        '        Start-Job -ScriptBlock {' & @CRLF & _
        '            param($psexecCommand, $psexecArgs)' & @CRLF & _
        '            Start-Process -FilePath $psexecCommand -ArgumentList $psexecArgs -NoNewWindow -Wait' & @CRLF & _
        '        } -ArgumentList $psexecCommand, $psexecArgs' & @CRLF & _
        '    }' & @CRLF & _
        '}' & @CRLF & _
        '$null = Wait-Job -Job $jobs' & @CRLF & _
        '$jobs | Remove-Job'

    FileWrite($psScriptPath, $psScriptContent)
    RunWait(@ComSpec & ' /c powershell.exe -ExecutionPolicy Bypass -File "' & $psScriptPath & '"', "", @SW_HIDE)
    MsgBox(64, "Info", "Jobs wurden erfolgreich ausgeführt.")
    FileDelete($psScriptPath)
EndFunc

Alles anzeigen

lg und eine schöne Woche

**Musashi**

Zitat von casi4712

ich hatte es auch schon mit "'" pfad "'" versucht...

Versuche es mal mit """ :

Beispiel zur Anschauung :

ConsoleWrite('Path = ' & """Path\with spaces\Viewer.exe""" & @CRLF)

casi4712

Hallo und danke erst einmal, du meintest es so:

Local $splayer = """c:\Program Files\VideoLAN\VLC\vlc.exe"""

Leider passiert wieder nichts,

**AspirinJunkie**

Ich denke das Problem liegt eher in dem resultierenden Powershell-Skript.
Das Skript welches du erzeugst sieht für mich - obwohl ich keine Ahnung von PS habe - nicht richtig aus:

Code

+ " -f -c start "" " + $batchFilePath

1. wird der Inhalt vom Dateipfad nach den doppelten Anführungszeichen mit einem Leerzeichen dazwischen gesetzt. Das könnte je nach Pfadparser Probleme bereiten.
2. Es gibt keine abschließenden doppelten Anführungszeichen.

Es scheint also weniger ein Problem auf AutoIt-Seite zu sein.
Ich würde daher empfehlen dir das PS-Skript mal ausgeben zu lassen und mal händisch auszuführen.
Dann sollte dir der ps-Interpreter schon sagen was da noch im argen liegt.

Eine AutoIt-Sache dennoch:

AutoIt

RunWait(@ComSpec & ' /c powershell.exe [...]

Ich verstehe den Sinn von @ComSpec hier noch nicht ganz.
Du willst ja etwas mit Powershell ausführen lassen und eben nicht mit der cmd.exe.
Warum schleifst du dann aber den Powershell-Aufruf erst noch durch den Kommandozeileninterpreter cmd.exe anstatt powershell direkt aufzurufen?

**Moombas**

JPG: Genrell, macht es Sinn parameter ohne inhalt (start "") zu übergeben? oder ist das wie Aspirinjunkie angesprochen hat, ggf. eher ein Formatirungsfehler?

MP3: gehört der Parameter -Idummy nicht ggf zu vlc und muss ggf. mit in die "" eingekapselt werden? (da bin ich mir aktuell unsicher!)

Aber generell stimme ich AspirinJunkie zu:

Lass das Skript die entsprechende batch erzeugen und führe sie dann mal manuell aus in einem PS fenster und schaue was er dir dann als Fehlermeldung raus wirft bzw. ggf. siehst du dann auch shcon den Formtierungsfehler in der batch

casi4712

AutoIt

jo danke erst einmal für Euer Feedback, also das generierte Script was im Normalfall funktioniert sieht wie folgt aus, as sollte dann ja nur nach den Parametern, hier -i , sattt dem Powershell der Pfad von vlv drin stehen, möglicherweise ist es ein Problem von Powershell:

Code

$hostFilePath = "C:\Users\Riegert\Desktop\construction\lists\host3.txt"
$computers = Get-Content -Path $hostFilePath
$username = "Administrator"
$password = "passwort"
$securePassword = ConvertTo-SecureString $password -AsPlainText -Force
$credential = New-Object System.Management.Automation.PSCredential($username, $securePassword)
$batchFilePath = "\\Fusselbox\Fusselbox\Scripts\Inventory-Network.ps1"
$psexecCommand = "C:\windows\psexec.exe"
$jobs = foreach ($computer in $computers) {
    if ($computer -ne $env:COMPUTERNAME) {
        $psexecArgs = "-u $username -p $password \\" + $computer + " -i powershell -NoProfile -ExecutionPolicy Bypass -File " + $batchFilePath
        Start-Job -ScriptBlock {
            param($psexecCommand, $psexecArgs)
            Start-Process -FilePath $psexecCommand -ArgumentList $psexecArgs -NoNewWindow -Wait
        } -ArgumentList $psexecCommand, $psexecArgs
    }
}
$null = Wait-Job -Job $jobs


$jobs | Remove-Job

Alles anzeigen

mit mp3 option und intigrierten Playerprogramm:

Code

$hostFilePath = "C:\Users\Riegert\Desktop\construction\lists\host3.txt"
$computers = Get-Content -Path $hostFilePath
$username = "Administrator"
$password = "passwort"
$securePassword = ConvertTo-SecureString $password -AsPlainText -Force
$credential = New-Object System.Management.Automation.PSCredential($username, $securePassword)
$batchFilePath = "\\Fusselbox\Fusselbox\Scripts\tuer.mp3"
$psexecCommand = "C:\windows\psexec.exe"
$jobs = foreach ($computer in $computers) {
    if ($computer -ne $env:COMPUTERNAME) {
        $psexecArgs = "-u $username -p $password \\" + $computer + " -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy" + $batchFilePath
        Start-Job -ScriptBlock {
            param($psexecCommand, $psexecArgs)
            Start-Process -FilePath $psexecCommand -ArgumentList $psexecArgs -NoNewWindow -Wait
        } -ArgumentList $psexecCommand, $psexecArgs
    }
}
$null = Wait-Job -Job $jobs
$jobs | Remove-Job

Alles anzeigen

Der Befehl der als Einzelbefehl geht ist folgender, an der Reihenfolge ist da aber grundsätzlich nichts anders:

$command = '"' & $PsExecPath & '" -d -i -s \\' & $Computer & ' -u ' & $Username & ' -p "' & $Password & '" "' & $Player & '" -Idummy "' & $file & '"'

lg und schöne Woche noch

**Moombas**

Aber Zeile 11 funktioniert so auch? Kann ich mir gerade schwerlich vorstellen:

"..." + $var + "..."c:\Program Files\VideoLAN\VLC\vlc.exe"..." + $var2

Müsste das nicht eher so sein:

"..." + $var + '...' + $var2

wobei '...' = ' -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy '

Das führt zu:

"-u $username -p $password \\" + $computer + ' -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy ' + $batchFilePath

Also 2 " müssen in ' geändert werden und hinter -Idummy muss ein Leerzeichen hinzugefügt werden. (Sorry für das Englisch bevor )

**AspirinJunkie**

Zitat von casi4712

also das generierte Script was im Normalfall funktioniert

Die beiden gezeigten Skripte haben ja auch nicht das von mir beschriebene Problem.

**Moombas**

Wie gesagt, der Fehler liegt (mind.) in Zeile 11.
Als erstes sein Beispiel ausgeführt (nur Zeile 11), dann die gleiche Zeile nur mit meinen Änderungen (ja ich habe das Leerzeichen nach dem -Idummy im Test vergessen, solltest du aber setzen )

Also in deinem Skript Zeile 30 anpassen (ungetestet): Return "' " & $params & ' "' & $splayer & '"' & " -Idummy '"

casi4712

so danke noch mal für das Feedback, ich habe das jetzt dahingehend angepasst, hoffe hatte dich richtig verstanden, aber geht leider immer noch nicht:

AutoIt

Func GetExecutionParameter($filePath)
    ; PSEXEC Parameter
    Local $params = ""
    Local $viewer = 'C:\Program Files (x86)\Google\Picasa3\PicasaPhotoViewer.exe'

    If BitAND(GUICtrlRead($check_c), $GUI_CHECKED) Then $params &= " -f -c"
    If BitAND(GUICtrlRead($check_i), $GUI_CHECKED) Then $params &= " -i"
    If BitAND(GUICtrlRead($check_h), $GUI_CHECKED) Then $params &= " -h"
    If BitAND(GUICtrlRead($check_l), $GUI_CHECKED) Then $params &= " -l"
    If BitAND(GUICtrlRead($check_e), $GUI_CHECKED) Then $params &= " -e"
    If BitAND(GUICtrlRead($check_s), $GUI_CHECKED) Then $params &= " -s"
    If BitAND(GUICtrlRead($check_x), $GUI_CHECKED) Then $params &= " -x"
    If BitAND(GUICtrlRead($check_v), $GUI_CHECKED) Then $params &= " -v"
    If BitAND(GUICtrlRead($check_d), $GUI_CHECKED) Then $params &= " -d"

    If StringRight($filePath, 4) = ".ps1" Then
        Return $params & ' powershell -NoProfile -ExecutionPolicy Bypass -File '
    ElseIf StringRight($filePath, 4) = ".bat" Then
        Return $params & ' -i cmd /c '
    ElseIf StringRight($filePath, 4) = ".cmd" Then
        Return $params & ' -i cmd /c '
    ElseIf StringRight($filePath, 4) = ".exe" Then
        Return $params & ' -i '
    ElseIf StringRight($filePath, 4) = ".vbs" Then
        Return $params & ' cscript.exe '
    ElseIf StringRight($filePath, 4) = ".jpg" Then
        If GUICtrlRead($check_i) = $GUI_UNCHECKED Then
			Return $params & ' start "" '
        Else
            Return $params & ' cmd /c '
        EndIf


	ElseIf StringRight($filePath, 4) = ".mp3" Then ;Audiofile
        $splayer = 'c:\Program Files\VideoLAN\VLC\vlc.exe' ; geändert
       Return $params & ' "' & $splayer & '" -Idummy ' ;Leerzeichen eingefügt


    Else
        Return $params & ' cmd /c '
    EndIf
EndFunc

Func ExecuteParallelBatch($batchFilePath)
Global $username, $password, $PsExecPath, $hostFilePath
Local $psScriptPath = @TempDir & "\temp_script.ps1"
Local $psexecArgs
Local $psScriptContent
Local $player = """ C:\Program Files (x86)\Google\Picasa3\PicasaPhotoViewer.exe"""
;Local $splayer = 'c:\Program Files\VideoLAN\VLC\vlc.exe'
; Bestimme den Parameter für den Ausführungstyp basierend auf der Dateiendung
Local $parameter = GetExecutionParameter($batchFilePath)
MsgBox(64, "Info", $Parameter)
; Powershell-Skript für parallele Jobausführung
$psScriptContent = '$hostFilePath = "' & $hostFilePath & '"' & @CRLF & _
'$computers = Get-Content -Path $hostFilePath' & @CRLF & _
'$username = "' & $username & '"' & @CRLF & _
'$password = "' & $password & '"' & @CRLF & _
'$securePassword = ConvertTo-SecureString $password -AsPlainText -Force' & @CRLF & _
'$credential = New-Object System.Management.Automation.PSCredential($username, $securePassword)' & @CRLF & _
'$batchFilePath = "' & $batchFilePath & '"' & @CRLF & _
'$psexecCommand = "' & $PsExecPath & '"' & @CRLF & _
'$jobs = foreach ($computer in $computers) {' & @CRLF & _
' if ($computer -ne $env:COMPUTERNAME) {' & @CRLF & _
' $psexecArgs = "-u $username -p $password \\" + $computer + "' & $parameter & '" + $batchFilePath' & @CRLF & _
' Start-Job -ScriptBlock {' & @CRLF & _
' param($psexecCommand, $psexecArgs)' & @CRLF & _
' Start-Process -FilePath $psexecCommand -ArgumentList $psexecArgs -NoNewWindow -Wait' & @CRLF & _
' } -ArgumentList $psexecCommand, $psexecArgs' & @CRLF & _
' }' & @CRLF & _
'}' & @CRLF & _
'$null = Wait-Job -Job $jobs' & @CRLF & _
'$jobs | Remove-Job'

FileWrite($psScriptPath, $psScriptContent)
RunWait(@ComSpec & ' /c powershell.exe -ExecutionPolicy Bypass -File "' & $psScriptPath & '"', "", @SW_HIDE)
MsgBox(64, "Info", "Jobs wurden erfolgreich ausgeführt.")
FileDelete($psScriptPath)
EndFunc

Alles anzeigen

lg und danke noch mal für die Mithilfe

**Moombas**

nein das ist nicht richtig., ich habe dir oben das Beispiel mit dran gepackt gehabt, musste aber dennoch was korrigieren. Also deine Zeile 36 müsste so aussehen:

Return "' " & $params & ' "' & $splayer & '"' & " -Idummy '" -> Return ' -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy '

casi4712

Hm, ist doch echt zum Schweinemelken, ich habs jetzt so korrigiert wie du sagtest:

AutoIt

	ElseIf StringRight($filePath, 4) = ".mp3" Then ;Audiofile
        ;$splayer = '"c:\Program Files\VideoLAN\VLC\vlc.exe"'
       Return ' -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy '
    Else

die resultierende PS Zeile sieht ja eigentlich gut aus

$psexecArgs = "-u $username -p $password \\" + $computer + " -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy " + $batchFilePath

passiert leider immer noch nix, werd langsam wahnsinnig;)

Achso noch was, habe das resultierende mal in ISE geöffnet, da wird folgendes bemeckert:

Code

$jobs = foreach ($computer in $computers) {
    if ($computer -ne $env:COMPUTERNAME) {
        $psexecArgs = "-u $username -p $password \\" + $computer + " -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy " + $batchFilePath
        Start-Job -ScriptBlock {
            param($psexecCommand, $psexecArgs)
            Start-Process -FilePath $psexecCommand -ArgumentList $psexecArgs -NoNewWindow -Wait

Das c:\Programm wird unterstrichen, Meldung: Unerwartetes Token "c:\Program" in Ausdruck oder Anweisung.
+ CategoryInfo : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : UnexpectedToken

lg

**AspirinJunkie**

Zitat von casi4712

die resultierende PS Zeile sieht ja eigentlich gut aus
$psexecArgs = "-u $username -p $password \\" + $computer + " -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy " + $batchFilePath

Wo sieht die denn gut aus?
Die doppelten Anführungszeichen sind ja gar nicht escaped im String und geöffnete Anführungszeichen ohne korrespondierendes schließendes Anführungszeichen glaube ich kaum, dass dies in PS so korrekt ist.
Wenn schon dann sollte es am Ende eher so aussehen:

Code

$psexecArgs = "-u " + $username + " -p " + $password + " \\" + $computer + " -i ""c:\Program Files\VideoLAN\VLC\vlc.exe"" -Idummy """ + $batchFilePath + '"'

casi4712

Stimmt, wie gesagt habs ja dann auch noch gesehen dass das so nicht hinhaut, deswegen der Nachtrag. Ok du hast Recht wenn ich im resultierenden PS Script diese Zeile ersetze dann funzt es, aber wie übergebe ich das nun korrekt in meiner ElsIf Anweisung an $params, damit das so erzeigt wird?

lg und recht herzlichen Dank

**AspirinJunkie**

Oben Return $params & ' ""' & $splayer & '"" -Idummy ' und unten ' $psexecArgs = "-u " + $username + " -p " + $password + " \\" + $computer + "' & $parameter & '""" + $batchFilePath + ''"''' & @CRLF & _

casi4712

Donnerwetter, das war ja eine Zangengeburt lach, es funz jetzt, nur komisch dass die anderen Dateitypen funktioniert hatten.

lg und schönen Abend noch an alle Helfer

**Moombas**

casi4712 : Du ignorierst ja auch die ganze Zeit was ich geschrieben habe...

$psexecArgs = "-u $username -p $password \\" + $computer + " -i "c:\Program Files\VideoLAN\VLC\vlc.exe" -Idummy " + $batchFilePath

Die beiden unterstrichenen habe ich dir so NICHT angegeben, sondern dort müssten bei meinem Beispiel ' rein... aber egal jetzt...

casi4712

ok hatte ich übersehen, sorry dass mit den vielen '"''" kann einen schon wahnsinnig machen, ich versuche gerade am ende nach Batchfile noch eine Variable aus Autoit reinzunehmen, die ans ende gehört, aber breche mir da wieder die Ohren bei. Wo fehlt denn jetzt was:

AutoIt

'$psexecArgs = "-u " + $username + " -p " + $password + " \\" + $computer + "' & $parameter & '""" + $batchFilePath + ''" + "' & $Zparams & '''"'' & @CRLF & _

lg und schöne Woche noch

**Moombas**

Bin mir nicht 100%ig sicher, da ich nicht sicher bin welche dieser Variablen nun aus deinem psscript sind (und somit mit in die "" sollen) und welche aus Autoit sind (und somit nicht mit in die "" sollen). Aber auf Basis wie du es oben geschrieben hast würde ich es so machen:

AutoIt

'$psexecArgs = "-u " + $username + " -p " + $password + " \\" + $computer + " ' & $parameter & ' " + $batchFilePath + " " + "' & $Zparams & '"' & @CRLF & _

Ergebnis:

$psexecArgs = "-u " + $username + " -p " + $password + " \\" + $computer + " Deine-$parameter " + $batchFilePath + " " + "Deine-$Zparams"

->

$psexecArgs = "-u " + USERNAME + " -p " + PASSWORD + " \\" + COMPUTER + " Deine-$parameter " + C:/PATH/ + " " + "Deine-$Zparams"

->

-u USERNAME -p PASSWORD \\COMPUTER Deine-$parameter C:/PATH/ Deine-$Zparams