Discussion :

[kstou2001]

Bonjour,

Voilà mon souci. Je cherche à modifier la valeur d'une variable que je passe en argument d'une fonction.

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
maFonction
{
    if  ping -c 3 192.168.xxx.xxx
    then
         $1=true
    fi
 
    echo ${var1} #devrait renvoyer true si ping ok
}
var1=false
var2=false
 
maFonction '$var1'
...
maFonction '$var2'

Quand je teste mon script, j'obtiens ce genre d'erreur
test.sh: line 86: $var1=true: command not found

Est-ce que quelqu'un a une idée? La solution alternative est d'utiliser case mais c'est beaucoup plus long.

merci d'avance

edit:j'ai complété la fonction que j'utilise

[al1_24]

Et en faisant comme ça, ça ne fonctionnerait pas mieux ?

Code :

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

maFonction 'var1'

[Aiglon13]

Bonjour,

Voilà mon souci. Je cherche à modifier la valeur d'une variable que je passe en argument d'une fonction.

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
maFonction
{
    if ...
    then
     $1=true
    fi
 
 
}
var1=false
var2=false
 
maFonction '$var1'
...
maFonction '$var2'

Quand je teste mon script, j'obtiens ce genre d'erreur
test.sh: line 86: $var1=true: command not found

Est-ce que quelqu'un a une idée? La solution alternative est d'utiliser case mais c'est beaucoup plus long.

merci d'avance

Bonjour,

As-tu essayé avec :

Code :

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

$1="true"

ou

Code :

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

1
2
typeset var=$1
var="true"

et de lancer tes fonctions comme ça :

Code :

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

maFonction $var1

(sans les apostrophes)

[zipe31]

Salut,

Une fonction se déclare soit comme ça function maFonction { cmd; }, soit comme ça maFonction () { cmd; } (note les parenthèses dans le second cas).

Et d'autres part si tu encadres tes variables par des quotes simples maFonction '$var1' elles ne seront pas interprétées
Il faut obligatoirement les entourées de quotes doubles maFonction "${var1}". Les entourées également d'accolades est une bonne habitude à prendre.

[kstou2001]

Merci pour la réponse mais le problème est identique.

[Aiglon13]

Et ceci :

mafonction ${var1} ?

[kstou2001]

Et d'autres part si tu encadres tes variables par des quotes simples maFonction '$var1' elles ne seront pas interprétées
Il faut obligatoirement les entourées de quotes doubles maFonction "${var1}". Les entourées également d'accolades est une bonne habitude à prendre.

Dans mon cas je ne souhaite pas utiliser la valeur de la variable mais son nom pour lui affecter ensuite une valeur.

Mettre des doubles quotes me génère cette erreur
test.sh: line 86: false=true: command not found

[kstou2001]

Bonjour,

As-tu essayé avec :

Code :

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

$1="true"

ou

Code :

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

1
2
typeset var=$1
var="true"

et de lancer tes fonctions comme ça :

Code :

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

maFonction $var1

(sans les apostrophes)

maFonction $var1
ou
maFonction "$var1"
=
maFonction false

J'ai rajouté un echo $1 dans maFonction. Il n'y a qu'avec les simple quote que je récupère le nom de ma variable. Par contre impossible de lui affecter une valeur.

[zipe31]

Dans mon cas je ne souhaite pas utiliser la valeur de la variable mais son nom pour lui affecter ensuite une valeur.

Alors le "$" est inutile devant

Quid de ta déclaration de fonction ? L'as-tu modifié ?

Sans savoir ce que fait ta fonction exactement, difficile d'interpréter le résultat de ton erreur

[kstou2001]

Alors le "$" est inutile devant

Quid de ta déclaration de fonction ? L'as-tu modifié ?

Sans savoir ce que fait ta fonction exactement, difficile d'interpréter le résultat de ton erreur

ma fonction va pinger un serveur. S'il répond la variable passe à true. (je venais de compléter maFonction pour être plus parlant)

J'ai déjà essayé d'appeler la fonction de la manière suivante mais c'est pas mieux
maFonction 'var1'
erreur:essai.sh: line 6: var1=true: command not found==> pourtant c'est ce que je souhaite: que var1 passe à true.

[zipe31]

Affiche ton code complet, on n'est pas devin

[kstou2001]

Affiche ton code complet, on n'est pas devin

Il est en début de post. Je teste sur ce code avant de l'appliquer à mon vrai script.
En adaptant l'adresse IP par une en service on devrait avoir
echo ${var1} qui affiche true.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
maFonction
{
    if  ping -c 3 192.168.xxx.xxx
    then
         $1=true
    fi
 
    echo ${var1} #devrait renvoyer true si ping ok
}
var1=false
var2=false
 
maFonction '$var1'

[zipe31]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#!/bin/bash

maFonction () {
    if  ping -c 1 192.168.xx.xx &>/dev/null
    then
         eval $1=true
    fi
     echo ${var1} 
}
var1=false
 
maFonction 'var1'

[jlliagre]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function maFonction
{
    if  ping -c 3 192.168.1.253
    then
         eval $1=true
    fi
    echo ${var1} #devrait renvoyer true si ping ok
}
 
var1=false
var2=false
 
maFonction var1
maFonction var2

[N_BaH]

Code :

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

1
2
3
4
5
6
7
8
9
maFonction() {
    ping -c 3 192.168.1.253 >/dev/null 2>&1 && echo true
}
 
var1=false
var2=false
 
var1=$(maFonction)
var2=$(maFonction)

à mon avis, le raisonnement de kstou2001 n'est pas correct.

Il nous faudrait le script complet pour voir comment utiliser la fonction, si elle est vraiment nécessaire, de même que ces variables.

Il faudrait aussi nous dire quel interpréteur de commandes tu utilises.

[jlliagre]

Code :

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

1
2
3
maFonction() {
    ping -c 3 192.168.1.253 >/dev/null 2>&1 && echo true || echo false
}

[N_BaH]

très juste jlliagre, la substitution de commande écrase la précédente valeur de la variable, même si la commande échoue.

[kstou2001]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function maFonction
{
    if  ping -c 3 192.168.1.253
    then
         eval $1=true
    fi
    echo ${var1} #devrait renvoyer true si ping ok
}
 
var1=false
var2=false
 
maFonction var1
maFonction var2

Merci. Grâce à la commande eval, je peux affecter la valeur à une variable passée en argument.

[kstou2001]

à mon avis, le raisonnement de kstou2001 n'est pas correct.

Il nous faudrait le script complet pour voir comment utiliser la fonction, si elle est vraiment nécessaire, de même que ces variables.

Il faudrait aussi nous dire quel interpréteur de commandes tu utilises.

En raccourci, j'avais besoin d'une fonction pour tester la connexion avec différents réseaux.
En plus approfondi, chaque réseau possède 2 liaisons, une principale et une de secours (valeurs passées en argument). Je teste d'abord le principal puis le secours. Je fais ça pour chaque réseau. Ensuite je regarde à l'intérieur du réseau et c'est pour ça que j'essayais de passer une variable en argument.

Le code complet pour résoudre mon problème était inutile sauf si on voulait trouver une solution de contournement.

Pour l'interpréteur de commande c'est le Bourne Shell.

[N_BaH]

eval est la solution de facilité, et elle comporte certains risques (qui n'apparaissent pas ici), qui me font souhaiter qu'on ne la propose pas pour des problèmes simples.
En général, l'utilisation d'eval souligne une erreur "algorithmique" (?).

Code monTestReseau :

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

1
2
3
4
5
#!/bin/sh
 
for adresse #sous-entendu «in "$@"»
do ping -c 1 $adresse && exploration_reseau $adresse
done

c'est plus compliqué que ça ?