Discussion :

[tom2669]

Bonsoir à tous,

J'ai rédigé un petit script qui me permet d'envoyer des SMS par rapport à la trame radio reçu.
Chaque numéro de téléphone correspond à un récepteur radio.

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
#!/bin/bash
 
# appel des scripts
 
send_sms.sh
 
#inscription des variables
 
emetteur_tel=( ["123456 Function"]="0612467890" ['24567 Function']="0786948172" ['4567890 Function]="07878987" )
 
 
#commande de décodage et de réception des trames radio
 
for line in `nc -l -u 7355 | sox -t raw -esigned-integer -b16 -r 48000 - -esigned-integer -b16 -r 22050 -t raw - | multimon-ng -t raw -a SCOPE -a POCSAG512 -a POCSAG1200 -a POCSAG2400 -f alpha -`
 
do
 
#isole le numéro d'émetteur sous forme 1234567 Function
 
emetteur_tel = `echo $line | cut -d : -f 3`
 
#isole le message contenu dans la trame
 
message = `echo $line | cut -d : -f 5`
 
#execute le script d'envoi de SMS suivant le numéro de récepteur et le message correspondant
 
./send_sms $emetteur_tel $message
 
...
echo $value>
done

Que pensez vous de mon script ?
C'est mon tout premier, je viens d'apprendre en le rédigeant

Merci par avance de votre aide

[N_BaH]

Bonjour,

Que pensez vous de mon script ?

• il n'est pas indenté
• le tableau associatif n'est pas déclaré, et ne sera donc pas reconnu comme tel
• il manque une apostrophe dans l'un de ses index
• des fois c'est des guillemets, des fois des apostrophes
• on ne parcours pas des lignes avec une boucle for (cf. Comment lire une fichier)
• il ne faut pas d'espaces autout d'une assignation
• la forme ` est désuète; la forme "moderne" ($()) est préférable
• il faut toujours mettre le développement des variables entre guillemets, à moins de savoir pourquoi

etc