Boucle FOR : ") etait inattendu"

Bonjour,

J'ai un petit launcher qui parcours une liste de hostname pour lancer un batch sur chaque poste avec le nom d'hote en parametre.
Il fonctionnait tres bien mais etait un peu lent, alors j'ai apporté une petite modification à mon batch dans le but d'accelerer son deroulement.
Maintenant il plante des le debut de l'execution avec le retour ") etait inattendu"
Je ne trouve pas mon erreur, votre aide serait très appréciée. :)

AVANT :

Code:

---

1 2 3 4 5 6 7 8

@echo off echo %time% --------------- LAUNCHER started>>log.txt for /f %%a in ('type hostnames.txt') do ( IF EXIST "\\%%a\C$\Windows\System32\config.txt" ( start starter.cmd %%a echo %time% $$$$$$$$$$ Started psexec for %%a $$$$$$$$$$>>log.txt ) )

---

APRES :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

@echo off echo %time% --------------- LAUNCHER started>>log.txt for /f %%a in ('type hostnames.txt') do ( PING -n 1 %%a | FIND "TTL" > NUL IF ERRORLEVEL 1 GOTO NEXT IF EXIST "\\%%a\C$\Windows\System32\fsdbupdate.exe" ( start /b starter.cmd %%a echo %time% $$$$$$$$$$ Started psexec for %%a $$$$$$$$$$>>log.txt ) :NEXT )

---

Le problème vient de l'étiquette :NEXT. Dans ce bloque de script il n'y a que ), d'ou l'erreur.

Même si le batch est un langage de programme Spaghetti il est bon de l'éviter tant que possible.
Petit tips, utilise le && et/ou le ||

Code:

---

1 2 3 4 5 6 7 8 9

@echo off echo %time% --------------- LAUNCHER started>>log.txt for /f %%a in ('type hostnames.txt') do ( PING -n 1 %%a | FIND "TTL" > NUL && IF EXIST "\\%%a\C$\Windows\System32\fsdbupdate.exe" ( start /b starter.cmd %%a echo %time% $$$$$$$$$$ Started psexec for %%a $$$$$$$$$$>>log.txt ) ) pause

---

ou directement

Code:

---

1 2 3 4 5 6 7 8 9

@echo off echo %time% --------------- LAUNCHER started>>log.txt for /f %%a in ('type hostnames.txt') do ( PING -n 1 %%a > NUL && IF EXIST "\\%%a\C$\Windows\System32\fsdbupdate.exe" ( start /b starter.cmd %%a echo %time% $$$$$$$$$$ Started psexec for %%a $$$$$$$$$$>>log.txt ) ) pause

---

salut,


il y a 2 problèmes dans ton code: le GOTO dans les FORINDO's et IF's et les étiquettes dans les blocs

dans des FORINDO/IF ne jamais utilisé de sauts GOTO car ça "casse" le contexte d'execution:

forindo

Code:

---

1 2 3 4 5 6

@echo off for /l %%a in (1 1 3) do ( goto :saut1 :saut1 echo %%a )

---

if/else

Code:

---

1 2 3 4 5 6 7 8

if walid==walid ( goto :saut2 echo 1 :saut2 echo 2 ) else ( echo 3 !!! )

---

les étiquettes dans des blocs ont leurs propres règles: une étiquette dans un bloc doit être suivi par une instruction/et ou étiquette "valide", valide ici veux dire

pas de )

Code:

---

1 2 3

( :A )

---

pas de ligne vide

Code:

---

1 2 3 4

( :B )

---

pas d'étiquette :: (lecteur)

Code:

---

1 2 3 4

( :C ::C )

---

cependant la seconde ligne, si c'est une étiquette valide, peux interpréter quelques caractère spéciaux tels que &>|()

Code:

---

1 2 3 4

( :D :E & (FOR %%A IN (1 2 3) DO @ECHO %%A ) | SORT /R )

---

ou encore:

Code:

---

1 2 3 4 5 6

( :F :G & SET /P INPUT=ENTER YOUR NAME: && CALL ECHO HELLO %%INPUT%% > ___file__.tmp type ___file__.tmp ) del ___file__.tmp

---