Discussion :

[GuillaumePr]

bonjour

je cherche à tester si un poste est connecté au réseau ou pas, et si non, que je sois prévenu par mail.
Dans un batch, J'utilise ping, errorlevel et blat.

ci dessous mon code : je reçois systématiquement un mail alors que le résultat du ping =0 donc ok.
Qu'est-ce qui ne va pas ? je pense à une erreur d'enchainement des commandes mais je ne vois pas où..

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

:INIT

rem ** Constantes
set Emetteur=xxxx@xxxx.com
set FichierDestinataire=e:\xxxx\TO.txt
set FichierCopie=e:\xxx\CC.txt
set CorpsDuMail=e:\xxx\BODY.txt
set SujetDuMail=ATTENTION, verifiez que le poste xxxx est allume - %DateDuJour%
set ServeurSMTP=xxxx
set Executable=\\xxxx\logiciel\blat\blat.exe


:ping
PING -n 1 xxxx | FIND "TTL" > NUL
IF not %ERRORLEVEL% ==1 goto envoi
else goto end



:ENVOI
%Executable% %CorpsDuMail% -u xxx\xxx -pw xxx -tf %FichierDestinataire% -cf %FichierCopie% -subject "%SujetDuMail%" -server %ServeurSMTP% -debug -f %Emetteur% 

:end
goto eof

Merci de votre aide.

Guillaume

[minnesota]

Salut,

Enlève les "==" du "IF not %ERRORLEVEL% ==1 goto envoi" et si tu veux utiliser le "else", tu dois le placer sur la même ligne.

[GuillaumePr]

salut Minnesota

j'ai fait comme tu as dis mais maintenant, je ne reçois plus aucun mail même qd errorlevel =1... : j'ai vérifié avec un poste non connecté qui renvoit un %errorlevel% à 1.

Je ne suis pas sur que mettre le Else sur la même ligne soit correcte, en effet, dans NotePad++, elle n'est plus reconnue comme une commande (écrit en bleu) mais comme du texte. Si on met Else à la ligne, c'est écrit en bleu de nouveau.

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

:INIT

rem ** Constantes
set Emetteur=xxxx@xxxx.com
set FichierDestinataire=e:\xxxx\TO.txt
set FichierCopie=e:\xxx\CC.txt
set CorpsDuMail=e:\xxx\BODY.txt
set SujetDuMail=ATTENTION, verifiez que le poste xxxx est allume - %DateDuJour%
set ServeurSMTP=xxxx
set Executable=\\xxxx\logiciel\blat\blat.exe


:ping
PING -n 1 xxxx | FIND "TTL" > NUL
IF not %ERRORLEVEL% 1 goto envoi else goto end



:ENVOI
%Executable% %CorpsDuMail% -u xxx\xxx -pw xxx -tf %FichierDestinataire% -cf %FichierCopie% -subject "%SujetDuMail%" -server %ServeurSMTP% -debug -f %Emetteur% 

:end
goto eof

[GuillaumePr]

Bon, j'ai trouvé

Code :

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

1
2
3
4
5
6
7

PING -n 1 **** | FIND "TTL" > NUL
IF %ERRORLEVEL% ==0 goto
end 
else goto 
ENVOI

comme çà, cela focntionne.

Merci

[minnesota]

Je ne suis pas sur que mettre le Else sur la même ligne soit correcte, en effet, dans NotePad++, elle n'est plus reconnue comme une commande

else n'est pas une commande forcément.

Voilà ce qu'il aurait fallu écrire :

Code :

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

if not %errorlevel% 1 (goto envoi) else goto end

ou encore

Code :

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

1
2
if not %errorlevel% 1 goto envoi
goto end

ou encore

Code :

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

1
2
if %errorlevel%==0 goto end
::"goto envoi" n'étant pas nécessaire ici.

ou encore...

Enfin tout, sauf ce que tu as écrit au-dessus

[GuillaumePr]

Merci de tes conseils. Je vais corriger.

Mais si c'est ce que j'ai écrit fonctionne, c'est que ce n'est pas si mal mais si ça ne respecte pas la bonne syntaxe.

Guillaume

[minnesota]

Ce qui fonctionne n'est pas infaillible. Mais ce qui est infaillible fonctionne toujours.

C'est de moi