je ne comprend pas trop ce que tu reproche à mon utilisation de 'expr'. Pourrais-tu développer ?

J'ai dit tout ce que j'avais à dire : utiliser une commande externe, pour quelque chose que sait faire le shell, est inutile.

C'est une spécificité des [[ : ils protègent l'expression de gauche, sans qu'on ait besoin de la mettre entre guillemets.
`case' non plus, n'a pas besoin d'avoir la variable utilisée entre guillemet

var="quelque chose"
case $var in
...

Watael, je ne comprend pas trop ce que tu reproche à mon utilisation de 'expr'. Pourrais-tu développer ?

Si j'utilisais le double crochet, ne faudrait il pas plutôt mettre des apostrophes ? Si par exemple un nom de fichier comprenait des espaces ?

 if [[ "$file" = *%* ]] 

Bonne journée

for boucle sur une liste.
Ici, si les noms de fichiers comportent un caractère qui figure dans l'IFS, ça va échouer.

!

`expr' n'est pas une commande interne. Elle est inutile avec un shell compatible POSIX en général, avec bash en particulier.
Tu pourrais tirer parti des possibilités étendues des [[

if [[ $file == *%* ]]

L'explication des options configurables avec shopt est dans le man bash.

Personnellement, je l'aurais comme ceci :

#!/bin/bash

for file in $(find . -name "*%*"); do
	contains=`expr index "$file" '%'`
	if [[ $contains != 0 ]]; then
		nname=${file/\%/\_};
		mv "$file" "$nname"
	fi
done

Puis-je savoir à quoi sert le shopt -s nullglob globstar ? La commande help shopt n'étant pas très explicite, je ne comprends pas.

Bonne soirée.

il faut pouvoir descendre dans les sous répertoires

nc13160 a écrit :

J'ai une arboresence avec de nombreux sous dossiers

shopt -s nullglob globstar
for i in ./**/*\%*; do mv "$i" "${i//\%/_}"; done

shopt -s globstar
rename 's/%/_/g' **

Rapide et efficace ?

for i in *\%*; do mv "$i" "${i/\%/_}"; done

et s'il ya +sieurs %:

for i in *\%*; do mv "$i" "${i//\%/_}"; done

find /chemin/ -type f -iname "*%*" -exec rename 's/%/_/g' {} \;

for boucle sur une liste.
Ici, si les noms de fichiers comportent un caractère qui figure dans l'IFS, ça va échouer.

for i in `find -name "*"`;do d=`echo $i | sed 's/%/_/g'`;if [ $i != $d ];then echo "$i - $d";fi;done

Les noms des fichiers à renommer vont s'afficher, sI ça te plais tu lances ça:

for i in `find -name "*"`;do d=`echo $i | sed 's/%/_/g'`;if [ $i != $d ];then mv $i $d;fi;done

renommer -> rename
LOL