Discussion :

[pitbac23]

Bonjour,
je suis occupé a réaliser mes premiers scripts Bash
Soyez donc svp indulgent quand a la repetition de mon code.
Je bloque sur une fonction qui devra renommer les fichiers s'ils ne sont pas vides
et ce dans une boucle for.
voici le code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#!/bin/bash
        x=1
                for file in *.avi* ;do
                if [[ ! -s "$file" ]]; then
                        echo " fichier vide verifier vos sources"
                        exit 1
 
                elif [[ -s "file" ]]; then
                        mv "$file" /*${file/%[par2][PAR2]/repa}=>substitution de test
                fi
                mv $file pearlharbor.$x
                ((x+=1))
                done

Donc pour chaque occurences contenant *.avi.* ,un test est fait pour voir si le fichier est vide,si au moins l' un d'entre eux est vide il quitte , car cela entrenerait un probleme d'avoir un fichier vide pour le reste du script.
s'il ne l'est pas il continu.
Je trouve ce code tres sale un niveau des MV et cela me conviens pas .

Ce que je voudrais est qu'il renomme les fichiers avi en $monchoix.$x
seulement avi peut avoir plusieur place dans le fichier et pas seulement en suffix :
exemple => test.avi , test.avi.ancienfichier test.avi.001 etc .....
j'avais pensé a $(ls -C | grep blablabla) mais j'entend souvent dans les forum que ls n'est pas fait pour le script et est tres deconseillé.
J'aurais donc besoin d'un peu d'éclaircissement .
Merci

[Sve@r]

Donc pour chaque occurences contenant *.avi.* ,un test est fait pour voir si le fichier est vide,si au moins l' un d'entre eux est vide il quitte , car cela entrenerait un probleme d'avoir un fichier vide pour le reste du script.

Pourquoi ne pas l'effacer et continuer ? Ou au pire ne pas le traiter ???
En tout cas c'est pas cool pour les fichiers suivants. Enfin c'est toi qui voit...

j'avais pensé a $(ls -C | grep blablabla) mais j'entend souvent dans les forum que ls n'est pas fait pour le script et est tres deconseillé.

Ah bon ???
Ben cela ne se dit pas dans les bons forums peuplés de bons unixiens. Tu peux utiliser dans ton script sans remord toute commande existante (et même créer la tienne si elle n'existe pas).

Ce que je voudrais est qu'il renomme les fichiers avi en $monchoix.$x
seulement avi peut avoir plusieur place dans le fichier et pas seulement en suffix :
exemple => test.avi , test.avi.ancienfichier test.avi.001 etc .....
J'aurais donc besoin d'un peu d'éclaircissement .
Merci

Bah, je tape ça en live (sans tester) mais ça devrait le faire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#!/bin/sh
i=1
ls *avi* |while read file
do
    if test ! -s "$file"
    then
        echo "$file vide => on le passe"
        continue
    fi
 
    echo "remplace $file par $monchoix.avi.$i"
    mv "$file" "$monchoix.avi.$i"
    i=`expr $i + 1`
done

Si le avi peut aussi être en majuscule, remplacer "ls *avi* |while read file" par ls |grep -y avi |while read file

[pitbac23]

Rebonjour,
Merci de m'avoir répondu avec une telle précision.
j'ai testé ton code seulement il y a l'air d'avoir un probleme avec read
il lit son entrée standard mais il bloque
le code je l'ai un peu modifié le voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#!/bin/bash
set -x
set test
        x=1
                ls | grep -i *avi* | while read fichier; do
                if [[ ! -s "$fichier" ]]; then
                        echo "les fichiers vides vont etre effaces"
                        rm -f $fichier                   
                        continue
                fi
                mv $fichier $1.$x
                ((x+=1))
                done

le set -x me donne un + a read fichier mais plus rien ensuite.

Pourquoi ne pas l'effacer et continuer ? Ou au pire ne pas le traiter ???
En tout cas c'est pas cool pour les fichiers suivants. Enfin c'est toi qui voit...

l'effacer serait une solution et je l'ai donc retenue.

Je ne suis pas un foudre de l'anglais mais en lisant cette page que j'ai compris partiellement , j'en ai deduis que ls n'était pas aproprié.
http://mywiki.wooledge.org/ParsingLs

Merci

[pitbac23]

Re bonjour,
alors j'ai trouvé pourquoi cela n'allait pas .
read n'était pas alimenter ce qui causait son blocage .
Je me suis dis des lors que cela ne pouvait venir que du ls
j'ai tester l'expression simple et il ne me retournait aucun fichier.
je l'ai changé en ERE et ca marche
je post ci dessus le code qui fonctionne

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#!/bin/bash
set -x
set test
        x=1
                ls | grep -E '+(avi)' | while read fichier; do
                if [[ ! -s "$fichier" ]]; then
                        echo "les fichiers vides vont etre effaces"
                        rm -f $fichier                   
                        continue
                fi
                mv $fichier $1.$x
                ((x+=1))
                done

Je remercie tout le monde de leur aide et plus particulierement Svear

[Sve@r]

Re bonjour,
alors j'ai trouvé pourquoi cela n'allait pas .
read n'était pas alimenter ce qui causait son blocage .

Ben en fait si le "ls *avi*" ne renvoie rien, ça ne bloque pas mais ça ne rentre pas non plus dans le while quoi...

Je ne suis pas un foudre de l'anglais mais en lisant cette page que j'ai compris partiellement , j'en ai deduis que ls n'était pas aproprié.
http://mywiki.wooledge.org/ParsingLs

Si on lit mieux, il dit qu'en fait si t'as des caractères bizarre dans tes noms de fichiers le ls renverra autre chose ce qui peut poser souci. Il conseille d'ailleurs de ne jamais faire "for file in `ls`" mais c'est la syntaxe qui est maladroite et non le ls lui-même. Mais c'est du shell extrème quoi...

[pitbac23]

salut svear

c'est exactement ce que j'avais comme probleme avec la boucle for que j'avais precedemment .
exemple: les fichiers avait comme nom =>Pear extensio.avi Pear extension.avi.1
il y avait 30 fichiers
le ls dans la boucle for me rapportait 62 fichiers :-)
au faite il les interpretait comme
Pearl
exentensio.avi
Pearl
extension.avi.1
etc ....
donc il passait a la ligne sur le separateur espace
meme avec les guillements "$i" ce passait pas.

[Sve@r]

salut svear

c'est exactement ce que j'avais comme probleme avec la boucle for que j'avais precedemment .
exemple: les fichiers avait comme nom =>Pear extensio.avi Pear extension.avi.1
il y avait 30 fichiers
le ls dans la boucle for me rapportait 62 fichiers :-)

Le for sépare ses éléments sur l'espace => d'où le ls |while read => le read sépare ses éléments sur le <return> qui se trouve en fin de ligne