Discussion :

[R3MIX]

Bonjour à tous,

J'ai une erreur, ou plutôt un avertissement lors de l'execution de cette ligne :

Code :

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

if [ $( echo "$mot_concatene" | cut -c 1 ) = *[[:upper:]]* ]

Pouvez vous me dire pourquoi ça n'a pas l'air d'aimer la balise upper ?

Merci.

[N_BaH]

Bonjour,

l'opérateur = effectue des comparaisons lexicales : il compare des mots, ou des phrases entre guillemets.
le test POSIX [, qui est un alias de le commande interne test, ne permet pas l'emploi de Motifs génériques (cf. man bash); il faut utiliser [[.

pourquoi des astérisques autour de [[:upper:]], alors qu'il n'y a qu'un seul caractère testé ?

[jack-ft]

[HS]
@N_BaH: Pour prendre le premier caractère d'une variable, est-il plus efficace de faire $( echo "$mot_concatene" | cut -c 1 ) ou bien "${mot_concatene%${mot_concatene#?}}" (qui est beaucoup plus lisible!) ?
[/HS]

[zipe31]

Salut,

[HS]
@N_BaH: Pour prendre le premier caractère d'une variable, est-il plus efficace de faire $( echo "$mot_concatene" | cut -c 1 ) ou bien "${mot_concatene%${mot_concatene#?}}" (qui est beaucoup plus lisible!) ?
[/HS]

Perso je dirai : ${mot_concatene::1}

[R3MIX]

Tout d'abord, merci de me répondre

En mettant ceci :

Code :

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

if [[ $( echo "$mot_concatene" | cut -c 1 ) = [[:upper:]] ]]

j'obtiens le message suivant lors de l'execution : [[: not found

Vous avez raison pour les *, je les ai enlevé.

[jack-ft]

Perso je dirai : ${mot_concatene::1}

Je sais bien qu'on est dans un forum linux/gnu, mais j'ai des contraintes de portabilité... et cette syntaxe n'est pas complètement universelle. En particulier, sous AIX/ksh88:

Code :

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

1
2
$ echo ${toto::1}      
ksh: ${toto::1}: 0403-011 The specified substitution is not valid for this command.

En tout cas, c'est une excellente initiative que de mentionner cette alternative pour linux/bash

[jack-ft]

j'obtiens le message suivant lors de l'execution : [[: not found

Quel shell utilises-tu?
Quelle version?

[R3MIX]

Voici ce que j'utilise :

GNU bash, version 4.2.37(1)-release (i686-pc-linux-gnu)

[disedorgue]

Bonjour,

Bizarre, chez moi, cela fonctionne...

[HS]
sinon si c'est uniquemment le premier caractère qui interresse, une autre façon de faire:

Code :

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

if $(echo "App" | grep -q '^[[:upper:]]') ; then echo OK; fi

[/HS]

[zipe31]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$ cat foo.sh 
#! /bin/bash
 
read -p "Entrez un nom de ville : " nom
 
if [[ ${nom::1}  =~ [[:upper:]] ]]; then
	echo "OK. ${nom} commence bien par la lettre capitale \"${nom::1}\"." 
	else 
	echo "Désolé, mais ${nom} ne commence pas par une lettre capitale \"${nom::1}\"."
	echo "Bonne orthographe : \"${nom^}\"."
fi

Code :

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

1
2
3
4
5
6
7
8
$ ./foo.sh 
Entrez un nom de ville : Toulouse
OK. Toulouse commence bien par la lettre capitale "T".
 
$ ./foo.sh 
Entrez un nom de ville : paris
Désolé, mais paris ne commence pas par une lettre capitale "p".
Bonne orthographe : "Paris".

[R3MIX]

Le problème vient du fait que j'utilise :

Code :

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

#!/bin/sh

et pas

Code :

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

#! /bin/bash

Je vais quand même changer les tests puisque vous dites que c'est plus optimisé.

[jack-ft]

Plutôt que de couper le 1er caractère, on peut aussi tester tout le mot (comme proposé par disedorgue):

Code :

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

1
2
3
4
5
6
7
8
# Solution ksh approximative (ne matche pas les majuscules accentuées À É and co):
if [[ "$mot_concatene" = [A-Z]* ]]; then echo ok ; fi
# Solution bash (et certains ksh):
if [[ "$mot_concatene" = [[:upper:]]* ]]; then echo ok ; fi
# Solution bash (d'après zipe31):
if [[ "$mot_concatene" =~ [[:upper:]]* ]]; then echo ok ; fi
# Solution ksh/bash de disedorgue:
if $(echo "$mot_concatene" | grep -q '^[[:upper:]]') ; then echo ok; fi

[disedorgue]

Attention, regarde bien, je ne test que le premier caractère :
'^[[:upper:]]'

[zipe31]

Le problème vient du fait que j'utilise : #!/bin/sh et pas #! /bin/bash

Ça sent le dash à plein nez ça

Que te renvoie la commande suivante readlink -f /bin/sh ?

[R3MIX]

/bin/dash

C'est gênant ?

[disedorgue]

[RECREATION]
Et une autre pour s'amuser (dans la première, on check le 1er caractère, dans la deuxieme on check le deuxieme:

Code :

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

1
2
3
4
5
6
$ echo $TITI
Tata
$ expr substr "$TITI" 1 1  : '[[:upper:]]'>/dev/null && echo ok || echo nook
ok
$ expr substr "$TITI" 2 1  : '[[:upper:]]'>/dev/null && echo ok || echo nook
nook

Pour les non connaisseurs, ici, comme on peut le voir, il y a un if,then,else implicite
[/RECREATION]

[ok.Idriss]

Bonsoir.

/bin/dash

C'est gênant ?

Disons que dash est assez light et ne propose pas autant de souplesse que bash ou ksh pour ce genre de choses (regexp dans les tests...). Bon tu dois toujours pouvoir bricoler avec grep.

Code :

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

if grep ... >/dev/null 2>&1 # ou avec l'option -q

Mais bon si tu disposes de bash (et c'est le cas sur les Debian based), je te conseil d'utiliser bash si cela t'es possible afin de profiter de la syntaxe des doubles crochets et de l'opérateur =~ qui font des merveilles (ça évites d'utiliser entre autres des commandes externes).

Cordialement,
Idriss

[N_BaH]

Pour prendre le premier caractère d'une variable, est-il plus efficace de faire $( echo "$mot_concatene" | cut -c 1 ) ou bien "${mot_concatene%${mot_concatene#?}}" (qui est beaucoup plus lisible!) ?

àma, moins il y a de commandes externes, plus un script est performant.