Discussion :

[azahiri]

Bonjour les amis,

J'ai un truc qui me rend dingue.... Voilà le problème. A vrai dire j'ai deux problèmes:

1er Problème:

J'ai une fonction ChangeChar :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function ChangeChar {
	# Function: This function update char $1 with char $2
	# List of parameters:
	#          - $1: Char to be updated
	#          - $2: New value of the char
	#          - $3: File to be updated
	# ------------------------------------
	sed "s/^${1}=.*/${1}=${2}/" ${3} > ${3}.tmp
	cp -f ${3}.tmp ${3}
	rm -f  ${3}.tmp
}

Le problème c'est que quand le 2ème argument "${2}" est une chaine vide, il me donne des messages d'erreur :

Code :

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

sed: -e expression #1, char 40: unknown option to `s'

pour régler ce problème j'ai fait un truc du genre :

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
function ChangeChar {
	# Function: This function update char $1 with char $2
	# List of parameters:
	#          - $1: Char to be updated
	#          - $2: New value of the char
	#          - $3: File to be updated
	# ------------------------------------
	if [ -z ${2} ]
	then
		sed "s/^${1}=.*/${1}=/" ${3} > ${3}.tmp
	else 
		sed "s/^${1}=.*/${1}=${2}/" ${3} > ${3}.tmp
	fi
	cp -f ${3}.tmp ${3}
	rm -f  ${3}.tmp
}

mais ça ne règle pas le problème malheureusement.

2eme Problème:
ça concerne toujours la même fonction. En fait, avec cette fonction je parcours 3 fichiers, pour chacun, quand je trouve le pattern que je veux remplacer, et bien je le remplace sinon ça doit continuer le traitement sans générer d'erreur.

Le soucis, c'est que, quand il parcours les 3 fichiers, et qu'il ne trouve pas un pattern dans l'un d'eux, il me générer un message d'erreur :

Code :

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

sed: -e expression #1, char 40: unknown option to `s'

Merci de votre aide

[frp31]

normal si une des variable n'est pas alimentée que ça plante.

tu dois tester le contenu des variables avant le sed et si c'est sans valeur il faut la valoriser à null

genre

var=""

ça t'obliges à changer un peu ton script pour ne pas utiliser directement $1/$2/$3 mais bon la bonne pratique est de ne jamais utiliser nativement les paramètres de toute façon et de les utiliser uniquement pour valoriser des variables qui elles tu as le droit de les passer à des commandes

[zipe31]

Salut,

Il serait peut être mieux de nous montrer à quoi ressemble tes patterns ainsi que le contenu de ton fichier xxx.tmp, sans quoi malgré tous nos dons de voyance et autres boules de cristal, on coince un tantinet soit peu

[jack-ft]

Le problème c'est que quand le 2ème argument "${2}" est une chaine vide, il me donne des messages d'erreur :

Code :

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

sed: -e expression #1, char 40: unknown option to `s'

Alors je me lance pour le concours de boules de cristal!

As-tu oublié de positionner le 'set -u' de rigueur?
As-tu oublié de tester ton code en mode debug?
As-tu utilisé ChangeChar sans guillemetiser les arguments?

Code :

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

1
2
3
4
ChangeChar  $a $b $c        # Pas bien !
set -u                      # Bien !
typeset -ft ChangeChar      # Bien !
ChangeChar  "$a" "$b" "$c"  # Bien !

J'ai gagné?

[jack-ft]

Autre possibilité:
Les variables 1 et 2 peuvent-elles contenir le séparateur que tu as choisi pour la substitution (/) ?
Si oui, as-tu oublié de quoter ce caractère?

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
function ChangeChar {
	# Function: This function update char $1 with char $2
	# List of parameters:
	#          - $1: Char to be updated
	#          - $2: New value of the char
	#          - $3: File to be updated
	# ------------------------------------
	typeset s=/
	typeset from=$(echo "$1" | sed -e "s:${s}:\\\\${s}:g")
	typeset to=$(  echo "$2" | sed -e "s:${s}:\\\\${s}:g")
 
	sed "s${s}^${from}=.*${s}${to}=${s}" ${3} > ${3}.tmp
	cp -f ${3}.tmp ${3}
	rm -f  ${3}.tmp
}

[frp31]

Code :

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

1
2
3
4
 
	cp -f ${3}.tmp ${3}
	rm -f  ${3}.tmp
}

heu ??? plutôt

Code :

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

mv ${3}.tmp ${3}

ou alors

Code :

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

cp -f ${3}.tmp ${3} && rm -f  ${3}.tmp

non ?

[azahiri]

Re les amis,

Merci beaucoup de votre aide. Ça m'a été utile.

Finalement avant de faire l'appelle a la fonction ChangeChar j'ai fait un test sur la variable "qui sera le $2" si elle est vite je lui donne 0. et dans la fonction je fait un test sur le $2 et ça marche nickel.

Merci encore.

[jack-ft]

Finalement avant de faire l'appelle a la fonction ChangeChar j'ai fait un test sur la variable "qui sera le $2" si elle est vite je lui donne 0. et dans la fonction je fait un test sur le $2 et ça marche nickel.

Puis-je en déduire que la variable passée à ChangeChar n'est pas entourée de guillemets?
Si c'est le cas, j'ai bien remporté le concours de boules de cristal!

Plutôt que de tester sa vacuité et de la remplacer par 0, as-tu essayé de l'entourer de guillemets?

[azahiri]

oui j'ai essayé les guillemets j'ai même déclaré une variable au début de la fonction ChangeChar qui prend $2 comme valeur et je fais un test dessus si c'est vide ou, rien à y faire....

[N_BaH]

Code :

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

1
2
3
$ maFonc() { echo "${2:-zero}";}
$ maFonc foo "" baz
zero

!

[zipe31]

Re-

Re les amis,

Merci beaucoup de votre aide. Ça m'a été utile.

Finalement avant de faire l'appelle a la fonction ChangeChar j'ai fait un test sur la variable "qui sera le $2" si elle est vite je lui donne 0. et dans la fonction je fait un test sur le $2 et ça marche nickel.

Merci encore.

Je suis désolé mais je reste persuadé que l'erreur renvoyée par sed ne provient pas d'une variable vide ou non déclarée.

Code :

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

sed "s/^${1}=.*/${1}=${2}/" ${3} > ${3}.tmp

Le problème c'est que quand le 2ème argument "${2}" est une chaine vide, il me donne des messages d'erreur :

Code :

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

sed: -e expression #1, char 40: unknown option to `s'

Dans le code ci-dessus, la variable $2 vide ne générera pas d'erreur, et son action se résumerait à cette syntaxe sed "s/^${1}=.*/${1}=/" ${3} > ${3}.tmp qui effacerait donc l'équivalent de .*

La seule chose qui puisse générer cette forme d'erreur c'est, comme l'a soulignée jack-ft, la présence du séparateur par défaut dans le contenu de ta variable, mais comme tu n'a pas répondu à ma demande précédente, difficile de l'affirmer avec exactitude

Tentative de reproduction :

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
$ A='var';B="chat"
 
$ echo 'var=chien' | sed "s/^${A}=.*/${A}=${B}/"
var=chat
 
$ A='var';B=""
 
$ echo 'var=chien' | sed "s/^${A}=.*/${A}=${B}/"
var=
 
$ A='var';B="/"
 
$ echo 'var=chien' | sed "s/^${A}=.*/${A}=${B}/"
sed: -e expression n°1, caractère 16: option inconnue pour `s'
 
$ A='var';B="chat/"
 
$ echo 'var=chien' | sed "s/^${A}=.*/${A}=${B}/"
sed: -e expression n°1, caractère 20: option inconnue pour `s'