Discussion :

[codezero1]

Bonjour,

J'ai ces 2 fonctions dans un script qui permettent de savoir si les 2 fichiers concernés dans un dossier sont bien présent .
La commande "ls" serait plus facile à écrire , mais comme il se pourrait qu'elle pose des problèmes de portabilité , j'ai fais un clone avec "find" .

Code :

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

function f1 { find  -maxdepth 1 -name 'actions.txt' ; find -maxdepth 1 -name 'answers.txt'; }

Code :

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

function f2 { echo -e "./actions.txt\n./answers.txt"; }

J'essaye de comparer leur résultat mais sans succés :

Code :

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

1
2
if [ $(f2) = $(f1) ];then echo ok;fi
bash: [: ./answers.txt: binary operator expected

[disedorgue]

Bonjour,

L'erreur rencontré lors du check est parce que tu n'encadres pas tes appelles de fonction par des guillemets et donc le test voit plusieurs chaines au lieu d'en voir une seule de chaque coté du signe, donc la bonne syntaxe serait:

Code :

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

if [ "$(f2)" = "$(f1)" ];then echo ok;fi

Maintenant, ceci te contraint de faire autant de find que de fichier à tester, car le find affiche les fichiers dans l'ordre d'apparition sur le FS.

Pourquoi, tu n'utilises pas directement l'option -f:

Code :

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

if [ -f "./actions.txt" -a -f "./answers.txt" ];then echo ok;fi

[jlliagre]

LE "./" n'est plus requis et si les noms sont simples comme ici, on peut aussi enlever les guillemets :

Code :

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

if [ -f actions.txt -a -f answers.txt ];then echo ok;fi

[disedorgue]

L'éternel débat du cas spécifique vis à vis du cas général

[jlliagre]

Oui, en utilisant des variables au lieu de constantes, ./ et les guillemets sont pleinement justifiés.