Retrouver la taille des commandes sed dans un script

Bonjour,
Mon but est d'ecrire une regle d'un script ( https://github.com/belaran/pmb celui ci ) qui est un script de controle de qualité de script bash .
La rêgle que je veux implementer doit reperer et evaluer la taille des commandes sed du fichier script pour avertir lorsqu'elle est supérieur a 20 caractere :

example : echo "truc" | sed -e /s/truc/machin/p' <= OK
echo "truc" | sed 's/machin/trucunpeutroplongpourlaregle/p' <= ERREUR

j'imagine qu'il faut le faire a l'aide de sed lui même ou de grep... mais je n'ai pas réussi a le faire, voici ce que j'ai essayé :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

pmb_check_sed_length() {   local source_file=${1} local nb_max_characters=${PMB_NB_MAX_CHARACTERS_BY_SED:-20}   nb_line=1 cat "${source_file}" | \ while read line do sed_length=$(echo ${line} | sed -e "s/sed[-[a-z][A-Z] ]*['\"]{1}(.*)['\"]{1}/\1/p"| wc -c) if [[ ${sed_length} -gt ${nb_max_characters} ]]; then echo "${source_file},${nb_line},Le sed à la ligne numéro ${nb_line} est trop long: taille > ${nb_max_characters},${line}" fi nb_line=$(expr "${nb_line}" + 1) done }

---

mais je n'arrive pas a extraire la partie entre parenthese du sed et rien d'autre...

Merci d'avance pour votre aide !

Bonjour,
Voici une petite aide, mais par contre je ne pense pas que tous les parsing d'une commande sed sont là...

Code:

---

1 2 3 4

$ echo "sed 's/machin/trucunpeutroplongpourlaregle/p'" | grep -qo "sed[-a-zA-Z ]* ['\"].\{20,\}['\"]" && echo ERREUR || echo OK ERREUR $ echo "sed 's/machin/truc/p'" | grep -qo "sed[-a-zA-Z ]* ['\"].\{20,\}['\"]" && echo ERREUR || echo OK OK

---

ATTENTION: ici je fais un echo ERREUR si le retour du grep est vrai, et un echo OK s'il retourne faux.

Merci beaucoup pour ton aide !
La solution est plutôt simple pourtant j'ai pas reussi tout seul , merci !
Du coup il faudrait coupler ça avec une recuperation de chaque sed de ligne au cas ou il y en ai plusieurs par ligne , c'est ça ?

En modifiant quelque peut la regex et en n'utilisant plus le connecteur -q du grep, tu peux obtenir la liste des sed recherchés qui se trouvent sur la même ligne en un seul grep:

Code:

---

1 2 3

$ echo "titi ; sed 's/machin/truc/p' ; tata ; sed 's/machin1/truc1unpeutroplongpourlaregle/p' ; tutu; sed 's/machin3/truc3/p' ; toto ; sed 's/machin2/truc2unpeutroplongpourlaregle/p'" | grep -o "sed[-a-zA-Z ]* ['\"][^'\"]\{20,\}['\"]" sed 's/machin1/truc1unpeutroplongpourlaregle/p' sed 's/machin2/truc2unpeutroplongpourlaregle/p'

---

Reste plus qu'à mettre ça comme argument dans une boucle...

Merci beaucoup !!!
Je mets résolu.

Citation:

---

Envoyé par disedorgue

Voici une petite aide, mais par contre je ne pense pas que tous les parsing d'une commande sed sont là...

---

Effectivement, on doit pouvoir trouver pas mal de contre-exemples, comme:

Code:

---

1 2	$ echo "sed 's/machin/truc/p' | commande 'un peu trop longue'" | grep -qo "sed[-a-zA-Z ]* ['\"].\{20,\}['\"]" && echo ERREUR || echo OK ERREUR

---

Je crois que c'est un problème difficile d'arriver à bien gérer tous les cas de manière statique.
Il faudrait probablement écrire un vrai parser...
Bonne chance!

Par contre, en dynamique, ça me paraît beaucoup plus simple à écrire, par exemple, en masquant, dans le PATH, /bin/sed par un script /chemin/vers/mon/sed qui examine le contenu de ses arguments avant d'appeler le vrai /bin/sed.

Une version rudimentaire (qui ne marche qu'avec un "sed" à un argument et ne traite pas les options comme "-e"):

Code:

---

1 2 3 4 5

#!/bin/sh   test ${#1} -ge 20 && echo ERREUR || echo OK   /bin/sed "$@"

---

Mais ça pose une question de spécifications. La commande "sed" suivante est-elle considérée comme longue?

Code:

---

1 2	exp="s/machin/trucunpeutroplongpourlaregle/p" sed -e "$exp"

---

À moins de pas mal d'efforts, la version statique détectera une commande courte alors que la version dynamique (comme celle proposée) détecte une commande longue et a donc l'avantage de pouvoir traiter des cas tordus qui voudraient tenter de tricher pour échapper à la règle.

Idem pour une commande comme:

Code:

---

1 2	truc=trucunpeutroplongpourlaregle sed -e "s/machin/$truc/p"

---

Cela dit, déterminer la qualité d'une instruction "sed" par la longueur de la commande n'est pas forcément le meilleur critère.
En effet, il peut exister des commandes "sed" de quelques caractères vraiment difficiles à comprendre et maîtriser...

Oui, j'ai corrigé ce souci dans mon deuxième post:

Code:

---

1 2	$ echo "sed 's/machin/truc/p' | commande 'un peu trop longue'" | grep -qo "sed[-a-zA-Z ]* ['\"][^'\"]\{20,\}['\"]" && echo ERREUR || echo OK OK

---

Mais, ça n'empêchera pas que cela ne couvrira pas tous les cas.

Oui je suis tout a fait d'accord avec toi jack-ft ! un sed peut etre très long est pourtant simple a comprendre et à maintenir alors que d'autre trucs court peuvent etre completement "tordus" .
C'est juste demandé dans l'exercice de GNU/Linux magazine a titre d'exemple pour travailler le bash .
La solution de modifier le sed ne rentrerait pas dans le cadre de l'exercice en fait ...

J'ai pensé à cette commande qui semble regler l'erreur que tu as mis en évidence , mais je ne sais pas si il ya encore des-contre exemples avec celle ci :

Code:

---

1 2	echo "sed 's/machin/truc/p' | commande 'un peu trop longue'" | grep -qo "sed[-a-zA-Z ]* ['\"][^'\"]\{20,\}['\"]" && echo ERREUR || echo OK OK

---

Ah je n'ai pas été assez vite ^^
Désolé du coup je repête ce que disedorgue disait...

Merci à vous deux en tout cas !