Discussion :

[Jerniti]

Bonjour,
svp j'aurai besoin d'aide sur un script ksh.
j'ai des fichiers sous cette formes :

HST_HST35G02_EDP_R0+45_G02R00C00_5909_I1523254.DATAE00
HST_HST35G02_EDP_H5+63_G02R00C00_5799_I1600063.DATAE00
HST_HST35G02_EDP_Q2+44_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_T1+31_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+14_G02R00C00_5801_I1500213.DATAE00

parmi ces fichiers par exemple je dois déplacer ceux qui ont une combinaison incorrect dans un répertoire poubelle.

dans mon script j'ai déclarer un tableau qui contient toutes les combinaisons correctes, ensuite j'ai fait une boucle pour le parcourir et parcourir les fichiers que je veux traiter et une condition qui permet de détecter les fausses combinaison et les déplacer :

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
18
19
20
21
#!/bin/ksh
DRCFR="1G+91 2G+93 3G+91 4G+94 A0+59 B2+33 C0+54 E0+21 F4+59 H3+63 H5+07 J1+83 J2+83 J4+07 K2+34 L1+54 M1+44 N2+45 P0+33 Q2+35 R0+14 S0+21 T1+31 U1+77 U2+79 U3+74 U6+78 W4+77"
set -A TABDRCFR $DRCFR
FICREPEDP=$(find /data/flf/35gchg/files/rejetChg/incft_HST/ -type f | grep "HST_HST35G02_EDP")
set -A TABFICREPEDP $FICREPEDP
i=0
for FICEDP in ${TABFICREPEDP[@]}
do
while [ $i -lt ${#TABDRCFR[@]} ]
do
affichEDP=`ls $FICEDP | cut -c60-64`
if [ $affichEDP != `echo ${TABDRCFR[$i]}` ]
then
mv $FICEDP /data/flf/35gchg/files/rejetChg/pcr_poubelle_HST/
else
echo OK
break
fi
i=$((i+1))
done
done

mais malheureusement quand j'exécute le script, ce dernier déplace tous les fichiers ...
quelqu'un peut m'aider sur ce script svp ?

Merci

[jack-ft]

mais malheureusement quand j'exécute le script, ce dernier déplace tous les fichiers ...
quelqu'un peut m'aider sur ce script svp ?

Pour voir, as-tu essayé de mettre un echo devant le mv ? C'est-à-dire :

Code :

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

echo mv $FICEDP /data/flf/35gchg/files/rejetChg/pcr_poubelle_HST/

ou bien, carrément, un set -x au début du script.


Et, attention, on n'utilise pas ls dans un script...

[eliot_net]

Bonjour.
En lisant ton script, je vois au moins deux problèmes dans l'algorithme.

Code :

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

1
2
3
4
Pour tous les fichiers possibles (ligne 7)
    Pour toutes les combinaisons de la liste (ligne 9)
        Si le nom du fichier ne contient pas la combinaison (ligne 12)
            déplacer le fichier

Dès que la liste de combinaisons aura au moins deux éléments distincts, il y aura au moins une raison de déplacer le fichier... et donc tous les fichiers sont déplacés. Il faut inverser le sens du test ligne 12.

L'incrémentation de i (ligne 19) est à l'intérieur de la double boucle alors que l'initialisation est à l'extérieur de cette double boucle. Dès que le premier fichier aura été testé sur toute la liste de combinaisons, i aura une valeur > taille de la liste....

[N_BaH]

• pourquoi ne pas déclarer DRCFR tout de suite sous la forme d'un tableau ?
• pourquoi ne pas déclarer FICREPEDP tout de suite sous la forme d'un tableau, et en utilisant le Développement des chemins (glogging), plutôt que find | grep qui est une mauvaise pratique, puisque find peut utiliser nativement des regex et la Substitution de commandes ne protègent pas les noms des fichiers (blancs, alinéas...) ?

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

je n'utilise pas KSH (d'ailleurs, pourquoi KSH ?), je serais surpris qu'il ne soit pas possible de faire aussi simplement qu'en bash une simple boucle for sur le Développement des chemins du format de noms de fichiers souhaité...

PS: n'oublie pas d'indenter ton script. Ça aide la lecture.
mais je comprends que tu ne veuilles pas qu'on lise ça.

[Flodelarab]

Bonjour

Code :

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

1
2
3
4
5
6
$ cat fichiers.txt
HST_HST35G02_EDP_H5+63_G02R00C00_5799_I1600063.DATAE00
HST_HST35G02_EDP_Q2+44_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+14_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+45_G02R00C00_5909_I1523254.DATAE00
HST_HST35G02_EDP_T1+31_G02R00C00_5801_I1500213.DATAE00

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
18
19
20
21
22
23
24
25
26
27
28
29
$ cat motifs.txt
1G+91
2G+93
3G+91
4G+94
A0+59
B2+33
C0+54
E0+21
F4+59
H3+63
H5+07
J1+83
J2+83
J4+07
K2+34
L1+54
M1+44
N2+45
P0+33
Q2+35
R0+14
S0+21
T1+31
U1+77
U2+79
U3+74
U6+78
W4+77

Code :

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

1
2
3
4
$ grep -vf motifs.txt fichiers.txt                                                                                                                                                                                                                          
HST_HST35G02_EDP_H5+63_G02R00C00_5799_I1600063.DATAE00
HST_HST35G02_EDP_Q2+44_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+45_G02R00C00_5909_I1523254.DATAE00

Code :

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

grep -vf motifs.txt fichiers.txt | xargs -ITT mv TT poubelle/

[N_BaH]

Pourquoi KSH ? le projet exige son utilisation

sur quel OS doit tourner le script ?
es-tu tenu à une version particulière de Ksh ?

[Jerniti]

Bonjour.
En lisant ton script, je vois au moins deux problèmes dans l'algorithme.

Code :

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

1
2
3
4
Pour tous les fichiers possibles (ligne 7)
    Pour toutes les combinaisons de la liste (ligne 9)
        Si le nom du fichier ne contient pas la combinaison (ligne 12)
            déplacer le fichier

Dès que la liste de combinaisons aura au moins deux éléments distincts, il y aura au moins une raison de déplacer le fichier... et donc tous les fichiers sont déplacés. Il faut inverser le sens du test ligne 12.

L'incrémentation de i (ligne 19) est à l'intérieur de la double boucle alors que l'initialisation est à l'extérieur de cette double boucle. Dès que le premier fichier aura été testé sur toute la liste de combinaisons, i aura une valeur > taille de la liste....

Merci pour ton retour.

j'ai inversé le test ligne 12, cela veux dire :

Code :

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

1
2
3
4
5
6
7
           if [ $affichEDP == `echo ${TABDRCFR[$i]}` ]
            then	
			echo OK
		else
		mv $FICEDP /data/flf/35gchg/files/rejetChg/pcr_poubelle_HST/
		   break
            fi

Ensuite j'ai initialisé la variable i à l'intérieur de la double boucle, mais toujours la même chose, il me déplace tous les fichiers

PS : je suis débutant en KSH, donc j'ai essaye de comprendre la logique

• pourquoi ne pas déclarer DRCFR tout de suite sous la forme d'un tableau ?
• pourquoi ne pas déclarer FICREPEDP tout de suite sous la forme d'un tableau, et en utilisant le Développement des chemins (glogging), plutôt que find | grep qui est une mauvaise pratique, puisque find peut utiliser nativement des regex et la Substitution de commandes ne protègent pas les noms des fichiers (blancs, alinéas...) ?

je n'utilise pas KSH (d'ailleurs, pourquoi KSH ?), je serais surpris qu'il ne soit pas possible de faire aussi simplement qu'en bash une simple boucle for sur le Développement des chemins du format de noms de fichiers souhaité...

PS: n'oublie pas d'indenter ton script. Ça aide la lecture.
mais je comprends que tu ne veuilles pas qu'on lise ça.

Merci pour ton retour.

J'avais déclaré au départ directement un tableau, mais ça n'a pas fonctionné (erreur de syntaxe et je n'arrive pas à le savoir), du coup je suis passé par une variable qui contient les combinaisons correctes et je l'ai affecté à un tableau.

Pourquoi KSH ? le projet exige son utilisation

PS : je suis débutant en KSH donc j'essaie de comprendre la logique

[eliot_net]

Merci pour ton retour.

j'ai inversé le test ligne 12, cela veux dire :

Code :

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

1
2
3
4
5
6
7
           if [ $affichEDP == `echo ${TABDRCFR[$i]}` ]
            then	
			echo OK
		else
		mv $FICEDP /data/flf/35gchg/files/rejetChg/pcr_poubelle_HST/
		   break
            fi

Ensuite j'ai initialisé la variable i à l'intérieur de la double boucle, mais toujours la même chose, il me déplace tous les fichiers

PS : je suis débutant en KSH, donc j'ai essaye de comprendre la logique

Là, tu as bien inversé le test (en mettant == à la place de !=), mais tu as échangé également la commande dans le then avec la commande dans le else: il n'est pas étonnant que le résultat ne soit pas meilleur qu'avant.
Définis bien ce que tu veux faire, et ensuite tu pourras le coder. Dans son message, Flodelarab te donne une solution immédiate qui se sert de deux fichiers (celui des "motifs" et celui des "noms de fichiers"). Tape "man grep" pour les options -v et -f, et "man xargs"...

[disedorgue]

En bash, en activant:

Code :

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

shopt -s extglob

on peut avoir ceci:

Liste complète des fichiers présents dans le répertoire:

Code :

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

1
2
3
4
5
6
$ printf "%s\n" *
HST_HST35G02_EDP_H5+63_G02R00C00_5799_I1600063.DATAE00
HST_HST35G02_EDP_Q2+44_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+14_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+45_G02R00C00_5909_I1523254.DATAE00
HST_HST35G02_EDP_T1+31_G02R00C00_5801_I1500213.DATAE00

commande pour ne garder que ceux qui ne réponde pas au pattern de la globbing:

Code :

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

1
2
3
4
$ printf '%s\n' HST_HST35G02_EDP_!(1G+91*|2G+93*|3G+91*|4G+94*|A0+59*|B2+33*|C0+54*|E0+21*|F4+59*|H3+63*|H5+07*|J1+83*|J2+83*|J4+07*|K2+34*|L1+54*|M1+44*|N2+45*|P0+33*|Q2+35*|R0+14*|S0+21*|T1+31*|U1+77*|U2+79*|U3+74*|U6+78*|W4+77*)
HST_HST35G02_EDP_H5+63_G02R00C00_5799_I1600063.DATAE00
HST_HST35G02_EDP_Q2+44_G02R00C00_5801_I1500213.DATAE00
HST_HST35G02_EDP_R0+45_G02R00C00_5909_I1523254.DATAE00

Donc, pas besoin de boucle, un simple mv devrait suffire...

En principe, cela fonctionne aussi en ksh (93) et je ne crois pas qu'il faille activer quoi que ce soit (mais j'ai pas moyen de tester pour l'heure).