Script de renommage en sh avec contrainte d'environnement

Alors encore merci pour toutes les infos, le lien que j'irai consulter (EDIT: en effet c'est bien expliqué ^^).
ça commence à bien fonctionner et je vous en remercies.

Cependant j'ai encore des questions ^^ et surtout des interrogations.

Avec ce cas de figure :

Citation:

---

Saison 01, Episode 06 [N°006] - La Momie.avi

---

Dans le :

Code:

---

case "${LESS_FILENAME}" in

---

Si je met dans cet ordre, je n'ai pas de problème, l'épisode match sur la premier cas

Code:

---

saison?[0-9][0-9],?episode?[0-9][0-9]*.???)

---

Code:

---

*[0-9][0-9][0-9]*.???)

---

en revanche, si je les met dans le sens inverse

Code:

---

*[0-9][0-9][0-9]*.???)

---

Code:

---

saison?[0-9][0-9],?episode?[0-9][0-9]*.???)

---

il match également sur le premier cas et je ne comprend pas pourquoi ?
et du coup je me retrouve avec la saison à 0 et avec un seul chiffre.

j'ai deux autre cas, si un fichier ne correspond à aucun cas il est supprimé, c'est pas top! dois-je mettre un cas en dernier spécifiant de ne rien faire ?
et lorsque un fichier nommé [qote]xxxx xxxxx - s02 e04.avi[/qote] match avec le cas :

Code:

---

*s[0-9][0-9]*e[0-9][0-9]*.???)

---

Mais j'obtiens le résultat suivant également sans comprendre pourquoi !

Citation:

---

Sxxxx xxxxxx - s02 e04.aviExxxx xxxxxx - s02 e04.avi.avi

---

Je me retourne les neurones depuis ce matin à comprendre mes différent cas, mais sans succès.

Dois je obligatoirement passé par un "case" pour pouvoir renommer mes différent type de fichier ? Ou existe t'il une autre possibilité en faisant plus général ?


autre chose j'ai essayer de faire une fonction pour effectuer ceci de manière générique

Code:

---

1 2 3 4

if [ "$SAISON" -le 9 ]; then SAISON="0""$SAISON"; fi

---

j'ai trouver en cherchant la nomenclature suivante :

Code:

---

1 2 3 4

NOM_DE_FONCTION() { COMMANDS }

---

j'ai donc fait ceci :

Code:

---

1 2 3 4 5 6 7 8

F_ADD_NUM() { if [ "$1" -le 9 ]; then VALUE="0""$1"; return VALUE fi }

---

et dans la parti de mon code :

Code:

---

1 2 3 4 5 6

*[0-9]?[0-9][0-9]*.???) # cas xxxxx2x23.xxx echo "cas 3 : $LESS_FILENAME" SAISON=$(echo "${LESS_FILENAME}" | sed -e 's:.*\([0-9][0-9]\).[0-9][0-9].*\....$:\1:') EPISODE=$(echo "${LESS_FILENAME}" | sed -e 's:.*[0-9][0-9].\([0-9][0-9]\).*\....$:\1:') F_ADD_NUM "$SAISON" ;;

---

je l'ai testé de diverse façons, mais sans succès.
Savez-vous pourquoi ?

Citation:

---

Envoyé par fifyx

Avec ce cas de figure Saison 01, Episode 06 [N°006] - La Momie.avi :

Si je met dans cet ordre, je n'ai pas de problème, l'épisode match sur la premier cas

Code:

---

1 2	saison?[0-9][0-9],?episode?[0-9][0-9]*.???) *[0-9][0-9][0-9]*.???)

---

en revanche, si je les met dans le sens inverse

il match également sur le premier cas et je ne comprend pas pourquoi ?

---

Eh oui!
Le nom Saison 01, Episode 06 [N°006] - La Momie.avi matche les 2 expressions!
La première, de manière évidente, et la 2ème à cause de [N°006] où il reconnaît 3 chiffres précédés et suivis de n'importe quel nombre de caractères quelconques!

L'intérêt du case est justement de tester les cas un par un en s'arrêtant sur le bon.
Du coup, il faut effectivement les classer du plus particulier au plus général!

Citation:

---

j'ai deux autre cas, si un fichier ne correspond à aucun cas il est supprimé, c'est pas top! dois-je mettre un cas en dernier spécifiant de ne rien faire ?

---

Oui! Pour le debug et l'évolution de ton programme, tu peux/dois mettre en dernière clause quelque chose comme:

Code:

---

1 2 3 4 5 6

case "${filename}" in <snip> *) echo "Type de fichier inconnu '${filename}'" 1>&2 exit 1 ;; esac

---

Citation:

---

et lorsque un fichier nommé "xxxx xxxxx - s02 e04.avi" match avec le cas : *s[0-9][0-9]*e[0-9][0-9]*.???) Mais j'obtiens le résultat suivant également sans comprendre pourquoi !

Code:

---

Sxxxx xxxxxx - s02 e04.aviExxxx xxxxxx - s02 e04.avi.avi

---

---

Je préférerais que tu cites toute la clause (le match et la substitution).
Ce serait plus facile pour comprendre d'où vient le problème.

En tout cas, c'est la marque que la substitution n'a pas matché!

Code:

---

saison=$(echo "${filename}" | sed -e "s:${regexp}:\1:")

---

si ${filename} ne matche pas ${regexp}, alors le sed ne change rien et du coup saison contient l'intégralité du filename.
de même pour épisode, ce qui donne le résultat que tu observes.

Citation:

---

Dois je obligatoirement passé par un "case" pour pouvoir renommer mes différent type de fichier ? Ou existe t'il une autre possibilité en faisant plus général ?

---

Tu peux faire une méthode plus générale, mais c'est moins souple!

Par exemple, si tu es sûr de trouver 2 chiffres puis encore 2 chiffres, tu peux essayer:

Code:

---

1 2 3 4 5

regexp='^[^0-9]*\([0-9][0-9]\)[^0-9][^0-9]*\([0-9][0-9]\)[^0-9].*' saison=$(echo "${filename}" | sed -e "s:${regexp}:\1:") episode=$(echo "${filename}" | sed -e "s:${regexp}:\2:")   echo "${filename}" | sed -e "s:${regexp}:Episode \2 de la saison \1:"

---

Citation:

---

autre chose j'ai essayer de faire une fonction pour effectuer ceci de manière générique

Code:

---

1 2 3 4

if [ "$SAISON" -le 9 ]; then SAISON="0""$SAISON"; fi

---

---

Fonctionnellement, ce n'est pas une bonne idée car le code était prévu pour chercher et retourner 2 chiffres!
S'il n'en trouve qu'un, c'est qu'il y a un bug et il vaut mieux corriger le bug plutôt que de corriger le résultat de la fonction beuguée!!! :calim2:

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9

F_ADD_NUM() { if [ "$1" -le 9 ]; then VALUE="0""$1"; return VALUE fi } F_ADD_NUM "$SAISON"

---

je l'ai testé de diverse façons, mais sans succès.
Savez-vous pourquoi ?

---

Oui. Il y a plusieurs raisons.

Il faut un dollar devant VALUE dans return $VALUE.
La fonction 'return' n'admet pas de chaîne, mais un entier (code status), donc elle ne peut pas transmettre "01".
Dans le cas où le nombre est > 9, elle retourne automatiquement 0.
Si return marchait, il faudrait écrire: F_ADD_NUM "$SAISON" ; SAISON=$? pour récupérer le status.
En l'occurrence (et juste pour information), il vaudrait mieux faire quelque chose comme:

Code:

---

1 2 3 4 5

F_ADD_NUM() { if [ "$1" -le 9 ] ; then echo "0$1" ; else echo "$1" ; fi } SAISON=$(F_ADD_NUM "$SAISON")

---

Mais il serait plus simple d'utiliser "printf" qui fait tout ça très bien.