Discussion :

[Invité]

Bonjour,
J'ai une question très bête mais qui me bloque depuis ce matin.
Je souhaiterais passer en paramètre d'une fonction :

Code :

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

sed '1,10d' |sed '$d'|sed '$d'|sed '$d'| sed 's/,$//'| sed "s/^\ *//g"| sed "s/\" *//g"| sed "s/\:*//g"

Le problème c'est que ça ne marche pas j'ai essayé les guillemets les côtes, les crochets....

Mais rien ne marche est-ce que cela est au moins possible ?

Merci et bonne après midi

[Flodelarab]

Bonjour

Un très bon exercice pour toi. Comprendre les quotes.
Les 'simple quotes' protègent de toute interprétation.
Les "doubles quotes" protègent, mais laissent passer des interprétations comme les variables, par exemple.

Code :

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

1
2
3
4
5
6
7
8
$ var=toto
$ echo blabla$var
blablatoto
$ echo 'blabla$var'
blabla$var
$ echo "blabla$var"
blablatoto
$

Ton expression contient des 'simple quotes' et des "double quotes". Donc il va falloir être minutieux dans la protection. Tu crois que les quotes du milieu sont protégés alors qu'ils sortent juste de la protection.

• Tu peux laisser les simple quotes dans les double quotes, et inversement.
• Si tu veux un simple quote dans des simple quotes, tu sors, tu écris le caractère et tu rentres.
  

  Code :

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

  1
  2
  $ echo 'J'\''ai un texte qui ne doit pas être interprété.'
  J'ai un texte qui ne doit pas être interprété.

• Si tu veux un double quote dans des double quotes, tu échappes le caractère.
  

  Code :

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

  1
  2
  $ echo "J'ai un \"texte\" qui ne doit pas être interprété."
  J'ai un "texte" qui ne doit pas être interprété.

[Invité]

Bonjour,
Merci beaucoup pour ta réponse.
Effectivement lors de mes tests je faisais le travail à moitié.
J'ai donc apporté les modifs de la façon suivante, j'ai entouré le code de guillemets pour l'interprétation du code

Code :

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

1
2
 
"sed '\''1,10d'\'' |sed '\''$d'\''|sed '\''$d\''|sed '\''$d'\''| sed '\''s/,$//'\''| sed \"s/^\ *//g\"| sed \"s/\" *//g\"| sed \"s/\:*//g\""

Quand je fais en écho sur le paramètre j'obtiens :

Code :

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

1
2
 
sed '\''1,10d'\'' |sed '\'''\''|sed '\''\''|sed '\'''\''| sed '\''s/,$//'\''| sed "s/^\ *//g"| sed "s/" *//g"| sed "s/\:*//g"

Est ce que j'ai mal identifié et modifier l'interprétation des caractères spéciaux ?

Je me retrouve avec l'erreur:
Sed : -e expression #1, char 1: unknown command: `' '

Pourtant je n'ai rien comme Sed -e

Update : le problème ne semble concerner que les simples quotes. Les guillemets sembles parfait

Merci

[N_BaH]

en général, quand une commande se pipe à elle-même (commande | commande), une seule instance de cette commande devrait suffire.

il faudrait donc d'abord que tu simplifies ta ligne de commande.

[Invité]

Salut,
Alors pour le coup je n'ose pas trop c'est un bout de code qu'on m'a donné et totalement fonctionnel pour passer les tests.
J'aurais peur de toucher quelque chose et perdre de l'info

[Flodelarab]

Dans la ligne initiale, je ne vois pas pourquoi utiliser des double quotes, toujours dangereux avec sed, puisque $ est souvent présent, potentiellement interprété comme une variable. C'est d'ailleurs ce qui, selon moi, à motiver l'utilisation des simple quotes dans les 4 premières commandes, puisque $d est interprété comme une variable et pas comme "à la dernière ligne, détruire". Les scripts sed doivent toujours être entourés de simple quotes, sauf en cas de besoins spécifiques.

J'ai donc apporté les modifs de la façon suivante, j'ai entouré le code de guillemets pour l'interprétation du code

Code :

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

"sed '\''1,10d'\''

Aucune modification n'est nécessaire si le signe n'est pas le même. Là, tu modifies un simple quote, alors que la protection est assurée par un double quote. Faute.

[Invité]

Alors voilà ma nouvelle version

Code :

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

1
2
 
sed '\'1,10d'\' |sed '\'$d'\'|sed '\'$d'\'|sed '\'$d'\'| sed '\'s/,$//'\'| sed "s/^\ *//g"| sed "s/\" *//g"| sed "s/\:*//g"'

Je vais me permettre de vous expliquer ma démarche car je ne suis pas sûr de tout saisir

J'avais délimiter le code avec des guillemets car sinon cela me dit qu'il ne trouve pas la caractère de fin (avec l'exemple ci dessous et en utilisant des simples quotes)

Après je ne comprends le double simple quote par exemple Sed '\' ' 1,10d '\' '

Après je délimite tout ce qui est entre simple quote mais pas ce qui est entre guillemets

Bon après midi

[Flodelarab]

C'est bon, sauf qu'il manque le simple quote du début.

[Invité]

Oui erreur de saisie de ma part lorsque j'ai fait le post.

Par contre lorsque je lance le script j'ai les messages :
Unexpected EOF while looking for matching ` ' '
Unexpected end of file

C'est initialement pour ça que j'avais mis les guillemets

[N_BaH]

c'est un bout de code qu'on m'a donné

il ne faut pas accepter tout ce qu'on te donne.
un jour, un vieux monsieur va vouloir te donner des bonbons...

[Invité]

Ahahah, oui on est bien d'accord, néanmoins il s'agit d'un bout de code de mon prof donc je lui fais (peut-être à tord) confiance.
Après oui il y a de la répétition dans le début avec le sed '$d' mais comment "optimiser" sans perte ?
De toute façon pour l'instant ça ne marche pas avec les simples quotes

[N_BaH]

on ne sait pas ce que ça traite, on ne va donc pas pouvoir optimiser.

le problème principal du code de ton premier message (outre les "auto-pipes"), c'est qu'un Développement de paramètre n'est pas réalisé entre apostrophes.

[Invité]

Salut,
Ca voudrait dire que pour que le paramètre soit pris en compte il faut que je laisse les simples quotes en début et fin du paramètre et que je mette des guillemets au niveau des pipes ?

[Flodelarab]

Mais c'est pas encore réglé ce problème ?

Code :

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

1
2
3
4
5
6
7
8
9
10
$ function montest
> {
> echo $1
> }
$ montest toto
toto
$ montest "sed '1,10d' |sed '\$d'|sed '\$d'|sed '\$d'| sed 's/,$//'| sed 's/^ *//g'| sed 's/\" *//g'| sed 's/:*//g'"
sed '1,10d' |sed '$d'|sed '$d'|sed '$d'| sed 's/,$//'| sed 's/^ *//g'| sed 's/" *//g'| sed 's/:*//g'
$ montest 'sed '\''1,10d'\'' |sed '\''$d'\''|sed '\''$d'\''|sed '\''$d'\''| sed '\''s/,$//'\''| sed "s/^\ *//g"| sed "s/\" *//g"| sed "s/\:*//g"'
sed '1,10d' |sed '$d'|sed '$d'|sed '$d'| sed 's/,$//'| sed "s/^\ *//g"| sed "s/\" *//g"| sed "s/\:*//g"

De plus, cette question sent mauvais depuis le début. Ça sent les mauvaises pratiques. Tu transmets une chaîne de caractères à une fonction ? Et après, quoi ? Tu l'exécutes ? Avec eval ? Ce qui veut dire que n'importe qui transmettant n'importe quelle chaîne de caractères peut faire exécuter n'importe quel code dangereux à ton ordinateur ? C'est ça le projet ? C'est mauvais. C'est une faille de sécurité. Il ne faut pas faire cela.

Quel est le vrai énoncé complet ?

[Invité]

Bonjour,
Je vais réessayer pour voir si ça tourne comme il faut chez moi.

Concernant l'explication :
Je dois faire des curls au travers d'une boucle. Le Curl va récupérer des données qui vont être restucturé dans le format json et stocké dans un fichier.
Il m'était demandé de rejouer les curls en échec.
Nous avons donc fait rajouté dans la fonction Curl cette ligne (grâce à l'aide du forum):
Json_reformat< fichier | $formatage >résultats.json

$formatage est le fameux paramètre "Sed ...| Sed...|"
J'ai décidé de faire comme ça car dans le code j'ai trois curl pour lequel le json reformat n'est pas le même.
Je passe donc la valeur reformat en paramètre

[N_BaH]

Code :

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

Json_reformat< fichier | $formatage >résultats.json

mauvaise pratique.
utilise une fonction à la place d'une variable :

Code :

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

1
2
3
4
5
6
7
8
9
10
formatage() {
   case $1 in
   format_1) ...
   ;;
   format_2) ...
   ;;
   format_3) ...
   ;;
   esac
}

...

Code :

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

curl ... | refomat | formatage format_N

ça sent l'usine à gaz !

[Invité]

Oui oui une véritable usine à gaz.
Je vais étudier ta proposition, merci

[Invité]

Bonjour à tous,
J'ai un peu réfléchi à ta proposition sur l'utilisation d'une fonction.
Je vois et comprends effectivement l'intérêt de cette pratique néanmoins en raison de l'implémentation actuelle du code ça me paraît difficile.
Effectivement j'appelle dans le code la fonction Curl (celle qui permet de vérifier que le Curl a fonctionné et si ce n'est pas le cas rejouer la requête)avec comme paramètre l'URL. Pour cette raison je ne peux pas appeler une fonction dans une fonction.
Après ce n'est pas l'utilisateur qui saisit les paramètres tout est déjà dans le code

[N_BaH]

vérifier que le Curl a fonctionné et si ce n'est pas le cas rejouer la requête)avec comme paramètre l'URL. Pour cette raison je ne peux pas appeler une fonction dans une fonction.

je ne vois pas le rapport.

Après ce n'est pas l'utilisateur qui saisit les paramètres tout est déjà dans le code

ça doit être "conditionnable"...

[Flodelarab]

je ne peux pas appeler une fonction dans une fonction.

Code :

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

1
2
3
4
5
6
7
8
9
10
$ function niveau_3
> {
> echo "Ici, le niveau 3"
> }
$ function niveau_2 { echo "Ici, le niveau 2"; niveau_3; }
$ function niveau_1 { echo "Ici, le niveau 1"; niveau_2; }
$ niveau_1
Ici, le niveau 1
Ici, le niveau 2
Ici, le niveau 3

On peut toujours appeler une fonction dans une fonction.