stoepsu77
Sep 09, 2020
view1821
view17
0
Goto Top

Goto geht einfach nicht

GermansolvedQuestionBatch, ShellDevelopment
Hallo zusammen

Ich habe eine Batch datei gebaut, die einfach den Goto Befehl nicht macht.
Ich sehe aber den Fehler einfach nicht.

So hoffe ich doch, dass mir hier jemand den zündenden Hinweis geben kann.
set vorhanden=
set ABC=\\xxx\SYSVOL\xxx\Scripts\Apps\Start\StartLayout.xml
set XYZ=C:\Tools\Einrichten\Users\AllUsers\AppData\Local\Microsoft\Windows\Shell\LayoutModification.xml

for /f "tokens=3" %i in ('reg query "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "StartLayoutFile" ') do set "vorhanden=%i"  
echo %vorhanden%

if %vorhanden%==%ABC% goto C
if %vorhanden%==%XYZ% goto Server

:C
REG ADD "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "StartLayoutFile"  /t REG_EXPAND_SZ /d "C:\Tools\Einrichten\Users\AllUsers\AppData\Local\Microsoft\Windows\Shell\LayoutModification.xml" /f  
goto End

:Server
REG ADD "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "StartLayoutFile"  /t REG_EXPAND_SZ /d "\\xxx\SYSVOL\xxx\Scripts\Apps\Start\StartLayout.xml" /f  
goto End

:End

Wenn ich z.B.
if %vorhanden%==%ABC% mit echo stimmt prüfe, gibt er mir stimmt aus, wennin der registry der ABC Wert steht.
Ebenso bei XYZ.
Also ist der eine oder andere IF wahr. Aber warum springt er dann nicht mit goto an den richtigen Ort?
Es wird einfach immer alles verarbeitet.

Er soll aber wechseln.
Also wenn ABC Pfad in der registry steht soll er wechseln auf XYZ Pfad uns.

hat jemand einen Rat für mich?

Danke
BeMe
comment-contentCommentShareShare
Share on Facebook Share on Twitter Share on Reddit Share on Linkedin

Content-Key: 603183

Url: https://administrator.de/contentid/603183

Printed on: May 4, 2024 at 18:05 o'clock

17 Comments
Latest comment
Goto Top
Member: TK1987
TK1987 Sep 09, 2020 updated at 14:27:20 (UTC)
Goto Top
Moin,

Zitat von @stoepsu77:
if %vorhanden%==%ABC% goto C
if %vorhanden%==%XYZ% goto Server
ich würde die Vergleich mit If immer in Anführungszeichen setzen, sowas macht oft Probleme.

Ansonsten...
hat jemand einen Rat für mich?
... ist wie immer der beste Rat: Lass das Batch-rumgemurkse sein und arbeite lieber mit Powershell
$PfadA     = "C:\Tools\Einrichten\Users\AllUsers\AppData\Local\Microsoft\Windows\Shell\LayoutModification.xml"  
$PfadB     = "\\xxx\SYSVOL\xxx\Scripts\Apps\Start\StartLayout.xml"  
$Schlüssel = 'Registry::HKCU\Software\Policies\Microsoft\Windows\Explorer'  

switch ((gp $Schlüssel).StartLayoutFile) {
  $PfadA {
    write-host -n "Schlüssel wird umgestellt auf '$PfadB'... "  
    try {
      Set-Itemproperty -Path $Schlüssel -Name 'StartLayoutFile' -Value $PfadB -EA stop  
      write-host -f green "erfolgreich."  
      } catch {write-host -f red $_.Exception.Message}
    }
  $PfadB {
    write-host -n "Schlüssel wird umgestellt auf '$PfadA'... "  
    try {
      Set-Itemproperty -Path $Schlüssel -Name 'StartLayoutFile' -Value $PfadA -EA stop  
      write-host -f green "erfolgreich."  
      } catch {write-host -f red $_.Exception.Message}
    }
  }

write-host -n "`nBeliebige Taste zum Beenden... "  
[void][console]::ReadKey($true)

Gruß Thomas
Member: Dr.Bit
Dr.Bit Sep 09, 2020 at 14:15:27 (UTC)
Goto Top
Muß es in Zeile 9 nicht else statt if heißen?

🖖
Member: stoepsu77
stoepsu77 Sep 09, 2020 at 14:45:45 (UTC)
Goto Top
Hallo TK1987

Danke!
Einfach mal kurz was für mich geschrieben. Toll!

Ich bin leider nicht so fit in Powershell.

Das Ding funktioniert super nur noch ein kleines Problem.
Wenn es den Schlüssel noch gar nicht gibt, müsste er mit den PfadB angaben angelegt werden plus noch ein weiter anderer Schlüssel angelegt werden.

Aber eben ich und Powershell sind noch keine Freunde.

Kannst du mir da noch mal helfen?

Danke
BeMe
Member: stoepsu77
stoepsu77 Sep 09, 2020 at 14:46:22 (UTC)
Goto Top
@Dr.Bit

else hilft nicht

trotzdem Danke

BeMe
Member: TK1987
TK1987 Sep 09, 2020 at 14:54:20 (UTC)
Goto Top
Zitat von @stoepsu77:
Wenn es den Schlüssel noch gar nicht gibt, müsste er mit den PfadB angaben angelegt werden
Kein Problem
plus noch ein weiter anderer Schlüssel angelegt werden.
Als der wäre?
Member: stoepsu77
stoepsu77 Sep 09, 2020 at 15:06:59 (UTC)
Goto Top
Bei mir hätte der so ausgesehen
REG ADD "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "LockedStartLayout" /t REG_DWORD /d "1" /f
Member: TK1987
Solution TK1987 Sep 09, 2020 updated at 16:00:01 (UTC)
Goto Top
$PfadA     = "C:\Tools\Einrichten\Users\AllUsers\AppData\Local\Microsoft\Windows\Shell\LayoutModification.xml"  
$PfadB     = "\\xxx\SYSVOL\xxx\Scripts\Apps\Start\StartLayout.xml"  
$Schlüssel = 'Registry::HKCU\Software\Policies\Microsoft\Windows\Explorer'  


if (Test-Path $Schlüssel) {
  switch ((gp $Schlüssel).StartLayoutFile) {
    $PfadA {
      write-host -n "Schlüssel wird umgestellt auf '$PfadB'... "  
      try {
        Set-Itemproperty -Path $Schlüssel -Name 'StartLayoutFile' -Value $PfadB -EA stop  
        write-host -f green "erfolgreich."  
        } catch {write-host -f red $_.Exception.Message}
      }
    $PfadB {
      write-host -n "Schlüssel wird umgestellt auf '$PfadA'... "  
      try {
        Set-Itemproperty -Path $Schlüssel -Name 'StartLayoutFile' -Value $PfadA -EA stop  
        write-host -f green "erfolgreich."  
        } catch {write-host -f red $_.Exception.Message}
      }
    }
  }
else {
  write-host -n "Schlüssel '$Schlüssel' wird erstellt... "  
  try {
    md -force $Schlüssel -EA Stop >$NULL 
    write-host -f green "erfolgreich."  
    write-host -n "Setze 'StartLayoutFile' auf '$PfadB'... "  
    Set-Itemproperty -Path $Schlüssel -Name 'StartLayoutFile' -Value $PfadB -EA stop  
    write-host -f green "erfolgreich."  
    write-host -n "Setze 'LockedStartLayout' auf 1... "  
    Set-Itemproperty -Path $Schlüssel -Name 'LockedStartLayout' -Value 1 -EA stop  
    write-host -f green "erfolgreich."  
    } catch {write-host -f red $_.Exception.Message}
  }

write-host -n "`nBeliebige Taste zum Beenden... "  
[void][console]::ReadKey($true)
Member: stoepsu77
stoepsu77 Sep 10, 2020 at 06:50:12 (UTC)
Goto Top
Hallo TK1987

Du bist mein Held, das weisst Du?

Vielen vielen dank.

Leider funktioniert Dein Script nicht, wenn der Schlüssel
LockedStartLayout und StartLayoutFile nicht in der Registry sind.
Die werden leider nicht eingetragen.

Leider verstehe ich Dein Script nur wage, hab also keinen Plan woran es liegen könnte.

Danke
BeMe
Member: TK1987
TK1987 Sep 10, 2020 at 07:09:40 (UTC)
Goto Top
Bei mir funktioniert das wunderbar.
Welche Fehlermeldung wird den bei dir ausgegeben?
Member: stoepsu77
stoepsu77 Sep 10, 2020 at 07:27:47 (UTC)
Goto Top
keine. er macht einfach keine Einträge.
Member: TK1987
TK1987 Sep 10, 2020 updated at 07:55:58 (UTC)
Goto Top
Zitat von @stoepsu77:
keine.
Entweder steht in grün erfolgreich dahinter, dann hat er die Werte des Schlüssels auch gesetzt, oder es wird in rot dahinter eine Meldung ausgegeben.
Wenn überall in grün erfolgreich dahinter steht, dann öffne mal Powershell und poste hier die Ausgabe von
gp 'Registry::HKCU\Software\Policies\Microsoft\Windows\Explorer'
Member: stoepsu77
stoepsu77 Sep 10, 2020 at 08:12:53 (UTC)
Goto Top
Ich hab versucht das Ganze mal etwas zu Dokumentieren.


Wenn die beiden Einträge bereits in der Registry sind, funktioniert der Wechsel perfekt.
Nur ohne die Einträge passiert irgendwie nix. (Bild)

registry
Mitglied: 145916
145916 Sep 10, 2020 updated at 08:25:01 (UTC)
Goto Top
Zitat von @stoepsu77:

Hallo zusammen

Ich habe eine Batch datei gebaut, die einfach den Goto Befehl nicht macht.
Ich sehe aber den Fehler einfach nicht.

Hier fehlen die doppelten Prozentzeichen für die Laufvariable, so funktioniert das nur auf der Konsole nicht aber innerhalb einer Batchdatei face-wink, außerdem berücksichtigt es keine Leerzeichen im Value
for /f "tokens=3" %i in ('reg query "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "StartLayoutFile" ') do set "vorhanden=%i"
for /f "tokens=2,*" %%a in ('reg query "HKEY_CURRENT_USER\Software\Policies\Microsoft\Windows\Explorer" /v "StartLayoutFile" ') do set "vorhanden=%%b"
Member: TK1987
Solution TK1987 Sep 10, 2020 at 08:40:56 (UTC)
Goto Top
Achsooo xD. Ich glaube wir haben da aneinander vorbei gredet.

Zur Info: Als Registry-Schlüssel bezeichnet man z.B. HKCU\Software\Policies\Microsoft\Windows\Explorer.

StartLayoutFile hingegen ist nur der Wert eines Schlüssels (nicht der Schlüssel selbst).

Was ich also gemacht habe, war mit If zu überprüfen, ob der Schlüssel existiert. Was du (nehme ich mal an wolltest), war hingegegen zu überprüfen, ob der Wert des Schlüssels existiert.

Daher
Vermutlich wird das Skript abgebrochen
Nein, dann käme eine Fehlermeldung. Es wird ausgeführt - und das volkommen korrekt. Der Schlüssel selbst existiert, nur der Wert halt nicht.

Ändere das If-Statement (Zeile 6) bitte wie folgt ab:
if ((gp $Schlüssel -EA si).StartLayoutFile) {
Dann dürfte alles wie gewünscht funktionieren.

BTW: Powershell-Skripte ausführen geht am besten so:

Leg das Skript in irgend einen Ort, wo es nicht stört.
Dann drücke Rechtsklick > kopieren und gehe an den Ort, von wo aus du es ausführen möchtest. Hier drückst du Rechtsklick > Verknüpfung einfügen.
Auf diese Verknüpfung Rechtsklick > Eigenschaften und bei Ziel vor den Pfad zum Skript noch Powershell -EP ByPass -File eintragen.
In deinem jetziegn Fall stände bei Ziel dann
Powershell -EP ByPass -File "C:\Users\g.muster\Desktop\Profile_a.Ps1"
Member: stoepsu77
stoepsu77 Sep 10, 2020 at 08:42:57 (UTC)
Goto Top
Danke eaglerfinder

ich bin im Moment noch auf Konsolen Ebene.
Deine Verbesserung hab ich ausprobiert. bringt leider auch nichts.

Danke
Member: stoepsu77
stoepsu77 Sep 10, 2020 at 08:51:32 (UTC)
Goto Top
Sorry für die Verwirrung.

Jetzt klappt alles genau so wie es sein soll!

Vielen vielen Dank!
Du bist echt super.
Mitglied: 145916
145916 Sep 10, 2020 updated at 09:19:04 (UTC)
Goto Top
Zitat von @stoepsu77:
Deine Verbesserung hab ich ausprobiert. bringt leider auch nichts.
Klappt hier einwandfrei, natürlich inkl. Verbesserungen wie Anführungszeichen in den IF-Vergleichen ...
GermansolvedQuestionBatch, ShellDevelopment
Hotly discussed
LinuxeroWirguard with systemd and nftablesLinuxero - 7 CommentsFrankUbuntu 24.04 LTS Noble Numbat availableFrankadmtechDeveloper diary: Release 6.1admtech