Discussion :

[justdoit08]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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
)

[ericlm128]

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 batch :

Sélectionner tout - Visualiser dans une fenêtre à part

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 batch :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[I'm_HERE]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

if/else

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
(
:A
)

pas de ligne vide

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
(
:B

)

pas d'étiquette :: (lecteur)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

ou encore:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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