Discussion :

[Guiona]

Salut @ll

Je suis en train de faire un batch qui reboote tous les postes d'un réseau dont le nom est de la forme AP254001, AP254002, AP254003,.... sous Windows NT4

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
@ECHO OFF
CLS

SET const=AP254

FOR /l %%a IN (0,1,3) DO (
FOR /l %%b IN (0,1,9) DO (
FOR /l %%c IN (0,1,9) DO (

ECHO "PING du poste %const%%%a%%b%%c"
PING %const%%%a%%b%%c
IF "%ERRORLEVEL%"=="0" GOTO PINGOK
GOTO NORESEAU

:PINGOK
rem Si le nom du poste se trouve dans le fichier Noreboot.txt on le reboote pas
FIND %const%%%a%%b%%c "Noreboot.txt"
IF "%ERRORLEVEL%"=="1" GOTO REBOOT
GOTO FIN

:REBOOT
ECHO "reboot du poste %const%%%a%%b%%c"
shutdown \\%const%%%a%%b%%c /R /Y /C
GOTO FIN

:NORESEAU
ECHO "Le poste n'est pas sur le reseau"
GOTO FIN

:FIN

)
)
)

Une erreur s'est glissé dans mon batch mais je la trouve pas

Erreur de syntaxe incorrecte

Si vous trouver dites le moi.

Si vous voyez aussi un moyen d'optimiser mon code n'hésitez pas.

Merci d'avance

[Reisubar]

Teste en remplaçant :

Code :

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

1
2
3
FOR /l %%a IN (0,1,3) DO ( 
FOR /l %%b IN (0,1,9) DO ( 
FOR /l %%c IN (0,1,9) DO (

par :

Code :

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

FOR /l %%a IN (0,1,3) DO ( FOR /l %%b IN (0,1,9) DO ( FOR /l %%c IN (0,1,9) DO (

Je te conseille d'utiliser PowerBatchpour mettre au point tes batchs : il a un assistant exécution pas à pas qui permet de visualiser où le Batch "plante" si besoin.

[Laurent Dardenne]

essaie comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
@ECHO OFF 
CLS 

SET const=AP254

FOR /l %%a IN (0,1,3) DO FOR /l %%b IN (0,1,9) DO FOR /l %%c IN (0,1,9) DO Call :Reboot %%a %%b %%c
Goto :eof

:Reboot
ECHO "PING du poste %const%%1%2%3" 
PING %const%%1%%2%3
IF "%ERRORLEVEL%"=="0" GOTO PINGOK 
GOTO NORESEAU 

:PINGOK 
rem Si le nom du poste se trouve dans le fichier Noreboot.txt on le reboote pas 
FIND %const%%1%%2%3 "Noreboot.txt" 
IF "%ERRORLEVEL%"=="1" GOTO REBOOT 
GOTO FIN 

:REBOOT 
ECHO "reboot du poste %const%%%a%%b%%c" 
shutdown \\%const%%1%%2%3 /R /Y /C 
GOTO FIN 

:NORESEAU 
ECHO "Le poste n'est pas sur le reseau" 
GOTO FIN 

:FIN 
Goto :eof

Dans ton contexte tu peux utiliser :

Code :

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

1
2
IF ERRORLEVEL 0 GOTO PINGOK

[Guiona]

Merci de votre aide.

J'essaye ça demain et vous tiens au courant!!!

Juste une petite question

Code :

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

Goto :eof

Pourquoi tu mets ceci?

[Laurent Dardenne]

Le 1er termine le batch principale

Le second termine le "sous programme" :Reboot
Syntaxe pour W2k et XP seulement.

[Guiona]

Donc dans mon cas je n'ai pas besoin de le mettre vu que je suis sous NT4

[Laurent Dardenne]

vérifie si cela fonctionne mais je ne pense pas.
Dans ce cas tu dois utiliser 2 batch.

[Guiona]

J'ai essayé sans

Code :

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

Goto :eof

et cela fonctionne parfaitement.

Merchi de votre aide

[Guiona]

Il faut que je mette

Code :

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

Goto :eof

Sinon mes boucles faisait un tour de trop:

AP254000
AP254001
...
AP254399
AP254

Maintenant cela s'arrête bien à AP254399

:p

[Laurent Dardenne]

Je t'ai laissé le découvrir
A+