Discussion :

[laloupiote]

Bonjour,

J'ai le bout de script suivant

Code :

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

1
2
3
4
5
fichier=$(kdialog --multiple --separate-output --getopenfilename /home/user "*.doc *.docx *.rtf *.sxw *.odt *.ods *.xls *.odm")
if [ "$?" = "1" ];
then exit 
fi
/usr/bin/libreoffice $fichier &

Si je choisis plusieurs fichiers dans la boîte de dialog kdialog, ma variable fichier ressemble à ceci

Code :

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

1
2
[mer. 06 février (~) user] echo $fichier                                                                            
/home/user/test1.odt /home/user/test2.odt

Mon script marche bien.

Mais si je choisis un fichier dont le nom contient des espaces, mon script ne fonctionne plus. Alors je mets $fichier entre guillemets dans la dernière ligne de mon script. Il marche bien pour ouvrir les fichiers avec espace. Mais alors, il ne fonctionne plus quand je veux ouvrir plusieurs fichiers.

Comment faire pourqu'il fonctionne à tous les coups ?

Codialement,

[frp31]

ouops... effectuvement j'avais loupé la ligne de fin

[jack-ft]

@frp1: es-tu sûr d'avoir lu les dernières lignes de son post?

@laloupiote: A tout hasard, essaie ça:

Code :

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

/usr/bin/libreoffice $(echo "${fichiers}" | sed -e 's: :\\ :g' -e 's:\\ /: /:g') &

PS: [Limitation] si ça marche (on ne sait jamais!), ça beuguera sûrement dans le cas où le nom d'un des répertoires du chemin se termine par une espace ou (peut-être) si le nom d'un des fichiers se termine par une espace... à tester...

[frp31]

sinon y'a la méthode à l'arrache qui marche :

Code :

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

1
2
3
4
5
6
7
8
9
10
sh-4.2$ echo $fichier
/local/wip2/Annuaire_correspondants.xls 
[...] # je vous epargne la liste des 62 fichiers....
/wip2/planning muscu.xls /local/wip2/relever ip.xls /local/wip2/suivi-frais-APM.xls
sh-4.2$ for i in "${fichier}" ; do libreoffice $i &  
> done
[2] 9064
 
[2]+  Fini                    libreoffice $i
sh-4.2$

et là effectivement espaces ou pas ça ouvre bien tous les fichiers sans exceptions. (me suis farci les 62 clics pour fermer les fenêtres !! j'avais oublier de vérifier combien il y en avait avant de faire le test.....)

[jack-ft]

J'ai le bout de script suivant

Code :

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

1
2
3
4
fichier=$(kdialog --multiple --separate-output --getopenfilename /home/user "*.doc *.docx *.rtf *.sxw *.odt *.ods *.xls *.odm")
if [ "$?" = "1" ];
then exit 
fi

[chipotage]
La variable $? est le status retourné par la commande précédente.
En conséquence de quoi, c'est TOUJOURS un entier.
L'entourer de guillemets peut faire penser que c'est une chaîne de caractères, ce qui nuit à la lisibilité.
En conséquence de quoi, je te recommande fortement de le remplacer (toutes choses égales par ailleurs) par:

Code :

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

1
2
3
4
fichier=$(kdialog --multiple --separate-output --getopenfilename /home/user "*.doc *.docx *.rtf *.sxw *.odt *.ods *.xls *.odm")
if [ $? -eq 1 ];
then exit 
fi

Tu peux même utiliser la notation abrégée:

Code :

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

fichier=$(kdialog --multiple --separate-output --getopenfilename /home/user "*.doc *.docx *.rtf *.sxw *.odt *.ods *.xls *.odm") || exit

[/chipotage]

[laloupiote]

Merci pour vos réponse, ça marche !
J'aime le chipotage, ça instruit
Cordialement,

[jack-ft]

Merci pour vos réponse, ça marche !

Tu dis que ça marche...
Quelle méthode as-tu utilisée?
Ta réponse pourra aider d'autres lecteurs!

[laloupiote]

Tu dis que ça marche...
Quelle méthode as-tu utilisée?
Ta réponse pourra aider d'autres lecteurs!

Code :

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

/usr/bin/libreoffice $(echo "${fichiers}") &

Cette méthode fonctionne bien, mais sans la commande sed. Avec la commande sed, ça modifie le nom des fichiers à espace et libreoffice me répond qu'ils n'existent pas.

Code :

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

1
2
for i in "${fichier}" ; do libreoffice $i &  
done

Cette méthode fonctionne également. C'est celle que j'ai retenue, parce que c'est celle dont j'ai compris le fonctionnement en premier.

Encore merci.

[N_BaH]

ça ne devrait pas fonctionner :

Code :

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

1
2
3
$ f="foo bar baz"
$ for i in "$f"; do echo $i; done
foo bar baz