Discussion :

[N_BaH]

Code :

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

test -e $dir/*

ça pas faire du joli s'il y a plusieurs fichiers sous $dir, et/ou si $dir contient un espace dans son nom.

En revanche, il me renvoie bien tous les fichiers mais y'a til possibilité de renvoyer le répertoire dans le quel se trouve le fichier ? Ne faut il pas mettre un -type d ? Par exemple le fichier flag appelé fichiertest.flag est dans un répertoire appelé "repertoire". J'aurai souhaité que le chemin complet s'affiche ainsi que toutes les infos de permissions date ect s'affichent je pense qu'un ls -lR serait pas mal mais comment l'imbriquer dans la commande. Ah ah telle est la question !!!

non.
adapte l'option -fprintf qui permet l'affichage de nombreuses information concernant les fichiers "remontés" par la commande find.

[Sve@r]

ça pas faire du joli s'il y a plusieurs fichiers sous $dir

Ah zut, ça j'y avais pas pensé !!!

et/ou si $dir contient un espace dans son nom.

Ah là c'est plus facile => [ -e "$dir"/* ]. En revanche effectivement le premier souci reste plus problématique

[disedorgue]

peut être quelque chose du genre (pas testé) :

Code :

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

1
2
shopt -s extglob
[ -e "$(set -- "$dir/"!(*/) ; echo $1)" ]

Ici, les doubles quotes à l'intérieur des doubles quotes sont bien prises en compte (au moins en bash)

[N_BaH]

Code :

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

1
2
$ fics=( "$dir"/* )
$ ((${#fics[*]})) && echo "fichiers présents sous \"$dir/\"" || echo "pas de fichiers sous \"$dir/\""

enfin, presque, parce que pour être sûr qu'il y a effectivement des fichiers normaux sous $dir/, il faudrait faire une boucle for :

Code :

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

for f in "$dir"/*; do test -f "$f" && { echo "fichier(s) présents sous \"$dir/\""; break;}; done; test -f "$f" || echo "pas de fichiers sous \"$dir/\""

on pourrait remplir le tableau complètement dans la boucle for, et tester le nombre d'élément si on veut réutiliser le tableau...

[papajoker]

En général je préfère mettre en premier les cas d'echec+exit

Bah, chacun voit le verre à sa façon (à moitié vide ou à moitié plein), moi, tant que la bouteille n'est pas vide

Mais, en général, je fais 36 tests au début du script (bon paramètres, bon droits, fichiers existes, ...) donc une sructure type "if not xxx exit" est beaucoup plus adéquate.

[disedorgue]

Je ne vois pas où se situe le problème:

Code :

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

1
2
3
4
$ true || { echo ko ; false ; } && { echo ok ; true; }
ok
$ false || { echo ko ; false ; } && { echo ok ; true; }
ko

Et rien n'empête de mettre la négation sur le test pour que cela soit le && qui traite le exit.

[Sve@r]

Je ne vois pas où se situe le problème:

C'est que c'est souvent plus aisé d'écrire

Code bash :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if souci1; then
	echo "souci1"
	exit 1
fi
if souci2; then
	echo "souci2"
	exit 2
fi
if souci3; then
	echo "souci3"
	exit 3
fi
 
... ici enfin le traitement et toute la place de l'écrire sur 500 lignes si nécessaire...

On peut moduler, affiner, changer l'ordre des soucis ou les invalider si on veut shunter certaines branches. Mettre une valeur distincte dans le exit donnera aussi un point possible de référence pour l'appelant, bref...

[N_BaH]

ça devient lourd s'il faut préciser le statut de chaque liste de commandes parce que la dernière commande pourrait faire continuer d'exécuter la suite conditionnelle implicite.
alors, en effet, il vaut mieux utiliser une structure conditionnelle explicite (if...then...elif...else..fi).

[disedorgue]

Donc ceci:

Code :

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

1
2
3
souci1 && { echo souci1 ; exit 1; }
souci2 && { echo souci2 ; exit 2; }
souci3 && { echo souci3 ; exit 3; }

[disedorgue]

ça devient lourd s'il faut préciser le statut de chaque liste de commandes parce que la dernière commande pourrait faire continuer d'exécuter la suite conditionnelle implicite.
alors, en effet, il vaut mieux utiliser une structure conditionnelle explicite (if...then...elif...else..fi).

ça je suis d'accord, mais pas une fausse excuse du genre il vaut mieux traiter les cas d'échec en premier.

[Sve@r]

Donc ceci:

Code :

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

1
2
3
souci1 && { echo souci1 ; exit 1; }
souci2 && { echo souci2 ; exit 2; }
souci3 && { echo souci3 ; exit 3; }

Oui bien sûr, la forme importe peu. Si tu préfères les accolades... C'est juste que j'ai pas l'habitude de les utiliser que j'ai explicité if...fi

mais pas une fausse excuse du genre il vaut mieux traiter les cas d'échec en premier.

Ben c'est un peu l'impression que donne ton dernier code. J'ai sincèrement l'impression que le travail dit "travail utile" va être écrit après et donc qu'il a traité les cas d'échec en premier et que tu as trouvé ça tout naturel de l'écrire dans ce sens.

[disedorgue]

Peut-être, personnellement, le bout de code que j'ai écris se basait sur le code du post #15

[saga974]

Hello,

DOnc la avec ce bout de code j'arrive à trouver des infos interessantes cela m'avait été proposé mais j'avais pas réussi à l'appliquer . Maintenant c'est bon. Cela donne une idée plus précise :

Code :

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

1
2
3
4
5
6
7
8
9
10
#!/bin/bash
 
repRecherche=/poche/guibs/sftpusers/
log=/poche/guibs/log/fichierlog.txt
if test ! -e $repRecherche/*; then
	echo "Aucun fichier n'existe"
	exit 1
fi
echo "Des fichiers existent"
find "$repRecherche" -printf "%a\t %c\t %t\t %p\n\n" -type f \( -name "*.flag" -o -name "*.tar.gz" -o -name "*.zip" -o -name "*.txt" \) > "$log"

En parcourant des forums, j'ai trouvé ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#/bin/bash
divider===============================
divider=$divider$divider
 
header="\n %-10s %8s %10s %11s\n"
format=" %-10s %08d %10s %11.2f\n"
 
width=43
 
printf "$header" "ITEM NAME" "ITEM ID" "COLOR" "PRICE"
 
printf "%$width.${width}s\n" "$divider"
 
printf "$format" \
Triangle 13  red 20 \
Oval 204449 "dark blue" 65.656 \
Square 3145 orange .7

Code :

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

1
2
3
4
5
6
7
$ ./table 
 
 ITEM NAME   ITEM ID      COLOR       PRICE
===========================================
 Triangle   00000013        red       20.00
 Oval       00204449  dark blue       65.66
 Square     00003145     orange        0.70

Je me suis dit le résultat affiché est présentable. Pourriez-vous m'aiguillez à appliquer dans mon script le même genre de présentation s'il vous plait

Bah, chacun voit le verre à sa façon (à moitié vide ou à moitié plein), moi, tant que la bouteille n'est pas vide

[saga974]

Hello,

Donc la avec ce bout de code j'arrive à trouver des infos intéressantes cela m'avait été proposé mais j'avais pas réussi à l'appliquer . Maintenant c'est bon. Cela donne une idée plus précise :

Code :

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

1
2
3
4
5
6
7
8
9
10
#!/bin/bash
 
repRecherche=/poche/guibs/sftpusers/
log=/poche/guibs/log/fichierlog.txt
if test ! -e $repRecherche/*; then
	echo "Aucun fichier n'existe"
	exit 1
fi
echo "Des fichiers existent"
find "$repRecherche" -printf "%a\t %c\t %t\t %p\n\n" -type f \( -name "*.flag" -o -name "*.tar.gz" -o -name "*.zip" -o -name "*.txt" \) > "$log"

En parcourant des forums, j'ai trouvé ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#/bin/bash
divider===============================
divider=$divider$divider
 
header="\n %-10s %8s %10s %11s\n"
format=" %-10s %08d %10s %11.2f\n"
 
width=43
 
printf "$header" "ITEM NAME" "ITEM ID" "COLOR" "PRICE"
 
printf "%$width.${width}s\n" "$divider"
 
printf "$format" \
Triangle 13  red 20 \
Oval 204449 "dark blue" 65.656 \
Square 3145 orange .7

Code :

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

1
2
3
4
5
6
7
$ ./table 
 
 ITEM NAME   ITEM ID      COLOR       PRICE
===========================================
 Triangle   00000013        red       20.00
 Oval       00204449  dark blue       65.66
 Square     00003145     orange        0.70

Je me suis dit le résultat affiché est présentable. Pourriez-vous m'aiguillez à appliquer dans mon script le même genre de présentation s'il vous plait

[Sve@r]

Code :

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

1
2
3
4
if test ! -e $repRecherche/*; then
	echo "Aucun fichier n'existe"
	exit 1
fi

Alors si tu as bien lu tous les posts (même si certains ont dévié), ça c'est une grosse connerie car si le dossier contient plus d'un fichier, le test -e part en couilles. Je sais que c'est moi qui te l'ai donnée mais ça n'empêche pas.
Donc je rattrappe ma bourde via cette autre solution qui, elle, fonctionne nickel

Code :

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

1
2
3
4
if test -n "$(find "$repRecherche" -maxdepth 0 -empty)"; then
	echo "Aucun fichier n'existe"
	exit 1
fi

L'option "-empty" n'affiche le retour que si le dossier est vide. Donc si chaine affichée (ce que teste l'option "-n"), alors dossier vide. Accessoirement tu remarqueras les guillemets autour de "$repRecherche" pour gérer les noms avec espace.

Pourriez-vous m'aiguillez à appliquer dans mon script le même genre de présentation s'il vous plait

Ca se fait avec les formats de printf. Le format "%d" affiche un nombre, le format "%05d" affiche un nombre avec 5 zéros à gauche, le format "%5d" affiche un nombre avec 5 espaces à gauche. Te suffit de tester en direct (ex printf "%5d" 123) puis quand tu trouves ce qui te convient tu l'inclus dans le script.

[N_BaH]

Code :

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

find "$repRecherche" -type f \( -name "*.flag" -o -name "*.tar.gz" -o -name "*.zip" -o -name "*.txt" \)  -fprintf "%a\t %c\t %t\t %p\n\n" "$log"

ce n'est pas grand-chose, mais quand une commande propose de faire le boulot du shell...

[saga974]

J'ai appliqué ton bout de code ca fonctionne très bien.

Le maxdepth 0 signifie bien qu'il ira chercher tous les répertoires et sous répertoires ainsi que leurs fichiers à partir de SFTPUSERS c'est ca ?
Pour le format en fait pour bien cibler l'attendu en gros c'est est il possible de mettre un header dans le résultat renvoyé dans le fichierlog.txt afin d'avoir quelque chose de plus présentable ? Le printf en effet s'occupe bien d'ordonner dans un premier temps mais je cherche a affiner la forme disons.

Egalement, j'ai vu ton conseil sur la commande printf et l'affichage des nombres. J'avais commencé à l'appliquer pour obtenir l'heure du genre 15:46:35 et enlever les résidus de chiffre en trop mais du coup ca m'enleve l'année

Code :

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

find "$repRecherche" -printf "%a\t %.19c\t %t\t %p\n\n" -type f \( -name "*.flag" -o -name "*.tar.gz" -o -name "*.zip" -o -name "*.txt" \) > "$log"

Sais-tu comment enlever les résidus de chiffre après l'heure tout en conservant l'année qui reste judicieuse ?

[N_BaH]

si tu veux choisir les informations d'horodatage à afficher, utilise plutôt %A... %C... %T... (cf.man find)

[Sve@r]

Le maxdepth 0 signifie bien qu'il ira chercher tous les répertoires et sous répertoires ainsi que leurs fichiers à partir de SFTPUSERS c'est ca ?

Certainement pas !!! On a besoin de descendre tout au fond d'un dossier pour savoir s'il est vide ???
maxdepth signifie que la profondeur maximale (max=maximum et depth=profondeur) sera de 0 sous-dossiers, donc qu'il ne reste qu'au niveau du dossier demandé, ce qui est largement suffisant pour vérifier s'il est vide ou pas.

Pour le format en fait pour bien cibler l'attendu en gros c'est est il possible de mettre un header dans le résultat renvoyé dans le fichierlog.txt afin d'avoir quelque chose de plus présentable ?

Ben tu commences par écrire le header, puis tu fais le find.
Tu peux regrouper l'ensemble dans des parenthèses

Code bash :

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

1
2
3
4
(
	echo header
	find ...
) >log

Ca évite de mettre 500 redirections

Sais-tu comment enlever les résidus de chiffre après l'heure tout en conservant l'année qui reste judicieuse ?

Directement non. On peut rediriger le find dans une commande X (ou un groupe de commande, ou une fonction) qui se chargera de mettre au propre. Après tout, c'est le principe de base d'Unix de n'avoir que des commandes qui ne font qu'une chose mais qui la font bien, et en les chainant on peut faire ce qu'on veut.

Code bash :

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

1
2
3
4
5
6
7
8
9
10
mise_au_propre() {
	while read ligne; do
		echo "$ligne"
	done
}
 
(
	echo header
	find ... |mise_au_propre
) >log

Bon ici la "mise au propre" ne fait que réécrire ce qu'elle reçoit mais charge à toi de coder ça pour qu'elle te donne ce que tu veux.

[N_BaH]

Directement non.

si si. cf. mon précédent message