Discussion :

[bessonnet]

bonjour,
voici mon script :

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
36
37
38
#!/bin/sh
# bin/telnet
 
while [ 1 ];do
{
 
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 $MACHINE";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";
 
   while [ $PING ] && [ i!=720 ];do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done;
 
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e "$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e "${CMD[5]}\15";) | telnet $MACHINE $PORT;
 
   while [ ! $PING ];do
   {
      sleep 5;
   }
   done;
 
}
done;
exit 0;

ce script doit me permettre de "pinguer" en continue un switch et si au bout d'une heure une on peu toujours le "pinguer" alors il faut fermer un port du switch.
mon probleme est que mon script ne marche pas.
il me met : too many arguments line 20 line 32.
je ne comprends pas pourquoi ? Auriez vous une idée ? Pouvez vous m'aider s'il vous plait ?

[bessonnet]

sur ligne 20 et ligne 32 se trouve :
ligne 20

Code :

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

1
2
3
4
5
6
while [ $PING ] && [ i!=720 ];do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done;

ligne 32

Code :

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

1
2
3
4
5
while [ ! $PING ];do
   {
      sleep 5;
   }
   done;

voila je prends toute les remarques et commentaires constructif qui me permettront d'avancer.
je remercie tous ceux qui ont pris le temps de lire ce topic.
Cordialement

[Arnaud F.]

Déjà dans dans ta seconde boucle while, sache que la comparaison ne se fait pas avec != mais -ne (pour Not Equal donc ta ligne deviendra

Code :

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

while [ $PING  && i -ne 720 ];do

Et il ne faut pas mettre d'accolades non plus dans la boucle while

[edit] il faut faire l'opérateur logique dans les crochets aussi (j'ai modifié le code)

[edit2] pas de ; après le done

[bessonnet]

et pour :

Code :

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

while [ ! $PING ];do

j'aurais donc :

Code :

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

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci

[bessonnet]

j'ai donc effectué les modifications,
pour la ligne 32, au premier tour l'erreur à disparue, au tour d'après elle est revenue, la meme : "too many arguments".
pour la ligne 20 elle s'est changée en : [:missing`]'
voila je m'y connais pas encore assez pour tout comprendre.
si vous pouviez continiuer à me mettre sur la bonne voie je vous en serais reconnaissante.
merci

[gorgonite]

pourquoi pas plutôt cela pour le test de ping ???

Code :

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

 [$PING -ne 0]

[Arnaud F.]

pourquoi pas plutôt cela pour le test de ping ???

Code :

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

 [$PING -ne 0]

+1

j'avais meme pas fait attention à ça ...

[Arnaud F.]

et pour :

Code :

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

while [ ! $PING ];do

j'aurais donc :

Code :

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

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci

Non car quand tu utilise -ne il faut comparer par rapport à une valeur or si tu fais ça, ce n'est pas le cas

Suis ce qu'a écris gorgonite

P.S: Pense à le modifier dans tes 2 boucles

[bessonnet]

merci,
j'ai changé dans les deux boucles, mais les erreurs persistes !
peu etre y à t'il des erreurs ailleurs que sur ces deux lignes là ? parce que là je ne vois vraiment pas ...
quand je fais : "sh -n ping.sh" il ne m'affiche aucune erreur, c'est quand je le lance (sh ping.sh) qu'il commence son execution et en meme temps il m'affiche ces deux erreurs.
au finale il tourne en rond avec les deux erreurs.

auriez vous d'autres idées ?

[Arnaud F.]

Oui, une seule, post nous ton code mis à jour maintenant, qu'on voit réellement ce que tu à modifié