Script qui n'en fait qu'à sa tête

Version imprimable

Bonjour,

j'ai fait un script qui permet de renommer les fichiers par leurs dates. Je ne comprend pas. La bouche se fait une première fois sur le premier fichier. Mais sur les fichiers suivants, elle ne s'exécute pas.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
#!/bin/bash
clear
#set -x
###
###TRAITEMENT DE LA DATE ===============================================
gcc="000000"
client="00000000"
fichier1="fichier1"
ddate="000000"
nom_fichier="nom_fichier"
cd /home/christophe/bl-clients/temp/
 
for fichier1 in $(ls *.pdf)
do
	###TRAITEMENT DE LA DATE ==========================================
	for col1 in 9 10 11 12 13
	do
		if (( $ddate == 000000 ))
		then
			ddate=$(pdfgrep --max-count 1 "Date" $fichier1 | awk -v col2=$col1 '{print $col2 }')
 
			len=`echo $ddate |awk '{print length}'`
 
			if [ $len -eq 7 ];
			then
			dateAA=$(echo $ddate | cut -b 6-7)
			dateMM=$(echo $ddate | cut -b 3-4)
			dateJJ=$(echo $ddate | cut -b 1)
			dateJJ="0$dateJJ"
			fi
 
			if [ $len -eq 8 ];
			then
			dateAA=$(echo $ddate | cut -b 7-8)
			dateMM=$(echo $ddate | cut -b 4-5)
			dateJJ=$(echo $ddate | cut -b 1-2)
			fi
 
			ddate=20$dateAA$dateMM$dateJJ
			if [[ "$ddate" =~ [0-9]{8} ]]
			then
				ddate=$ddate
			else
				ddate="000000"
			fi
		fi
	done 
	###FIN TRAITEMENT DE LA DATE ==========================================
 
	nom_fichier=BL_$ddate\_$gcc\_$client.pdf
 
	echo "fichier="$nom_fichier
	read -p "enter to continue"
 
done

j'ai mis le script. j'explique vaguement le fonctionnement . Il cherche une date sur une ligne. ensuite il l'a cute bref un joli moulin à café.

merci de votre aide.

23/05/2023, 15h56
Sve@r
Bonjour

Citation:

Envoyé par chrisl

Mais sur les fichiers suivants, elle ne s'exécute pas.

Chez-moi elle s'exécute. Donc problème réglé

Mais puisque tu nous as mis un script, autant indiquer ce qui ne va pas
- cd /home/christophe/bl-clients/temp/:? Et si on veut travaillers ailleurs...?
- for fichier1 in $(ls *.pdf) => quand N_BaH verra ça... :fessee: => for fichier1 in *.pdf!!!
- len=`echo $ddate |awk '{print length}'` => c'est bizarre, de partout tu as utilisé la notation récente et là, brusquement, tu passes en old-school...:weird:
- echo "fichier="$nom_fichier => les guillemets acceptent les variables => echo "fichier=$nom_fichier"
- if [ $len -eq 7 ]; then... if [ $len -eq 8 ]; then... => case...esac
- nom_fichier=BL_$ddate\_$gcc\_$client.pdf => c'est quoi ces backslashes?
23/05/2023, 17h24
N_BaH

en simplifiant le code tu y verras peut-être plus clair.

la déclaration préalable des variables n'est pas indispensable, en dehors des tableaux associatifs pour qui c'est une question de vie ou de mort, ou pour typer une variable numérique.

pourquoi for fichier1 ? quand la boucle en sera au deuxième fichier tu vas aller éditer le script pour changer en fichier2 ? :lol:
simplement for fichier in *.pdf, et je ne reviens pas sur le ls : tu as déjà reçu la fessée de par t par Sve@r. :lol:
mais

Citation:

ON N'UTILISE PAS ls DANS UN SCRIPT !!!

Code:

if (( $ddate == 000000 ))

si une ddate commence par un zéro, tu vas avoir une belle erreur.
bien qu'une date semble être un nombre pour un humain, il est préférable de la comparer en tant que chaîne.

Code:

len=`echo $ddate |awk '{print length}'`

non, non :

Code:

len=${#ddate}

mais ça sert à rien :

Code:

IFS='/' read jour mois an <<<"$ddate"

et hop ! une bonne partie du code en moins. :)

Code:

nom_fichier=BL_$ddate\_$gcc\_$client.pdf

non,non plus :

Code:

nom="BL_${ddate}_${gcc}_$client.pdf"

tiens! pour la route :for col in {9..13}, et pas col1 :roll:

Bonjour merci de vos réponses. j'ai corrigé le script suivant vos indications.

le code est plus lisible :)

le voila :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#!/bin/bash
clear
#set -x
###
###TRAITEMENT DE LA DATE ===============================================
for fichier in *.pdf
do
echo "fichier_en_cours-$fichier"
	###TRAITEMENT DE LA DATE ==========================================
	for col in {9..13}
	do
		if (( $ddate == 000000 ))
		then
			ddate=$(pdfgrep --max-count 1 "Date" $fichier | awk -v col2=$col '{print $col2 }')
 
			len=${#ddate}
 
			if [ $len -eq 7 ];
			then
			dateAA=$(echo $ddate | cut -b 6-7)
			dateMM=$(echo $ddate | cut -b 3-4)
			dateJJ=$(echo $ddate | cut -b 1)
			dateJJ="0$dateJJ"
			fi
 
			if [ $len -eq 8 ];
			then
			dateAA=$(echo $ddate | cut -b 7-8)
			dateMM=$(echo $ddate | cut -b 4-5)
			dateJJ=$(echo $ddate | cut -b 1-2)
			fi
 
			ddate=20$dateAA$dateMM$dateJJ
			if [[ "$ddate" =~ [0-9]{8} ]]
			then
				ddate=$ddate
			else
				ddate="000000"
			fi
		fi
	done 
	###FIN TRAITEMENT DE LA DATE ==========================================
 
	nom="BL_${ddate}_${gcc}_$client.pdf"
 
	echo "fichier="$nom
	read -p "enter to continue"
 
done

mais j'ai toujours ce satané bug à la noix que je ne trouve pas.
Il prend pas la date des autres fichiers.
J'ai mis le résultat de la commande en dessous :

Code:

1
2
3
4
5
6
7
8
9
10
11
 
fichier_en_cours-BL-20230403-588955.pdf
fichier=BL_20230403_000000_00000000.pdf
enter to continue
fichier_en_cours-BL-20230404-590658-02650052.pdf
fichier=BL_20230403_000000_00000000.pdf
enter to continue
fichier_en_cours-BL-20230411-589829-02637952.pdf
fichier=BL_20230403_000000_00000000.pdf
enter to continue
christophe@srv-ged01:~/bl-clients/temp$

la variable "20230403" ne change pas. helas.

merci de votre aide.

24/05/2023, 10h27
jack-ft

Citation:

Envoyé par chrisl

la variable "20230403" ne change pas. helas.

[sarcasme]
Normal ! Le script n'en fait qu'à sa tête !
Autrement dit:

Citation:

le script ne fait jamais ce que je veux, il fait toujours ce que je lui demande ! (référence connue)

[/sarcasme]

Plus sérieusement, à la ligne 34 ddate=20$dateAA$dateMM$dateJJ, tu mets 8 chiffres dans ddate.
À la ligne suivante if [[ "$ddate" =~ [0-9]{8} ]], tu testes si ddate contient 8 chiffres, ce qui, justement, est le cas.
Dans la clause then ddate=$ddate, tu mets dans ddate la valeur de ddate (histoire que le CPU ne s'endorme pas, je suppose).
Et, après la fin de boucle, au fichier suivant, à la ligne 13 if (( $ddate == 000000 )), tu testes si ddate contient 000000, ce qui n'est certainement pas le cas puisqu'elle contient les 8 chiffres de la date du premier fichier, en conséquence de quoi, le programme n'entre pas dans la clause then destinée à changer la date, si bien que la date ne sera jamais changée.

Donc le résultat observé est conforme à ce que tu lui demandes !

J'avoue que j'ai du mal à comprendre la raison de cette date à 000000 :koi:

Y a-t-il une (bonne) raison de ne pas recalculer systématiquement la date pour chaque fichier (c'est-à-dire de supprimer le test if (( $ddate == 000000 ))) ?
24/05/2023, 10h30
Sve@r

Citation:

Envoyé par jack-ft

Plus sérieusement, à la ligne 34...

Moi j'ai mis :plusser: pour avoir eu le courage de lire et d'analyser ce script... :applo:
24/05/2023, 10h38
N_BaH

que retourne pdfgrep pour un de ces fichiers ?

note qu'il est aussi maladroit d'exécuter 5 fois pdfgrep, alors qu'il serait plus économique de parcourir la sortie d'une seule exécution.
26/05/2023, 08h37
chrisl

Citation:

Envoyé par N_BaH

que retourne pdfgrep pour un de ces fichiers ?

note qu'il est aussi maladroit d'exécuter 5 fois pdfgrep, alors qu'il serait plus économique de parcourir la sortie d'une seule exécution.

Bonjour, merci de ton intervention

pdfgrep parcoure le fichier pdf en cherchant une date.
si la date est trouvé, il l'a renomme dans le fichier.

par contre mon soucis c'est :

le script garde la première valeur lors du deuxième fichier. je ne comprend pas.
26/05/2023, 11h11
N_BaH

ton script est écrit aussi bien que tu le pouvais, et ça n'aide pas à la lecture de son comportement; d'autant moins qu'on n'a pas de fichier exemple pour reproduire le dysfonctionnement que tu observes.

si les pdf que tu traites ne sont pas 'TOP SECRET" peux-tu le mettre en pièce jointe ? ou alors peux-tu en créer un similaire, et de la même manière, nous le mettre en pièce jointe ?
il faut qu'on voit le script en action.
là, je lirais un lorem ipsum ce serait pareil : les mots sont corrects, ça ne signifie pas que ça ait du sens. :)
27/05/2023, 07h16
jack-ft
Citation:

Envoyé par chrisl

pdfgrep parcoure le fichier pdf en cherchant une date.

Oui, bien sûr, mais à quoi ressemble ou peut ressembler le format de cette date dans tes fichiers pdf ?

Citation:

par contre mon soucis c'est :

le script garde la première valeur lors du deuxième fichier. je ne comprend pas.

Essaie d'intercaler une initialisation de ddate entre les lignes 10 et 11, de la manière suivante:
Code:

1 2 3 ###TRAITEMENT DE LA DATE ========================================== ddate="000000" for col in {9..13}
Ça devrait aider à régler le problème (c'est toujours une bonne idée d'initialiser les variables...)