[sed] Enlever des lignes

Bonjour,

je voudrais enlever plusieurs ligne d'un fichier mais il me dit ceci:

Code:

---

sed: -e expression n°1, caractère 65: expression régulière d'adresse inachevée

---

voici la partie du script qui enlève des lignes du fichier $nom, lues dans "fichier"

Code:

---

1 2 3 4 5 6 7 8

tmp="$(zenity --file-selection --multiple --filename /media/$USER/WALKMAN/MUSIC/)" echo $tmp>fichier tr "|" "\n" <fichier>fichier2 cut -d/ -f 6- fichier2 > fichier for i in "$(cat fichier)";do sed -i '/'"${i//\//\\/}"'/d' "$nom" done rm fichier fichier2;;

---

quelqu'un a une idée?

d'abord, ce n'est pas comme ça (for i in "$(cat fichier)";do) qu'on lit un fichier en shell, mais comme ça :

Code:

---

while read line; do : whatever with "$line"; done

---

parce que for lit des mots, pas des lignes.

ensuite, le recours aux fichiers temporaires est une mauvaise pratique : cela requiert des accès disques qui ralentissent le script, tout comme l'emploi de commandes externes (tr, cut...).

enfin, je te l'ai déjà demandé, il nous faut des données (saisie, extrait représentatif de fichier) pour analyser le comportement du script.

zenity retourne le nom des fichier,avec leur chemein,séparés par une barre verticale
par exemple:

Code:

---

/media/mathieu/WALKMAN/MUSIC/bon_funk/calif_so.mp3|/media/mathieu/WALKMAN/MUSIC/bon_funk/carleen_.mp3|/media/mathieu/WALKMAN/MUSIC/bon_funk/darkness.mp3|/media/mathieu/WALKMAN/MUSIC/bon_funk/freedom_.mp3

---

et le fichier duquel supprimer des lignes ?

c'est un fichier m3u. chaque ligne un fchier avec un chemin relatif

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

cat /media/mathieu/WALKMAN/MUSIC/dolly.m3u Dolly\Dolly-Plein_air\01.Un_peu_de_mer.mp3 Dolly\Dolly-Plein_air\02.Cest_pour_toi.mp3 Dolly\Dolly-Plein_air\03.Comment_taire.mp3 Dolly\Dolly-Plein_air\04.Il_fait_si_bon.mp3 Dolly\Dolly-Plein_air\05.Matins_dencre.mp3 Dolly\Dolly-Plein_air\06.To_Be_Loved.mp3 Dolly\Dolly-Plein_air\07.Un_homme_éternel.mp3 Dolly\Dolly-Plein_air\08.Under_the_Bridge.mp3 Dolly\Dolly-Plein_air\09.Liquide_électrique.mp3 Dolly\Dolly-Plein_air\10.U_Cant_Hide.mp3 Dolly\Dolly-Plein_air\11.God.mp3 Dolly\Dolly-Plein_air\12.Que_du_bô.mp3 Dolly\Dolly-Plein_air\13.Tim.mp3 Dolly\Dolly-Plein_Air_Bonus\01.Elles_dorment_déjà.mp3 Dolly\Dolly-Plein_Air_Bonus\02.Allways_on_my_mind.mp3 Dolly\Dolly-Plein_Air_Bonus\03.Sweet_Jane.mp3 Dolly\Dolly-Plein_Air_Bonus\04.You_cant_hide.mp3 Dolly\Dolly-Plein_Air_Bonus\05.Un_peu_de_mer.mp3 Dolly\Dolly-Plein_Air_Bonus\06.Matins_dencre.mp3 Dolly\Dolly-Plein_Air_Bonus\07.Lhiver.mp3 Dolly\dolly - sunday afternoon\001 No One But You.mp3 Dolly\dolly - sunday afternoon\002 Angel (Version Anglaise).mp3 Dolly\dolly - sunday afternoon\003 Don't Lose That Feeling.mp3 Dolly\dolly - sunday afternoon\004 Sometimes.mp3 Dolly\dolly - sunday afternoon\005 Sunday Afternoon.mp3 Dolly\dolly - sunday afternoon\006 Un Jour De RИves.mp3 Dolly\dolly - sunday afternoon\007 Years Go By.mp3 Dolly\dolly - sunday afternoon\008 Nothing.mp3 Dolly\dolly - sunday afternoon\009 I Love Your Hands.mp3 Dolly\dolly - sunday afternoon\010 Colouring.mp3 Dolly\dolly - sunday afternoon\011 Love And Money.mp3 Dolly\dolly - sunday afternoon\012 Never.mp3 Dolly\dolly - sunday afternoon\013 RВgis.mp3 Dolly\Tous des stars\Disc 1 - 01 - Tatutia.mp3 Dolly\Tous des stars\Disc 1 - 02 - Au paradis.mp3 Dolly\Tous des stars\Disc 1 - 03 - Tous des stars.mp3 Dolly\Tous des stars\Disc 1 - 04 - Machines.mp3 Dolly\Tous des stars\Disc 1 - 05 - Il était une fois.mp3 Dolly\Tous des stars\Disc 1 - 06 - Manga Tears.mp3 Dolly\Tous des stars\Disc 1 - 07 - 7..mp3 Dolly\Tous des stars\Disc 1 - 08 - Tout va bien.mp3 Dolly\Tous des stars\Disc 1 - 09 - Faut y croire.mp3 Dolly\Tous des stars\Disc 1 - 10 - Assez de nous.mp3 Dolly\Tous des stars\Disc 1 - 11 - Les Bulles.mp3

---

la barre oblique est un antislash car ce baladeur fonctionne comme ça
une idée?

ben oui. :roll: les motifs ne correspondent pas !
il faut supprimer le chemin vers le répertoire (ou le remplacer par le chemin affiché dans le fichier), et remplacer les slashes par des antislashes.

sans utiliser tr,cut...?
comment?

en fait le fichier temporaire travaille avec des slash et quand vient le moment d'enregistrer dan le baladeur, c'est à ce moment qu'il change les slash par des antislash
ça marche quand on enlève juste un fichier du m3u

alors pourquoi nous donner le m3u du baladeur ?
il nous faut celui sur lequel le sed est appliqué.

les fichiers temporaires seraient avantageusement remplacés par un tableau :

Code:

---

1 2 3

$ wDir="/media/$USER/WALKMAN/MUSIC/" $ IFS='|' read -ra liste < <(zenity --file-selection --multiple --filename "$wDir") $ for i in "${liste[@]}"; do i=${i##*$wDir"}; sed /'"${i//\//\\/}"'/d' "$m3u"; done

---

mais sans savoir précisément sur quoi et avec quoi on travaille, il n'est pas dit que ça fonctionne tel quel !

le fichier du baladeur est un exemple. le fichier sur lequel on travaille est pareil, juste \ remplacé par /
merci pour ces commandes. Sinon, pour la troisième commande, le shell attend une suite

le nom du fichier m3u est donné par l'utilisateur et est récupéré dans la variable $nom

Code:

---

for i in "${liste[@]}"; do i=${i##*"$wDir"}; sed '/'"${i//\//\\/}"'/d' "$m3u"; done

---

ceci donne le contenu du fichier "$m3u"
le script doit effacer plusieurs lignes, indiquées par l'utilisateur avec zenity
une idée?

Citation:

---

Envoyé par matser

le fichier du baladeur est un exemple. le fichier sur lequel on travaille est pareil, juste \ remplacé par /

---

ah, et c'est à moi de faire la conversion pour faire ton travail !?

Citation:

---

ceci donne le contenu du fichier "$m3u"

---

probablement pas en entier...

Citation:

---

le script doit effacer plusieurs lignes, indiquées par l'utilisateur avec zenity

---

tu ne vois rien près de la commande sed, qui pourrait empêcher la suppression souhaitée ?

je vais potasser un peu les tableau et le paramètres en bash et je reviendrais si il y a problème

à la place de la boucle for, qui exécute autant de fois sed que l'utilisateur a sélectionné de fichiers
on peut faire comme ça :

Code:

---

1 2	$ regex=$(IFS='|'; echo "${liste[*]//\//\\/}") # la regex aura cette forme : chemin\/fichier1|chemin\/fichier2|chemin\/fichierN. la | a valeur de OU $ sed -E -i '/'"$regex"'/d' "$m3u" # sed est exécuté une seule fois

---

ça va accélérer l'exécution du script.

Citation:

---

Envoyé par N_BaH

ah, et c'est à moi de faire la conversion pour faire ton trava

---

ben non, c'est déjà fait sinon le m3u du baladeur n'existerai pas

Code:

---

sed -i 's/\//\\/g' "$m3u"

---

j'ai pas encore exécuté les dernières commande, mais je vais le faire de suite

Bonjour :coucou:

Code:

---

1 2 3 4 5

zenity --file-selection --multiple --filename /media/$USER/WALKMAN/MUSIC/ | awk -vRS='|' 1 /media/mathieu/WALKMAN/MUSIC/bon_funk/calif_so.mp3 /media/mathieu/WALKMAN/MUSIC/bon_funk/carleen_.mp3 /media/mathieu/WALKMAN/MUSIC/bon_funk/darkness.mp3 /media/mathieu/WALKMAN/MUSIC/bon_funk/freedom_.mp3

---

Et pour filtrer les lignes d'un fichier dans un autre fichier, on peut penser à grep, (ou remplir awk de regex)

Code:

---

grep -vf aExclure.txt fichierCible.txt

---

Donc si tu es flemmard :

Code:

---

zenity --file-selection --multiple --filename /media/$USER/WALKMAN/MUSIC/ | awk -vRS='|' 1 | grep -vf aExclure.txt -

---

Citation:

---

Envoyé par matser

ben non, c'est déjà fait sinon le m3u du baladeur n'existerai pas

Code:

---

sed -i '/\//\\/' "$m3u"

---

j'ai pas encore exécuté les dernières commande, mais je vais le faire de suite

---

ce que je voulais dire, c'est que tu ne nous fournis pas le bon fichier, et que ce n'est pas à nous de convertir le fichier du baladeur (avec des \) au format UNIX, pour pouvoir t'aider.

je ne comprends pas ce que tu fais.

• tu récupères le fichier .m3u du baladeur vers ton PC
• tu convertis les antislashes (\) en slashes (/)
• tu sélectionnes les fichiers présents sur ton PC que tu veux supprimer du .m3u
• tu convertis les slashes (/) en antislashes (\)
• tu renvoies le .m3u vers le baladeur

c'est ça ?i ll faut nous expliquer.

je n'utilise pas zenity, donc je ne sais pas... il n'y a pas moyen d'utiliser le .m3u pour faire une liste de cases à cocher ?

c'est presque ça:

• donner un nom à la nouvelle liste
• ajouter des mp3 du disque dur ou autre
• possibilité d'en enlever, puis en ajouter d'autres etc...
• copier le m3u dans le baladeur après avoir converti / en \

@Flodelarab

Code:

---

1 2 3

$ m3u=nom de la liste $ zenity --file-selection --multiple --filename /media/$USER/WALKMAN/MUSIC/ | awk -vRS='|' 1 > fichier $ grep -vf fichier "$m3u"

---

après ça, les lignes de "$m3u" n'ont pas été enlevé.
faut-il faut enlever le "/media/$USER/WALKMAN/MUSIC/" de "fichier"?

Citation:

---

Envoyé par N_BaH

je n'utilise pas zenity, donc je ne sais pas... il n'y a pas moyen d'utiliser le .m3u pour faire une liste de cases à cocher ?

---

Tout à fait, il faut "parser" le m3u et mettre les titres et numéro de ligne dans une liste
format : FALSE no_ligne "titre"

Code:

---

1 2 3 4 5 6 7 8

choix=(FALSE 1 "pierre" FALSE 2 "paul" TRUE 3 "arlette") # ici, ligne 3 est l'intruse response=$(zenity --list --checklist --separator=" " --width=600 --height=450 \ --title "Filtre musiques" \ --text="Sélectionner les musiques à supprimer" \ --column="Supprimer ?" --column="id" --column="Musique" \ "${choix[@]}" \ ); echo "${response[@]}"

---

ici, $response est la liste des lignes à supprimer (ou garder... suivant le label du dialogue)

Citation:

---

Envoyé par matser

après ça, les lignes de "$m3u" n'ont pas été enlevé.
faut-il faut enlever le "/media/$USER/WALKMAN/MUSIC/" de "fichier"?

---

Code:

---

zenity | awk -vRS='|' -F/ '{print $NF;}' | grep -vf - "$m3u"

---